Skip to content

Don't include ext description in IChatSessionsExtensionPoint#273595

Merged
mjbvz merged 1 commit intomicrosoft:mainfrom
mjbvz:dev/mjbvz/classical-tyrannosaurus
Oct 27, 2025
Merged

Don't include ext description in IChatSessionsExtensionPoint#273595
mjbvz merged 1 commit intomicrosoft:mainfrom
mjbvz:dev/mjbvz/classical-tyrannosaurus

Conversation

@mjbvz
Copy link
Copy Markdown
Collaborator

@mjbvz mjbvz commented Oct 27, 2025

This isn't part of the contribution so the types are incorrect

This isn't part of the contribution so the types are incorrect
Copilot AI review requested due to automatic review settings October 27, 2025 16:44
@mjbvz mjbvz enabled auto-merge October 27, 2025 16:44
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 removes the extensionDescription field from the IChatSessionsExtensionPoint interface, as it was incorrectly included in the contribution type definition. The extension description is not part of the contribution schema itself but is metadata about the extension providing the contribution.

Key Changes:

  • Removed extensionDescription field from IChatSessionsExtensionPoint interface
  • Updated internal storage to keep extension metadata separate from contribution data
  • Updated all references to access extension metadata through the new storage structure

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/vs/workbench/contrib/chat/common/chatSessionsService.ts Removed extensionDescription field and unused import from the IChatSessionsExtensionPoint interface
src/vs/workbench/contrib/chat/browser/chatSessions.contribution.ts Refactored internal storage to separate contribution data from extension metadata, and updated all references throughout the service

@mjbvz mjbvz merged commit 541d997 into microsoft:main Oct 27, 2025
17 checks passed
@vs-code-engineering vs-code-engineering Bot locked and limited conversation to collaborators Dec 11, 2025
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.

3 participants