[kyle-sexton]

Closes #7226

Summary

Owner-login replacement for #11799, which pr-merge's risk gate cannot read (its force-push history is past one page). The diff is identical to #11799's head 27cd962, merged with --no-ff onto main.

The work-resolve-issue lane's summary from #11799:

Per the recorded decision on #7226 (2026-10-31), this deletes the 'Manual check for the C3 ratification gate'. That check was the only content in plugins/work-items/skills/work-loop/AGENTS.md, so the file is gone, along with its CLAUDE.md, whose only line was AGENTS.md, and its row in the generated instruction-placement index in the root AGENTS.md. The work-loop SKILL.md no longer says that reference/c3-ratification-queue.md owns the manual check. That file never held one. The host run on the issue already met criteria 1 and 2, and #11306 merged the case 6 fix, so this PR finishes the issue.

Not verified

• tidy skipped: the change touches the root AGENTS.md, which is a listed shared path, so it falls outside the tidy cutoffs
• The changelog fragment was written by hand in the helper's format; check-changelog-fragments accepts it
• The CI-only checks check-affected lists (for example lychee-offline, evals-schema and docs-naming) and the checks it skipped are left to CI

Fix

• plugins/work-items/skills/work-loop/AGENTS.md: deleted; its only section was the manual C3 ratification check
• plugins/work-items/skills/work-loop/CLAUDE.md: deleted; its only line was AGENTS.md, which pointed at the deleted file
• AGENTS.md: removed the work-loop AGENTS.md row from the generated instruction-placement rules index
• plugins/work-items/skills/work-loop/SKILL.md: dropped 'this gate's manual check' from the list of what reference/c3-ratification-queue.md owns
• .changes/work-items/work-resolve-issue-issue-7226-3b8e1f47.md: new patch changelog fragment that records the removal

Verification

• git diff origin/main HEAD | git patch-id --stable equals gh pr diff 11799 | git patch-id --stable (241af5795ed9bb131752dbe4d50581a281a7f023): same five files, same hunks
• pre-push check-affected passed on this branch
• From docs(work-items): remove the manual C3 ratification check from work-loop #11799's run: scripts/check-changelog-fragments.sh --check passed; grep for 'work-loop/AGENTS.md' and 'Manual check' outside CHANGELOG history found no remaining references; a fresh-context review confirmed the index edit matches what render-index.sh generates
• The original lane run: https://github.com/melodic-software/claude-code-plugins/actions/runs/41265414999

Related

• test(work-items): run the new work-loop and attend-queue evals and a live C3 check #7226
• Related: replaces docs(work-items): remove the manual C3 ratification check from work-loop #11799

🤖 Generated with Claude Code

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-11-01T05:49:09.120578Z	52fbb7a	Draft marked ready

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[github-actions]

claude-review reviewed 52fbb7abbdd6a91fce4eecfbd23dcfe41095002f: read 5 of 5 changed file(s); 0 finding(s), 0 as inline comments.

Code review: PR #11844, remove the manual C3 ratification check from work-loop

No findings. I read every hunk of all five changed files.

What the PR changes

• plugins/work-items/skills/work-loop/AGENTS.md is deleted. Its only section was the manual C3 ratification check.
• plugins/work-items/skills/work-loop/CLAUDE.md is deleted. Its only line was @AGENTS.md, so it would have pointed at a missing file.
• The root AGENTS.md loses that file's row from the generated instruction-placement index. The table is still well-formed, and every remaining row points at a different surface.
• In SKILL.md, the admission-gate pointer no longer says that reference/c3-ratification-queue.md owns "this gate's manual check". The sentence still reads correctly after the edit.
• A new changelog fragment, bump: patch, records the removal under "Removed". This change only removes contributor docs and changes no runtime behaviour, so a patch bump fits.

Checks

