mirror of
https://github.com/cursor/plugins.git
synced 2026-09-14 20:00:00 +08:00
e8d856f027
* docs(pstack): port the September skill audit cuts from the internal tree Port of the 2026-09-01 to 2026-09-07 audit program on the internal skill tree. Deletion-only density cuts and the mannered prose pass applied wherever the OSS sentence is the same sentence. Two new principle leaves, Attack the Premise and Test Behavior Not Implementation, with their index lines. Critique mode removed from how and its callers. PR-body briefing rules in opening-a-pr and technical-writing. Model pins stay on the setup-pstack role mechanism. No internal path, slug, or name added. * docs(pstack): restore the autopilot chooser rule that multi-phase-plan cites multi-phase-plan step 4 picks between autopilot-full and autopilot-stack "per the rule at the end of playbooks/autopilot-stack.md". The density pass removed that paragraph and left the pointer dangling. Restore the paragraph verbatim. The chooser is a rule, not a restatement, and no other file states it. Addresses the Bugbot finding on PR #329. --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com>
2.5 KiB
2.5 KiB
name, description, disable-model-invocation
| name | description | disable-model-invocation |
|---|---|---|
| no-comments | Spawn Comment Sicko, fix accepted findings, and offer encodings for claimed constraints. | true |
No comments
Spawn Comment Sicko. Act on accepted findings.
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
- Spawn
Taskwithsubagent_type: "Comment Sicko". Pass the scope. Do not restate its rules. - Inspect its report and diff. Reject application-code edits, scope escapes, exception-protected deletions, misstated
MUST KILLreasons, 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 actionableMUST KILLs. Restore deletions only with exact exceptions and scoped proof. Before accepting thinIMPORTANTordo not removekills or keeps, run/howor/whyon 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. - 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
/architectonce 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. Neither authorizes widening the fence nor fixing instances outside it. Never bolt on symptom guards.
- Constraint comments say
do not remove,do not change wording, ortalk 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.