fix: shutdown context + double IPC registration #36

Merged

Virgil merged 727 commits from fix/codex-review-findings into dev

2026-03-24 22:28:42 +00:00

Virgil commented

2026-03-24 22:28:30 +00:00

Member

Fixes Codex review findings 1-3.

Co-Authored-By: Virgil virgil@lethean.io

Fixes Codex review findings 1-3. Co-Authored-By: Virgil <virgil@lethean.io>

Virgil added 1 commit 2026-03-24 22:28:31 +00:00

fix: shutdown context, double IPC registration 95076be4b3

- Run() uses context.Background() for shutdown (c.context is cancelled)
- Stoppable closure uses context.Background() for OnShutdown
- WithService delegates HandleIPCEvents to RegisterService only

Fixes Codex review findings 1, 2, 3.

Co-Authored-By: Virgil <virgil@lethean.io>