Skip to content

diag(TEMP): stderr trace span IDs through chat.message + session.idle#79

Merged
Alezander9 merged 1 commit into
mainfrom
alex/v4-bcode-laminar-trace-spans
May 17, 2026
Merged

diag(TEMP): stderr trace span IDs through chat.message + session.idle#79
Alezander9 merged 1 commit into
mainfrom
alex/v4-bcode-laminar-trace-spans

Conversation

@Alezander9
Copy link
Copy Markdown
Member

@Alezander9 Alezander9 commented May 17, 2026

rc4 trace inspection still showed orphan turn parent despite the new async shutdown hook + forceFlush running cleanly (forceFlush done in 0ms). Need granular instrumentation: which span did chat.message create? did session.idle find and end it? did session.idle's forceFlush actually export it? Adds stderr writes covering both branches. Will be removed in the cleanup commit before v0.1.8 proper.


Summary by cubic

Add temporary stderr logs in bcode-laminar to trace turn span IDs across chat.message, session.idle, and server.instance.disposed. This helps debug orphan turn parent spans and confirm forceFlush/export behavior.

  • New Features
    • chat.message: log created turn span ID and session; log skip when subagent or an open span exists.
    • session.idle: log ending span ID and session; time forceFlush; log errors; log when no span exists and still forceFlush.
    • server.instance.disposed: log count of open turn spans ended.
    • Temporary diagnostics; will be removed before v0.1.8.

Written for commit 1b58d8b. Summary will update on new commits. Review in cubic

rc4 trace still shows orphan turn parent despite shutdown hook running cleanly. Need to confirm chat.message fires + which span ID it creates, and whether session.idle's forceFlush actually exports it. Add stderr lines covering both branches plus their span IDs. To be removed with the rest of the diagnostics before v0.1.8 proper.
@Alezander9 Alezander9 merged commit 1b18379 into main May 17, 2026
@Alezander9 Alezander9 deleted the alex/v4-bcode-laminar-trace-spans branch May 17, 2026 18:24
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