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 missing-mirror guard for SQLite-backed novel commands, ensuring that when the local store hasn't been populated yet (i.e., the user hasn't run sync), the command returns a clear human-readable hint to stderr and an empty JSON array for machine/agent callers rather than a SQLite open failure.

• skills/printing-press/SKILL.md: Adds standalone guidance prose and copyable guard snippet (placed after dryRunOK, required-input, and dbPath resolution gates), and injects the same guard into the full command skeleton immediately before store.OpenWithContext.
• internal/cli/printing_press_skill_test.go: Adds TestPrintingPressSkillSQLiteNovelCommandsGuardMissingMirror to assert the presence of all key strings and that the guard snippet precedes store.OpenWithContext(ctx, dbPath) in the document.

Confidence Score: 5/5

Safe to merge — the change is purely additive documentation and guidance, with no modification to existing logic or interfaces.

Both changed files are documentation and tests. The SKILL.md prose correctly positions the guard after all earlier exit gates (help probe, dry-run, required-input, dbPath resolution), the skeleton reflects that order verbatim, and the new test pins the key strings and their relative ordering in the document. No runtime code paths are altered.

No files require special attention.

Important Files Changed

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[RunE entry] --> B{bare invocation?}
    B -- yes --> C[return cmd.Help]
    B -- no --> D{dryRunOK?}
    D -- yes --> E[print dry-run msg and return nil]
    D -- no --> F{required input missing?}
    F -- yes --> G[cmd.Usage and return usageErr exit-2]
    F -- no --> H[resolve dbPath]
    H --> I{os.IsNotExist dbPath?}
    I -- yes --> J[print hint to stderr]
    J --> K{asJSON or agent?}
    K -- yes --> L[print empty array to stdout]
    K -- no --> M[return nil]
    L --> M
    I -- no --> N[store.OpenWithContext]
    N --> O[query SQLite and scan results]
    O --> P{asJSON or not terminal?}
    P -- yes --> Q[json.Encode results]
    P -- no --> R[human table output]

_{Reviews (4): Last reviewed commit: "fix(skills): document sqlite mirror exit..." | Re-trigger Greptile}

Merge Queue Status

• ✅ Entered queue — 2026-06-07 22:48 UTC · Rule: default
• ✅ Checks started · on draft merge queue: embarking main (88617bb), #2818 and #2819 together #2847
• ❌ Checks failed
• 🚫 Left the queue — 2026-06-07 22:58 UTC · at f70dd8d092d9b83f3fb0c4393b83386761021b42

This pull request spent 9 minutes 19 seconds in the queue, including 9 minutes 2 seconds running CI.

Reason

The merge conditions cannot be satisfied due to failing checks

Failing checks:

• ❌ build-and-test (job log)

Hint

You may have to fix your CI before adding the pull request to the queue again.
If you update this pull request, to fix the CI, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio queue comment.

@Mergifyio queue

Merge Queue Status

• ✅ Entered queue — 2026-06-07 23:13 UTC · Rule: default
• ✅ Checks passed · on draft merge queue: embarking main (d0a5d24), #2817, #2826 and #2819 together #2851
• ✅ Merged — 2026-06-07 23:37 UTC · at f70dd8d092d9b83f3fb0c4393b83386761021b42 · squash

This pull request spent 23 minutes 8 seconds in the queue, including 22 minutes 43 seconds running CI.