feat(gateway,cli): windows compilation support by araza008 · Pull Request #2496 · NVIDIA/OpenShell

araza008 · 2026-07-27T14:43:51Z

Summary

compile the OpenShell gateway and CLI natively on Windows MSVC for x86-64 and arm64. It is build support only. It does not add functional Windows support for MXC, the sandbox supervisor, or Windows services.

Related Issue

Changes

adds windows mise tasks
supports bundled-z3 with optional system z3
gates unix-only dependencies so the geteway and cli compile on windows
adds linting and pre-commit tasks

Testing

mise run --skip-tools windows:build:x64
mise run --skip-tools windows:build:arm64

Checklist

Follows Conventional Commits
Commits are signed off (DCO)
Architecture docs updated (if applicable)

copy-pr-bot · 2026-07-27T14:43:55Z

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

github-actions · 2026-07-27T14:44:02Z

All contributors have signed the DCO ✍️ ✅
_{Posted by the DCO Assistant Lite bot.}

araza008 · 2026-07-27T17:33:14Z

I have read the DCO document and I hereby sign the DCO.

jamieknvidia · 2026-07-27T23:12:18Z

I have read the DCO document and I hereby sign the DCO.

drew · 2026-07-28T05:03:59Z

/ok to test d0bcaca

gburachas · 2026-07-28T16:14:00Z

I have read the DCO document and I hereby sign the DCO.

araza008 · 2026-07-29T16:04:08Z

/ok to test 15c622a

araza008 · 2026-07-29T16:55:59Z

/ok to test 7c96e78

drew · 2026-07-29T20:29:00Z

/ok to test 7c96e78

araza008 · 2026-07-29T21:47:34Z

/ok to test 923ecf0

araza008 · 2026-07-31T03:46:12Z

/ok to test c1d5e98

araza008 · 2026-07-31T05:08:32Z

/ok to test d1fa290

pimlock · 2026-07-31T22:32:02Z

/ok to test d1fa290

araza008 · 2026-08-04T08:59:36Z

/ok to test 60c2340

pimlock · 2026-08-04T23:05:26Z

/ok to test 97ce1f3

Signed-off-by: Shailendra Singh <shailendras@nvidia.com> Signed-off-by: Akber Raza <akberr@nvidia.com>

Signed-off-by: Akber Raza <akberr@nvidia.com>

Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>

Signed-off-by: Akber Raza <akberr@nvidia.com>

araza008 · 2026-08-06T02:12:35Z

/ok to test b2e23e0

Signed-off-by: Piotr Mlocek <1116309+pimlock@users.noreply.github.com>

pimlock · 2026-08-06T23:44:42Z

+  x64:
+    runs-on: windows-2025
+    steps:
+      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
+      - uses: jdx/mise-action@e6a8b3978addb5a52f2b4cd9d91eafa7f0ab959d # v4.2.0
+        with:
+          install: false
+          experimental: true
+      - uses: dtolnay/rust-toolchain@master
+        with:
+          toolchain: "1.95.0"
+          targets: x86_64-pc-windows-msvc
+      - run: mise run --skip-tools windows:check:x64
+      - run: mise run --skip-tools windows:build:x64
+      - run: mise run --skip-tools windows:test:x64
+      - run: mise run --skip-tools windows:test:unsupported:x64


This job takes about 80 mins to run (sample run) and it will run on each PR and merge to main. This is too long especially for PRs. We'd need to explore adding cache, so deps are not rebuilt from scratch for each run.

We could make this to be a manual trigger for now and introduce cache and add it to PR runs later.

Signed-off-by: Piotr Mlocek <1116309+pimlock@users.noreply.github.com>

pimlock · 2026-08-08T00:14:58Z

/ok to test 31c47d2

github-actions · 2026-08-08T01:01:35Z

Label test:e2e applied for 31c47d2. Open the existing run and click Re-run all jobs to execute with the label set. The run will execute the standard E2E suite after building the required gateway and supervisor images once. The matching required CI gate status on this PR will flip green automatically once the run finishes.

Signed-off-by: Piotr Mlocek <1116309+pimlock@users.noreply.github.com>

araza008 requested review from a team, derekwaynecarr, maxamillion and mrunalp as code owners July 27, 2026 14:43

drew changed the title ~~Windows compilation~~ feat(gateway,cli): windows compilation support Jul 28, 2026

araza008 force-pushed the windows_compilation branch from d0bcaca to 6d0f148 Compare July 29, 2026 14:56

matthewgrossman reviewed Jul 31, 2026

View reviewed changes

Comment thread .agents/skills/build-openshell-mxc-windows/SKILL.md Outdated

This comment was marked as outdated.

Sign in to view

araza008 force-pushed the windows_compilation branch from 602d192 to 97ce1f3 Compare August 4, 2026 18:54

pimlock reviewed Aug 4, 2026

View reviewed changes

araza008 force-pushed the windows_compilation branch from 17484be to 833d8eb Compare August 6, 2026 01:08

shailendra-nv added 3 commits August 5, 2026 20:15

chore(windows): gate Unix-only workspace code for MSVC

34a0323

