From 455a4a355d4226d2b84e8472f0e3b5b1a25b9872 Mon Sep 17 00:00:00 2001 From: Eric Rakestraw Date: Sun, 25 Oct 2026 12:37:33 +0000 Subject: [PATCH] Map ConfigKit consolidation callers and coverage --- docs/roadmap/implementation.md | 34 ++++++++++++++++++++++++++++++++-- 1 file changed, 32 insertions(+), 2 deletions(-) diff --git a/docs/roadmap/implementation.md b/docs/roadmap/implementation.md index 3288283..ab74dd9 100644 --- a/docs/roadmap/implementation.md +++ b/docs/roadmap/implementation.md @@ -1,6 +1,6 @@ # Consolidation Implementation Plan -Status: ready for implementation. Follow Stage 1 through Stage 20 in order. +Status: in progress. Stage 1 is complete; follow Stage 2 through Stage 20 in order. The target executor is a Luna coding agent working through one end-to-end sprint. ## Authority, Baseline, And Execution @@ -121,7 +121,7 @@ traversal-count test, extended fuzz campaign, CI wait, or manual platform check required by this plan. Add narrow behavior tests only for consequential gaps identified below that existing tests do not already cover. -## Stage 1 — Establish The Starting Map +## Stage 1 — Establish The Starting Map (Complete) **Work:** @@ -143,6 +143,36 @@ identified below that existing tests do not already cover. **Finish:** a concrete caller/coverage map and any actual coverage gaps. No production change or empty commit is needed for orientation alone. +Progress notes (2026-10-25): the worktree was clean at `8ab1568`, immediately +after the planning baseline `8a99e61`; the audited production baseline +`5959537` is present. `selectOptions` is called by the ordinary and grouped +load entry points. The `assembleBaseInput...` wrappers converge on +`assembleBaseInputTrackedForRole`, reached from ordinary loading, layered main +loading, and underlay-group acquisition. The tracked import wrappers converge +on `loadImportedDocumentsTrackedWithProcessing`; production callers are input +acquisition and underlay-group acquisition, while older convenience variants +also have direct test callers. `contextsForAcquiredSources` is shared by ordinary +loading, layered assembly, input acquisition, and overlay application. Selected +overlay loading is reached from ordinary overlay application and layered input +assembly. Both inheritance adapters, for ordinary and identified sequences, +feed `applyInheritanceScopeTemplates`. + +Existing behavioral owners cover repeated/cyclic resources and cross-tier reuse +(`internal/engine/imports_test.go`, `facade_equivalence_test.go`); default and +enabled source-version modes (`internal/engine/source_version_test.go`, +`facade_source_version_mode_test.go`); selected/absent discovery +(`internal/engine/discovery_load_test.go`, `facade_discovery_test.go`); ordinary +and layered overlays (`internal/engine/selected_overlay_test.go`, +`facade_layered_test.go`); item-local helper scopes (`internal/engine/imports_test.go`, +`facade_import_lifecycle_test.go`); inheritance identity/history +(`internal/engine/inheritance_identity_history_test.go`, +`internal/engine/inheritance_history_test.go`); inline fields +(`internal/engine/schema_test.go`, `internal/engine/projection_test.go`); and +detached results and callback ownership (`internal/engine/metadata_test.go`, +`internal/engine/callback_ownership_test.go`, `facade_diff_test.go`). No specific +coverage gap surfaced in this map; later focused checks remain assigned in the +validation groups above. + ## Stage 2 — Remove Unused Reverse Bridges And Share Real Facade Conversion **Owners:** root `conversion.go`, `records_test.go`, `lifecycle.go`, `layered.go`,