fix: harden process termination and retain print task #67

Merged
gronod merged 1 commits from fix/process-wait-and-print-task into develop 2026-09-10 00:18:06 +01:00
Owner

Add waitUntilExit() fallbacks to ProcessManager.runStreaming and runCaptured so fast child exits cannot race past the terminationHandler. Keep the existing terminationHandler, 2-second forceKill/forceFinalize watchdog, parent write-end close, ARGYLL_NOT_INTERACTIVE=1, chartread pre-kill q\n + ~500 ms, killAll, and readDataToEndOfFile() output collection.

Retain TargetWorkflowViewModel.printTask across printAllPages / printPage so the unstructured Task is not dropped before it resumes.

Add two ProcessManagerTests covering fast-exit and stderr-only captured runs.

Target: develop
Branch: fix/process-wait-and-print-task

Add `waitUntilExit()` fallbacks to `ProcessManager.runStreaming` and `runCaptured` so fast child exits cannot race past the `terminationHandler`. Keep the existing `terminationHandler`, 2-second `forceKill`/`forceFinalize` watchdog, parent write-end close, `ARGYLL_NOT_INTERACTIVE=1`, chartread pre-kill `q\n` + ~500 ms, `killAll`, and `readDataToEndOfFile()` output collection. Retain `TargetWorkflowViewModel.printTask` across `printAllPages` / `printPage` so the unstructured `Task` is not dropped before it resumes. Add two `ProcessManagerTests` covering fast-exit and stderr-only captured runs. Target: `develop` Branch: `fix/process-wait-and-print-task`
gronod added 1 commit 2026-09-10 00:14:43 +01:00
fix: harden process termination and retain print task
macOS CI / build-and-test (push) Canceled after 0s
macOS CI / package (push) Canceled after 0s
1d9c6ae334
Add waitUntilExit() fallbacks to ProcessManager.runStreaming and
runCaptured so fast child exits cannot race past the terminationHandler.
Retain TargetWorkflowViewModel.printTask across printAllPages/printPage
to prevent the unstructured Task from being dropped before it resumes.
Add two ProcessManager tests covering fast-exit and stderr-only
captured runs.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
gronod added the
Priority
Critical
1
Project/ICCery-v2Bug/Architecture
labels 2026-09-10 00:14:52 +01:00
gronod merged commit 18767c2212 into develop 2026-09-10 00:18:06 +01:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: gronod/iccery-v2-mac#67