caspar-sweep

name	caspar-sweep
description	Light pre-commit cleanup pass — diff sanity, log/dead-code hygiene on changed files, strict lint, run related + broad tests, then conventional-commit. Use when wrapping up uncommitted or just-finished work before check-in ("sweep", "clean up and commit", "tidy this diff"), standalone or as the final phase of caspar-clean. Do NOT use for forensic dead-code removal across the codebase (use caspar-prune), for authoring new tests (use caspar-test), or as a full validation gate — this is a fast hygiene+commit pass, not deep review.
user-invocable	true

sweep

Prepare uncommitted or recently-changed work for check-in: a fast, formulaic hygiene pass, then descriptive conventional commits. No subagents, no approval gates — execute each step and move on.

Inputs

• $ARGUMENTS (optional scope/intent hint). Read the live working tree at runtime (git status, git diff) — do not assume a prior phase ran.

Working Set

Changed files only. Opportunistic, not forensic — do not hunt beyond the diff.

Outputs + DONE

One or more conventional commits covering the changes. DONE when:

• Diff reviewed: no unintended edits, no out-of-scope staged files, no secrets/keys/credentials/PII.
• Debug/temp logging removed; intentional logs (errors, key state transitions) kept at production-appropriate levels.
• Commented-out code, session-introduced TODO/FIXME/HACK, and hardcoded test values resolved or documented.
• Lint passes with zero violations fixed by suppression — refactor structural issues, never eslint-disable or equivalent. .gitignore covers temp/build/IDE artifacts.
• Related + broader test suite run and green; failures caused by these changes fixed.
• Work committed as type(scope): description conventional commits, split by concern.

Method / guardrails

1. Diff sanity — scan full diff for accidental edits, stray staged files, and secrets.
2. Log + code hygiene — strip debug logging, commented-out code, leftover TODO/FIXME from this work, hardcoded test values; keep intentional logs.
3. Opportunistic dead code (changed files only) — orphaned imports, unused vars/functions, debugger statements. No deep investigation.
4. Strict lint — fix all violations by correcting code; YOU MUST NOT suppress or --no-verify. Refactor for size/complexity thresholds.
5. Test — run tests related to changed files (co-located, importers, shared modules) plus the broader suite; fix change-caused failures. Do NOT author new tests — this is a sweep, not a test pass.
6. Commit — group by concern into conventional commits.
   • Types: feat · fix · refactor · test · chore · docs · style · perf.
   • Subject answers what changed and why; include scope (feat(auth): add token refresh on 401). Optional body for motivation/trade-offs.
   • One concern per commit — if it spans concerns, split it. Treat commits as durable context for future readers/LLMs; never fix: updates / refactor: clean up.

Handoff

Next Steps: one-line footer suggesting the natural follow-on (e.g. caspar-rebase to tidy history, or push/PR) grounded in the actual repo state.

Escalate-If

• A secret/credential appears in the diff — stop, surface it, do not commit until resolved.
• Lint or a test failure can only be cleared by suppression or by changing behavior beyond this diff — stop and ask rather than --no-verify.