Found during review of PR #12 (pre-existing, not caused by the merge).
packages/node-sdk/src/v2/config-mapper.ts KIMI_CONFIG_DOMAINS (line 23) whitelists which KimiConfig domains survive the v1→v2 mapping. Several newer domains are not in the list and are silently dropped: disabledSkills, extraAgentDirs, secondaryModel, persistDefaultModel, agentsMdExpandIncludes.
The code comment at line 52 already admits (secondaryModel, extraAgentDirs, ...) are dropped. Impact: SDK consumers on the v2 engine lose these settings without any warning.
Fix: add the missing domains to KIMI_CONFIG_DOMAINS (each maps to an existing v2 config section), or map them explicitly.
Found during review of PR #12 (pre-existing, not caused by the merge).
packages/node-sdk/src/v2/config-mapper.tsKIMI_CONFIG_DOMAINS(line 23) whitelists whichKimiConfigdomains survive the v1→v2 mapping. Several newer domains are not in the list and are silently dropped:disabledSkills,extraAgentDirs,secondaryModel,persistDefaultModel,agentsMdExpandIncludes.The code comment at line 52 already admits (
secondaryModel,extraAgentDirs, ...) are dropped. Impact: SDK consumers on the v2 engine lose these settings without any warning.Fix: add the missing domains to
KIMI_CONFIG_DOMAINS(each maps to an existing v2 config section), or map them explicitly.