Skip to content

fix(memos-local-plugin): serialize Hermes viewer daemon startup#1773

Merged
hijzy merged 1 commit into
MemTensor:mem-agent-0514from
hijzy:agent/fix-hermes-daemon-port-race
May 20, 2026
Merged

fix(memos-local-plugin): serialize Hermes viewer daemon startup#1773
hijzy merged 1 commit into
MemTensor:mem-agent-0514from
hijzy:agent/fix-hermes-daemon-port-race

Conversation

@hijzy
Copy link
Copy Markdown
Collaborator

@hijzy hijzy commented May 20, 2026

Summary

  • add a cross-process startup lock around the Hermes viewer daemon launch path
  • re-probe :18800 after acquiring the lock so concurrent no-viewer sessions reuse the daemon
  • keep --no-viewer stdio bridges out of the viewer daemon pid file

Tests

  • /Users/jiang/MyProject/MemOS-jiang/.venv/bin/ruff format apps/memos-local-plugin/adapters/hermes/memos_provider/daemon_manager.py apps/memos-local-plugin/tests/python/test_bridge_client.py
  • /Users/jiang/MyProject/MemOS-jiang/.venv/bin/ruff check apps/memos-local-plugin/adapters/hermes/memos_provider/daemon_manager.py apps/memos-local-plugin/tests/python/test_bridge_client.py
  • python3 -m unittest tests.python.test_bridge_client
  • npm run lint
  • npx vitest run tests/unit/bridge/methods.test.ts tests/unit/bridge/stdio.test.ts

@hijzy hijzy merged commit 3942ef3 into MemTensor:mem-agent-0514 May 20, 2026
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