Skip to content

[Bug Fix] Make execution subagent model exp based instead of simple.#308642

Open
vikramnitin9 wants to merge 2 commits intomainfrom
vikram/exec_sub_exp_based
Open

[Bug Fix] Make execution subagent model exp based instead of simple.#308642
vikramnitin9 wants to merge 2 commits intomainfrom
vikram/exec_sub_exp_based

Conversation

@vikramnitin9
Copy link
Copy Markdown
Member

The execution subagent model is not set as experiment-based, which means that our flights are not using the correct models. Fixed that, and also changed the default execution subagent tool calling limit to 10.

Supercedes #308570.

@bhavyaus - tagging you for approval.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes execution subagent configuration so model selection is experiment-based (ensuring flights pick up the intended model), and updates the default execution subagent tool-calling limit.

Changes:

  • Switch chat.executionSubagent.model to an experiment-based config and read it via getExperimentBasedConfig(...).
  • Increase default chat.executionSubagent.toolCallLimit from 5 to 10.
  • Mark execution subagent settings as experiment-driven in the extension configuration (onExp tag).
Show a summary per file
File Description
extensions/copilot/src/platform/configuration/common/configurationService.ts Makes ExecutionSubagentModel experiment-based and raises default tool-call limit to 10.
extensions/copilot/src/extension/prompt/node/executionSubagentToolCallingLoop.ts Reads execution subagent model via experiment-based configuration lookup.
extensions/copilot/package.json Updates defaults/tags for execution subagent settings to reflect experiment-based behavior and new limit.

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 0

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.

2 participants