diff --git a/DEV_LOOP_STATUS.txt b/DEV_LOOP_STATUS.txt index d33593c..e60840c 100644 --- a/DEV_LOOP_STATUS.txt +++ b/DEV_LOOP_STATUS.txt @@ -1,36 +1,25 @@ -============================================================================= -REVIEW-BOT DEV LOOP STATUS — 2026-05-15 04:08 UTC -============================================================================= +Last updated: 2026-05-15 (dev-loop run) +Coverage (origin/main): 54.1% cmd/review-bot -OVERALL STATUS: ✅ PR OPEN +## Open Issues +- #143: bug: doc-map config loaded from PR branch (untrusted) → IN PR #153 +- #150: fix: validateDocmapPath — add EvalSymlinks → IN PR #152 +- #154: refactor: extract shared base-args helper in main_test.go (LOW PRIORITY, deferred NIT) -Active Work: - - PR #140: test(#139): improve cmd/review-bot coverage 44.6% → 49.3% - State: open, labeled: ready, self-reviewed - Branch: issue-139 +## Closed This Run +- #144: bug: dev-loop merged PR autonomously → closed (fixed by #148 pure shell dispatch) +- #145: bug: merged despite REQUEST_CHANGES → closed (fixed by #148 pure shell dispatch) +- #146: missing subprocess tests → closed (fixed by PR #151 + comments) +- #147: coverage <50% → closed (54.1% on origin/main) -Test Results (last full run, worktree): - - All 6 packages: PASS ✅ - - Build: ✅ clean - - Vet: ✅ clean +## Open PRs (waiting for review/merge by Aaron) +- #151: test(#146): add InvalidDocMapPath/File tests (base: main) — labels: ai-review +- #152: fix(#150): EvalSymlinks dir-symlink bypass (base: main) — labels: needs-review +- #153: feat(#143): doc-map-trusted-ref (base: main, rebased on issue-146) — labels: needs-review -Coverage (post-change): - - cmd/review-bot: 49.3% (was 44.6%) - - review: 91.9% - - budget: 92.0% - - github: 86.3% - - gitea: 85.2% - - llm: 81.3% +## Merge Order +Recommended: #152 first (no deps), then #151, then #153 (rebased on issue-146, no conflict) -Repository (main): - - Branch: main (up to date with origin — 1e3d86b) - - Working tree: clean - - Open issues: 1 (#139, addressed by PR #140) - - Open PRs: 1 (#140, ready for review) - -System Health: ✅ GREEN - ✓ All tests passing - ✓ No warnings - ✓ PR ready for merge - -============================================================================= +## Notes +- PR #153 is rebased on issue-146 (which is the base for PR #151). Merge #151 before #153. +- PR #154 (refactor) is low priority — deferred NIT from PR #151 review.