Skip to content

fix(shell): preserve pathname expansion in scripts - #705

Draft
RSS1102 wants to merge 1 commit into
voidzero-dev:mainfrom
RSS1102:rss1102/fix-package-script-shell-globs
Draft

fix(shell): preserve pathname expansion in scripts#705
RSS1102 wants to merge 1 commit into
voidzero-dev:mainfrom
RSS1102:rss1102/fix-package-script-shell-globs

Conversation

@RSS1102

@RSS1102 RSS1102 commented Aug 30, 2026

Copy link
Copy Markdown

Motivation

Package scripts run by package managers rely on the platform shell for pathname expansion. Vite Task's static command path flattened unquoted patterns such as packages/*/src into literal arguments, so vp run behaved differently from pnpm run and tools received the unexpanded pattern.

Fixes #573.

Summary

  • fall back to the platform shell when a program or argument contains unquoted pathname-expansion characters
  • keep quoted, escaped, and assignment-value patterns on the static command path
  • cover parser decisions, execution plans, and Unix/Windows package-script behavior with regression snapshots
  • document the user-visible fix in the changelog

@RSS1102
RSS1102 force-pushed the rss1102/fix-package-script-shell-globs branch 2 times, most recently from 60d5073 to 477b3b7 Compare August 31, 2026 02:20
Co-authored-by: GPT-5 Codex <codex@openai.com>
@RSS1102
RSS1102 force-pushed the rss1102/fix-package-script-shell-globs branch from 477b3b7 to 150b596 Compare August 31, 2026 03:23
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.

vp run does not preserve shell semantics for package.json scripts

1 participant