Skip to content

Split Pi and Hermes Dynamo plugins#7

Merged
ishandhanani merged 5 commits into
mainfrom
idhanani/pi-hermes-plugin-layout
Jun 22, 2026
Merged

Split Pi and Hermes Dynamo plugins#7
ishandhanani merged 5 commits into
mainfrom
idhanani/pi-hermes-plugin-layout

Conversation

@ishandhanani

Copy link
Copy Markdown
Collaborator

This splits the repo into standalone pi-plugin/ and hermes-plugin/ packages. It keeps the Pi provider work isolated while adding the smallest Hermes middleware needed to stamp Dynamo trajectory headers from Hermes session_id.

How This Was Implemented

  • Moved the existing Pi provider package, tests, scripts, and skill under pi-plugin/.
  • Added hermes-plugin/ with one middleware hook that sets x-dynamo-trajectory-id when Hermes provides session_id.
  • Updated CI and repo docs to use the new plugin paths.
Walkthrough
  • pi-plugin/: existing Pi provider package and validation assets now live under one folder.
  • hermes-plugin/__init__.py: registers an llm_request middleware and preserves explicit Dynamo trajectory headers.
  • .github/workflows/integration-smoke.yml: points npm install and smoke script execution at pi-plugin/.
  • README.md / CLAUDE.md: document the two-plugin repo layout.

Validation

  • cd pi-plugin && npm run check
  • cd pi-plugin && npm test
  • cd pi-plugin && npm run build
  • python3 -m unittest discover -s hermes-plugin/tests
  • git diff --check

Signed-off-by: Ishan Dhanani <ishandhanani@gmail.com>
Signed-off-by: Ishan Dhanani <ishandhanani@gmail.com>
Signed-off-by: Ishan Dhanani <ishandhanani@gmail.com>
Signed-off-by: Ishan Dhanani <ishandhanani@gmail.com>
Signed-off-by: Ishan Dhanani <ishandhanani@gmail.com>
@ishandhanani ishandhanani marked this pull request as ready for review June 22, 2026 19:51
@ishandhanani ishandhanani merged commit bc42ed8 into main Jun 22, 2026
1 check passed
@ishandhanani ishandhanani deleted the idhanani/pi-hermes-plugin-layout branch June 22, 2026 19:51
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