Skip to content

mark chatModes contribution point as proposed#268866

Merged
aeschli merged 2 commits intomainfrom
aeschli/gradual-emu-619
Sep 29, 2025
Merged

mark chatModes contribution point as proposed#268866
aeschli merged 2 commits intomainfrom
aeschli/gradual-emu-619

Conversation

@aeschli
Copy link
Copy Markdown
Contributor

@aeschli aeschli commented Sep 29, 2025

For #251580

Copilot AI review requested due to automatic review settings September 29, 2025 13:50
@aeschli aeschli enabled auto-merge (squash) September 29, 2025 13:50
@aeschli aeschli self-assigned this Sep 29, 2025
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 marks chat-related extension contribution points (chatPromptFiles, chatInstructions, and chatModes) as proposed APIs by adding the required API enablement check.

  • Adds proposed API validation for chat prompt file contributions

private handle(extensionPoint: extensionsRegistry.IExtensionPoint<IRawChatFileContribution[]>, contributionPoint: ChatContributionPoint) {
extensionPoint.setHandler((_extensions, delta) => {
for (const ext of delta.added) {
checkProposedApiEnabled(ext.description, 'chatParticipantPrivate');
Copy link

Copilot AI Sep 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The API identifier 'chatParticipantPrivate' appears incorrect for chat prompt files contributions. Based on the context that this handles 'chatPromptFiles', 'chatInstructions', and 'chatModes' contribution points, the API identifier should likely be something like 'chatPromptFiles' or a more generic chat-related identifier rather than 'chatParticipantPrivate'.

Suggested change
checkProposedApiEnabled(ext.description, 'chatParticipantPrivate');
checkProposedApiEnabled(ext.description, contributionPoint);

Copilot uses AI. Check for mistakes.
@vs-code-engineering vs-code-engineering bot added this to the September 2025 milestone Sep 29, 2025
@aeschli aeschli changed the title mark chatPromptFiles, chatInstructions and chatModes as proposed mark chatModes contribution point as proposed Sep 29, 2025
@aeschli aeschli merged commit 6c76b64 into main Sep 29, 2025
28 checks passed
@aeschli aeschli deleted the aeschli/gradual-emu-619 branch September 29, 2025 14:07
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Nov 13, 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