| name | optimize-code-reuse |
| description | Code duplication and helper extraction audit. Identifies reimplemented utilities, promotes shared helpers, extracts constants, and consolidates domain utilities. Use to reduce code duplication and improve maintainability. |
| allowed-tools | Bash, Read, Edit, Write, Glob, Grep, Agent, Skill |
| skills | ["dancing-bear-rules"] |
Optimize Code — Code Reuse Pass
Delegates to workflows/code/optimize-code.yaml with skip_checks=complexity,coverage,security,arch,
running only the reuse scan dimension. Findings are applied and rechecked automatically.
When to Use
- After implementing a feature to consolidate similar code
- When you notice the same function exists in multiple places
- Pre-PR deduplication check
- User says "reduce duplication", "extract helpers", or "consolidate code"
How to Run
Bootstrap source_root and test_path from context:
source_root: the <domain>/ directory of the files you just wrote or changedtest_path: the corresponding tests/<domain>/ directorypr_number: optional — if you have an open PR, pass it to scope file detection to that PR's diff
IMPORTANT: Use the /workflow skill — do NOT call ./bin/workflow run --execute directly. It only writes dispatch files and exits (status=pending). The /workflow skill is what actually spawns agents, waits for results, and handles human gates.
Skill(skill="workflow", args="--workflow workflows/code/optimize-code.yaml --params source_root=<domain>/ --params test_path=tests/<domain>/ --params skip_checks=complexity,coverage,security,arch")
With a PR number:
Skill(skill="workflow", args="--workflow workflows/code/optimize-code.yaml --params pr_number=314 --params source_root=<domain>/ --params test_path=tests/<domain>/ --params skip_checks=complexity,coverage,security,arch")
What Runs
Focuses on the reuse scan dimension — other dimensions can be added via skip_checks.
The workflow runs the full correction and recheck cycle:
- pre-check-auth — verify GitHub and qlty connections
- scan-changed-files — identify .py files changed vs main (or PR diff)
- mps-scan-reuse — shared utility reuse, bare string literals, domain duplicate detection
- mps-collate-findings — collate reuse findings into unified report
- apply-corrections — fix all critical + auto-fixable minor reuse findings
- recheck-lint —
bin/ruff-resolve.sh check + tests to confirm fixes
- human-gate — present reuse summary, ask to commit/open PR
Workflow Params
| Param | Default | Description |
|---|
source_root | "" | Path prefix for changed source files, e.g. workflow/ |
test_path | "" | Test directory to run, e.g. tests/workflow/ |
pr_number | "" | Optional PR number — scopes file detection to that PR's diff |
skip_checks | complexity,coverage,security,arch | Passed automatically — controls which scan dimensions run |
auth_domains | "github,qlty" | Comma-separated auth pre-flight services |
