Skip to content

Remove noisy log#311034

Merged
roblourens merged 1 commit intomainfrom
roblou/decent-aphid
Apr 17, 2026
Merged

Remove noisy log#311034
roblourens merged 1 commit intomainfrom
roblou/decent-aphid

Conversation

@roblourens
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 17, 2026 16:07
@roblourens roblourens enabled auto-merge (squash) April 17, 2026 16:07
@roblourens roblourens self-assigned this Apr 17, 2026
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 reduces log/telemetry noise from chat request cancellation no-ops by skipping the “no pending request” reporting path when the cancellation is triggered by archiving an agent session.

Changes:

  • Suppress chat.stopCancellationNoop telemetry and the corresponding info log when cancelCurrentRequestForSession(..., 'archive') finds no pending request.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/common/chatService/chatServiceImpl.ts Adds a source check to avoid emitting no-op cancel logs/telemetry during archive-driven cancellations.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@github-actions
Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: 7d58583b Current: 8204533c

Changed (2)

chat/aiCustomizations/aiCustomizationManagementEditor/McpBrowseMode/Light
Before After
before after
editor/inlineCompletions/other/JumpToHint/Dark
Before After
before after

@roblourens roblourens merged commit e44fc34 into main Apr 17, 2026
30 checks passed
@roblourens roblourens deleted the roblou/decent-aphid branch April 17, 2026 16:40
@vs-code-engineering vs-code-engineering Bot added this to the 1.117.0 milestone Apr 17, 2026
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