Skip to content

fix(cli): resolve Windows Alchemy executable shims - #280

Draft
AmanVarshney01 wants to merge 2 commits into
mainfrom
codex/verify-windows-alchemy
Draft

fix(cli): resolve Windows Alchemy executable shims#280
AmanVarshney01 wants to merge 2 commits into
mainfrom
codex/verify-windows-alchemy

Conversation

@AmanVarshney01

@AmanVarshney01 AmanVarshney01 commented Sep 8, 2026

Copy link
Copy Markdown
Member

Problem

Composer only looked for node_modules/.bin/alchemy. Windows package managers can install alchemy.exe or alchemy.cmd without that extensionless file, so Composer incorrectly reports DEPLOY.ALCHEMY_BIN_MISSING before launching anything.

Fix

Resolve the nearest installed Windows executable in order: alchemy.exe, alchemy.cmd, then the extensionless shim. Continue searching ancestors for hoisted installations; POSIX behavior is unchanged. No PATH/global fallback, application workaround, or dependency change. Update the deployment guide and core-concepts skill with the resolution contract.

Verification

  • Local CLI typecheck passed.
  • Local resolver/runner tests: 12 passed; 2 Windows-only regressions skipped on macOS.
  • Windows regressions cover cmd/exe-only installations and execution of a cmd shim with spaces and a literal ampersand in argv through the existing cross-spawn runner.
  • The existing Windows CI matrix is running; native Windows verification is still pending. No new workflow.

Related owning-repo fix

prisma/prisma-cli#250 fixes the consolidated CLI adapter that launches the resolved command. Its native Windows regression reproduced spawn EINVAL on npm.cmd before the fix.

Limits

This does not claim all Windows assembly or deployment failures share this cause, or enable local Windows dev support. No production deployment was performed for this change.

Regression intentionally fails on Windows until resolver supports installed cmd/exe shims. Existing POSIX resolver tests pass.

Signed-off-by: Aman Varshney <amanvarshney.work@gmail.com>
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Comment @coderabbitai help to get the list of available commands.

@pkg-pr-new

pkg-pr-new Bot commented Sep 8, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@prisma/composer@280
npm i https://pkg.pr.new/@prisma/composer-cli@280
npm i https://pkg.pr.new/@prisma/composer-prisma-cloud@280

commit: 9f85af1

Local resolver/runner tests and CLI typecheck pass. Windows execution is verified by the existing CI matrix, pending at commit time.

Signed-off-by: Aman Varshney <amanvarshney.work@gmail.com>
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.

1 participant