docs(adr): catch up README index for ADR-0029 through 0033#4774
Merged
Conversation
PR #4771 (ADR-0033 positioning vs ECC) explicitly deferred the ADR README index update with: 'the index is stale at 0028; separate cleanup'. This PR is that cleanup. Added 6 entries that were present in the repo but missing from the index: - ADR-0029 (solo-dev-first refocus, 2026-05-16) - ADR-0030 network isolation (Accepted, 2026-05-22) - ADR-0030 agent auth RFC (Proposed, 2026-06-05) [duplicate number] - ADR-0031 budgets API design (Proposed, 2026-05-25) - ADR-0032 multi-agent architecture (Approved, 2026-05-30) - ADR-0033 positioning vs ECC (Proposed, 2026-06-20) Notes: - ADR-0030 has two real files sharing one number; linked both with disambiguated titles rather than silently renumbering (Boss call). - Pre-existing duplicates at ADR-0024 and ADR-0025 are out of scope (separate Boss decision; flagged in the index note). - All status/date/issue values verified against the actual ADR headers. - All 6 link targets exist on origin/develop.
Contributor
There was a problem hiding this comment.
β LGTM β surgical docs index cleanup, all 9 gates pass.
Verification (independent):
- All 6 added ADR files exist on
origin/develop(verified viagh api contents/...) - Status / Date / Issue values match actual ADR headers (spot-checked 0030, 0031, 0032, 0033)
- ADR-0033 cross-links resolve: #4768 (CLOSED/COMPLETED), ADR-0023, ADR-0029, ADR-0006 all exist
- Duplicate-numbering note (ADR-0030 a/b, pre-existing ADR-0024/0025) is honest and out-of-scope for this PR
- Diff is +8/-0 in
docs/adr/README.mdonly β no code, no config, no API surface touched
Gates:
- Review β full diff read β
- No conflicts β single file, no overlap risk β
- CI green β 15/15 SUCCESS, 2 SKIPPED by design (test-matrix, auto-label-test) β
- No regressions β markdown only, no code surface β
- Unit tests β N/A (docs-only, per 2026-06-09 durable rule) β
- E2E/UAT β N/A (docs-only) β
- Documented β this IS the documentation β
- Security clean β markdown, no secrets, no .gitignore violations β
- Targets
developβ β (notmain)
Functional Code Gate: N/A β no code touched, no test surface to verify.
No-Fly List (ECC differentiation): N/A β pure docs, no architectural decisions, no marketing copy, no harness-specific shortcuts.
Squash-merging to develop.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
PR #4771 (ADR-0033 positioning vs ECC) shipped with an explicit out-of-scope line in the PR body:
That cleanup was punted. This PR is that cleanup.
What
1 file changed, +8/-0 β
docs/adr/README.mdonly.Added 6 entries to the index that exist on
origin/developbut were unlinked:Plus a short note flagging the ADR-0030 duplicate-numbering problem and the pre-existing duplicates at ADR-0024 and ADR-0025.
Verification
origin/develop(verified vials docs/adr/)npm run docsnot applicable (markdown only)Decisions / non-decisions
Out of scope (intentionally not addressed here)
Closes the docs gap introduced by #4771's punt.