Skip to content

chore(ci): bump socket-registry refs to f1b40c99 (npm-banner-validation fix)#615

Merged
John-David Dalton (jdalton) merged 1 commit intomainfrom
chore/registry-sha-bump-f1b40c99
Apr 24, 2026
Merged

chore(ci): bump socket-registry refs to f1b40c99 (npm-banner-validation fix)#615
John-David Dalton (jdalton) merged 1 commit intomainfrom
chore/registry-sha-bump-f1b40c99

Conversation

@jdalton
Copy link
Copy Markdown
Contributor

Bumps SocketDev/socket-registry workflow/action pins to f1b40c99.

Fixes a regression introduced by the previous propagation SHA (0371e83f, merged via #611). That commit added a runtime guard that calls npm view @socketsecurity/lib version to compute the floor; when the response was a Socket Firewall banner instead of a version string, the comparison function exploded with [: <banner>: integer expression expected.

f1b40c99 validates npm view output against a plain-semver regex before using it, falling back to the hardcoded floor (5.24.0) when the response isn't semver. Same defensive check applied to the consumer's installed version.

socket-sdk-js already pins @socketsecurity/lib at 5.24.0 from #611 — this bump is mechanical, no consumer code changes.

Test plan

  • CI pipeline (check + matrix tests) passes
  • Audit GitHub Actions check passes

…on fix)

The previous propagation SHA (0371e83f, merged via #611) shipped a
guard step whose version_lt function exploded when npm view returned
a Socket Firewall banner string instead of a version. f1b40c99
validates npm view output as semver before using it, falling back to
the hardcoded floor (5.24.0) when the response isn't plain semver.
@jdalton John-David Dalton (jdalton) merged commit cbdfb82 into main Apr 24, 2026
11 checks passed
@jdalton John-David Dalton (jdalton) deleted the chore/registry-sha-bump-f1b40c99 branch April 24, 2026 20: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.

2 participants