Skip to content

typescript: suppress Copilot Chat crash notification#325405

Open
roblourens wants to merge 1 commit into
mainfrom
roblou/agents/suppress-notification-copilot-chat
Open

typescript: suppress Copilot Chat crash notification#325405
roblourens wants to merge 1 commit into
mainfrom
roblou/agents/suppress-notification-copilot-chat

Conversation

@roblourens

Copy link
Copy Markdown
Member

Summary

  • suppress JS/TS language service crash notifications when Copilot Chat is the only contributing TypeScript server plugin
  • continue reporting other contributing plugins and preserve existing no-plugin notifications

Validation

  • npm run gulp compile-extension:typescript-language-features
  • npm run precommit

(Written by Copilot)

Do not show the JS/TS language service crash notification when Copilot Chat is the only contributing TypeScript server plugin. Continue reporting other plugins and preserve the existing no-plugin notification.\n\n(Written by Copilot)\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 11, 2026 01:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Suppresses TypeScript crash notifications when GitHub Copilot is the sole contributing server plugin.

Changes:

  • Filters Copilot from crash attribution.
  • Preserves notifications for other plugins and no-plugin crashes.
  • Deduplicates extension IDs in prompts.
Show a summary per file
File Description
extensions/typescript-language-features/src/typescriptServiceClient.ts Updates crash-notification plugin filtering and reporting.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Medium

@roblourens roblourens enabled auto-merge (squash) July 11, 2026 02:08
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.

3 participants