Repository navigation
feat(hooks): add Empryo session capture - #341
Merged
Merged
Conversation
- Route Empryo session logs to a new empryo host adapter as harness empryo - Stop sends the latest finished turn; SessionEnd sends every turn within the JSON cap - Skip the Claude transcript fallback, PreCompact flush, and in-session extraction for Empryo logs - Share payload building and the recall capture call in hooks/lib/hosts/capture-door.ts - Document the Empryo integration and remove the new hook files on uninstall
- Mark the omp Curated LoA tasks done with PR #340 - Add Empryo MCP registration, injection check, and hyphenated search bug
- Stop sends each tab's latest turn finished within 10 minutes of the newest - Stop no longer relies on SessionEnd, which the installer does not register - Run the Claude and Empryo Stop tests from a hooks copy without src - Drop source-grep assertions in favor of that run - Narrow Empryo skill docs: plugin installs do not expose skills to Empryo
…e stale duplicates
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What Changed
Risk Assessment
✅ Low: The adapter is bounded, reuses the existing capture boundary, and introduces no substantiated material defects in the reviewed paths.
Testing
Focused adapter tests, the production build, native Empryo capture/resume, database forwarding, adversarial hook checks, MCP retrieval, and lifecycle cleanup passed after resolving harness setup issues. CLI and persisted-state evidence is retained; disposable processes and data were removed. No rendered UI was changed or exercised.
Evidence: Product transcript
Evidence: Native Empryo commands and hook completion
~/.no-mistakes/evidence/01M5BB78BDJEKXV3P5K2S4HTTH/hook-commands.jsonl)Evidence: SessionEnd persisted memory
Evidence: Oversized capture persisted memory
Evidence: MCP search response
Evidence: Installation and uninstall transcript
Pipeline
Updates from git push no-mistakes
⏭️ **Intent** - skipped
Step was skipped.
✅ **Rebase** - passed
✅ No issues found.
🔧 **Review** - 1 issue found → auto-fixed ✅
docs/EMPRYO_INTEGRATION.md:26- Document hook environment forwarding for custom databases. With RECALL_DB_PATH=/data/recall.db set when launching Empryo and the same path configured for MCP, capture still falls back to ~/.agents/Recall/recall.db because Empryo filters custom hook environment variables unless explicitly forwarded. This either writes to another initialized database or loses capture when that database is absent. The affected path is hooks/RecallExtract.ts:1006 → hooks/lib/hosts/capture-door.ts:68; MCP independently receives the path configured here. Add setup and troubleshooting instructions for forwarding RECALL_DB_PATH (or legacy MEM_DB_PATH) through EMPRYO_HOOK_ENV. See Empryo’s hook environment contract.🔧 Fix applied.
✅ Re-checked - no issues remain.
✅ **Test** - passed
✅ No issues found.
bun install --frozen-lockfile --ignore-scriptsIsolatedbun test tests/hooks/empryo-capture.test.tsIsolatedbun run buildVerified the official Empryo 3.10.1-beta bundle signature and SHA-256 before running it locally.bun .local-test-empryo/native-final.tsbun .local-test-empryo/hook-e2e.tsbun .local-test-empryo/mcp-readback.tsbun .local-test-empryo/lifecycle-e2e.tsVerified disposable children had exited and removed test homes, databases, downloaded runtime, temporary drivers, and build output.✅ **Document** - passed
✅ No issues found.
✅ **Lint** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.