Skip to content

Run the CLI CI job on Linux, Windows and macOS - #1537

Merged
helpcodeai merged 1 commit into
mainfrom
ci/cli-os-matrix
Nov 5, 2026
Merged

helpcodeai merged 1 commit into
mainfrom
ci/cli-os-matrix

Conversation

@helpcodeai

Copy link
Copy Markdown
Contributor

What

Turns the single-OS CLI (test, build) job into a 3-OS matrix: ubuntu-latest, windows-latest, macos-latest.

Why

@anythingmcp/cli is published to npm and installed on all three platforms. It has OS-specific code paths (e.g. browserCommand returns open / rundll32 / xdg-open, and the config dir resolves via XDG_CONFIG_HOME/home dir), but CI only ever ran it on Ubuntu, so Windows/macOS regressions could ship unnoticed. The unit suite already asserts the Windows browser-command form; this makes the whole suite + build actually run on Windows and macOS each PR.

Details

  • strategy.matrix.os over the three runners, fail-fast: false so one OS failing still reports the others.
  • defaults.run.shell: bash for the job — Git Bash exists on the Windows runner, so the final test ! -e packages/cli/dist/cli.test.js guard stays portable without rewriting it.
  • npm ci --ignore-scripts: the CLI's only runtime dep is pure JS, so skipping workspace install scripts (Prisma engine download, native backend builds) keeps the job fast and avoids failing on an unrelated backend dependency on Windows/macOS. The CLI test/build touch no backend code.

Shell-syntax sub-tests for shells not present on a runner (e.g. zsh/fish on Windows) self-skip, as fish already does on Linux.

No source changes; CI-only.

@helpcodeai
helpcodeai enabled auto-merge (squash) November 5, 2026 13:55
@helpcodeai
helpcodeai merged commit 539c2b1 into main Nov 5, 2026
17 checks passed
@helpcodeai
helpcodeai deleted the ci/cli-os-matrix branch November 5, 2026 14:01
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant