Repository navigation
feat: support Windows process entry points - #38
Merged
Merged
Conversation
SKTT1Ryze
marked this pull request as draft
October 22, 2026 09:57
- Read native Windows executable and argv metadata for WFP TLS policy matching. - Recognize WorkBuddyAI.exe and packaged daemon and extensionless CLI entries. - Add native/parser tests, a WFP TLS probe, and configuration documentation. - Validate 492 Windows tests, strict Clippy, formatting, and local WFP probes. Signed-off-by: chunchi.che@lexmount.com Co-Authored-By: GPT-6 <noreply@anthropic.com>
- Recognize WorkBuddy executable aliases directly in NodeEntryPoint and explain their Node execution mode. - Remove PackagedElectronEntryPoint and move its regression tests into the Node resolver. - Validate 126 broker tests, strict Clippy, and formatting. Signed-off-by: chunchi.che@lexmount.com Co-Authored-By: GPT-6 <noreply@anthropic.com>
- Move CommandBuffer into the Windows native process-query implementation. - Use a transparent MaybeUninit wrapper with stable aligned heap storage and explicit unsafe initialization conversion. - Preserve bounds validation and test partial initialization, invalid ranges, and ownership moves. - Validate 127 broker tests, strict Clippy, and formatting. Signed-off-by: chunchi.che@lexmount.com Co-Authored-By: GPT-6 <noreply@anthropic.com>
- Own native command-line storage in Arguments and expose borrowed UTF-16 slices through ArgumentsIter. - Decode arguments directly without intermediate per-argument copies. - Cover storage ownership, repeated iteration, unpaired surrogates, and argument count limits. - Validate 130 broker tests, strict Clippy, formatting, and a compile-fail lifetime check. Signed-off-by: chunchi.che@lexmount.com Co-Authored-By: GPT-6 <noreply@anthropic.com>
- Remove the arbitrary 4096-argument limit and construct Arguments directly from the native count. - Update documentation and verify that 4097 arguments are accepted. - Validate 130 broker tests, strict Clippy, and formatting. Signed-off-by: chunchi.che@lexmount.com Co-Authored-By: GPT-6 <noreply@anthropic.com>
- Replace manual module lookup and function-pointer conversion with a typed dlopen Container. - Initialize the process API once with LazyLock and retain its library for subsequent queries. - Add Windows-only dlopen dependencies and remove the unused LibraryLoader feature. - Validate 130 broker tests, including native process queries, strict Clippy, and formatting. Signed-off-by: chunchi.che@lexmount.com Co-Authored-By: GPT-6 <noreply@anthropic.com>
- Consume the UTF-16 Vec directly and append its NUL terminator without cloning the input. - Update argument readers and tests to transfer buffer ownership. - Replace the global process API cache with a local dlopen Container for each query. - Validate 130 broker tests, strict Clippy, and formatting. Signed-off-by: chunchi.che@lexmount.com Co-Authored-By: GPT-6 <noreply@anthropic.com>
- Verify argument storage survives consumed input vectors with either capacity. - Exercise command buffers at allocation boundaries and maximum UTF-16 length. - Run 512 concurrent native queries through repeated library and handle drops. Validation: 500 regression tests passed, along with Windows WFP end-to-end checks for daemon passthrough, CLI interception, and unmatched-entry passthrough. Signed-off-by: chunchi.che@lexmount.com Co-Authored-By: GPT-6 <noreply@anthropic.com>
SKTT1Ryze
marked this pull request as ready for review
October 22, 2026 09:59
SKTT1Ryze
force-pushed
the
codex/windows-process-entry-points
branch
from
October 22, 2026 10:00
64708e9 to
d1c9786
Compare
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
1 participant
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.
Signed-off-by: chunchi.che@lexmount.com