[PR #15494] Remove the telemetry for VT sequences #30652

Open
opened 2026-01-31 09:42:07 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/15494

State: closed
Merged: Yes


This removes the telemetry tracking which counted how many times each VT
sequence was executed, and how many times there were "failures". This
information isn't needed any more, and we were reaching the limit of how
many sequences we could track anyway.

Essentially what's been removed is the TermTelemetry class, but we are
still tracking some statemachine telemetry in the ParserTracing class.
And since that used the same trace logging provider as TermTelemetry,
I've now moved that definition into the tracing.cpp file.

The code still compiles and runs without exploding.

Closes #15482

**Original Pull Request:** https://github.com/microsoft/terminal/pull/15494 **State:** closed **Merged:** Yes --- This removes the telemetry tracking which counted how many times each VT sequence was executed, and how many times there were "failures". This information isn't needed any more, and we were reaching the limit of how many sequences we could track anyway. Essentially what's been removed is the `TermTelemetry` class, but we are still tracking some statemachine telemetry in the `ParserTracing` class. And since that used the same trace logging provider as `TermTelemetry`, I've now moved that definition into the `tracing.cpp` file. The code still compiles and runs without exploding. Closes #15482
claunia added the pull-request label 2026-01-31 09:42:07 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#30652