← ClaudeAtlas

craft-fixlisted

The Craftsman standard for driving fixes against an existing `craft-audit` workspace — "fix the findings", "fix SEC-003", "fix the security findings", "work through the audit", "start the climb". This is an ACTION skill, not a domain: it picks findings off the master tracker's climb sequence, re-verifies each one is still real, gets the user's approval, and executes a batched fix with a regression test — it never re-audits or re-ranks. REQUIRES an existing `.craftsman/` workspace at the project root to fire. Without one: a scoped fix request naming no audit ("fix this bug", "harden this endpoint") goes to the relevant domain craft skill directly (e.g. `craft-security`, `craft-backend`); a whole-project assessment request ("is my app production-ready", "what should I fix before I ship") goes to `craft-audit` first, since there's nothing to fix from without an audit. Trigger even when the user names no finding ID — "start the climb" or "work through the audit" with no argument means the top 5 open items from th
atifgul99/craftsman-marketplace · ★ 1 · AI & Automation · score 65
Install: claude install-skill atifgul99/craftsman-marketplace
# Craftsman Fix — driving fixes off an audit `craft-audit` finds and ranks; this skill **fixes**. It reads the `.craftsman/` workspace another run already built, re-verifies the picks are still real, gets sign-off, and executes a tightly scoped batch — then hands back to a re-run of `craft-audit` to confirm. It never re-audits, never re-ranks, and never marks a finding `fixed` itself — only a fresh audit pass can do that, because "not seen ≠ fixed" is the whole point of the tracker. **Handoffs (state both explicitly when this skill doesn't fire):** - No `.craftsman/` at the project root → this skill does not apply. A scoped, no-audit fix request goes straight to the relevant domain craft skill. A whole-project assessment goes to `craft-audit` — run that first, then come back here. - `.craftsman/` exists but the user wants a *new* assessment, not a fix → that's `craft-audit` re-run territory, not this skill. ## The loop Depth for every step lives in `references/fix-protocol.md` — this is the compact sequence. 0. **Precheck.** Locate `.craftsman/` at the project root. - Missing → stop. Tell the user to run `craft-audit` first (give them the trigger phrase, e.g. "is my app production-ready"). - **Concurrency marker (mandatory before any mutation).** If `.craftsman/.run-in-progress` exists, an audit is claiming the workspace (or left a stale marker). **Refuse by default** — do not mutate `findings.md`, the master tracker, or any other `.craftsman/`