Skip to content

Sessions: telemetry fix#308093

Merged
osortega merged 1 commit intomainfrom
osortega/extraordinary-mollusk
Apr 6, 2026
Merged

Sessions: telemetry fix#308093
osortega merged 1 commit intomainfrom
osortega/extraordinary-mollusk

Conversation

@osortega
Copy link
Copy Markdown
Contributor

@osortega osortega commented Apr 6, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 6, 2026 22:37
@osortega osortega self-assigned this Apr 6, 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 fixes Sessions (Agents window) telemetry for the “Run” split-button dropdown by ensuring the interaction events are logged from the actual UI click handlers used by the titlebar dropdown, rather than from command handlers that may not be invoked in that flow.

Changes:

  • Move logSessionsInteraction(..., 'addTask' | 'generateNewTask') from the Action2.run() implementations to the dropdown action run callbacks used by RunScriptActionViewItem.
  • Store ITelemetryService on RunScriptActionViewItem so it can be used consistently by dropdown actions and passed to the dropdown widget.
Show a summary per file
File Description
src/vs/sessions/contrib/chat/browser/runScriptAction.ts Relocates “Add Task” / “Generate New Task” telemetry logging to the titlebar dropdown action handlers to ensure events fire when users click those items.

Copilot's findings

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

@osortega osortega merged commit 66c2fef into main Apr 6, 2026
23 checks passed
@osortega osortega deleted the osortega/extraordinary-mollusk branch April 6, 2026 23:37
@vs-code-engineering vs-code-engineering bot added this to the 1.116.0 milestone Apr 6, 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