Skip to content

feat: promote allowSubAgents to top-level with default true - #276

Merged
ranxianglei merged 2 commits into
masterfrom
2026-08-04_allow-subagents-default
Aug 5, 2026
Merged

feat: promote allowSubAgents to top-level with default true#276
ranxianglei merged 2 commits into
masterfrom
2026-08-04_allow-subagents-default

Conversation

@ranxianglei

Copy link
Copy Markdown
Owner

Summary

Promotes allowSubAgents from the experimental namespace to a top-level config field, and changes the default from false to true.

Changes

  • Top-level field: config.allowSubAgents: boolean (was config.experimental.allowSubAgents)
  • Default: true (was false) — subagent compression is now on by default
  • Backward compat: old experimental.allowSubAgents configs still work (top-level takes priority)
  • Updated 4 hook sites: index.ts:111, hooks.ts:89/110/166
  • Type validation: added boolean check for top-level allowSubAgents in validateConfigTypes
  • Updated schema: top-level allowSubAgents + deprecated experimental.allowSubAgents
  • Updated docs: README, README.zh-CN, CONFIGURATION, CONFIGURATION.zh-CN, AGENTS.md, TESTING.md

Why

The feature is stable and well-tested. Subagent sessions benefit from compression just like main sessions.

Dual-Agent Review

  • Oracle: APPROVE — backward compat verified, type-safe, all consumers updated
  • Explore: APPROVE — exhaustive grep confirms no missed references, merge logic correct

Test Results

  • Typecheck: PASS
  • Tests: 954/954 PASS
  • Build: PASS (386.62 KB)

- Move allowSubAgents from experimental namespace to top-level config
- Change default from false to true (subagent compression is now on by default)
- Full backward compat: old experimental.allowSubAgents still read
- Update hooks.ts (3 sites), index.ts (1 site), config.ts, config-validation.ts
- Update dcp.schema.json, README.md, README.zh-CN.md, CONFIGURATION.md/zh-CN
- Update AGENTS.md §2.4 default config
…G.md

Oracle review finding: validateConfigTypes validated the deprecated
experimental.allowSubAgents but not the new preferred top-level key.
Added boolean type check for top-level allowSubAgents.

Also updated TESTING.md config examples to use new top-level form.
@ranxianglei
ranxianglei merged commit 8f59d9f into master Aug 5, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant