mcpField
N
no-comments

no-comments

github.com/ericlitmanTesting & quality53847 installsUpdated 1w ago
Repository

About no-comments

A skill for reviewing code comments: it asks the comment-sicko subagent to inspect specified files or the current diff, then filters findings, applies accepted fixes, and reports unresolved work.

Key features

  1. 01Inspect specified files or the current diff
  2. 02Delegate review to the comment-sicko subagent
  3. 03Audit reports and reject out-of-scope findings
  4. 04Fix accepted issues and report outcomes

Use cases

  1. 01Review comments in the current branch diff
  2. 02Handle comment-related findings under project rules
  3. 03Resolve tool names in Codex and other non-Claude runtimes

Install

Copy the command and run it in a terminal.

$npx skills add https://github.com/ericlitman/open-pstack --skill no-comments

After installing, type/no-commentsin Claude Code to confirm it loaded.

Details

From the project README

No comments

Spawn comment-sicko. Act on accepted findings.

Defer to comment-sicko's fresh perspective.

Platform note. On Codex or another non-Claude runtime, the comment-sicko subagent and the Claude tool names below are Claude defaults. Resolve them via codex-tools.md.

Scope

Use the caller's files or diff. Otherwise use the current diff against the base branch, default main, including the working tree.

Steps

  1. Spawn an Agent with subagent_type: "comment-sicko". Pass the scope. Pick each subagent's effort for the job, per the Subagents section of the poteto-mode skill. Do not restate its rules.
  2. Inspect its report and diff. Reject application-code edits, scope escapes, exception-protected deletions, misstated MUST KILL reasons, and flags that treat kept intentional code as guilty. Reshape flags on our-code surprises stay actionable. Do not restore those comments. A keep survives only with proof it is about something we cannot change. Audit missed scoped lint and TypeScript suppressions. Correctness or safety suppressions stay actionable MUST KILLs. Restore deletions only with exact exceptions and scoped proof. Before accepting thin IMPORTANT or do not remove kills or keeps, run /how or /why on their symbol. 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, report it open, and fail /no-comments.
  3. Fix trivial accepted flags directly by deleting a dead path, dropping a parameter, or using the real API. If any fix needs a shape, run /architect once for the accepted set and surrounding code. Stop at the sketch. Architect shapes. Step 4 implements.
  4. 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. The principle-fix-root-causes and principle-redesign-from-first-principles skills guide intent only. Neither authorizes widening the fence nor fixing instances outside it. Never bolt on symptom guards.
  5. Constraint comments say do not remove, do not change wording, or talk to X before changing. Leave keeps about things we cannot change. Offer the cheapest in-scope type, runtime, test, or CI lint. Wait for interactive approval. Unattended and eval require caller pre-approval. If approved, encode then delete. Otherwise delete, report the constraint open, and sketch out-of-scope work.
  6. Report the deletion count, restored comments, reruns, architect sketch, fixes, encoding offers, encodings, unenforced constraints, and other open work.

Comments

Markdown supported