Skip to content

fix(console): preserve inference sessions - #43124

Merged
fwang merged 4 commits into
devfrom
r2-session-ids
Aug 18, 2026
Merged

fix(console): preserve inference sessions#43124
fwang merged 4 commits into
devfrom
r2-session-ids

Conversation

@adamdotdevin

Copy link
Copy Markdown
Member

Summary

  • preserve the validated OpenCode session header when the legacy Zen route forwards to the managed inference gateway
  • continue stripping the session and other OpenCode metadata from direct provider requests
  • cover managed inference, direct provider, and empty-session behavior

Root cause

The Zen handler captured the session for legacy metrics, then unconditionally removed the header before forwarding. Requests routed through managed inference therefore emitted canonical generation events without a session ID, leaving the R2 generation table mostly null after the live-source handoff.

Impact

New managed-inference generation events retain their OpenCode session ID, allowing R2-backed stats to calculate session counts and per-session metrics correctly after deployment.

Validation

  • bun test test/providerHeaders.test.ts
  • bun typecheck from packages/console/app
  • repository pre-push typecheck: 30 packages passed

@adamdotdevin
adamdotdevin marked this pull request as ready for review August 17, 2026 23:31
@fwang
fwang merged commit 4e81a0b into dev Aug 18, 2026
5 checks passed
@fwang
fwang deleted the r2-session-ids branch August 18, 2026 04:48
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