[object Object]

← back to Cli Printing Press

fix(skills): polish agent reports skipped findings with reasoning

c4befa79bc45c3d9d27a03967db07750f3a480ff · 2026-04-04 15:45:44 -0700 · Trevin Chow

The polish-worker agent now distinguishes three categories:
- fixes_applied: what changed
- skipped_findings: issues found but deliberately not fixed, with
  reasoning (e.g., scorer bug, needs domain context)
- remaining_issues: issues tried but couldn't resolve

This lets the caller surface skipped findings so the user can decide
whether to address them manually.

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

Files touched

Diff

commit c4befa79bc45c3d9d27a03967db07750f3a480ff
Author: Trevin Chow <trevin@trevinchow.com>
Date:   Sat Apr 4 15:45:44 2026 -0700

    fix(skills): polish agent reports skipped findings with reasoning
    
    The polish-worker agent now distinguishes three categories:
    - fixes_applied: what changed
    - skipped_findings: issues found but deliberately not fixed, with
      reasoning (e.g., scorer bug, needs domain context)
    - remaining_issues: issues tried but couldn't resolve
    
    This lets the caller surface skipped findings so the user can decide
    whether to address them manually.
    
    Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 agents/polish-worker.md | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/agents/polish-worker.md b/agents/polish-worker.md
index e34162c2..6ed3be54 100644
--- a/agents/polish-worker.md
+++ b/agents/polish-worker.md
@@ -158,12 +158,22 @@ govet_before: <N>
 govet_after: <N>
 fixes_applied:
 - <one-line description of each fix>
+skipped_findings:
+- <finding>: <why you chose not to fix it>
 remaining_issues:
-- <one-line description of each unfixed issue>
+- <one-line description of each issue you tried to fix but couldn't>
 ship_recommendation: <ship|ship-with-gaps|hold>
 ---END-POLISH-RESULT---
 ```
 
+The three lists serve different purposes:
+- **fixes_applied**: what changed — the caller displays these
+- **skipped_findings**: issues you found but deliberately did not fix, with reasoning
+  (e.g., "verify classifies `stale` as read — scorer bug, not a CLI problem",
+  "README cookbook section is generic — needs domain context from research brief").
+  The caller surfaces these so the user can decide whether to address them manually.
+- **remaining_issues**: issues you tried to fix but couldn't resolve
+
 Ship recommendation logic:
 - `ship`: verify >= 80%, scorecard >= 75, no critical failures
 - `ship-with-gaps`: verify >= 65%, scorecard >= 65, non-critical gaps remain

← af759a27 feat(skills): extract polish protocol into polish-worker age  ·  back to Cli Printing Press  ·  fix(skills): polish agent picks useful Quick Start commands, afcd3bd6 →