Skip to content

better autopilot wording + if no tools were called in the last round#312458

Merged
justschen merged 2 commits intomainfrom
justin/sylveon
Apr 25, 2026
Merged

better autopilot wording + if no tools were called in the last round#312458
justschen merged 2 commits intomainfrom
justin/sylveon

Conversation

@justschen
Copy link
Copy Markdown
Collaborator

… we dont continue

fix issue where autopilot was continuing even though it was basically done

Copilot AI review requested due to automatic review settings April 24, 2026 23:40
@justschen justschen enabled auto-merge (squash) April 24, 2026 23:41
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 adjusts the Copilot extension’s tool-calling autopilot loop to stop continuing when the model appears “done” (i.e., returns no tool calls), and updates the user-facing autopilot progress wording accordingly.

Changes:

  • Update shouldAutopilotContinue to stop nudging when the model returns a text-only response with no tool calls, and to stop after a prior nudge produced no tool calls.
  • Refine autopilot progress messages (tool-call-limit extension, error recovery, and “task not complete” continuation).
  • Extend the autopilot unit tests to cover the new stopping behaviors and update test helpers to support text responses.
Show a summary per file
File Description
extensions/copilot/src/extension/intents/test/node/toolCallingLoopAutopilot.spec.ts Updates mocks and adds tests to validate autopilot stops when no tools are called (including text-only responses).
extensions/copilot/src/extension/intents/node/toolCallingLoop.ts Adds new autopilot stop conditions for “no tools called” scenarios and updates autopilot progress wording.

Copilot's findings

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

Comment thread extensions/copilot/src/extension/intents/node/toolCallingLoop.ts Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@justschen justschen merged commit 0725586 into main Apr 25, 2026
26 checks passed
@justschen justschen deleted the justin/sylveon branch April 25, 2026 00:55
@vs-code-engineering vs-code-engineering Bot added this to the 1.118.0 milestone Apr 25, 2026
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.

3 participants