Skip to content

fix(package): repair moved global bin symlinks#2104

Open
fordlyn wants to merge 1 commit intovolta-cli:mainfrom
fordlyn:fix/package-bin-symlink-repair
Open

fix(package): repair moved global bin symlinks#2104
fordlyn wants to merge 1 commit intovolta-cli:mainfrom
fordlyn:fix/package-bin-symlink-repair

Conversation

@fordlyn
Copy link
Copy Markdown

@fordlyn fordlyn commented May 6, 2026

Repair Unix package bin symlinks after moving a global package install from the staging directory into the final package image directory.

Some packages rewrite npm global bin entries during postinstall to point at absolute paths inside the staging directory. After Volta persists that directory into tools/image/packages, those symlinks become stale and the generated Volta shim fails to execute.

Only symlinks in the package bin directory that point back into the staging directory are rewritten. Regular files, relative symlinks, and external symlinks are left unchanged.

Also clear _VOLTA_TOOL_RECURSION in test sandboxes so smoke and acceptance tests do not accidentally inherit a parent shim-recursion state.

Repair Unix package bin symlinks after moving a global package install from the staging directory into the final package image directory.

Some packages rewrite npm global bin entries during postinstall to point at absolute paths inside the staging directory. After Volta persists that directory into tools/image/packages, those symlinks become stale and the generated Volta shim fails to execute.

Only symlinks in the package bin directory that point back into the staging directory are rewritten. Regular files, relative symlinks, and external symlinks are left unchanged.

Also clear _VOLTA_TOOL_RECURSION in test sandboxes so smoke and acceptance tests do not accidentally inherit a parent shim-recursion state.

Co-authored-by: Codex <noreply@openai.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