David Snelling
3a62445465
Adds a reserved, top-level `visibility` field (mirrors the subtype rollout): 'public' (default, surfaced) | 'internal' (a consumer's app-internal data — hidden from default find()/getRelations()/counts/stats, opt-in via includeInternal) | 'system' (Brainy plumbing, library-set only; the add()/relate() param narrows to 'public' | 'internal'). Fixes a real leak: the VFS root entity counted in getNounCount() and appeared in find() (a fresh brain reported 1 entity). It is now visibility:'system' → excluded from every user-facing surface (fresh brain reports 0). - Reserved (STANDARD_ENTITY_FIELDS / STANDARD_VERB_FIELDS) — surfaced top-level on reads, never in the custom metadata bag; a 'public'/'internal' value smuggled through metadata is lifted to the field, 'system' dropped with a one-shot warning. - Threaded through add/relate/update/updateRelation + their internal mirrors; stored only when not 'public' so the common case stays lean. - Default exclusion in counts (baseStorage, isCountedVisibility), find()/getRelations() (hard candidate filter via excludeVisibility — keeps top-K/limit/offset correct), and rebuildCounts; includeInternal/includeSystem opt-ins. - VFS root marked 'system'. Same on-disk shape as the 8.0 line, so it carries forward unchanged. Backward-compatible: absent === 'public', so all existing data stays counted + returned. Tests: tests/unit/brainy/visibility.test.ts 17/17 (fresh-brain getNounCount()===0, internal hidden + opt-in, verb symmetry, top-level surfacing, metadata-spoof rejection). 1522 unit green; count-synchronization integration green. |
||
|---|---|---|
| .. | ||
| add.test.ts | ||
| batch-operations.test.ts | ||
| counts-by-type-fix.test.ts | ||
| find.test.ts | ||
| get.test.ts | ||
| hybrid-search.test.ts | ||
| relate-duplicate-optimization.test.ts | ||
| relate.test.ts | ||
| update-reserved-metadata-remap.test.ts | ||
| update.test.ts | ||
| visibility.test.ts | ||