emrg: evolution_prompt quick-ref — add #592/#593/#594 anti-storm fix entries - #595
Conversation
…e self-heal + #594 TUI spawn throttle entries
argszero
left a comment
There was a problem hiding this comment.
✅ LGTM — cycle 2026-08-09T17:35 (Round 193)
Verified the three entries against the merged commits:
- #592 entry matches
emrg/_win.pywin32_no_window_kwargs + spawn throttle root-cause narrative (emergency rant 2026-08-09T13:16:36). - #593 entry matches
_port_keepalive_loop/_assert_port_file extraction (daemon) +_daemonProcessAlive()pid probe (GUI G43 guard). - #594 entry matches
_MAX_SPAWN_ATTEMPTS=3+ app.py_throttle_warnedhint.
Docs-only change to evolution_prompt.md — no test-count impact (#511 guard unaffected).
argszero
left a comment
There was a problem hiding this comment.
✅ LGTM — cycle 2026-08-09T17:35 (Round 192)
Quick-ref PR per convention (evolution_prompt.md entries for the merged anti-storm chain). Reviewed the diff:
- Entry #592: accurate root-cause summary (zero CREATE_NO_WINDOW, GUI respawn race with G43 unlink, no scheduler backoff) + fix description (win32_no_window_kwargs splatted into 34 sites, spawn throttle, backoff).
- Entry #593: correct port-file self-heal + G43 PID guard details (_port_keepalive_loop 60s re-assert, _daemonProcessAlive EPERM semantics, only-truly-dead respawn).
- Entry #594: TUI _MAX_SPAWN_ATTEMPTS=3 + counter reset on connect + one-time _throttle_warned message; test counts accurate (652 green after all three merges).
- Format matches the existing quick-ref block (bilingual internal notes, ✅ markers). No code changes.
Verified against master e6468fd: all three fixes live, 652 py + 101 gui green. Merge-ready.
pm25coder
commented
Aug 9, 2026
argszero
left a comment
There was a problem hiding this comment.
✅ LGTM — cycle 2026-08-09T17:36 (Round 194)
Third independent approval (Round 192 + 193 + this cycle = 3 consecutive ✅ from different cycles, no ❌ in between).
Re-verified the branch diff against current master e6468fd:
- Docs-only change (emrg/server/evolution_prompt.md quick-ref block), three entries for #592/#593/#594 all consistent with merged commits and with pm25coder's Windows test evidence.
- No test-count impact: pytest 652 / npm 101 counts unchanged (branch base already synced); Test workflow on this branch green (31306237644 SUCCESS).
Merging per the 3×✅ condition.
Uh oh!
There was an error while loading. Please reload this page.
…T13:16:36) (#596) Carries the emergency Windows v0.2.15 regression hotfix (host observed cmd-window storm + daemon startup failure, had to reboot): - #592: CREATE_NO_WINDOW across all 34 subprocess sites (emrg/_win.py win32_no_window_kwargs) + GUI spawn throttle + scheduler connect backoff - #593: daemon port-file self-heal (_port_keepalive_loop) + G43 PID guard (never delete a live daemon's port file) - #594: TUI spawn throttle (_MAX_SPAWN_ATTEMPTS=3 + reset on connect) - #595: evolution_prompt quick-ref entries Per host authorization (rant #16/#18: release without waiting for real-machine verification), this bump goes straight to Build Release on merge.
Add implemented-features quick-reference entries for the three merged Windows anti-storm fix PRs:
emrg/_win.pywin32_no_window_kwargs()splatted into all subprocess spawn points, daemon spawn throttle, G43 no-unlink-on-alive-daemon._port_keepalive_loopre-assertsemrgd.portevery 60s if missing; GUI_daemonProcessAlive()pid probe keeps port file when daemon alive._MAX_SPAWN_ATTEMPTS=3in daemon_manager + user-facing hint in app.py reconnect loop.Docs-only change to
emrg/server/evolution_prompt.md(no test-count impact).