emrg: evolution_prompt quick-ref — add #592/#593/#594 anti-storm fix entries - #595
Merged
Conversation
…e self-heal + #594 TUI spawn throttle entries
argszero
commented
Aug 9, 2026
argszero
left a comment
Owner
Author
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
commented
Aug 9, 2026
argszero
left a comment
Owner
Author
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.
Collaborator
argszero
commented
Aug 9, 2026
argszero
left a comment
Owner
Author
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.
argszero
added a commit
that referenced
this pull request
Aug 9, 2026
…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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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).