merge queue: checking #331 on main (a2773cc) - #334
Closed
mergify[bot] wants to merge 3 commits into
Closed
Conversation
65 of 185 merged PRs in this repo touch engine/hooks/, 13 of 28 hooks needed post-ship repair, and 37 PRs did nothing but repair a shipped detector. gh-write-verification hit two already-known kinds in three days (#322, #323, #324). create-skill never mentions hooks, CONTRIBUTING has no hook section, and 10 of 31 hooks have no docs/ecosystem.md row. The playbook is a verbatim 20-step ordered list, copied into a todolist before task-specific work; a step that does not apply stays in the list marked `skip: <reason>`. Steps 4-10 are the seven recurring defect kinds, one per step. Steps 14-18 are the install, README, and inventory tail that this repo measurably drops. Step 20 hands off to make-pr. Every numbered step cites the PRs that motivated it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…authoring a detector follows one written, ordered sequence whose steps come from the repair history rather than from memory. Review lane: behavior Safety invariant: the playbook adds no new mechanical gate and changes no existing hook, so a detector that passes CI today still passes; it can only change what a future author does before opening a PR. Slice rationale: the sequence has to exist before anything can route to it, and its steps are derived from thirty-seven specific PRs, so writing it is a separate reviewable claim from wiring the routing. Architectural effect: adds the repo's second playbook package; its final step calls the existing make-pr skill rather than duplicating publication. Goal: add product/skills/ship-a-detector with a playbook carrying the ordered steps that the thirty-seven repair PRs imply. Effectiveness measurement: each of the seven recurring defect kinds maps to a numbered step, and each step cites the PR numbers that motivated it, so a reader can check the step against the incident rather than trusting the author. Motivation: gh-write-verification shipped this month and hit two of the seven known kinds within three days, authored by an agent with the whole corpus available; the knowledge existed and was not reachable. Alternative considerations: adding the steps to create-skill was rejected because that skill covers skills rather than hooks and mixing them would bury both; a new hook was rejected because the gap is a missing order, not a missing check. Implementation details: the seven kinds are target and scope resolution, false positive on quoted or fenced content, missed near-miss shape, retry-loop behaviour under stop_hook_active, first-failure exit hiding a second check, unreadable input reported as clean, and lifecycle state across turns. Non-goals: does not modify any existing hook, does not add a CI gate, does not change create-skill, does not reimplement what make-pr does. Layer: domain Feature state: active Files: product/skills/ship-a-detector/** (new) Change types: new skill, new playbook Acceptance criteria: check_skill_test_coverage.py passes for the new skill, and every numbered step cites at least one PR number. Exit code: 0
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎉 This pull request has been checked successfully and will be merged soon. 🎉
#331 is queued for merge on branch main (a2773cc).
This pull request has been created by Mergify to check the mergeability of #331.
You don't need to do anything. Mergify will close this pull request automatically when it is complete.
Required conditions of queue rule
admin-bypassfor merge:check-success = lintcheck-success = testRequired conditions to stay in the queue:
-draftbase=mainlabel=admin-bypass