[codex] feat(cli): add built-in subagents#12012
[codex] feat(cli): add built-in subagents#12012yzlu0917 wants to merge 2 commits intocontinuedev:mainfrom
Conversation
💡 Codex ReviewLine 42 in 4ec8449 The new workspace filter is guarded by a truthy check, so passing ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
0c87bf1 to
d0ef33c
Compare
Summary
config.yamlsetupplanner,researcher, andreviewersubagents when no explicitsubagentmodels are configuredWhy
The beta subagent tool already exists in the CLI, but trying it currently requires internal-only
subagentmodel configuration. This change makes the feature usable out of the box by cloning the current chat model into a few specialized built-in subagents with hard-coded prompts inextensions/cli.Validation
npm test -- src/services/ModelService.test.ts src/tools/subagent.test.tsnpm run lintinextensions/cli(passes with two existing repo warnings in unrelated files)Closes #9550
Summary by cubic
Adds built-in CLI subagents (
planner,researcher,reviewer) so the beta Subagent tool works out of the box while still honoring configuredsubagentmodels (closes #9550). Also fixes empty workspace filtering and standardizes OpenRouter attribution headers.New Features
ModelService.getSubagentModelsprefers configured subagents; otherwise uses built-ins cloned from the active chat model. Tests cover precedence, fallback, and the no-base-model case.@continuedev/openai-adaptersOPENROUTER_HEADERS; switched toX-OpenRouter-Titleand addedX-OpenRouter-Categories, applied in the coreOpenRouterclient.Bug Fixes
workspaceDirectorynow returns only no-workspace sessions; added test coverage.Written for commit 0b953cb. Summary will update on new commits.