No comments
Follow the portable runtime contract for reviewer-role selection and child briefs.
Spawn Comment Sicko. Act on accepted findings.
Authoring agents defend comments. Defer to Comment Sicko's fresh perspective.
Scope
Use the caller's files or diff. Otherwise use the current diff against the base branch, default main, including the working tree.
Steps
- Use the host's specialized Comment Sicko reviewer when available. Otherwise launch a fresh canonical
reviewer child for the no-comments objective. Pass the scope and a standalone brief; do not use host-specific task fields or silently claim the specialized reviewer ran.
- 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, invoke the sibling how or why skill 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 this skill.
- Fix trivial accepted flags directly by deleting a dead path, dropping a parameter, or using the real API. If any fix needs a shape, invoke the sibling architect skill once for the accepted set and surrounding code. Stop at the sketch. Architect shapes. Step 4 implements.
- 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: fix real causes, redesign as if requirements always existed, never bolt on symptom guards. Neither authorizes widening the fence nor fixing instances outside it.
- 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.
- Report the deletion count, restored comments, reruns, architect sketch, fixes, encoding offers, encodings, unenforced constraints, and other open work.
1---2name: no-comments3description: Spawn Comment Sicko, fix accepted findings, and offer encodings for claimed constraints.4---56# No comments78Follow the [portable runtime contract](../pstack-pi/references/runtime.md) for reviewer-role selection and child briefs.910Spawn Comment Sicko. Act on accepted findings.1112Authoring agents defend comments. Defer to Comment Sicko's fresh perspective.1314## Scope1516Use the caller's files or diff. Otherwise use the current diff against the base branch, default `main`, including the working tree.1718## Steps19201. Use the host's specialized Comment Sicko reviewer when available. Otherwise launch a fresh canonical `reviewer` child for the no-comments objective. Pass the scope and a standalone brief; do not use host-specific task fields or silently claim the specialized reviewer ran.212. 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 KILL`s. Restore deletions only with exact exceptions and scoped proof. Before accepting thin `IMPORTANT` or `do not remove` kills or keeps, invoke the sibling **how** or **why** skill 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 this skill.223. Fix trivial accepted flags directly by deleting a dead path, dropping a parameter, or using the real API. If any fix needs a shape, invoke the sibling **architect** skill once for the accepted set and surrounding code. Stop at the sketch. Architect shapes. Step 4 implements.234. 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: fix real causes, redesign as if requirements always existed, never bolt on symptom guards. Neither authorizes widening the fence nor fixing instances outside it.245. 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.256. Report the deletion count, restored comments, reruns, architect sketch, fixes, encoding offers, encodings, unenforced constraints, and other open work.
shrimpwtf