[object Object]

← back to Cli Printing Press

fix(skills): auto-polish after dogfood testing when fixes were applied

e7f756b2238ffb04e9f6ec5147daa2913c156714 · 2026-04-04 15:12:48 -0700 · Trevin Chow

If dogfood testing applied any fixes, automatically run one verify +
scorecard pass before promoting. Dogfood fixes can introduce dead code
or break verify checks. Don't ask — the user opted into quality by
choosing to test. Report the before/after delta inline.

Skip the auto-polish when all dogfood tests passed first try (no fixes
needed).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Files touched

Diff

commit e7f756b2238ffb04e9f6ec5147daa2913c156714
Author: Trevin Chow <trevin@trevinchow.com>
Date:   Sat Apr 4 15:12:48 2026 -0700

    fix(skills): auto-polish after dogfood testing when fixes were applied
    
    If dogfood testing applied any fixes, automatically run one verify +
    scorecard pass before promoting. Dogfood fixes can introduce dead code
    or break verify checks. Don't ask — the user opted into quality by
    choosing to test. Report the before/after delta inline.
    
    Skip the auto-polish when all dogfood tests passed first try (no fixes
    needed).
    
    Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 skills/printing-press/SKILL.md                     |  8 ++++--
 .../printing-press/references/dogfood-testing.md   | 30 ++++++++++++++++++++++
 2 files changed, 36 insertions(+), 2 deletions(-)

diff --git a/skills/printing-press/SKILL.md b/skills/printing-press/SKILL.md
index 161eaccb..c47ec363 100644
--- a/skills/printing-press/SKILL.md
+++ b/skills/printing-press/SKILL.md
@@ -1269,8 +1269,12 @@ test plan execution, inline fix workflow, and reporting.
 - Fix issues as you find them, not at the end. The value is discovering bugs in
   context.
 - Note whether each fix is CLI-specific or a machine issue (feeds the retro).
-- If a fix changes the sync, data layer, or output pipeline, re-run the relevant
-  shipcheck tool (`verify` or `scorecard`) once after all dogfood fixes.
+
+**After dogfood completes:** If any fixes were applied during testing, automatically
+run one polish pass (re-run `verify`, `scorecard`, fix regressions) before promoting.
+The dogfood fixes may have introduced dead code, broken a verify check, or shifted
+the scorecard. Don't ask — just run it. The user opted into quality by choosing to
+test. Report the before/after delta inline.
 
 Write:
 
diff --git a/skills/printing-press/references/dogfood-testing.md b/skills/printing-press/references/dogfood-testing.md
index f68b1a47..1b92d556 100644
--- a/skills/printing-press/references/dogfood-testing.md
+++ b/skills/printing-press/references/dogfood-testing.md
@@ -139,6 +139,36 @@ Dogfood Results: <cli-name>
     - [list issues for the retro]
 ```
 
+### Step 5: Auto-polish (if fixes were applied)
+
+If any fixes were applied during dogfood (M > 0), automatically run one
+verification pass. Do not ask the user — they opted into quality by choosing
+to test. Dogfood fixes can introduce dead code, break verify checks, or shift
+the scorecard.
+
+```bash
+# Rebuild after dogfood fixes
+go build -o "$CLI_NAME" ./cmd/"$CLI_NAME"
+gofmt -w .
+
+# Re-run verification tools
+printing-press verify --dir "$CLI_DIR" $SPEC_FLAG --fix
+printing-press scorecard --dir "$CLI_DIR" $SPEC_FLAG
+```
+
+Fix any regressions introduced by dogfood fixes (typically: unused imports
+after removing code, dead functions from refactoring). Report the delta:
+
+```
+Post-dogfood polish:
+  Verify:    86% → 93% (+7%)
+  Scorecard: 92 → 94 (+2)
+  Fixed: removed 2 dead functions, fixed 1 verify regression
+```
+
+If no fixes were applied during dogfood (everything passed first try), skip
+this step.
+
 ### Common failure patterns
 
 | Symptom | Likely cause | Fix location |

← bc320846 fix(cli): FTS trigger safety, envelope unwrapping, dogfood t  ·  back to Cli Printing Press  ·  fix(skills): polish always runs after shipcheck, not just af d509976c →