Skip to content

feat: builtin subagents#10830

Open
uinstinct wants to merge 10 commits intocontinuedev:mainfrom
uinstinct:cli-default-subagents
Open

feat: builtin subagents#10830
uinstinct wants to merge 10 commits intocontinuedev:mainfrom
uinstinct:cli-default-subagents

Conversation

@uinstinct
Copy link
Copy Markdown
Contributor

@uinstinct uinstinct commented Feb 25, 2026

Description

Add 2 builtin subagents: navigator and code reviewer.

  • is added automatically only when anthropic provider is present with a local api key
  • also fix tool calls hung in subagent
  • fix nested subagent execution
  • put subagent tool out of beta

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • [] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Continue Tasks: ❌ 7 failed — View all


Summary by cubic

Adds built-in Navigator and Generalist subagents (auto-added when using Anthropic with a local API key) and enables the Subagent tool by default. Improves tool result handling in subagents and blocks nested subagent calls.

  • New Features

    • Built-in subagents: navigator and generalist; included when Anthropic has a direct API key.
    • Subagent tool is always on; removes the --beta-subagent-tool flag.
    • Refined subagent tool description with dynamic listing of available subagents.
  • Bug Fixes

    • Tool call results are recorded in subagents even without ChatHistoryService; no more hangs.
    • Prevents nested subagent invocation with a clear error.

Written for commit 96cc2f8. Summary will update on new commits.

@uinstinct uinstinct requested a review from a team as a code owner February 25, 2026 13:57
@uinstinct uinstinct requested review from sestinj and removed request for a team February 25, 2026 13:57
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Feb 25, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 8 files

Copy link
Copy Markdown
Contributor

@sestinj sestinj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For now, I think we should still make sure that we have a way to disable sub-agents, even if we have them on by default, just in case we need to roll something back. e.g. CN_DISABLE_SUBAGENTS=1.

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Mar 1, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 4 files (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="extensions/cli/src/tools/subagent.test.ts">

<violation number="1" location="extensions/cli/src/tools/subagent.test.ts:6">
P2: Mock path no longer matches the imported module path, so getAgentNames/getSubagent won’t be mocked and vi.mocked(...).mockReturnValue may fail or run real logic.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

@uinstinct uinstinct requested a review from sestinj March 10, 2026 10:27
@RomneyDa RomneyDa mentioned this pull request Mar 25, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants