Skip to content

feat(vscode): Add telemetry for charting tool - #12878

Merged
cosi-conda merged 6 commits into
mainfrom
feature/charts-telemetry
Aug 5, 2026
Merged

feat(vscode): Add telemetry for charting tool#12878
cosi-conda merged 6 commits into
mainfrom
feature/charts-telemetry

Conversation

@cosi-conda

@cosi-conda cosi-conda commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Issue

  • Add telemetry to charting tool to capture the event in which the charting is used.
  • Add a unit test for this event in the telemetry tests.

Implementation

Leverage the @kilocode/kilo-telemetry existing tracking function to capture a new chart event.

How to Test

Manual/local verification

  1. Open a kilo session in the VS code extension
  2. Ask to render a sample chart such as a bar chart:
    "Can you make a sample bar chart?"
image
  1. Executing this should capture a chart tool event
  2. Verify the event can be seen remotely

Checklist

  • Issue linked above, or exception explained
  • Tests/verification described
  • Screenshots/video included for visual changes, or marked N/A
  • Changeset considered for user-facing changes
  • I personally reviewed the diff and can explain the changes, including any AI-assisted work.

@cosi-conda cosi-conda changed the title feat(charts-telemetry): Add telemetry for charting tool leveraged in VS Code extension feat(charts-telemetry): VS Code extension - Add telemetry for charting tool Aug 4, 2026
@cosi-conda cosi-conda changed the title feat(charts-telemetry): VS Code extension - Add telemetry for charting tool feat(vscode): Add telemetry for charting tool Aug 4, 2026
@cosi-conda
cosi-conda marked this pull request as ready for review August 5, 2026 20:26
Comment thread packages/opencode/src/kilocode/tool/chart.ts
@kilo-code-bot

kilo-code-bot Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 1
Issue Details (click to expand)

SUGGESTION

File Line Issue
packages/opencode/src/kilocode/tool/chart.ts 70 Telemetry.trackToolUsed only fires on successful renders — failed validations emit no Tool Used event, unlike codebase_search which tracks all invocations
Files Reviewed (2 files)
  • packages/opencode/src/kilocode/tool/chart.ts - 1 issue
  • packages/kilo-telemetry/src/__tests__/telemetry.test.ts - 0 issues

Fix these issues in Kilo Cloud


Reviewed by kimi-k3 · Input: 53.9K · Output: 8.9K · Cached: 519K

Review guidance: REVIEW.md from base branch main

@cosi-conda
cosi-conda merged commit 00594eb into main Aug 5, 2026
41 of 44 checks passed
@cosi-conda
cosi-conda deleted the feature/charts-telemetry branch August 5, 2026 22:03
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
feat(vscode): Add telemetry for charting tool
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.

2 participants