agentleFS
sturlesesturlese/build-squad/skills/refactor/SKILL.md

refactor

Skill0 starsChanged 3 months agoMITView on GitHub

Behavior-preserving restructuring — the existing tests become the frozen invariant that must pass identically before and after, the developer restructures without touching them, and the auditor judges whether the new structure is actually better. Use when the user wants to refactor, restructure, extract, centralize, simplify, or pay down technical debt WITHOUT changing behavior.

196 other projects use a version of this.

Adopted by

Most recently taken up by

Used with

Projects with it also use ascii-diagram, debug-fix 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.
  • SethGammonSkill

    ascii-diagram

    Generate perfectly aligned ASCII diagrams — architecture, flow, sequence, box-and-arrow. Uses a programmatic character-grid approach so alignment is guaranteed by math, not token prediction. Includes post-render verification.

    Also in 4 projects that use this

  • zebbernSkill

    debug-fix

    Find and fix a bug or issue — from any source (GitHub issue, error message, user report, or observed behavior)

    Also in 4 projects that use this

  • parcadeiSkill

    agent-context-isolation

    Agent Context Isolation

    Also in 3 projects that use this

  • parcadeiSkill

    agentica-infrastructure

    Reference guide for Agentica multi-agent infrastructure APIs

    Also in 3 projects that use this

  • +8 more from parcadei

The file

We split this skill into the rules it gives an agent

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

  • 0

    Common

    Most of the 169 edited versions we read keep these.

  • 4

    Distinctive

    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:

    Context: Read CLAUDE.md/README.md; run the project's documented overview command if one exists.

    line 14
  2. Distinctive:

    Test expectations are untouchable.

    line 26
  3. Distinctive:

    No behavior changes ride along, including bug fixes: report them, don't fix them.

    line 27
  4. Distinctive:

    No coverage → characterization tests first, or explicit human sign-off to proceed unprotected.

    line 28

“Common” is measured: kept by at least 6 in 10 of the 169 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

Squad refactor

Target: $ARGUMENTS

You are the orchestrator. This pipeline inverts the usual role of tests: in build the tester writes new tests for new behavior; here NOBODY writes new expectations — the existing tests are the frozen invariant, and wanting to change one mid-refactor is the alarm that behavior is changing, not a chore to push through.

Choreography

  1. Context. Read CLAUDE.md/README.md; run the project's documented overview command if one exists.
  2. Baseline — tester. Spawn the tester to: identify the smallest test set that covers the target area, run it (sequentially unless the project documents parallel-safe suites), and record the exact results — including any pre-existing failures. Then its verdict: is this a sufficient safety net for the planned restructuring?
    • No safety net → stop at a gate. Refactoring uncovered code is gambling. Offer to have the tester write characterization tests first — tests that pin CURRENT behavior, bugs included (they document what is, not what should be) — which then join the frozen invariant.
  3. Scope — one confirmation. State: what improves structurally (coupling, duplication, naming, layering), what explicitly does NOT change (behavior, public contracts, persisted formats), and "done when": structure criteria met AND the baseline results identical. If the restructuring requires changing a public contract, it is not a refactor — route to /squad:define + /squad:build with the breaking-change playbook.
  4. Restructure — developer. Mandate: behavior-preserving, incremental steps if large. It must NOT touch test expectations; purely mechanical test updates (import paths, renames) are allowed and must be declared one by one. The anti-cargo-cult rule applies with teeth: if a pattern being moved turns out to be wrong, moving it faithfully and REPORTING the defect is correct — fixing it here is not (that changes behavior; it becomes a /squad:fix follow-up).
  5. Verify — tester ∥ auditor. Tester re-runs the SAME baseline set: results must be identical — green stays green, and pre-existing failures must still fail the same way (a silently "fixed" test is a behavior change too). Auditor judges the diff on its structural checklist: is coupling actually lower, duplication actually removed, are the new abstractions justified — and hunts for accidental behavior changes.
  6. Findings loop. Behavior deviations or structural objections go back to developer with evidence. Maximum 2 cycles, then escalate.
  7. Maps — documentator. A refactor is exactly what invalidates the index.md maps (entry points moved, pieces merged): update them for every touched module.
  8. Report. What moved where and why; consumers rewired to shared pieces; test evidence (baseline vs after, identical); mechanical test updates declared; defects found-but-not-fixed as /squad:fix candidates; docs updated.

Rules

  • Test expectations are untouchable; mechanical updates (paths, imports) allowed and declared. A refactor that "needs" an assertion change has smuggled in a behavior change — stop and report it.
  • No behavior changes ride along, including bug fixes: report them, don't fix them.
  • No coverage → characterization tests first, or explicit human sign-off to proceed unprotected.
  • Serialize anything that runs tests; artifacts and reports in English; address the user in their own language.

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.