agentleFS
rolandtolnayrolandtolnay/llm-toolkit/skills/boy-scout/SKILL.md

boy-scout

Skill1 starsChanged 3 months agoMITView on GitHub

Re-read the flow this session built or fixed for cheap, worthwhile fixes and behavior that would surprise a user or developer. Use before concluding a task, when something still feels worth fixing.

3 other projects use a version of this.

Adopted by

Most recently taken up by

The file

Files
SKILL.md

Take a second look at the work this session built or fixed and leave it a little better than you found it. The flow is that work followed end to end through the paths it actually runs on, or whatever the invocation names. Right now you know this code better than anyone will again for a long time: the paths, the decisions behind them, and the corners that iteration touched. A fix that costs minutes now costs an investigation later. The pass works by switching roles: read the flow as its first caller or user would, not as its author, and treat anything that would surprise them as a candidate.

Invoking this skill asks for the pass, not for findings. "Nothing worth changing" is a complete answer, and padding the report with cosmetic findings is the failure. Do the pass yourself rather than delegating it; a fresh reviewer lacks the context that makes it cheap. The report goes to the user, who picks what to apply, so write it so they can decide without reopening the code.

Gate

A finding earns a place only when both hold:

  • Cheap — contained to the flow, no new abstraction, no redesign.
  • Worth it — a real user or developer is likely to hit it, and it costs them when they do. A rare glitch that self-heals fails the gate; a common path that blocks, loses work, or misleads passes it.

Decisions settled earlier in the session stay settled unless new evidence shows they are wrong. Problems outside the flow get one line at most, pointing at the skill that handles them: ripple-check when the same pattern repeats elsewhere, improve-codebase-architecture when the fix is structural.

Pass

The first pass tends to miss:

  • Unhappy paths — what happens when the flow is cancelled, fails partway, or is retried.
  • Silent failure — a fallback or an unknown that gets reported as success.
  • Drift — two places encoding the same knowledge differently, including docs versus behavior.
  • Leftovers from iterating — dead paths, vestigial parameters, names that describe an earlier version.

Reproduce a suspected bug when a quick script or test can; otherwise mark it [inference]. Stop when you have read the flow end to end as its user would and can either name the findings or say why none earn a change.

Report

Lead with the verdict: how many findings are worth doing now, or that none are. Then number each finding with where it lives, what happens and whom it surprises, the evidence (reproduced output or [inference]), and the cheap fix with its regression risk. Close with one line on what you examined and deliberately left alone, and why, then your recommendation.

Make no changes until the user picks, unless the invocation asked you to apply them; then implement the picked items.

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.