hu3rror/my-skills

no-comments

Audit the comments in a scope with the comment-sicko role, judge its findings, fix root causes, and encode claimed constraints.

Quelltext ansehen

Verwendung

  1. Installationsbefehl kopieren und das Quellpaket vor der Installation in der eigenen Agentenumgebung prüfen.
  2. Benötigte Werkzeuge, Zugangsdaten und Hosts im Original prüfen.
  3. Eingabe und gewünschtes Ergebnis beschreiben und die Ausgabe mit den Quellen abgleichen.
npx skills add https://github.com/hu3rror/my-skills --skill no-comments

Vor dem Start

  • Die Installation umfasst weder Modellabo noch API-Kontingent oder Dateizugriffsrechte.
  • Sterne und Installationen zeigen Aktivität, keine unabhängige Qualitäts- oder Sicherheitsprüfung.
Originales Skill-Dokument

Aus dem Quell-Repository gerendert; Überschriften, Beispiele, Code, Tabellen, Links und Bilder bleiben erhalten.

No comments

Spawn comment-sicko for an independent pass, then judge its report and fix. Reviewer and fixer stay separate so the audit has a second pair of eyes.

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. Run subagent once with comment-sicko and the scope in task, nothing else. The call is blocking: the full report arrives in one result, so no polling, and its rules are not restated.
  2. Judge the report against the 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 surprises in our own code stay actionable; those comments are not restored.
  • A keep survives only with proof it is about something we cannot change. An ambiguous kill is not restored; a refuted or still-ambiguous keep is deleted.
  • Audit scoped lint and TypeScript suppressions the report missed. Correctness or safety suppressions stay actionable MUST KILLs.
  • A restored deletion needs an exact exception and scoped proof.
  • For a thin IMPORTANT or do not remove verdict, run fact-finder on its symbol (or read the symbol and its call sites yourself) before judging.
  • Rerun one rejected report with the failure named. Reject a second, report it open, and fail this run.
  1. Fix trivial accepted flags directly: delete a dead path, drop a parameter, use the real API. When a fix needs a shape, consult codebase-design once for the accepted set and surrounding code, and stop at the sketch — it shapes, step 4 implements.
  2. Implement the smallest root-cause fix in scope and remove every named workaround. If the root cause is out of scope, land the smallest in-scope fix and report the rest open. The fence stays where it is.
  3. Constraint comments (do not remove, do not change wording, talk to X before changing) survive when they are about things we cannot change. Offer the cheapest encoding — type, runtime check, test, CI lint — and wait for approval: approved, encode then delete; otherwise delete, report the constraint open, and sketch the out-of-scope work.
  4. Report the deletion count, restored comments, reruns, design sketch, fixes, encoding offers, encodings, unenforced constraints, and other open work.

Prerequisite: pi-matt-subagent (the subagent tool plus the bundled comment-sicko and fact-finder roles).

Verwandte Aufgaben

aus demselben Repository

Weitere Skills

Alle Skills