Applies fixes from a prior review-llm-artifacts run, with safe/risky classification
Applies fixes from a prior review-llm-artifacts run, with safe/risky classification
/plugin marketplace add existential-birds/beagle/plugin install beagle@existential-birdsApply fixes from a previous review-llm-artifacts run with automatic safe/risky classification.
/beagle:fix-llm-artifacts [--dry-run] [--all] [--category <name>]
Flags:
--dry-run - Show what would be fixed without changing files--all - Fix entire codebase (runs review with --all first)--category <name> - Only fix specific category: tests|dead-code|abstraction|styleExtract flags from $ARGUMENTS:
--dry-run - Preview mode only--all - Full codebase scan--category <name> - Filter to specific category# Check for uncommitted changes
git status --porcelain
If working directory is dirty, warn:
Warning: You have uncommitted changes. Creating a git stash before proceeding.
Run `git stash pop` to restore if needed.
Create stash if dirty:
git stash push -m "beagle: pre-fix-llm-artifacts backup"
Check for existing review file:
cat .beagle/llm-artifacts-review.json 2>/dev/null
If file missing:
--all flag: Run review-llm-artifacts --all --json first/beagle:review-llm-artifacts first."If file exists, validate freshness:
# Get stored git HEAD from JSON
stored_head=$(jq -r '.git_head' .beagle/llm-artifacts-review.json)
current_head=$(git rev-parse HEAD)
if [ "$stored_head" != "$current_head" ]; then
echo "Warning: Review was run at commit $stored_head, but HEAD is now $current_head"
fi
If stale, prompt: "Review results are stale. Re-run review? (y/n)"
Parse findings from JSON and classify by fix_safety field:
Safe Fixes (auto-apply):
unused_import - Unused importstodo_comment - Stale TODO/FIXME commentsdead_code_obvious - Obviously unreachable codeverbose_comment - Overly verbose LLM-style commentsredundant_type - Redundant type annotationsRisky Fixes (require confirmation):
test_refactor - Test structure changesabstraction_change - Class/function extractioncode_removal - Removing functional codemock_boundary - Test mock scope changeslogic_change - Any behavioral modificationsIf --dry-run:
## Safe Fixes (would apply automatically)
| File | Line | Type | Description |
|------|------|------|-------------|
| src/api.py | 15 | unused_import | Remove `from typing import List` |
| src/models.py | 42 | verbose_comment | Remove 23-line docstring |
...
Otherwise, spawn parallel agents per category with Task tool:
Task: Apply safe fixes for category "{category}"
Files: [list of files with findings in this category]
Instructions: Apply each fix, preserving surrounding code. Report success/failure per fix.
Categories to parallelize:
style - Comments, formattingdead-code - Imports, unreachable codetests - Test-related safe fixesabstraction - Safe refactorsFor each risky fix, prompt interactively:
[src/services/auth.py:156] Remove seemingly unused authenticate_legacy() method?
This method has no callers in the codebase but may be used externally.
(y)es / (n)o / (s)kip all risky:
Track user choices:
y - Apply this fixn - Skip this fixs - Skip all remaining risky fixesDetect project type and run appropriate linters:
Python:
# Check if ruff config exists
if [ -f "pyproject.toml" ] || [ -f "ruff.toml" ]; then
ruff check --fix .
ruff format .
fi
# Check if mypy config exists
if [ -f "pyproject.toml" ] || [ -f "mypy.ini" ]; then
mypy .
fi
TypeScript/JavaScript:
# Check for eslint
if [ -f "eslint.config.js" ] || [ -f ".eslintrc.json" ]; then
npx eslint --fix .
fi
# Check for TypeScript
if [ -f "tsconfig.json" ]; then
npx tsc --noEmit
fi
Go:
if [ -f "go.mod" ]; then
go vet ./...
go build ./...
fi
# Python
if [ -f "pyproject.toml" ] || [ -f "pytest.ini" ]; then
pytest
fi
# JavaScript/TypeScript
if [ -f "package.json" ]; then
npm test 2>/dev/null || yarn test 2>/dev/null || true
fi
# Go
if [ -f "go.mod" ]; then
go test ./...
fi
## Fix Summary
### Applied Fixes
- [x] src/api.py:15 - Removed unused import `List`
- [x] src/models.py:42-64 - Removed verbose docstring
- [x] src/auth.py:156-189 - Removed dead method (user confirmed)
### Skipped Fixes
- [ ] src/services/cache.py:23 - User declined risky fix
- [ ] tests/test_api.py:45 - Test refactor skipped
### Verification Results
- Linter: PASSED
- Type check: PASSED
- Tests: PASSED (42 passed, 0 failed)
### Diff Summary
```bash
git diff --stat
On successful completion (all verifications pass):
rm .beagle/llm-artifacts-review.json
If any verification fails, keep the file and report:
Review file preserved at .beagle/llm-artifacts-review.json
Fix issues and re-run, or restore with: git stash pop
# Preview all fixes without applying
/beagle:fix-llm-artifacts --dry-run
# Fix only dead code issues
/beagle:fix-llm-artifacts --category dead-code
# Full codebase scan and fix
/beagle:fix-llm-artifacts --all
# Fix style issues only, preview first
/beagle:fix-llm-artifacts --category style --dry-run