Skip to content

fix: align Hermes restart UX#1586

Merged
hijzy merged 1 commit into
MemTensor:mem-agent-0424from
hijzy:fix/memos-local-plugin-hermes-restart
Apr 29, 2026
Merged

fix: align Hermes restart UX#1586
hijzy merged 1 commit into
MemTensor:mem-agent-0424from
hijzy:fix/memos-local-plugin-hermes-restart

Conversation

@hijzy
Copy link
Copy Markdown
Collaborator

@hijzy hijzy commented Apr 29, 2026

Summary

  • Keep the Hermes Memory Viewer daemon alive during config-save restart and terminate only the active hermes chat process.
  • Remove the passive settings-saved restart card path so config saves always show the blocking full-screen restart overlay.
  • Filter Hermes internal memory/skill review prompts so they are not persisted as user turns.

Test plan

  • /Users/jiang/MyProject/MemOS-jiang/.venv/bin/ruff check apps/memos-local-plugin/adapters/hermes/memos_provider/__init__.py apps/memos-local-plugin/tests/python/test_hermes_provider_pipeline.py
  • /Users/jiang/MyProject/MemOS-jiang/.venv/bin/ruff format --check apps/memos-local-plugin/adapters/hermes/memos_provider/__init__.py apps/memos-local-plugin/tests/python/test_hermes_provider_pipeline.py
  • /Users/jiang/MyProject/MemOS-jiang/.venv/bin/python -m unittest apps.memos-local-plugin.tests.python.test_hermes_provider_pipeline
  • npx tsc --noEmit --target ES2022 --module NodeNext --moduleResolution NodeNext --esModuleInterop --skipLibCheck server/routes/admin.ts
  • npm run build:web

Keep the Memory Viewer daemon alive when Hermes config changes, close only the active Hermes chat process, and remove the passive settings-saved restart card so the operator always sees the blocking restart overlay.
@hijzy hijzy merged commit 978161d into MemTensor:mem-agent-0424 Apr 29, 2026
@hijzy hijzy deleted the fix/memos-local-plugin-hermes-restart branch May 8, 2026 03:44
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