Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 require-ready-label-and-ci

Greptile Summary

This PR adds a shared blocked-API journal flow to the Printing Press skills, closing #2143. When an API attempt is held for a repeatable reachability or buildability blocker, users can now create a durable public-library journal entry so future runs warn before repeating the same dead-end attempt.

• skills/printing-press/SKILL.md: Extends Phase 0 to fetch and check blocked-apis.json alongside registry.json, and adds an "Add to blocked-API journal" option to the hold-path menu with guardrails limiting it to repeatable blockers.
• skills/printing-press-publish/SKILL.md: Adds --blocked-api-journal mode that updates only blocked-apis.json (not registry surfaces), using a safe jq-to-tmp-then-mv pattern with pre-move JSON validation.
• internal/pipeline/contracts_test.go: Adds three contract tests that pin the new skill instructions and journal-mode publish boundaries.

Confidence Score: 5/5

Safe to merge — changes are confined to skill instruction files and contract tests with no runtime code paths altered.

The changes add a new opt-in journal flow with independent failure handling (fetch failures are logged and skipped, not blocking), correct jq-to-tmp-then-validate-then-mv write ordering, and tight staged-file scope enforced by both the skill text and a NotContains contract assertion. Previously flagged issues (suffix combining both permanent and tracking_issue, jq validation order, missing example) are all addressed in this diff.

No files require special attention.

Important Files Changed

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["/printing-press <api>"] --> B["Phase 0: Local library check"]
    B -->|"No local match"| C["Fetch registry.json + blocked-apis.json"]
    C --> D{"blocked-apis.json\nfetch succeeded?"}
    D -->|"No / absent"| F["Log: journal check skipped\nProceed to registry check"]
    D -->|"Yes"| E{"Slug matches\nblocked entry?"}
    E -->|"No match"| F
    E -->|"Match found"| G["Prompt: API held — reason + tracking suffix\nStop here (recommended) / Proceed anyway"]
    G -->|"Proceed anyway"| F
    G -->|"Stop"| STOP1["End run"]
    F --> H["Registry check → Phase 1 research"]
    H --> I{"Ship result?"}
    I -->|"Ship"| SHIP["Promote + publish flow"]
    I -->|"Hold"| J["Hold-path menu\n1 Run retro\n2 Polish to retry\n3 Add to blocked-API journal\n4 Done for now"]
    J -->|"Option 3\n(reachability/buildability hold only)"| K["/printing-press-publish --blocked-api-journal <api>"]
    K --> L["Blocked API Journal Mode\nUpdate only blocked-apis.json via jq→tmp→mv\nCreate branch + PR"]
    L --> M["Report PR URL and stop\nNo registry / package / skill-mirror steps"]

_{Reviews (4): Last reviewed commit: "fix(skills): clean blocked journal temp ..." | Re-trigger Greptile}

Merge Queue Status

• ✅ Entered queue — 2026-06-07 19:05 UTC · Rule: default
• ✅ Checks passed · on draft merge queue: embarking main (015e629), #2805, #2809, #2807 and #2806 together #2820
• ✅ Merged — 2026-06-07 19:40 UTC · at cd3fa3d87b0489f7836756d5021d17a64f16b1a1 · squash

This pull request spent 34 minutes 53 seconds in the queue, including 34 minutes 38 seconds running CI.