fix: add context_enhancer symlink step and Mandatory Pre-Action Protocol#25
Merged
Merged
Conversation
setup.sh: - Phase 5b: symlink context_enhancer.py to Hermes scripts dir (fixes silent ModuleNotFoundError that disabled [qdrant] injection) - Phase 9: auto-apply execution-agent-protocol.md modifications/soul-rulebook.md: - New section: Mandatory Pre-Action Protocol (4-step enforcement) modifications/execution-agent-protocol.md: - Standalone protocol for automated application by setup.sh layers/07-ground-truth.md: - Document behavioral gap between knowing and executing Ground Truth rules
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
1 participant
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.
Two fixes discovered in production:
1. Qdrant injection silently broken since 2026-05-29
hooks.pyimportsfrom scripts.context_enhancer import ...which resolves to~/.hermes/scripts/. The file was never placed there — it lived in the Memory OS repo. Theexcept Exception: return []pattern swallowed theModuleNotFoundError, and[qdrant]context was never injected.Fix: setup.sh Phase 5b creates a symlink from the Memory OS repo to the Hermes scripts directory.
2. Ground Truth hierarchy correct but behaviorally inert
The hierarchy has been documented since 2026-05-31. The agent reads it every turn. Under time pressure, it still defaults to "resolve fast" instead of checking injected context first.
Fix: Mandatory Pre-Action Protocol — a 4-step mechanical sequence (inventory → match → use/declare → act) that runs before every tool call. Added to rulebook.md and to the Memory OS documentation.
Changes
setup.shmodifications/soul-rulebook.mdmodifications/execution-agent-protocol.mdlayers/07-ground-truth.md