Skip to content

fix: make frontend web dev script cross-platform#2982

Open
DS123-ally wants to merge 1 commit into
AgentWrapper:mainfrom
DS123-ally:agent/windows-dev-web-source-support
Open

fix: make frontend web dev script cross-platform#2982
DS123-ally wants to merge 1 commit into
AgentWrapper:mainfrom
DS123-ally:agent/windows-dev-web-source-support

Conversation

@DS123-ally

@DS123-ally DS123-ally commented Jul 22, 2026

Copy link
Copy Markdown

Summary

  • replace the POSIX-only VITE_NO_ELECTRON=1 vite ... dev:web script with a small Node launcher
  • keep VITE_NO_ELECTRON=1 behavior while allowing extra Vite arguments to pass through
  • document frontend source-development commands and the Windows npm.cmd workaround

Why

Windows PowerShell/cmd do not support VAR=value command syntax, so npm run dev:web fails for Windows contributors before Vite can start. The launcher sets the environment through Node instead of depending on shell-specific syntax.

Fixes # 2981

Validation

  • npm.cmd run dev:web -- --host 127.0.0.1 --port 5185 --strictPort reached Vite ready state on Windows
  • npm.cmd run typecheck

@somewherelostt

Copy link
Copy Markdown
Collaborator

Thanks for contributing to Agent Orchestrator.

This PR is being picked up by the current external contributor on-call pair:

If someone is already working on this, please continue as usual.
The on-call pair is added for visibility, tracking, and support, not to take over the work.
If you need help with review, direction, reproduction, or next steps, please tag @illegalcall and @Pulkit7070 here.

For faster context or live questions, you can also join the AO Discord.

Join the session here:
https://discord.gg/H6ZDcUXmq

Come by if you want to see what is being built, ask questions, or just hang around with the community.

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.

4 participants