Skip to content

fix: forward stdin through npm launcher#23

Merged
cafitac merged 1 commit intomainfrom
fix/npm-stdin-forwarding
Apr 30, 2026
Merged

fix: forward stdin through npm launcher#23
cafitac merged 1 commit intomainfrom
fix/npm-stdin-forwarding

Conversation

@cafitac
Copy link
Copy Markdown
Owner

@cafitac cafitac commented Apr 30, 2026

Summary

  • Forward stdin from the npm launcher into the delegated Python CLI process.
  • Add a real Hermes hook regression test that proves node bin/agent-memory.js hermes-pre-llm-hook ... receives hook JSON from stdin.
  • Extend published install smoke to exercise stdin-dependent hermes-pre-llm-hook through npm exec, uvx, and pipx surfaces so future releases QA the Hermes hook path after publish.

Test Plan

  • uv run pytest tests/test_npm_launcher.py tests/test_published_install_smoke.py -q
  • uv run pytest tests/ -q
  • uv run python scripts/check_release_metadata.py
  • uv run python scripts/smoke_release_readiness.py
  • npm pack --dry-run
  • git diff --check
  • node --check bin/agent-memory.js
  • static diff scan: 0 findings

@cafitac cafitac merged commit 50e5d96 into main Apr 30, 2026
2 checks passed
@cafitac cafitac deleted the fix/npm-stdin-forwarding branch April 30, 2026 06:20
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