Skip to content

Better activation info for chat context provider#289069

Merged
alexr00 merged 1 commit intomainfrom
alexr00/additional-minnow
Jan 20, 2026
Merged

Better activation info for chat context provider#289069
alexr00 merged 1 commit intomainfrom
alexr00/additional-minnow

Conversation

@alexr00
Copy link
Member

@alexr00 alexr00 commented Jan 20, 2026

Fixes #280643

Copilot AI review requested due to automatic review settings January 20, 2026 10:33
@alexr00 alexr00 enabled auto-merge (squash) January 20, 2026 10:33
@alexr00 alexr00 self-assigned this Jan 20, 2026
@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 20, 2026
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 improves the documentation for chat context provider activation by replacing the old generic activation event instruction with more specific and nuanced guidance. The change differentiates between workspace/resource-based context providers (which should use appropriate signal-based activation events) and explicit context providers (which are automatically activated when requested).

Changes:

  • Updated API documentation in the proposed chat context provider TypeScript definitions to provide clearer guidance on activation events
  • Added the onChatContextProvider activation event to the extension registry's autocomplete suggestions

Reviewed changes

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

File Description
src/vscode-dts/vscode.proposed.chatContextProvider.d.ts Updated JSDoc comments to provide more specific guidance on activation events for different types of chat context providers
src/vs/workbench/services/extensions/common/extensionsRegistry.ts Added autocomplete entry for the onChatContextProvider activation event

@alexr00 alexr00 merged commit b7b1562 into main Jan 20, 2026
27 of 28 checks passed
@alexr00 alexr00 deleted the alexr00/additional-minnow branch January 20, 2026 10:55
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.

Activation event not working

3 participants