agentleFS
Curious-KeeperCurious-Keeper/skills/no-comments/SKILL.md

no-comments

Skill1 starsChanged 57 days agoMITView on GitHub

Delete the comments in scope, fix the code they were covering for, and offer to encode any constraint they claimed.

68 other projects use a version of this, 50 of them edited.

Adopted by

Most recently taken up by

Used with

Projects with it also use poteto-help, setup-pstack and 10 more.

Other files people use alongside this one.

Files found in the same public projects as a version of this one, more often than they turn up anywhere else. Each project counts once, large collections count for less, and this file's own repository is left out.
  • ianclearySkill

    poteto-help

    Guides users through pstack setup, /poteto-mode, and picking the skill, playbook, or principle for a task. Type /poteto-help with a question.

    Also in 13 projects that use this

  • bnemaSkill

    setup-pstack

    Configure which Pi models pstack uses per role. Detects available provider/model specs and writes pstack's user configuration. Use for /setup-pstack, "configure pstack models", or changing pstack model choices.

    Also in 22 projects that use this

  • ianclearySkill

    principle-explain-the-number

    Apply before you trust, report, or act on a number you measured: a speedup, a regression, a throughput, a latency, or an eval result. Find what limits it, and rule out that it measured something other than the work you think.

    Also in 25 projects that use this

  • eryzerzSkill

    make-bot-ui

    Use when building a custom UI (page, dashboard, buttons) that should wake a Grok Bot over a webhook, when the user must provide a webhook sender key, or when exposing that UI on Tailscale.

    Also in 16 projects that use this

  • TheOnlyFusionCubeSkill

    maintain-formal-verification-skill

    Bring a project's formal verification back in line with the code after the code changes: find the commits that touched verified code, update the model, harness, or properties to describe the code as it is now, rerun, and say whether each failure is drift in the verification or a bug in the code. Use for /maintain-formal-verification-skill, \"is the model still true to the code\", or after merges and refactors that touched verified code without rechecking it.

    Also in 19 projects that use this

  • TheOnlyFusionCubeSkill

    create-formal-verification-skill

    Generate a project-local skill that formally verifies the code where a bug costs the most and tests reach the least, in any language, with whichever verification tool fits. Use for /create-formal-verification-skill, \"model-check this race\", \"prove this invariant\", or when concurrency, retries, state machines, parsers, or unsafe code need more than tests.

    Also in 21 projects that use this

  • +1 more from TheOnlyFusionCube

The file

We split this skill into the rules it gives an agent

4 rules. 1 is common, 2 are distinctive, and 1 is very specific to this project.

  • 1

    Common

    Most of the 66 edited versions we read keep these.

  • 2

    Distinctive

    This author's own choices. Worth a look.

  • 1

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  1. Common:

    Use the caller's files or diff.

    line 15
  2. Very specific:

    Before accepting a thin IMPORTANT or do not remove kill or keep, investigate the named symbol yourself: git log and git blame on the lines, …

    line 35
  3. Distinctive:

    Remove every named workaround.

    line 45
  4. Distinctive:

    If approved, encode the constraint and then delete the comment.

    line 53

“Common” is measured: kept by at least 6 in 10 of the 66 edited versions of this file we could read. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

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

1. Spawn the sub-agent

Spawn one sub-agent with the brief in COMMENT-SICKO.md, pasted whole. Pass the scope. Do not restate its rules and do not add your own.

2. Audit the report and diff

Reject any of these:

  • edits to application code
  • deletions outside the scope
  • deletions of exception-protected comments
  • misstated MUST KILL reasons
  • flags that treat kept intentional code as guilty

Reshape flags on our-code surprises stay actionable, and their comments stay deleted. A keep survives only with proof it is about something we cannot change. Audit the scoped linter and type-checker suppressions it missed. Suppressions that protect correctness or safety stay actionable MUST KILLs. Restore a deletion only against an exact exception with scoped proof.

Before accepting a thin IMPORTANT or do not remove kill or keep, investigate the named symbol yourself: git log and git blame on the lines, the ADRs and CONTEXT.md, and any APP_MAP.json decisionsSettled or invariants. An ambiguous kill stands. A keep that is refuted or still ambiguous gets deleted.

Revert and rerun one rejected report, naming the failure. Reject a second and stop. Report it open and fail the run.

3. Sketch a shape where one is needed

Fix trivial accepted flags directly by deleting a dead path, dropping a parameter, or using the real API. If any fix needs a shape, call the Skill tool for codebase-design once for the accepted set and the surrounding code. Stop at the sketch. That skill shapes, step 4 implements.

4. Implement the smallest root-cause fix in scope

Remove every named workaround. Fix causes, never symptoms, and never bolt on a symptom guard. If the root cause is out of scope, land the smallest in-scope fix and report the rest open.

Where a fix has to land in a design written without it, call the Skill tool for redesign-from-first-principles. It guides intent only. It does not authorize widening the fence or fixing instances outside it.

5. Offer to encode the constraints

Constraint comments say do not remove, do not change wording, or talk to X before changing. Leave keeps about things we cannot change. For the rest, offer the cheapest in-scope type, runtime check, test, or CI lint that makes the constraint enforceable, and wait for the user to approve it. A non-interactive run needs that approval from the caller up front.

If approved, encode the constraint and then delete the comment. Otherwise delete the comment, report the constraint open, and sketch the out-of-scope work.

6. Report

Report the deletion count, restored comments, reruns, the design sketch, fixes, encoding offers, encodings made, unenforced constraints, and other open work.

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.