Skip to content

Conversation

@mikolalysenko
Copy link

@mikolalysenko mikolalysenko commented Dec 19, 2025

This PR replaces the patch subcommand with the @socketsecurity/socket-patch sub command. To get this in I have to publish a new release for the socket-patch module


Note

Cursor Bugbot is generating a summary for commit 3496bf1. Configure here.

- Replace inline patch implementation with @socketsecurity/[email protected]
- Use runPatch() from socket-patch/run for programmatic invocation
- Remove deleted handle-patch.mts, manifest-schema.mts, output-patch-result.mts
- Add SOCKET_PATCH_PROXY_URL environment variable support
- Forward socket-cli environment to socket-patch options
@socket-security
Copy link

socket-security bot commented Dec 19, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​socketsecurity/​socket-patch@​1.0.0100100100100100

View full report

@socket-security-staging
Copy link

socket-security-staging bot commented Dec 19, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​socketsecurity/​socket-patch@​1.0.0100100100100100

View full report

@jdalton
Copy link
Contributor

jdalton commented Dec 19, 2025

@mikolalysenko

Failure reason:
specifiers in the lockfile don't match specifiers in package.json:

@mikolalysenko
Copy link
Author

I have to publish the new version of the patch CLI in order for the dependency to load.

@jdalton jdalton merged commit 44655ac into v1.x Dec 22, 2025
8 checks passed
@jdalton jdalton deleted the feature/use-socket-patch-run branch December 22, 2025 19:48
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.

3 participants