Signed-off-by: Shailendra Singh <shailendras@nvidia.com> Signed-off-by: Akber Raza <akberr@nvidia.com>

feat(windows): stub unsupported compute drivers

b17118e

Signed-off-by: Shailendra Singh <shailendras@nvidia.com> Signed-off-by: Akber Raza <akberr@nvidia.com>

ci(windows): add MSVC mise build lane

db42c57

Signed-off-by: Shailendra Singh <shailendras@nvidia.com> Signed-off-by: Akber Raza <akberr@nvidia.com>

araza008 and others added 17 commits August 5, 2026 20:16

fix(windows): restore ARM64 Ninja discovery

d275242

Signed-off-by: Akber Raza <akberr@nvidia.com>

refactor(windows): separate platform crate roots

b7fc4e1

Signed-off-by: Akber Raza <akberr@nvidia.com>

fix(windows): restore proto include cfg gating

25659b9

Signed-off-by: Akber Raza <akberr@nvidia.com>

refactor: address lint errors

6d557da

fix(windows): add preflight check for proxy auth file path

b2c6d4d

docs(windows): update GitHub checkout guidance

c88bab2

Signed-off-by: Akber Raza <akberr@nvidia.com>

fix(windows): restore CI after dependency updates

0eda7b1

Signed-off-by: Akber Raza <akberr@nvidia.com>

fix(mise): repair Windows sccache lock entry

b40c2b0

Signed-off-by: Akber Raza <akberr@nvidia.com>

fix(windows): reconcile validation after rebase

26cd924

Signed-off-by: Akber Raza <akberr@nvidia.com>

refactor(server): exclude unsupported drivers on Windows

50c527e

Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>

refactor(server): isolate platform driver config

e788073

Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>

fix(windows): repair unsupported driver contract test

62b9a04

Signed-off-by: Akber Raza <akberr@nvidia.com>

fix(sandbox): remove stale dependencies

57c3ce3

Signed-off-by: Akber Raza <akberr@nvidia.com>

ci(windows): pin x64 workflow actions

c43103c

Signed-off-by: Akber Raza <akberr@nvidia.com>

ci(windows): align x64 Rust toolchain

932601a

Signed-off-by: Akber Raza <akberr@nvidia.com>

ci(windows): align ARM64 workflow setup

a988c4d

Signed-off-by: Akber Raza <akberr@nvidia.com>

refactor(windows): exclude unsupported runtime crates

b2e23e0

Signed-off-by: Akber Raza <akberr@nvidia.com>

araza008 force-pushed the windows_compilation branch from 833d8eb to b2e23e0 Compare August 6, 2026 02:11

pimlock added 4 commits August 6, 2026 16:34

refactor(windows): exclude unsupported crates at workspace boundary

4c6b3d3

Signed-off-by: Piotr Mlocek <1116309+pimlock@users.noreply.github.com>

refactor(server): gate builtin driver config by platform

c6fa0ed

Signed-off-by: Piotr Mlocek <1116309+pimlock@users.noreply.github.com>

fix(sandbox): restore crate documentation

eb7ea15

Signed-off-by: Piotr Mlocek <1116309+pimlock@users.noreply.github.com>

ci(windows): make build workflow manual

05fe62d

Signed-off-by: Piotr Mlocek <1116309+pimlock@users.noreply.github.com>

pimlock reviewed Aug 7, 2026

View reviewed changes

ci(windows): temporarily enable pull request builds

31c47d2

Signed-off-by: Piotr Mlocek <1116309+pimlock@users.noreply.github.com>

pimlock added test:e2e Requires end-to-end coverage os:windows Bug affects Windows hosts labels Aug 8, 2026

refactor(windows): remove unnecessary platform changes

a97927b

Signed-off-by: Piotr Mlocek <1116309+pimlock@users.noreply.github.com>

Uh oh!

Conversation

araza008 commented Jul 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Related Issue

Changes

Testing

Checklist

Uh oh!

copy-pr-bot Bot commented Jul 27, 2026

Uh oh!

github-actions Bot commented Jul 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

araza008 commented Jul 27, 2026

Uh oh!

jamieknvidia commented Jul 27, 2026

Uh oh!

drew commented Jul 28, 2026

Uh oh!

gburachas commented Jul 28, 2026

Uh oh!

araza008 commented Jul 29, 2026

Uh oh!

araza008 commented Jul 29, 2026

Uh oh!

drew commented Jul 29, 2026

Uh oh!

araza008 commented Jul 29, 2026

Uh oh!

araza008 commented Jul 31, 2026

Uh oh!

araza008 commented Jul 31, 2026

Uh oh!

pimlock commented Jul 31, 2026

Uh oh!

Uh oh!

This comment was marked as outdated.

This comment was marked as outdated.

araza008 commented Aug 4, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

pimlock commented Aug 4, 2026

Uh oh!

araza008 commented Aug 6, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

pimlock Aug 6, 2026

Choose a reason for hiding this comment

Uh oh!

pimlock commented Aug 8, 2026

Uh oh!

github-actions Bot commented Aug 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

araza008 commented Jul 27, 2026 •

edited

Loading

github-actions Bot commented Jul 27, 2026 •

edited

Loading