🤖 Fix zombie processes via explicit resource management

[ammar-agent]

Prevents zombie processes from exec() calls by wrapping them in a disposable resource that guarantees cleanup via TypeScript's using declarations.

Core Solution

All git operations now use explicit resource management:

using proc = execAsync("git status");
const { stdout } = await proc.result;
// Process is automatically cleaned up when scope exits

Key implementation details:

• Uses close event instead of exit to ensure all stdio streams are fully flushed before resolving
• Only kills processes that haven't exited naturally (checks exitCode and signalCode)
• Rejects when process is killed by signal (e.g., SIGTERM)

Additional Fixes

• Added .unref() to detached terminal spawns on macOS/Windows
• ESLint rule prevents future promisify(exec) usage
• Centralized in src/utils/disposableExec.ts to avoid duplication

Testing

• ✅ All unit tests pass (418 tests)
• ✅ All integration tests pass (74 tests)
• ✅ All E2E tests pass
• ✅ ESLint rule active

Generated with cmux

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.