feat(executor): add scoped directory helper - #676
Merged
Merged
Conversation
SaladDay
marked this pull request as ready for review
September 17, 2026 08:42
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.
The pinned native filesystem walk re-resolves pathnames during enumeration and collects directory names before applying its entry cap. It cannot establish the directory isolation required by the pending Files consumer.
Add a separately named, one-shot directory helper beside the existing executor launcher. It traverses directories without following symlinks, enumerates and stats through retained descriptors, applies the entry bound during scanning, and returns explicit truncation or a settled error. The native process API supplies transport and sandboxing. Installation and workspace authority remain the caller's responsibility.
This PR delivers only the helper prerequisite and its acceptance fixture. It does not enable the Codex adapter or public Files, modify Core/schema, patch stock native filesystem behavior, or change the model/tool loop. PR #675 remains draft and blocked pending safe adapter integration.
Validation:
make checkpassed in 517.87 seconds at8907c9dc; fresh independent whole-diff review found no actionable issues in all 11 changed files.Evidence:
~/.parsar/remediation/20260917/native-directory-isolation/on zju_a100_2. Later adapter/public integration still requires real-provider execution, fixed SDK and raw HTTP acceptance. No complete compatibility claim.