Skip to content

feat: support npm link symlinked dependencies#1281

Open
adrianstephens wants to merge 1 commit into
microsoft:mainfrom
adrianstephens:symlink-fix
Open

feat: support npm link symlinked dependencies#1281
adrianstephens wants to merge 1 commit into
microsoft:mainfrom
adrianstephens:symlink-fix

Conversation

@adrianstephens

Copy link
Copy Markdown

Add support for npm-linked dependencies by mapping resolved symlink target paths back to their symlink locations in node_modules.

Changes:

  • Implement mapSymlinkedDependencies() to handle npm link scenarios
  • Map both scoped (@scope/package) and unscoped packages
  • Deduplicate nested dependencies (only when symlinks are present)
  • Always follow symlinks for dependencies while respecting followSymlinks parameter for root package
  • Simplify follow symlinks logic with concise expression

The fix allows vsce to correctly package extensions that use npm link for local multi-package development workflows.

Add support for npm-linked dependencies by mapping resolved symlink target
paths back to their symlink locations in node_modules.

Changes:
- Implement mapSymlinkedDependencies() to handle npm link scenarios
- Map both scoped (@scope/package) and unscoped packages
- Deduplicate nested dependencies (only when symlinks are present)
- Always follow symlinks for dependencies while respecting followSymlinks
  parameter for root package
- Simplify follow symlinks logic with concise expression

The fix allows vsce to correctly package extensions that use npm link for
local multi-package development workflows.
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