Skip to content

ci(windows): compile the native workspace for ready pull requests - #4358

Merged
ScriptedAlchemy merged 17 commits into
masterfrom
copilot/replace-ci-windows-cargo-check
Oct 22, 2026
Merged

ScriptedAlchemy merged 17 commits into
masterfrom
copilot/replace-ci-windows-cargo-check

Conversation

Copilot AI commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Ready pull requests and pushes build every Bazel workspace target on native Windows, covering cfg(windows) code through the workspace build authority. Draft pull requests skip this heavy lane. Full Windows tests remain opt-in through workflow_dispatch with run_os=true.

The native Windows lane replaces the Linux Cargo cross-check and its Cargo/MinGW-only setup. The job retains the required Test Windows check name so branch protection recognizes its result.

Validation on 90e5390c5c10e2e172c6b4896a5404bc6da77f67: full hosted CI passed, including the native Windows workspace build under Test Windows, Linux fleet and aggregate, Clippy, shipped CLI, repository gates, dashboard, feature gates, and benchmark self-tests. The dispatch-only Windows test step was skipped as intended; the native workspace build executed successfully. Merge whitespace and commit-message checks passed.

Fixes #4324

@changeset-bot

changeset-bot Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 90e5390

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI linked an issue Oct 19, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Replace CI's Windows cargo check with Linux-hosted Bazel check Replace Windows Cargo check with Bazel CI Oct 19, 2026
Copilot AI requested a review from ScriptedAlchemy October 19, 2026 04:11
Copilot AI and others added 4 commits October 20, 2026 15:41
Co-authored-by: ScriptedAlchemy <25274700+ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: ScriptedAlchemy <25274700+ScriptedAlchemy@users.noreply.github.com>
@ScriptedAlchemy
ScriptedAlchemy force-pushed the copilot/replace-ci-windows-cargo-check branch from aa3d292 to b2bb1a6 Compare October 20, 2026 22:42
@ScriptedAlchemy
ScriptedAlchemy marked this pull request as ready for review October 21, 2026 06:27
devin-ai-integration[bot]

This comment was marked as resolved.

@ScriptedAlchemy ScriptedAlchemy left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pushed d74895d. Windows Bazel builds now honor the existing run-heavy scope gate, so draft PRs retain the lightweight path. Ready PRs and pushes compile the native Windows workspace; full Windows tests remain available through an explicit run_os dispatch. The workflow YAML parses, and diff/commitlint checks pass. Native Windows compilation remains assigned to CI.

@ScriptedAlchemy ScriptedAlchemy left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pushed ff2342a after the native CI failure reported four agent-hosts journeys timing out at their initial code-index generation.

The failing mount reaches a concrete pre-existing text-build admission defect: it added the watermark margin to the ledger reservation and then shrank it, so a build fitting the actual headroom could be refused by the larger temporary charge. This commit reuses the reviewed #4393 implementation that sizes and charges the build atomically under the canonical ledger, plus the cgroup-headroom regression already reviewed on #4379. It retains the existing ceiling, bounded reclamation, pressure/wakeup rules, and reservation owner.

The journey helper still uses its five-second deadline, 10 ms cadence, and original generation-ready condition. It now captures bounded park, worker, pressure, and ledger evidence inside that same deadline and signals the existing scheduler's cancellation on failure. Reporting after timeout performs no new readiness waits or diagnostic reads.

Validation: the transplanted production files match #4393 byte-for-byte; the added cgroup regression matches #4379. Independent source review, pinned rustfmt, diff checks, and commitlint pass. Native behavior remains for fresh CI. The old timeout log did not include enough admission state to prove this defect caused all four failures, so the diagnostic evidence remains necessary.

@ScriptedAlchemy
ScriptedAlchemy enabled auto-merge (squash) October 22, 2026 02:53
@ScriptedAlchemy ScriptedAlchemy changed the title Replace Windows Cargo check with Bazel CI ci(windows): compile the native workspace for ready pull requests Oct 22, 2026
@ScriptedAlchemy
ScriptedAlchemy merged commit e6a7bc9 into master Oct 22, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace CI's Windows cargo check with a Linux-hosted Windows Bazel check

2 participants