chore(#141): remove TODO.md — dev-loop artifact, not project documentation
PR Ready Gate / clear-labels (pull_request) Successful in 1s
CI / test (pull_request) Successful in 16s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Successful in 38s
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Successful in 1m22s
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Successful in 1m43s
PR Ready Gate / clear-labels (pull_request) Successful in 1s
CI / test (pull_request) Successful in 16s
CI / review (anthropic--claude-4.6-sonnet, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Successful in 38s
CI / review (gpt-5, gpt, GPT_REVIEW_TOKEN) (pull_request) Successful in 1m22s
CI / review (gpt-5, security, ., rodin/security-patterns, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Successful in 1m43s
This commit is contained in:
@@ -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
|
|
||||||
|
|
||||||
Reference in New Issue
Block a user