Skip to content

fix(mq-feeder): required check must be a job a PR head runs (re-cut of #6315 on current main, PMAT-6175) - #6429

Open
noahgift wants to merge 8 commits into
mainfrom
recut/6315-on-main
Open

noahgift wants to merge 8 commits into
mainfrom
recut/6315-on-main

Conversation

@noahgift

@noahgift noahgift commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

Re-cut of #6315 onto current main: its three commits cherry-picked, roadmap.yaml taken from main and regenerated. Supersedes #6315 (stays open for the cop's close quorum). Unarmed. CI/workflow-adjacent diff needs its quorum and non-author sign-off.

ont-delta: none (no ontology change)

Closes #6175

🤖 Generated with Claude Code

t and others added 3 commits October 27, 2026 05:19
… head runs (PMAT-6175)

The feeder required `x86-main`, a job that runs only on merge_group/push, so it
refused every CLEAN PR. required_is_mq_only/mqbranch read ci.yml; needs split no
longer cuts hyphenated job ids; always() jobs are not mq-only; check_required
reads jobs via here-string (no pipe into grep -q); a fixture row with an
`event_name == pull_request && != pull_request_target` if kills the
mqbranch pull_request-exemption mutant. Linear re-cut of #6187 (two merge
commits failed EMG-7).

PMAT-Ticket: PMAT-6175

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
(cherry picked from commit 33609d5)
… and the installer guard (Refs #6175)

PMAT-Ticket: PMAT-6175
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
(cherry picked from commit c0618b4)
…n main) (Refs #6175)

PMAT-Ticket: PMAT-6175
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
(cherry picked from commit 89e736a)
@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

§13.11 rung 1 — quorum shadow verdict

S13-SHADOW pr=6429 head=0022e44fb4c265599a5ab39eeab06508b3dfd840 verdict=REFUSE class=Q1 arm_rc=1

Shadow mode: this records a verdict and merges nothing. A refusal
to arm is not a block (§13 adds zero rows to §7) — the pull request is
exactly as green as it was.

…844 (Refs #6175)

The row would have been a new GATE on workspace-test with no C324 cover; quorum
split (triage-quorum-6429-gate-add-0550Z). Fragment dropped, check moved to
guards-nightly with its one LAB ticket; 843 text in the postmortem and both
roadmap copies reworded.

PMAT-Ticket: PMAT-6175

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@noahgift

Copy link
Copy Markdown
Member Author

quorum-review (AD-04): NOT agreed (auto_merge: checked=true was_armed=false disarmed=false)

{
 "ticket": "PMAT-6175",
 "head": "df178d8b6f9381c89877b08d2cf50025cfaadec1",
 "width": 3,
 "executor": "agy",
 "agreed": false,
 "auto_merge": {
  "checked": true,
  "was_armed": false,
  "disarmed": false,
  "note": "auto-merge not armed"
 },
 "lanes": [
  {
   "lane": 1,
   "verdict": "PASS",
   "findings": 3
  },
  {
   "lane": 2,
   "verdict": "PASS",
   "findings": 3
  },
  {
   "lane": 3,
   "verdict": "FAIL",
   "findings": 1
  }
 ]
}

t and others added 2 commits October 27, 2026 07:28
…ures ignore the caller's env (Refs #6175)

Measured: with GUARD_CHECK_NAME=x86-main exported, the old script failed
'incident workflow refused for pr-light' and the bare-run row; the pinned one passes.

PMAT-Ticket: PMAT-6175

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
A top-level pin hid a feeder default mutated to x86-main. Measured: that mutation is
now red on the bare run; the self-test fixtures stay hermetic under an exported name.

PMAT-Ticket: PMAT-6175

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@noahgift

Copy link
Copy Markdown
Member Author

quorum-review (AD-04): NOT agreed (auto_merge: checked=true was_armed=false disarmed=false)

{
 "ticket": "PMAT-6175",
 "head": "46ffcd25b83295a055fbf546336521badd84a6cb",
 "width": 3,
 "executor": "agy",
 "agreed": false,
 "auto_merge": {
  "checked": true,
  "was_armed": false,
  "disarmed": false,
  "note": "auto-merge not armed"
 },
 "lanes": [
  {
   "lane": 1,
   "verdict": "PASS",
   "findings": 3
  },
  {
   "lane": 2,
   "verdict": "PASS",
   "findings": 0
  },
  {
   "lane": 3,
   "verdict": "FAIL",
   "findings": 1
  }
 ]
}

…ck (Refs #6175)

PMAT-Ticket: PMAT-6175

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@noahgift

Copy link
Copy Markdown
Member Author

quorum-review (AD-04): NOT agreed (auto_merge: checked=true was_armed=false disarmed=false)

{
 "ticket": "PMAT-6175",
 "head": "af24b8c03d2160945110ea6c5f44c30f5f98e5ea",
 "width": 3,
 "executor": "agy",
 "agreed": false,
 "auto_merge": {
  "checked": true,
  "was_armed": false,
  "disarmed": false,
  "note": "auto-merge not armed"
 },
 "lanes": [
  {
   "lane": 1,
   "verdict": "PASS",
   "findings": 4
  },
  {
   "lane": 2,
   "verdict": "PASS",
   "findings": 5
  },
  {
   "lane": 3,
   "verdict": "FAIL",
   "findings": 1
  }
 ]
}

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

mq_feeder: the check it requires must be a job a PR head runs (checked at runtime, not only in --self-test)

1 participant