Skip to content

fix: use cross-env and shx for Windows-compatible npm scripts#28

Merged
Aditya-thesys merged 2 commits into
thesysdev:mainfrom
Akihiro2004:fix_installer
May 13, 2026
Merged

fix: use cross-env and shx for Windows-compatible npm scripts#28
Aditya-thesys merged 2 commits into
thesysdev:mainfrom
Akihiro2004:fix_installer

Conversation

@Akihiro2004
Copy link
Copy Markdown
Contributor

The npm scripts in both packages used Unix-only shell syntax that breaks on Windows. Running pnpm dev or building/packing the plugin on Windows would fail immediately.

Changes:

  • Added cross-env to claw-client devDependencies and updated the dev and start scripts to use it, so NEXT_OUTPUT=server gets set correctly on Windows instead of throwing a syntax error
  • Added shx to claw-plugin devDependencies and updated the bundle-ui script to use shx rm -rf and shx cp -r instead of bare Unix commands that cmd.exe does not understand

Test Plan

  1. Verified locally (pnpm ci passes)
  2. Manually tested the plugin end-to-end with a running OpenClaw gateway
  3. Ran pnpm dev on Windows and confirmed it starts without the NEXT_OUTPUT is not recognized error
  4. Ran pnpm prepack on Windows and confirmed the static files copy over without rm or cp failing

@Akihiro2004 Akihiro2004 changed the title fix: windows installer split(.) issue fix: use cross-env and shx for Windows-compatible npm scripts May 13, 2026
@Aditya-thesys
Copy link
Copy Markdown
Contributor

Hi @Akihiro2004 , thanks! for raising the fix.
Could you please remove package-lock.json file,since this repo is pnpm based.

@Akihiro2004
Copy link
Copy Markdown
Contributor Author

Hi @Akihiro2004 , thanks! for raising the fix. Could you please remove package-lock.json file,since this repo is pnpm based.

Sure.

This repo uses pnpm — package-lock.json should not be committed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Akihiro2004 Akihiro2004 reopened this May 13, 2026
Copy link
Copy Markdown
Contributor

@Aditya-thesys Aditya-thesys left a comment

Choose a reason for hiding this comment

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

nit: could you use latest cross env package version

@Aditya-thesys
Copy link
Copy Markdown
Contributor

thanks! for the contribution

@Aditya-thesys Aditya-thesys merged commit 777141d into thesysdev:main May 13, 2026
1 check passed
@zahlekhan
Copy link
Copy Markdown

Hey @Akihiro2004 thanks for the contribution.
We would love you on our Discord to share what you are using OpenClaw-OS for.
https://discord.com/invite/Pbv5PsqUSv

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.

3 participants