From 24247a855021ccccdc8ec56e6ff4475cd7cfff25 Mon Sep 17 00:00:00 2001 From: Rodin Date: Fri, 15 May 2026 07:03:40 +0000 Subject: [PATCH] =?UTF-8?q?chore(#141):=20update=20dev-loop=20status=20?= =?UTF-8?q?=E2=80=94=20ready=20for=20PR=20submission?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- TODO.md | 65 +++++++++++++++++++++++++++++++-------------------------- 1 file changed, 35 insertions(+), 30 deletions(-) diff --git a/TODO.md b/TODO.md index 9e8f5b3..977fc7d 100644 --- a/TODO.md +++ b/TODO.md @@ -1,42 +1,47 @@ -## Dev Loop Status: 2026-05-15 04:35 UTC +## Dev Loop Status: 2026-05-15 07:02 UTC -**Repository:** review-bot (rodin/review-bot on Gitea) -**Status:** ✅ OPTIMAL +**Issue:** #141 — validate-docmap subcommand +**Branch:** issue-141 +**Status:** ✅ READY FOR PR SUBMISSION -### Health Check +### Implementation Complete -- **Working tree:** clean -- **Branch:** main (up to date with origin) -- **Build:** ✅ passes (`go build ./cmd/review-bot`) -- **Tests:** ✅ ALL PASS (6/6 packages) -- **Vet:** ✅ clean -- **Open issues:** 0 -- **Open PRs:** 0 +- ✅ `cmd/review-bot/validatedocmap.go` — main subcommand implementation (139 lines) +- ✅ `cmd/review-bot/validatedocmap_test.go` — 9 test cases (all passing) +- ✅ `review/docmap.go` — exported `FileCoveredByDocMap` helper +- ✅ `cmd/review-bot/main.go` — integrated `validate-docmap` subcommand +- ✅ `README.md` — documented subcommand with examples and flags -### Recent Changes +### Test Results -- ✅ Merged PR #140: test coverage improvements (44.6% → 49.3%) -- ✅ PR #137 (doc-map feature) merged successfully with all reviews approved +``` +TestRunValidateDocmap_Clean ✅ +TestRunValidateDocmap_MissingDocmapFlag ✅ +TestRunValidateDocmap_BadYAML ✅ +TestRunValidateDocmap_StaleDocs ✅ +TestRunValidateDocmap_UncoveredFile ✅ +TestRunValidateDocmap_BothFailures ✅ +TestRunValidateDocmap_EmptyStdin ✅ +TestRunValidateDocmap_BlankLinesSkipped ✅ +TestRunValidateDocmap_DuplicateDocsDeduped ✅ +``` -### Coverage +### Pre-Push Checks -| Package | Coverage | -|---------|----------| -| cmd/review-bot | 49.3% ↑ | -| gitea | 85.2% | -| github | 86.3% | -| review | 92.0% | +- ✅ All packages pass: `go test ./...` (6/6) +- ✅ `go vet ./...` — clean +- ✅ `go build ./cmd/review-bot` — clean +- ✅ `scripts/check-deps.sh` — pass -### Status Summary +### Commits -All recent feature work (doc-map, coverage improvements) successfully integrated. Repository is stable and ready for next development cycle. +1. feat(#141): add validate-docmap subcommand +2. docs(#141): add validate-docmap subcommand to README +3. fix(#141): use stdinValidateDocmap in Clean test — avoid real os.Stdin dependency -### Next Priority +### Next Steps -- Continue increasing cmd/review-bot coverage (target: ≥60%) -- Monitor prod logs for edge cases -- VCS integration stable; GitHub + Gitea paths clear +- [ ] Create PR via gitea-rodin token (title: "feat(#141): add validate-docmap subcommand") +- [ ] Labels: none yet (AI reviewers will assign) +- [ ] Link issue #141 ---- - -_Dev-loop cycle complete at 04:35 UTC._