No comments
Spawn a read-only comment reviewer. Act on accepted findings.
Authoring agents defend comments. Defer to a fresh reviewer that did
not write the code.
Companion to comment-mode (granular feedback on prose drafts) and
deslop (broader AI-artifact strip). This skill is the pre-review
comment pass.
Authorized Scope
Apply this engine only within the user's requested task and existing explicit
authorization. Loading or delegating to it grants no additional authority.
Preserve report-only restrictions and the caller's target, host, provider, and
cost limits. Existing approval satisfies a gate only for the same actions and
scope; obtain approval before expanding them. Forward these limits to delegates.
Contract
Inputs:
- The caller's files or diff. Otherwise the current diff against trunk,
including the working tree.
Outputs:
- Deletion count, restored comments, reruns, fixes, encoding offers,
and open work
Creates/Modifies:
- Deletes accepted comments and applies the smallest in-scope
root-cause fixes
External Side Effects:
Confirmation Required:
- Before encoding a constraint comment into a type, test, or lint
- Unattended runs require caller pre-approval for encodings
Delegates To:
how or why when an IMPORTANT or do not remove comment is
thin
architect once, sketch only, when a fix needs a new shape
Steps
- Spawn a read-only review subagent on the fast cheap tier. Pass the
scope and references/comment-review.md.
Do not restate the rules.
- Inspect the report. Reject application-code edits, scope escapes,
exception-protected deletions, and flags that treat kept
intentional code as guilty. A keep survives only with proof it is
about something we cannot change. If a kill is ambiguous, do not
restore. If a keep is refuted or still ambiguous, delete it. Revert
and rerun one rejected report with the failure named. Reject a
second and fail this skill.
- Fix trivial accepted flags directly. If any fix needs a shape, run
architect once for the accepted set and stop at the sketch.
- Implement the smallest root-cause fix in scope. Remove every named
workaround. If the root cause is out of scope, land the smallest
in-scope fix and report the rest open.
- Constraint comments (
do not remove, talk to X before changing)
stay when they are about things we cannot change. Offer the cheapest
in-scope type, runtime, test, or CI lint. Wait for approval. If
approved, encode then delete. Otherwise delete, report the
constraint open, and sketch out-of-scope work.
- Report the deletion count, restored comments, reruns, architect
sketch, fixes, encoding offers, encodings, and open work.
No Comments procedure
Read no-comments procedure when running this workflow.
Apply the authorized scope and mode of this entry point to every step.
Resolve other skills through this distribution’s active catalog; resolve
resources relative to the installed skill directory.
1---2name: no-comments3description: Review comments on a diff, delete narration and workaround sermons, fix accepted findings, and offer encodings for claimed constraints. Use before review or when asked to strip comments.4license: MIT5---6
7# No comments
8
9Spawn a read-only comment reviewer. Act on accepted findings.
10
11Authoring agents defend comments. Defer to a fresh reviewer that did
12not write the code.
13
14Companion to `comment-mode` (granular feedback on prose drafts) and
15`deslop` (broader AI-artifact strip). This skill is the pre-review
16comment pass.
17
18## Authorized Scope
19
20Apply this engine only within the user's requested task and existing explicit
21authorization. Loading or delegating to it grants no additional authority.
22Preserve report-only restrictions and the caller's target, host, provider, and
23cost limits. Existing approval satisfies a gate only for the same actions and
24scope; obtain approval before expanding them. Forward these limits to delegates.
25
26## Contract
27
28Inputs:
29
30- The caller's files or diff. Otherwise the current diff against trunk,
31 including the working tree.
32
33Outputs:
34
35- Deletion count, restored comments, reruns, fixes, encoding offers,
36 and open work
37
38Creates/Modifies:
39
40- Deletes accepted comments and applies the smallest in-scope
41 root-cause fixes
42
43External Side Effects:
44
45- None beyond local edits
46
47Confirmation Required:
48
49- Before encoding a constraint comment into a type, test, or lint
50- Unattended runs require caller pre-approval for encodings
51
52Delegates To:
53
54- `how` or `why` when an `IMPORTANT` or `do not remove` comment is
55 thin
56- `architect` once, sketch only, when a fix needs a new shape
57
58## Steps
59
601. Spawn a read-only review subagent on the fast cheap tier. Pass the
61 scope and [references/comment-review.md](references/comment-review.md).
62 Do not restate the rules.
632. Inspect the report. Reject application-code edits, scope escapes,
64 exception-protected deletions, and flags that treat kept
65 intentional code as guilty. A keep survives only with proof it is
66 about something we cannot change. If a kill is ambiguous, do not
67 restore. If a keep is refuted or still ambiguous, delete it. Revert
68 and rerun one rejected report with the failure named. Reject a
69 second and fail this skill.
703. Fix trivial accepted flags directly. If any fix needs a shape, run
71 `architect` once for the accepted set and stop at the sketch.
724. Implement the smallest root-cause fix in scope. Remove every named
73 workaround. If the root cause is out of scope, land the smallest
74 in-scope fix and report the rest open.
755. Constraint comments (`do not remove`, `talk to X before changing`)
76 stay when they are about things we cannot change. Offer the cheapest
77 in-scope type, runtime, test, or CI lint. Wait for approval. If
78 approved, encode then delete. Otherwise delete, report the
79 constraint open, and sketch out-of-scope work.
806. Report the deletion count, restored comments, reruns, architect
81 sketch, fixes, encoding offers, encodings, and open work.
82
83## No Comments procedure
84
85Read [no-comments procedure](references/no-comments-procedure.md) when running this workflow.
86Apply the authorized scope and mode of this entry point to every step.
87Resolve other skills through this distribution’s active catalog; resolve
88resources relative to the installed skill directory.