Skip to content

Commit 3688e25

Browse files
author
codeo1io
committed
fix(dashboard): classify memory.external in ConfigEditor coverage manifest
The external-memory-backend branch added the memory.external schema subtree (provider/endpoint/banks/retain_sources/tags + recall sub-block) but did not add a corresponding entry to the dashboard's config-field-coverage manifest. The config-parity guard (config-parity.test.ts) enforces that every schema leaf is either RENDERED by the ConfigEditor form or listed in OMITTED_BY_DESIGN, so CI failed with 16 uncovered leaves. memory.external is USER-config-only and operator-configured (no form widgets exist for it), so the correct classification is OMITTED_BY_DESIGN with the whole subtree covered by the 'memory.external' prefix.
1 parent b462755 commit 3688e25

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

packages/dashboard/src/components/ConfigEditor/config-field-coverage.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,11 @@ export const OMITTED_BY_DESIGN: Readonly<Record<string, string>> = {
8383
"sidekick.system_prompt": "free-form prompt override; raw JSONC",
8484
"system_prompt_injection.skip_signatures":
8585
"free-form substring array; raw JSONC (no array widget in the form yet)",
86+
// External memory backend (Hindsight tee + recall). USER config only and
87+
// operator-configured (endpoint/api_key, bank routing, recall tuning); no
88+
// form widgets exist. Whole subtree is raw-JSONC by design.
89+
"memory.external":
90+
"external long-term memory backend (provider/endpoint/banks/retain_sources/tags + recall sub-block); USER config only, operator-configured, raw JSONC",
8691
};
8792

8893
/**

0 commit comments

Comments
 (0)