Skip to content

[Telemetry] Add @log open to quickly open traces - #2955

Merged
George Ng (GeorgeNgMsft) merged 9 commits into
mainfrom
dev/georgeng/add_trace_command
Sep 2, 2026
Merged

[Telemetry] Add @log open to quickly open traces#2955
George Ng (GeorgeNgMsft) merged 9 commits into
mainfrom
dev/georgeng/add_trace_command

Conversation

@GeorgeNgMsft

@GeorgeNgMsft George Ng (GeorgeNgMsft) commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Add log open <traceId|last> to quickly open traces in local Grafana view (opens in new window + chat link)

image image

Uploading image.png…

@GeorgeNgMsft
George Ng (GeorgeNgMsft) marked this pull request as ready for review September 1, 2026 23:51
@GeorgeNgMsft

Copy link
Copy Markdown
Contributor Author

Minor CICD fix here and doc changes waiting on other PRs to complete

@datduyng Dominic Nguyen (datduyng) 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.

George Ng (@GeorgeNgMsft), my suggestion

It's good practice to persist all trace id some how - either through storage or in memory (for the active session)

Example of claude jsonl

Image

The uuid is the server id

Comment thread ts/packages/dispatcher/dispatcher/src/context/commandHandlerContext.ts Outdated
Comment thread ts/packages/dispatcher/dispatcher/src/context/commandHandlerContext.ts Outdated
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: d3fa76ff-0a2e-424c-8c79-828ec4369319
…e_command

# Conflicts:
#	ts/docs/architecture/telemetry/local-telemetry-debugging.md
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: d3fa76ff-0a2e-424c-8c79-828ec4369319
@GeorgeNgMsft
George Ng (GeorgeNgMsft) added this pull request to the merge queue Sep 2, 2026
Merged via the queue into main with commit ff77ee4 Sep 2, 2026
27 checks passed
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