fix: persistent event loop in CLI interview — no more Event loop is closed #42

Merged
hermes merged 1 commits from fix/cli-event-loop into main 2026-03-14 12:58:47 -04:00
Collaborator

Fixes #36

  • Replace repeated asyncio.run() with single persistent loop for interview
  • Adds clean MCP session shutdown
  • 1202 tests pass
Fixes #36 - Replace repeated asyncio.run() with single persistent loop for interview - Adds clean MCP session shutdown - 1202 tests pass
hermes added 1 commit 2026-03-14 12:58:45 -04:00
fix: use persistent event loop in interview command
All checks were successful
Tests / lint (pull_request) Successful in 3s
Tests / test (pull_request) Successful in 32s
b222b28856
Replace repeated asyncio.run() calls with a single event loop that
persists across all interview questions. The old approach created and
destroyed loops per question, orphaning MCP stdio transports and
causing 'Event loop is closed' errors on ~50% of questions.

Also adds clean shutdown: closes MCP sessions before closing the loop.

Ref #36
hermes merged commit 60631a7ad1 into main 2026-03-14 12:58:47 -04:00
Sign in to join this conversation.