Skip to content

refactor: use Dynamo session identity#8

Merged
ishandhanani merged 2 commits into
mainfrom
idhanani/session-id
Jun 24, 2026
Merged

refactor: use Dynamo session identity#8
ishandhanani merged 2 commits into
mainfrom
idhanani/session-id

Conversation

@ishandhanani

@ishandhanani ishandhanani commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

Pi and Hermes now emit Dynamo's canonical session headers and session-based trace context with explicit subagent lineage. This keeps identity passive while aligning agent plugins with ai-dynamo/dynamo#10896.

How This Was Implemented

  • Renamed Pi headers, environment variables, trace fields, and public helpers from trajectory to session terminology.
  • Preserved root/child lineage through session_id and parent_session_id, including self-parent rejection for nested agents.
  • Removed Pi's generic session-affinity header switch so identity alone does not activate sticky routing.
  • Renamed the Hermes plugin and CI coverage to the session-only contract.
Walkthrough
  • pi-plugin/src/light/session.ts: derives root and child session identity from Pi and pi-subagents metadata.
  • pi-plugin/src/light/provider.ts: emits canonical Dynamo session headers without enabling affinity.
  • pi-plugin/src/light/tool-relay.ts: publishes session-based Dynamo tool-event context.
  • hermes-plugin: maps Hermes session_id to x-dynamo-session-id.

Validation

  • All completed test-plan items below passed locally.

Test plan

Signed-off-by: Ishan Dhanani <ishandhanani@gmail.com>
Signed-off-by: Ishan Dhanani <ishandhanani@gmail.com>
@ishandhanani ishandhanani force-pushed the idhanani/session-id branch from 56be5bb to d4400a3 Compare June 23, 2026 20:22
@ishandhanani ishandhanani marked this pull request as ready for review June 23, 2026 21:27
@ishandhanani ishandhanani merged commit 5f62703 into main Jun 24, 2026
1 of 2 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