Skip to content

past tense for optimizing tool selection#313989

Merged
justschen merged 1 commit into
mainfrom
justin/rowlet
May 3, 2026
Merged

past tense for optimizing tool selection#313989
justschen merged 1 commit into
mainfrom
justin/rowlet

Conversation

@justschen
Copy link
Copy Markdown
Collaborator

optimizing tool selection -> optimized tool selection

Copilot AI review requested due to automatic review settings May 3, 2026 20:48
@justschen justschen enabled auto-merge (squash) May 3, 2026 20:48
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 3, 2026

Screenshot Changes

Base: bedd833d Current: 96527e7d

Changed (1)

agentSessionsViewer/ApprovalRowLongLabel/Light
Before After
before after

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

Updates the tool-grouping progress UI text in the Copilot prompt pipeline to use a past-tense completion message once tool selection optimization finishes.

Changes:

  • Adjusted the in-progress message for tool selection optimization.
  • Added a past-tense completion message returned from the progress callback.
Show a summary per file
File Description
extensions/copilot/src/extension/prompt/node/defaultIntentRequestHandler.ts Updates the tool-selection optimization progress text to resolve to a past-tense completion message.

Copilot's findings

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

@justschen justschen merged commit 8419cd4 into main May 3, 2026
30 checks passed
@justschen justschen deleted the justin/rowlet branch May 3, 2026 21:50
@vs-code-engineering vs-code-engineering Bot added this to the 1.119.0 milestone May 3, 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