Context
Flight's Phase 2 review found that config is declared in CollectionEntityMap (collection-map.ts:31) but has no corresponding ConfigCollection class in collections.ts and no state.config property in SquadState.
Options
- Add a stub
ConfigCollection — read-only access to .squad/config.json via StorageProvider
- Remove
config from CollectionEntityMap — if config is not intended as a state-layer concern
Branch
Work should be done on diberry/sa-phase1-interface
Severity
Should-fix (non-blocking for Phase 3 CLI migration)
Context
Flight's Phase 2 review found that
configis declared inCollectionEntityMap(collection-map.ts:31) but has no correspondingConfigCollectionclass incollections.tsand nostate.configproperty inSquadState.Options
ConfigCollection— read-only access to.squad/config.jsonvia StorageProviderconfigfromCollectionEntityMap— if config is not intended as a state-layer concernBranch
Work should be done on
diberry/sa-phase1-interfaceSeverity
Should-fix (non-blocking for Phase 3 CLI migration)