Skip to content

feat: log session config on create#93

Merged
NathanFlurry merged 2 commits intomainfrom
feat-log-session-config-on-create
Feb 6, 2026
Merged

feat: log session config on create#93
NathanFlurry merged 2 commits intomainfrom
feat-log-session-config-on-create

Conversation

@NathanFlurry
Copy link
Member

Summary

  • Adds a session_created telemetry event that fires when a new session is created
  • Logs safe session config fields: agent, agent_mode, permission_mode, model, variant
  • Does not include workdir or any sensitive paths
  • Respects existing telemetry enabled/disabled flag (--no-telemetry)
  • Sends asynchronously via spawned task to avoid blocking session creation

Test plan

  • cargo check passes
  • Verify event reaches telemetry endpoint with correct fields
  • Verify no event is sent when --no-telemetry is set

@railway-app railway-app bot temporarily deployed to sandbox-agent / sandbox-agent-pr-93 February 6, 2026 07:12 Destroyed
@railway-app
Copy link

railway-app bot commented Feb 6, 2026

🚅 Deployed to the sandbox-agent-pr-93 environment in sandbox-agent

Service Status Web Updated (UTC)
website 😴 Sleeping (View Logs) Web Feb 6, 2026 at 7:52 am

@claude
Copy link

claude bot commented Feb 6, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@railway-app railway-app bot temporarily deployed to sandbox-agent / sandbox-agent-pr-93 February 6, 2026 07:45 Destroyed
Copy link
Member Author

NathanFlurry commented Feb 6, 2026


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge queue - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@NathanFlurry NathanFlurry merged commit 5dbfde5 into main Feb 6, 2026
4 of 5 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.

1 participant