Imported from shipshitdev/skills (
skills/no-comments/SKILL.md). Install upstream withnpx skills add shipshitdev/skills --skill no-comments. Copyright stays with the author (MIT).
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:
- None beyond local edits
Confirmation Required:
- Before encoding a constraint comment into a type, test, or lint
- Unattended runs require caller pre-approval for encodings
Delegates To:
howorwhywhen anIMPORTANTordo not removecomment is thinarchitectonce, 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
architectonce 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.