Skip to content

Refactor auto mode hints logic to include 'auto' in model hints for editor location#311331

Merged
jrieken merged 1 commit intomainfrom
joh/sticky-peacock
Apr 20, 2026
Merged

Refactor auto mode hints logic to include 'auto' in model hints for editor location#311331
jrieken merged 1 commit intomainfrom
joh/sticky-peacock

Conversation

@jrieken
Copy link
Copy Markdown
Member

@jrieken jrieken commented Apr 20, 2026

…ditor location

re github/copilot-api#31385

Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings April 20, 2026 08:56
@jrieken jrieken enabled auto-merge (squash) April 20, 2026 08:56
@jrieken jrieken self-assigned this Apr 20, 2026
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

Refactors the auto mode token request shaping so that when running in the editor (inline chat), the auto_mode.model_hints list always includes an 'auto' hint in addition to any experiment-provided hint, aligning with the “auto in editor location” requirement.

Changes:

  • Builds model_hints from the experiment value, defaulting to 'auto'.
  • For ChatLocation.Editor, appends 'auto' when the primary hint is not already 'auto'.
Show a summary per file
File Description
extensions/copilot/src/platform/endpoint/node/automodeService.ts Updates AutoModels request body to include 'auto' in editor model_hints while preserving existing behavior elsewhere.

Copilot's findings

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

Comment thread extensions/copilot/src/platform/endpoint/node/automodeService.ts
Comment thread extensions/copilot/src/platform/endpoint/node/automodeService.ts
@jrieken jrieken merged commit 8f579c2 into main Apr 20, 2026
30 checks passed
@jrieken jrieken deleted the joh/sticky-peacock branch April 20, 2026 09:25
@vs-code-engineering vs-code-engineering Bot added this to the 1.118.0 milestone Apr 20, 2026
@jrieken jrieken added the ~release-cherry-pick Trigger: cherry-pick this PR to the latest release branch label Apr 20, 2026
@vs-code-engineering vs-code-engineering Bot added release-cherry-pick Automated cherry-pick between release and main branches and removed ~release-cherry-pick Trigger: cherry-pick this PR to the latest release branch labels Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-cherry-pick Automated cherry-pick between release and main branches

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants