Skip to content

fix(runtime): helper-lifecycle follow-ups (#666, #667, #668) and a detached-reap regression #669

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ndycode merged 18 commits into from
Aug 13, 2026
+3,908 −271
Merged

fix(runtime): helper-lifecycle follow-ups (#666, #667, #668) and a detached-reap regression #669

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
52d92c1
fix(codex): stop runtime helpers from leaking past their idle timeout
possibilities Aug 11, 2026
f5bf873
fix(codex): address CodeRabbit round 1 on the helper-leak fix
possibilities Aug 11, 2026
1590cd1
test(codex): pin the sweep-retry test to the four-attempt budget, ord…
possibilities Aug 11, 2026
6bb2049
fix(codex): reap app helpers stranded by the detach grace
possibilities Aug 11, 2026
c18a5df
fix(codex): degrade an unreadable connection count to zero
possibilities Aug 11, 2026
c03af14
fix(codex): correct the connection-count comment and cover the reap o…
possibilities Aug 11, 2026
e57da41
fix(codex): accept only a positive socket count as evidence of a cons…
possibilities Aug 11, 2026
16a335f
test: add owned-PID probes for helper-lifecycle fixtures (#668)
ndycode Aug 13, 2026
bf0a749
fix(codex): reap only helpers that were never handed to a consumer
ndycode Aug 13, 2026
040e0c1
fix(runtime): stop unbind from orphaning helper owner files (#666)
ndycode Aug 13, 2026
7c10722
refactor(runtime): one identity-checked helper selector for both read…
ndycode Aug 13, 2026
109db04
docs: correct the runtime app helper status path in the README
ndycode Aug 13, 2026
51c5ca4
refactor(runtime): narrow the helper PID instead of casting it
ndycode Aug 13, 2026
c42bf54
fix(runtime): address the review on the helper-lifecycle follow-ups
ndycode Aug 13, 2026
e087c43
test: stress the helper lifecycle at the scale the leak report described
ndycode Aug 13, 2026
8f0b578
test: fix the review round on the stress suite
ndycode Aug 13, 2026
6b9f327
test: make the spawn-failure coverage exercise the helpers, not node
ndycode Aug 13, 2026
02e0c9b
test: cover the failed-spawn branch in withLivePid
ndycode Aug 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
docs: correct the runtime app helper status path in the README 
#664 moved helper status files to `runtime-rotation-app-helper.<pid>.json`
and updated `docs/reference/storage-paths.md`, but the storage table in
the README kept the pre-per-PID shared name.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015b4Lew3oHNEYmTZtg7zEWz
  • Loading branch information
@ndycode @claude
ndycode and claude committed Aug 13, 2026
commit 109db04ea0fd765ccb4ce45ab04f727b269e1674
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,7 @@ For remote or headless shells, prefer `codex-multi-auth login --device-auth`.
| Routing profiles | `~/.codex/multi-auth/routing-profiles.json` |
| Budget guards | `~/.codex/multi-auth/budget-guards.json` |
| Local client tokens | `~/.codex/multi-auth/local-client-tokens.json` |
| Runtime app helper status | `~/.codex/multi-auth/runtime-rotation-app-helper.json` |
| Runtime app helper status | `~/.codex/multi-auth/runtime-rotation-app-helper.<pid>.json` |
| Runtime app helper owner metadata | `~/.codex/multi-auth/runtime-rotation-app-helper-owner.<pid>.json` |
| Persistent app bind state/logs | `~/.codex/multi-auth/app-bind/` |
| Logs | `~/.codex/multi-auth/logs/codex-plugin/` |
Expand Down