Devin Review Fix
- 1 installs
- Updated July 2, 2026
- drdave-flexnetos/ai-top-utility
Fixes bugs flagged by Devin Review on PRs by reading review comments, triaging by severity, implementing surgical fixes, and verifying.
About
Reads Devin Review PR comments, triages Bug/Flag/Info findings, and implements minimal surgical fixes consistent with the repo. A developer uses it to address Devin Review findings on ai-top-utility PRs.
- Triages Devin findings by Bug/Flag/Info severity
- Surgical minimal-change fixes matching existing style
Devin Review Fix by the numbers
- 1 all-time installs (skills.sh)
- Ranked #982 of 1,352 Code Review & Quality skills by installs in the Skillselion catalog
- Data as of Jul 8, 2026 (Skillselion catalog sync)
npx skills add https://github.com/drdave-flexnetos/ai-top-utility --skill devin-review-fixAdd your badge
Show developers this skill is listed on Skillselion. Paste this into your README.
| Installs | 1 |
|---|---|
| Last updated | July 2, 2026 |
| Repository | drdave-flexnetos/ai-top-utility ↗ |
What it does
Fixes bugs flagged by Devin Review on PRs by reading review comments, triaging by severity, implementing surgical fixes, and verifying.
Files
Fix Devin Review bugs
Process for addressing Devin Review findings on ai-top-utility PRs.
Steps
1. Read the PR conversation Use git_view_pr to read all Devin Review comments. Look for severity markers:
Bug— must fixFlag— should investigate, may need fixInfo— informational, fix if low-cost
2. Triage by severity Create a todo list. Bugs first, then flags, then info items worth addressing.
3. Understand the context For each bug, read the cited file and surrounding code. Check:
- Does the bug match a known pattern from
.omc/skills/? - Is the fix consistent with how similar code works elsewhere in the repo?
- Does the fix touch a frozen file (
packaging/.deb-build/build.sh)?
4. Implement fixes surgically Per the Karpathy mandate: minimal changes, match existing style.
5. Verify
python3 -m py_compile <file>for Python changescd aitop && cargo clippy --workspace --release -- -D warningsfor Rust- Run the relevant test subset if possible
git diffto confirm only intended changes
6. Commit with clear messages Group by concern: fix(backend): use configurable train_dir in telemetry loop
Common Devin Review findings in this repo
| Pattern | Location | Fix |
|---|---|---|
| Hardcoded paths | backend/app.py | Use app.state.settings.train_dir or get_settings() |
| Regex injection | .omc/knowledge/build.sh | Use lambda replacement in re.sub() |
| stdin consumption | .claude/settings.json hooks | Ensure fallback chain (stdin -> env var -> exit 0) |
| Lazy import violation | backend/routers/*.py | Move heavy imports inside function bodies |
Related skills
Improve Codebase ArchitectureSafely deepen clusters of shallow modules into cohesive, testable units while respecting their external dependencies.566k196k
Caveman ReviewGet ultra-compressed, one-line code review comments that cut noise while keeping every actionable fix.270k94.7kCodebase DesignShared vocabulary for designing deep modules: improve a module's interface, find deepening opportunities, decide where a seam goes, make code more testable.266k196kCavecrewDelegate coding tasks to specialized subagents that return compressed output, keeping the main context window usable for much longer sessions.220k94.7kCode ReviewReviews a branch or PR diff on two axes at once: conformance to coding standards plus a code-smell baseline, and whether it actually implements the original spec.201k196k
Requesting Code ReviewDispatch a consistent, high-signal code reviewer subagent that catches plan deviations and quality issues before merging or continuing development.183k264k