-
Notifications
You must be signed in to change notification settings - Fork 0
P0-X: Fix Architectural Naming Drift (BLOCKS: correct design implementation) #56
Copy link
Copy link
Closed
Labels
P0: criticalCritical path — blocks all dependent workCritical path — blocks all dependent worklayer: cortexRetrieval orchestration (rank → expand → order)Retrieval orchestration (rank → expand → order)layer: daydreamerBackground consolidation (LTP/LTD, recalc)Background consolidation (LTP/LTD, recalc)layer: documentationAPI docs, developer guide, architecture diagramsAPI docs, developer guide, architecture diagramslayer: hippocampusIngest orchestration (chunk → embed → persist)Ingest orchestration (chunk → embed → persist)
Milestone
Metadata
Metadata
Labels
P0: criticalCritical path — blocks all dependent workCritical path — blocks all dependent worklayer: cortexRetrieval orchestration (rank → expand → order)Retrieval orchestration (rank → expand → order)layer: daydreamerBackground consolidation (LTP/LTD, recalc)Background consolidation (LTP/LTD, recalc)layer: documentationAPI docs, developer guide, architecture diagramsAPI docs, developer guide, architecture diagramslayer: hippocampusIngest orchestration (chunk → embed → persist)Ingest orchestration (chunk → embed → persist)
Why: The codebase uses the term "Metroid" to name the sparse proximity/neighbor graph (
MetroidNeighbor,MetroidSubgraph,metroid_neighbors,getInducedMetroidSubgraph,FastMetroidInsert,FullMetroidRecalc). This is architecturally incorrect. In CORTEX, a Metroid is a structured dialectical search probe{ m1, m2, c }— a concept that does not yet exist in the codebase at all. The proximity graph has nothing to do with Metroids. This naming collision will cause permanent confusion and make the MetroidBuilder impossible to implement cleanly without a rename.P0-X1: Rename
MetroidNeighbor→SemanticNeighborincore/types.tsstorage/IndexedDbMetadataStore.tsP0-X2: Rename
MetroidSubgraph→SemanticNeighborSubgraphincore/types.tsstorage/IndexedDbMetadataStore.tscortex/Query.tsP0-X3: Rename
MetadataStoreproximity graph methods:putMetroidNeighbors→putSemanticNeighborsgetMetroidNeighbors→getSemanticNeighborsgetInducedMetroidSubgraph→getInducedNeighborSubgraphneedsMetroidRecalc→needsNeighborRecalcflagVolumeForMetroidRecalc→flagVolumeForNeighborRecalcclearMetroidRecalcFlag→clearNeighborRecalcFlagstorage/IndexedDbMetadataStore.ts,cortex/Query.ts, and test filesP0-X4: Rename planned Hippocampus file
hippocampus/FastMetroidInsert.ts→hippocampus/FastNeighborInsert.tsFastNeighborInsert/insertSemanticNeighborsP0-X5: Rename planned Daydreamer file
daydreamer/FullMetroidRecalc.ts→daydreamer/FullNeighborRecalc.tsFullNeighborRecalc/runNeighborRecalcP0-X6: Rename IndexedDB object store from
metroid_neighbors→neighbor_graphDB_VERSIONinstorage/IndexedDbMetadataStore.tsapplyUpgradeto copy data from old store to new storeP0-X7: Update all documentation strings and JSDoc that use "Metroid neighbor" to use "semantic neighbor"
Exit Criteria: No source file uses "Metroid" to refer to the proximity graph. The term "Metroid" is reserved exclusively for the
{ m1, m2, c }dialectical probe type implemented incortex/MetroidBuilder.ts.