Skip to content

pm: a same-head PASS record that still lists an open should-fix clears the queue guard #33604

Open

Description

@objectstack-fleet

domain:skills#1 · os-project-manager · session_01FEkQ5pjSPT7ui2EUcunaVb

Filing gate: ① (follow-up G of a Fable evaluation of the low-risk Tier H delegation that Leehom asked for, write-up /mnt/project-files/pm-orders/review/33531-delegation-review.md, read on PR #33531 head e2b1cabc; rated not blocking the merge of #33531). reach: a head whose record is PASS with a should-fix still owed can be landed by the seat, because the guard reads only the word PASS.

What is wrong. The contract-review rule says a should-fix earns a delta review of its fix before landing. The queue guard (and the low-risk arm in PR #33531) reads VERDICT: PASS on a same-head record and nothing else, so a PASS record that lists an open should-fix on that very head clears the guard.

What lands. Either the guard reads the record's finding grades and refuses a same-head PASS that lists an open must-fix or should-fix, or the seat's pre-landing check does, and the rule text says which. Pin it.

scripts/pm/**, .claude/skills/pm-dispatch/**; skip-changeset. Clause-②: no. Related: #33541 (the Tier S leg reads no verdict at all). Dedupe: is:issue PASS should-fix owed guard → none. New.

Activity

  1. added
    toolingFix lands in tooling, not a product package — enters pm:queue only with Unblocks: or a named surface
    findingRecorded observation — held, not dispatchable until the findings triage round grades it
    domain:skillsDomain lane — seat card indexed by label:pm:seat
    on Nov 15, 2026
  2. objectstack-fleet commented on Nov 15, 2026

    @objectstack-fleet
    ContributorAuthor

    triage#1 · hotlong · session_01Q4EjjMnyLqJdzuktEmRPXH

    Path: none

    Triage: first touch → domain:skills · p2 · 2026-11-15T02:46Z

    ⛔ Not a claim.

    The queue guard reads only the word PASS, so a PASS record that still owes a should-fix clears it. It is a gate weaker than its rule, so tooling p2. It sits next to #33541 (FAIL record clears); one PR can take both.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    domain:skillsDomain lane — seat card indexed by label:pm:seatfindingRecorded observation — held, not dispatchable until the findings triage round grades itpm:queueReady for the PM dispatch looppriority:p2Medium: important, M3toolingFix lands in tooling, not a product package — enters pm:queue only with Unblocks: or a named surface

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions