Repository navigation
Refine Windows dev setup and add independent fork CI - #1
Merged
Merged
Conversation
DangerMouseUK
changed the title
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.
The Windows development script installs and verifies the repository's prerequisites without building Codex or changing Rust's global default. It supports Windows PowerShell 5.1 and PowerShell 7, native x64/ARM64 MSVC tools, and the repository's Rust and pnpm versions.
-CheckOnlyprevents implicit Rust downloads and persistent changes.Add independent fork CI on standard GitHub-hosted Linux, Windows and macOS runners. Every run checks repository policies, formatting, dependency advisories, spelling, mocked Windows setup installers, and native shell/path/PTY Rust packages. Rust source/build changes expand coverage to the full native workspace; SDK changes enable TypeScript and Python SDK checks against a Cargo-built CLI using checksum-verified public V8 artifacts. A single
Fork CI requiredgate rejects failures, cancellations and unexpected skips. OpenAI's infrastructure-dependent workflows are confined to upstream.Fix the stale Cargo manifest exception, normalize manifest paths on Windows, read Rust source as UTF-8 in the boundary checker, allow valid technical spelling tokens, and align the SDK formatter test with the existing Cargo formatter. Preserve successful SDK build caches when later tests fail. Fork CI uses Cargo shear's standard warning policy; unused required dependencies still fail.
Validation: GitHub Actions passes all 14 checks at
cb5b5f37c7. Pester passes 36 tests in each PowerShell version. Native Rust passes 263 tests on Linux, 207 on Windows and 254 on macOS. The TypeScript SDK passes 49 tests; Python passes 267 with 38 skipped. Repository policies, actionlint, formatting, dependency advisories and the required gate all pass. Locally, the eight coverage/gate tests and corrected formatter test pass; the worktree and whitespace checks are clean.Fresh-machine setup, native ARM64 installation and the full native Rust workspace have not been exercised for this PR. The fork suite does not replace upstream Bazel, source-built V8, cross-compilation or release/signing coverage.