Route shell commands through PowerShell -EncodedCommand to fix paths with spaces

[linghengqian]

• For Remove documentation regarding the lack of support for Windows 11 folder names containing spaces vfox-flutter#33 .
• Route shell commands through PowerShell -EncodedCommand to fix paths with spaces.

[linghengqian]

CC @Chance-fyi @aooohan @bytemain . Could you help approve the CI run?

[Copilot]

Copilot review overview

🟡 Changes recommended

The Windows Lua overrides break existing shell and file-handle contracts and contain incorrect pipe, EOF, and status handling.

Review effort: Balanced
Findings: 1 · 2 · 1

Open (4)

• Preserve cmd.exe syntax for Windows plugin commands · New
• Attach standard streams to Windows plugin processes · New
• Correctly assign os.Pipe reader and writer ends · New
• Validate the selected PowerShell executable correctly · New

What changed in this PR

Routes Windows shell execution through PowerShell encoded commands to support paths containing spaces.

Changes:

• Adds shared encoded PowerShell execution helpers.
• Updates junctions, upgrades, shims, and Lua command execution.
• Adds Windows-focused regression tests.

File	Description
internal/​shared/​util/​shell.go	Adds cross-platform shell helpers.
internal/​shared/​util/​shell_test.go	Tests command encoding and construction.
internal/​shared/​util/​file.go	Creates Windows junctions through PowerShell.
internal/​shared/​util/​file_windows_test.go	Tests junction paths containing spaces.
internal/​shared/​shim/​shim_windows.go	Escapes paths in generated shims.
internal/​plugin/​luai/​vm.go	Installs Windows Lua execution overrides.
internal/​plugin/​luai/​exec.go	Implements overridden Lua process APIs.
internal/​plugin/​development_output.go	Uses the shared shell helper in development mode.
cmd/​commands/​upgrade.go	Launches the upgrade script through PowerShell.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[aooohan]

Could you help approve the CI run?

Where is CI... I remember there was one before.

[linghengqian]

Could you help approve the CI run?

Where is CI... I remember there was one before.

• I’ve opened #713; I will address the review comments on the current PR after that one is merged.

[codecov]

Codecov Report

❌ Patch coverage is 31.00000% with 138 lines in your changes missing coverage. Please review.
✅ Project coverage is 35.00%. Comparing base (930f5be) to head (96f9c8f).

Files with missing lines	Patch %	Lines
internal/plugin/luai/exec.go	25.51%	102 Missing and 6 partials ⚠️
internal/shared/util/file.go	0.00%	15 Missing ⚠️
internal/shared/util/shell.go	65.71%	10 Missing and 2 partials ⚠️
internal/shared/util/shell_notwindows.go	0.00%	2 Missing ⚠️
cmd/commands/upgrade.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #712      +/-   ##
==========================================
- Coverage   35.09%   35.00%   -0.09%     
==========================================
  Files          92       95       +3     
  Lines        6514     6702     +188     
==========================================
+ Hits         2286     2346      +60     
- Misses       3922     4043     +121     
- Partials      306      313       +7     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[linghengqian]

Done.