Skip to content

Sdk session information#304

Open
sanjeed5 wants to merge 1 commit intofederated-sdk-release-candidatefrom
cursor/sdk-session-information-c088
Open

Sdk session information#304
sanjeed5 wants to merge 1 commit intofederated-sdk-release-candidatefrom
cursor/sdk-session-information-c088

Conversation

@sanjeed5
Copy link
Contributor

@sanjeed5 sanjeed5 commented Mar 6, 2026

Add guidance to the update-integration-example skill to print tracer.session_id for easier session dump retrieval.


Open in Web Open in Cursor 

Co-authored-by: Sanjeed <sanjeed5@users.noreply.github.com>
@cursor
Copy link

cursor bot commented Mar 6, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@snyk-io
Copy link

snyk-io bot commented Mar 6, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@claude
Copy link
Contributor

claude bot commented Mar 6, 2026

Claude finished @sanjeed5's task in 27s —— View job


Review of PR #304

  • Gather context and read changed files
  • Provide review feedback

LGTM — This is a clean, minimal documentation-only change to an internal skill file (.agents/skills/update-integration-example/SKILL.md). No SDK code is affected.

The PR adds two small additions to Step 4 (Capture Spans & Session Dump):

  1. A tip to print(tracer.session_id) after tracer init for easy retrieval (lines 126-129)
  2. A note to use the printed session ID as the <session_id> value in the session dump fetch (line 153)

Both additions are well-placed, clear, and improve the workflow for anyone following the skill instructions. No concerns.

@sanjeed5 sanjeed5 marked this pull request as ready for review March 10, 2026 11:13
@sanjeed5 sanjeed5 requested a review from a team March 10, 2026 11:13
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