Skip to content
This repository was archived by the owner on May 20, 2026. It is now read-only.

provide more terminal tools to subagent#5050

Closed
meganrogge wants to merge 1 commit into
mainfrom
merogge/sub-agent
Closed

provide more terminal tools to subagent#5050
meganrogge wants to merge 1 commit into
mainfrom
merogge/sub-agent

Conversation

@meganrogge
Copy link
Copy Markdown
Contributor

@meganrogge meganrogge commented Apr 9, 2026

part of microsoft/vscode#308048

Expand execution subagent's allowed tools to include get_terminal_output, send_to_terminal, and kill_terminal so it can actively manage terminals instead of relying on notifications (which are only passed to the main agent)

@meganrogge meganrogge self-assigned this Apr 9, 2026
@meganrogge meganrogge added this to the 1.116.0 milestone Apr 9, 2026
@meganrogge meganrogge enabled auto-merge April 9, 2026 17:32
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

This PR expands the execution subagent’s terminal capabilities by adding additional terminal-related tool names to the tool registry and allowing the execution subagent tool-calling loop to use them, aligning with the goal of enabling richer execution workflows (microsoft/vscode#308048).

Changes:

  • Add send_to_terminal and kill_terminal to the ToolName enum and categorize them for the virtual tool grouping system.
  • Allow the execution subagent tool-calling loop to use get_terminal_output, send_to_terminal, and kill_terminal in addition to run_in_terminal.
Show a summary per file
File Description
src/extension/tools/common/toolNames.ts Adds new terminal tool identifiers and categorizes them so they’re treated as built-in tools in the virtual tool grouping system.
src/extension/prompt/node/executionSubagentToolCallingLoop.ts Expands the execution subagent allowlist to include additional terminal tools.

Copilot's findings

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

@meganrogge meganrogge removed this from the 1.116.0 milestone Apr 10, 2026
@meganrogge meganrogge closed this Apr 14, 2026
auto-merge was automatically disabled April 14, 2026 16:49

Pull request was closed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants