agentleFS
theonly1metheonly1me/shadowclone/.claude/skills/scope-confirmed-changes/SKILL.md

scope-confirmed-changes

Skill25 starsChanged 32 days agoMITView on GitHub

Use when fixing a bug, a regression, or a confirmed review finding. Also use it when the user says "fix this bug", "it is broken", or "keep the change small". Reproduces the issue before any fix, with a test, a real command, or the real interface. After the fix, the same reproduction must pass. Keeps the change to confirmed, reachable behavior. Not for checking review comments before a fix (use `verify-review-findings`).

The file

Show its rules and text

We split this skill into the rules it gives an agent

19 rules. About 0 are common, 19 are distinctive, and 0 are very specific to this project.

  • ≈0

    Common · estimate

    Many files like this have a rule of this kind.

  • ≈19

    Distinctive · estimate

    This author's own choices. Worth a look.

  • 0

    Very specific

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

  • Common
  • Distinctive
  • Very specific
  1. Distinctive:

    After the fix, does the same reproduction show the expected result?

    line 19
  2. Distinctive:

    If you could not reproduce the issue, did you label the cause as inferred, with the data and code that support it?

    line 20
  3. Distinctive:

    Do all checks that passed before your change still pass?

    line 22
  4. Distinctive:

    If an approach caused a new regression, did you remove it instead of adding a second fix on top?

    line 23
  5. Distinctive:

    Write the issue as a concrete input, the wrong result, and the expected result.

    line 29
  6. Distinctive:

    Keep the reproduction and its wrong output, because you run it again after the fix.

    line 34
  7. Distinctive:

    Run the same reproduction.

    line 37
  8. Distinctive:

    Run the formatter and the linter on each touched file.

    line 38
  9. Distinctive:

    Read the diff. Remove each line that does not serve the fix or its proof.

    line 39
  10. Distinctive:

    If reproduction is impossible, say why.

    line 40
  11. Distinctive:

    Easy route: Read the code, guess that the copy check is wrong, and change it.

    line 45
  12. Distinctive:

    Evidence: Before the fix, the reproduction printed "Updated design-deep-modules to the bundled version (3 copies)." and the edit was gone.

    line 48
  13. Distinctive:

    Keep the unrelated edits of the user in the worktree.

    line 52
  14. Distinctive:

    Do not widen a public contract to fit an internal shortcut.

    line 53
  15. Distinctive:

    Do not add a guard for a case that cannot happen.

    line 55
  16. Distinctive:

    Put each later review round in its own commit.

    line 56
  17. Distinctive:

    If you chose to escape a check that flags false positives, do not add the same check somewhere else as a safety net.

    line 58
  18. Distinctive:

    If a change proves marginal or causes a regression, revert it.

    line 59
  19. Distinctive:

    Ask before a side change that the request does not cover.

    line 60

≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “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

Scope Confirmed Changes

Use when

You fix a bug, a regression, or a review finding that you confirmed. Only a reproduction proves that a bug is real. Only the same reproduction after the fix proves that the fix works.

Gates

  1. Did you reproduce the issue before you changed code, with a test, a real command, or the real interface? If the fix already exists, a test that fails when you undo the smallest part of the fix counts as the reproduction.
  2. After the fix, does the same reproduction show the expected result?
  3. If you could not reproduce the issue, did you label the cause as inferred, with the data and code that support it?
  4. Does each changed line serve the fix, or a test or check that the fix needs?
  5. Do all checks that passed before your change still pass?
  6. If an approach caused a new regression, did you remove it instead of adding a second fix on top?

Process

Write in the user's voice. Before you write, read ~/.agents/voice.md. If it does not exist, build it once from the user's merged pull requests, review comments, and commit messages. Use only text that the user wrote, and name those sources in your handoff. Never overwrite an existing or linked voice.md.

  1. Write the issue as a concrete input, the wrong result, and the expected result.
  2. Reproduce it in the cheapest way that shows the real symptom:
    • a failing unit or integration test,
    • the real command in a temporary home, or
    • the real interface, such as browser or desktop automation when the host has it.
  3. Keep the reproduction and its wrong output, because you run it again after the fix.
  4. For a performance problem, measure the cost of the suspect and compare it with the size of the symptom before you build a fix.
  5. Find the cause in the code that the reproduction runs. Fix it at the boundary that owns it.
  6. Run the same reproduction. It must now show the expected result.
  7. Run the formatter and the linter on each touched file. Then run the focused tests and the required check of the repository.
  8. Read the diff. Remove each line that does not serve the fix or its proof.
  9. If reproduction is impossible, say why. Reason from logs, data, and code, and label the conclusion as inferred.

Example

Situation: A user reports that sync lost an edit to an installed skill. All unit tests pass. Easy route: Read the code, guess that the copy check is wrong, and change it. Hidden cost: A guess can be wrong, and nothing proves the fix. The edit can still disappear in the case that the user hit. Best route: Reproduce the report. Install the skill, edit one copy, apply the build again, and run sync. The edit disappears. Fix the cause, and run the same steps again. Evidence: Before the fix, the reproduction printed "Updated design-deep-modules to the bundled version (3 copies)." and the edit was gone. After the fix, it printed "Kept your edited copy of design-deep-modules at ..." and the edit stayed.

Guardrails

  • Keep the unrelated edits of the user in the worktree.
  • Do not widen a public contract to fit an internal shortcut.
  • Refactor only when the fix or its proof needs it.
  • Do not add a guard for a case that cannot happen. Fix the cause of a case that can.
  • Put each later review round in its own commit.
  • List each defensive, style, or unrelated change as an optional follow-up. Never bundle it with the fix.
  • If you chose to escape a check that flags false positives, do not add the same check somewhere else as a safety net.
  • If a change proves marginal or causes a regression, revert it. Do not defend it.
  • Ask before a side change that the request does not cover. Do not replace the choice of the user with your own.
  • Say once what is wrong with a direction. After the user chooses, build that choice.
  • In plan mode, write the plan and wait for approval before you edit code, even for a small fix.

Completion

  • The reproduction command or test, with the wrong output before the fix.
  • The same reproduction after the fix, with the expected output.
  • The focused test command and the required check, with their result lines.
  • Each changed file, with the reason that it is in scope.
  • Each check that you did not run, with the reason. Never call a check green if you did not run it.
  • For an issue that you could not reproduce: the reason, and the evidence for the inferred cause.

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.