fix: align repo with MCP Split — register as_brain for Zero + update stale tests/docs#1
Merged
Merged
Conversation
…stale tests/docs - cli.py install-zero now registers BOTH 'eling' (Notion) and 'as_brain' (local brain) servers in Zero config. Previously only the Notion-only 'eling' server was added, so Zero never received its local memory brain (the whole point of the v0.7.3 MCP Split). - tests/test_think.py + test_verify_on_stop.py: tools moved to eling.as_brain.mcp_server as brain_*/brain_verify in the split; updated assertions to match. - README: architecture tree + verify-on-stop section reflect split (mcp_server.py = 5 Notion tools, as_brain/mcp_server.py = 20 brain_* tools). Full suite: 409 passed.
…erride + all-agents verify - Gap #1: as_brain captures MCP clientInfo.name from initialize handshake and uses it as the default source for brain_remember (auto agent attribution) - Gap #2: as_brain resolves ELING_HOME explicitly and passes it to Brain(home=) - Gap #3: ELING_VERIFY_ALL_AGENTS=1 opens verify-on-stop to every agent incl. Hermes - Add tests/test_universal_brain.py (13 tests) covering all three gaps - Update README / API.md / ARCHITECTURE.md / RELEASE_NOTES.md for the universal brain - Bump version to 0.8.0
PatrickNoFilter
added a commit
that referenced
this pull request
Jul 8, 2026
Merges fix/mcp-split-zerobrain-tests including v0.8.0 universal brain changes.
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.
Summary
The v0.7.3 MCP Split moved local-brain tools into
eling.as_brain.mcp_server(asbrain_*) and madeeling.mcp_serverNotion-only (5eling_*tools). The repo was left out of sync with that split in three places.Bug — Zero gets no brain (the whole point of the split)
cli.pyinstall-zeroonly registered theelingserver in Zero'sconfig.json. Zero therefore never received the localas_brainmemory server. Now it registers both servers (idempotent — only adds what's missing), so Zero actually gets its brain.Stale tests
tests/test_think.pyassertedeling_thinkexisted ineling.mcp_server→ now assertsbrain_thinkineling.as_brain.mcp_server.tests/test_verify_on_stop.pyassertedeling_verify→ nowbrain_verify.Stale docs
mcp_server.py — 22 tools(unified) → now reflects split (Notion 5 + brain 20).eling_verifyMCP examples →brain_verify(as_brain).Test plan
pytestfull suite: 409 passed (was 4 failing on stale assertions).eling install-zero --zero-config-dir /tmp/xverified to write botheling+as_braininto Zero config.eling.as_brain.mcp_serverstdio smoke-tested: 20brain_*tools, reads~/.eling(249 facts).Note for Hermes wiring
I separately added the
as_brainMCP server to~/.hermes/config.yaml(alongside the existingelingserver) so Hermes keeps its local-brain tools after the split. That change lives in local Hermes config, not this repo.