Repository navigation
docs: refresh §9 refactor state — R0–R7 COMPLETE (ADR-0007–0009 shipped) - #107
Merged
Merged
Conversation
…0009 shipped) The architecture refactor is done: importlinter is now a required CI gate with a clean 6-contract / 0-violation baseline. Update ROADMAP §9 accordingly. - §9.0 phase header + status table: R6 (#100) and R7 (#101–#106) marked complete; ADR-0007/0008/0009 → Accepted; ADR-0010 deferred. Reviewed against dev @ fe5f604. - Add "Completed" entries for R6, R7.1, R7.2, R7.3a–d and matching "Tests run" lines (incl. the R6 cv2/Qt subprocess-isolation incident and the R7.3d 6/0 baseline). - Dependency-graph note: 3,982 nodes / 11,433 edges post-R7 (net growth from new modules + deprecation shims; cross-layer imports 24 → 0). - R4/R5.2/R7 risk rows marked resolved; the "21 violations gating R7" notes rewritten to reflect the cleared baseline. - §9.2 diagram + prose: all phases ✅; Phase R6/R7 detail headers → COMPLETE. - New lessons: layers-contract forbids intra-band siblings (use forbidden contracts); fix-the-layer beats DI; precise contract beats blanket ban + waivers; subprocess-isolate cv2-importing tests. - "Next steps" rewritten: refactor complete; remaining items are optional follow-ups (BaseTermManager for metrics/curriculum is the recommended next slice; ADR flips; ≤400 cli target; test_cli split; shim removal; ADR-0010). (CLAUDE.md, plans/adr/0007–0009, and the ADR status flips are gitignored local artifacts, refreshed in the same pass; only ROADMAP.md is committed — matching docs PRs #93 / #96 / #99.) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
KraHsu
added a commit
that referenced
this pull request
Jun 5, 2026
docs: refresh §9 refactor state — R0–R7 COMPLETE (ADR-0007–0009 shipped)
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
1 participant
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.
What
Closing docs refresh: the R0–R7 architecture refactor is complete.
lint-importsis now a required CI gate (6 contracts, 0 violations). Updates ROADMAP §9 to reflect R6 + R7. Docs-only PR (matching #93 / #96 / #99).Changes (ROADMAP.md)
R0–R7 COMPLETE; status table marks R6 (refactor: relocate vecenv adapters under rl/vecenvs/ (ROADMAP §9 PR R6 / ADR-0007) #100) + R7 (feat: public extension API genelab.extensions + registry.Runnable (ROADMAP §9 PR R7.1 / ADR-0008) #101–ci: flip importlinter to a required gate with a clean 6/0 baseline (ROADMAP §9 PR R7.3d / ADR-0009) #106) complete; ADR-0007/0008/0009 →Accepted; reviewed againstdev @ fe5f604.layerscontracts forbid intra-band siblings → useforbiddencontracts; fix-the-layer beats dependency injection; precise contract beats blanket ban + waivers; subprocess-isolate cv2-importing tests.BaseTermManagerfor metrics/curriculum managers).Note
CLAUDE.mdandplans/adr/(the ADR-0007/0008/0009Acceptedflips + outcome notes, the updated invariants/lessons/status) are gitignored local artifacts, refreshed in the same pass for future local sessions; only the trackedROADMAP.mdis committed here.🤖 Generated with Claude Code