Repository navigation
Conversation
Forward stdin through worktree commands and finalize child state and budget reservations when executor startup fails. Retain recovery errors when process cleanup fails. Transfer internal results and native session artifacts independently of model file-read limits while preserving pinned workspace and regular-file checks. Cover real local, copied, Git worktree and Team candidate execution, large results, startup failures, path boundaries, descriptor cleanup and bridge compatibility.
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.
Member-authored workflows failed to start in worktree isolation because the environment did not forward the standard input payload. Startup exceptions could leave the child marked running with its budget reservation retained. Completed workflows could also fail delivery when their internal result or session artifact exceeded the model-facing 4 MiB file-read limit.
Forward binary stdin through WorktreeEnvironment and include executor creation in the existing child failure and cleanup handling. Transfer internal result and session files separately from model-facing reads. Local transfers reuse anchored regular-file access and stream the bytes, preserving path, symlink and descriptor protections. Existing Docker and VM transfers retain their checked routes, and ordinary file reads keep their original size limit.
The original regression set changed from eight failures and one pass to nine passes. Author validation passed 179 related cases, with one optional real-Docker test skipped. Independent review passed 198 cases with the same optional skip. Actual Local and Git worktree runs delivered results and session artifacts larger than 4 MiB, released reservations and ended their processes. Additional probes cover binary stdin, startup errors, cancellation, path escape, symlinks, FIFOs and descriptor cleanup. Repository Ruff, import constraints and dependency checks passed.