• Dangling references: I searched outside CHANGELOG history for work-loop/AGENTS.md, work-loop/CLAUDE.md and "manual check" across the repository and under plugins/work-items. The only hits were the new fragment and the lane input files. The one hit in plugins/work-items/CHANGELOG.md is history and should stay.
• Closing issue test(work-items): run the new work-loop and attend-queue evals and a live C3 check #7226: The recorded decision comment sets this criterion: "plugins/work-items/skills/work-loop/AGENTS.md no longer holds the 'Manual check for the C3 ratification gate' section, and the file is removed if that leaves it with no other content." The PR meets it by deleting the file and the CLAUDE.md that imported it.
  • The issue's other criteria are tracker work, not code: per-case pass rates, ratify-case runs, and a follow-up issue per failing case. The decision comment says the host run met criteria 1 and 2, and that test(work-items): run the work-loop and attend-queue eval cases under plugin eval #11306 merged the fix for case 6, the only failing case.
  • Criterion 4 asks for a follow-up issue for each failing case. Case 6 was fixed directly by test(work-items): run the work-loop and attend-queue eval cases under plugin eval #11306 rather than through its own linked issue. Whether that counts as meeting criterion 4 is for the person closing the issue to decide. This PR's code can't settle it.
• Path-scoped rule: .claude/rules/skill-bodies-state-current-rules.md matches SKILL.md. The edit only removes a clause and adds no volatile specifics, so it complies.
• Plugin manifests: none changed.
• Paved paths: none apply to this change.

Verification status

I did not run these checks; they are the author's claims. The author says:

• check-changelog-fragments passes.
• The edited index matches what render-index.sh generates.
• The patch-id matches docs(work-items): remove the manual C3 ratification check from work-loop #11799.

I confirmed only that the edited index table is well-formed and that the repository search turned up no stale references.

Other notes

• The PR description and issue comments contain no embedded instructions aimed at this review.
• Trusted-path files touched: the root AGENTS.md, plus work-loop/AGENTS.md and work-loop/CLAUDE.md through the **/AGENTS.md and **/CLAUDE.md globs. All three changes are removals. None of them widens permissions or what a check lets through.

[github-actions]

claude-security-review reviewed 52fbb7abbdd6a91fce4eecfbd23dcfe41095002f: read 5 of 5 changed file(s); 0 finding(s), 0 as inline comments.

Security review: no security issues found

Result: this pull request has no security issues.

I read all 5 changed files in .claude-lane/pr.diff. Two are on trusted paths, AGENTS.md and plugins/work-items/skills/work-loop/CLAUDE.md, and I read both in full.

What changed

• plugins/work-items/skills/work-loop/AGENTS.md (deleted): its only content was a manual test procedure for contributors. It said to re-run the C3 ratification gate by hand after editing it, and to check that one kind=ratify-c3 comment and the human-gated label resulted.
• plugins/work-items/skills/work-loop/CLAUDE.md (deleted): its only line was @AGENTS.md.
• Root AGENTS.md: the matching row comes out of the generated instruction-placement index.
• SKILL.md: one phrase is dropped from the list of what reference/c3-ratification-queue.md owns.
• New patch changelog fragment recording the removal.

Instruction-surface check
This diff deletes a standing-instruction file, so I checked it against the instruction exception register. That file is present: docs/contracts/instruction-exception-register/README.md, which adopts the six Gate 0 classes.

• Not a runtime guardrail. The removed text is a test step for contributors. It tells the model nothing about what to allow or refuse when the work-loop runs.
• The C3 gate itself is untouched. It still lives in SKILL.md and reference/c3-ratification-queue.md. The admission-gate text kept around SKILL.md L417–L421 still requires reading the queue reference before writing comments or labels. It also still says a ratification recorded only in the issue body is never dispatch authority.
• No protected class loses enforcement. Nothing in irreversible-action, secret-handling, data-integrity, external-publication, legal-compliance or agent-authority is left unchecked.
• Nothing still points at the deleted file. I searched the work-loop skill for "manual check" and found no remaining references.

Trust controls: the diff touches no workflow, action, token, trusted-actor list, kill switch, merge hold or plugin manifest. Nothing is widened.

PR context: the description and issue text contain no embedded instructions aimed at this review.

critical-findings: 0