feat(#141): validate-docmap subcommand — CI hard-fail on missing docmap coverage #142

Merged
rodin merged 11 commits from issue-141 into main 2026-05-15 07:39:22 +00:00
Showing only changes of commit 83a1835474 - Show all commits
-47
View File
1
@@ -1,47 +0,0 @@
## Dev Loop Status: 2026-05-15 07:02 UTC
**Issue:** #141 — validate-docmap subcommand
**Branch:** issue-141
**Status:** ✅ READY FOR PR SUBMISSION
### Implementation Complete
-`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
### Test Results
```
TestRunValidateDocmap_Clean ✅
TestRunValidateDocmap_MissingDocmapFlag ✅
TestRunValidateDocmap_BadYAML ✅
TestRunValidateDocmap_StaleDocs ✅
TestRunValidateDocmap_UncoveredFile ✅
TestRunValidateDocmap_BothFailures ✅
TestRunValidateDocmap_EmptyStdin ✅
TestRunValidateDocmap_BlankLinesSkipped ✅
TestRunValidateDocmap_DuplicateDocsDeduped ✅
```
### Pre-Push Checks
- ✅ All packages pass: `go test ./...` (6/6)
-`go vet ./...` — clean
-`go build ./cmd/review-bot` — clean
-`scripts/check-deps.sh` — pass
### Commits
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 Steps
- [ ] Create PR via gitea-rodin token (title: "feat(#141): add validate-docmap subcommand")
- [ ] Labels: none yet (AI reviewers will assign)
- [ ] Link issue #141