airkjw/claude-mem
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MAESTRO: Close PR #738 - parallel subprocess tracking system duplicates ProcessRegistry

Browse Source 
PR #738's QueryWrapper/QueryWrapperManager introduces a separate process tracking
system alongside the existing ProcessRegistry. Its orphan cleanup duplicates
killSystemOrphans() and killIdleDaemonChildren() already in main.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Alex Newman
2026-02-05 20:13:26 -05:00
parent 7be19b20cd
commit ad37629f0a
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Auto Run Docs/PR-Triage/PR-Triage-06.md
+2 -1
View File
@@ -17,7 +17,8 @@ These PRs address orphaned/zombie processes. This is a recurring theme — v9.0.
- [x] Review PR #847 (`fix: comprehensive observer session isolation without breaking auth` by @bigph00t). Files: ProcessRegistry.ts, SDKAgent.ts, EnvManager.ts, paths.ts, etc. (7 files). Steps: (1) `gh pr checkout 847` (2) v9.0.12 already fixed observer isolation (using SDK `cwd` option instead of CLAUDE_CONFIG_DIR). Check if this PR provides additional isolation beyond v9.0.12. (3) If it's substantially the same as what shipped in v9.0.12, close with: `gh pr close 847 --comment "Observer session isolation was addressed in v9.0.12 using SDK cwd option. If there are remaining isolation gaps, please describe the specific scenario and reopen."` (4) If it addresses real remaining gaps, review and merge.
> **Result: CLOSED** — All changes from this comprehensive PR were already shipped incrementally via three merged PRs: PR #845 (v9.0.12, `cwd` isolation replacing `CLAUDE_CONFIG_DIR`), PR #733 (`EnvManager.ts` with `buildIsolatedEnv()` for credential isolation), and the `CLAUDE_MEM_CLAUDE_AUTH_METHOD` setting. Current main branch `SDKAgent.ts` already uses both `cwd: OBSERVER_SESSIONS_DIR` and `env: isolatedEnv`. Closed with detailed explanation.
- [ ] Review PR #738 (`fix(worker): add subprocess lifecycle cleanup for zombie haiku agents` by @fedosov). Files: 8 source files + tests. Adds QueryWrapper/QueryWrapperManager for subprocess lifecycle management. Steps: (1) `gh pr checkout 738` (2) Check overlap with ProcessRegistry (v9.0.8) — does this add a parallel tracking system? That would be redundant. (3) If it extends ProcessRegistry, good. If it introduces a separate system, close and suggest integrating with ProcessRegistry. (4) Run `npm run build` and tests.
- [x] Review PR #738 (`fix(worker): add subprocess lifecycle cleanup for zombie haiku agents` by @fedosov). Files: 8 source files + tests. Adds QueryWrapper/QueryWrapperManager for subprocess lifecycle management. Steps: (1) `gh pr checkout 738` (2) Check overlap with ProcessRegistry (v9.0.8) — does this add a parallel tracking system? That would be redundant. (3) If it extends ProcessRegistry, good. If it introduces a separate system, close and suggest integrating with ProcessRegistry. (4) Run `npm run build` and tests.
> **Result: CLOSED** — Introduces a parallel process tracking system (QueryWrapper/QueryWrapperManager, 413 lines) with a separate `activeQueries` Map alongside the existing `processRegistry` Map. The orphan cleanup function (`cleanupOrphanedClaudeSubprocesses`) duplicates two existing functions: `killSystemOrphans()` (ppid=1 claude+haiku detection) and `killIdleDaemonChildren()` (idle daemon child cleanup, merged via PR #879). The periodic 15-minute cleanup is already covered by the 5-minute orphan reaper. Suggested the author contribute Windows orphan detection as a small extension to ProcessRegistry instead.
- [ ] Review PR #713 (`fix: prevent SDK subprocess accumulation` by @cjpeterein). Files: 17 files (very large PR). Steps: (1) `gh pr checkout 713` (2) This is a large PR touching many files — check how much overlaps with v9.0.8 ProcessRegistry work (3) Large PRs on process management are risky. If substantial overlap exists with shipped code, close with explanation. (4) If unique value remains, request the author to rebase and reduce scope.