Skip to content

Add support for rename inline suggestions#279147

Merged
benibenj merged 7 commits intomainfrom
benibenj/wet-skunk
Nov 24, 2025
Merged

Add support for rename inline suggestions#279147
benibenj merged 7 commits intomainfrom
benibenj/wet-skunk

Conversation

@benibenj
Copy link
Contributor

@benibenj benibenj commented Nov 24, 2025

Copilot Generated Description: Introduce support for inline suggestions that allow renaming symbols.

closes #277818

Copilot AI review requested due to automatic review settings November 24, 2025 11:19
@benibenj benibenj enabled auto-merge November 24, 2025 11:19
@benibenj benibenj self-assigned this Nov 24, 2025
@vs-code-engineering vs-code-engineering bot added this to the November 2025 milestone Nov 24, 2025
Copy link
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 introduces support for inline suggestions that enable automatic rename refactoring when accepting code completions. When an inline completion involves renaming a symbol, the system detects this and offers to perform the rename operation across the codebase.

Key changes include:

  • Added a new RenameSymbolProcessor class that analyzes inline completion edits to detect rename operations
  • Extended the inline completion API with a supportsRename property to indicate rename capability
  • Added telemetry tracking for rename operations (creation status, duration, timeout)

Reviewed changes

Copilot reviewed 11 out of 13 changed files in this pull request and generated 16 comments.

Show a summary per file
File Description
src/vscode-dts/vscode.proposed.inlineCompletionsAdditions.d.ts Added supportsRename property to proposed inline completion API
src/vs/workbench/api/common/extHostLanguageFeatures.ts Extended inline completion adapter to pass supportsRename flag to main thread
src/vs/workbench/api/browser/mainThreadLanguageFeatures.ts Added rename telemetry fields to lifetime summary
src/vs/monaco.d.ts Added supportsRename property and renamed InlineCompletionHint to IInlineCompletionHint, added rename telemetry properties
src/vs/editor/contrib/rename/browser/rename.ts Exported new prepareRename function for rename validation
src/vs/editor/contrib/inlineCompletions/browser/telemetry.ts Added telemetry classification for rename tracking fields
src/vs/editor/contrib/inlineCompletions/browser/model/renameSmbolProcessor.ts New processor class that detects and proposes rename refactorings from inline completions
src/vs/editor/contrib/inlineCompletions/browser/model/provideInlineCompletions.ts Added RenameInfo type and support for rename commands in suggestion data
src/vs/editor/contrib/inlineCompletions/browser/model/inlineSuggestionItem.ts Added rename-related properties and methods to suggestion items
src/vs/editor/contrib/inlineCompletions/browser/model/inlineCompletionsSource.ts Integrated rename processor into suggestion pipeline
src/vs/editor/contrib/inlineCompletions/browser/model/inlineCompletionsModel.ts Added rename command execution when accepting completions
src/vs/editor/contrib/inlineCompletions/browser/controller/commandIds.ts Added rename symbol command identifier
src/vs/editor/common/languages.ts Added supportsRename to InlineCompletion, renamed InlineCompletionHint to IInlineCompletionHint, added rename fields to LifetimeSummary

aiday-mar
aiday-mar previously approved these changes Nov 24, 2025
…bolProcessor.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
aiday-mar
aiday-mar previously approved these changes Nov 24, 2025
@benibenj benibenj merged commit bfc6d53 into main Nov 24, 2025
28 checks passed
@benibenj benibenj deleted the benibenj/wet-skunk branch November 24, 2025 14:53
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Jan 8, 2026
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.

Explore invoking semantic rename with NES

5 participants