-
Notifications
You must be signed in to change notification settings - Fork 965
fix: npx usage #613 #4630
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
+325
−37
Merged
fix: npx usage #613 #4630
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
9efd47d
fix(resolve-extends): add npx cache detection for package resolution
escapedcat 6a0969f
fix(load): add npx cache and search paths support for plugin resolution
escapedcat 3a380ce
fix: address PR feedback for npx cache resolution
escapedcat c69853e
fix(load): address review feedback from PR #4630
escapedcat 8459b69
fix(load): use resolvedPath for version lookup in debug mode
escapedcat 20f4d6e
fix(resolve-extends): add debug logging to silent catch blocks
escapedcat 56c12db
fix(load): sanitize error messages to avoid exposing internal paths
escapedcat 3700297
fix(load): validate searchPaths to ensure they are safe absolute paths
escapedcat 886d9f3
fix(load): maintain backward compatibility for loadPlugin signature
escapedcat 622297b
fix(load): walk up directory tree to find package.json
escapedcat 14debd7
fix(load): validate searchPaths is a directory
escapedcat b873b90
test(load): add tests for npx cache resolution and searchPaths valida…
escapedcat 972a0a0
test(resolve-extends): add test for resolveFromNpxCache behavior
escapedcat 5ff7004
fix: remove duplicate test names causing lint errors
escapedcat 996fa27
test(resolve-extends): remove ineffective test
escapedcat File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.