Repository navigation
Package risk scan (Socket) and pin GitHub Actions to commit SHAs (#922) - #930
Merged
dadadave80 merged 2 commits intoOct 23, 2026
Conversation
OpenSSF Scorecard's Pinned-Dependencies check scored 2/10 because most actions referenced mutable tags. Each reference now uses the commit SHA of its current release, with the version kept as a comment. The repoHygiene setup-bun check matches any pinned ref instead of the literal @v2. Refs Heliobond#922
Runs `socket ci` on pull requests to score added or changed packages for install scripts, obfuscated code, network access and new maintainers. Needs the SOCKET_CLI_API_TOKEN secret, and SOCKET_ORG as an optional variable. Forks do not receive secrets, so the scan is skipped there with a notice rather than reported as a pass. Thresholds come from the Socket dashboard policy. Refs Heliobond#922
|
@onyinyechinwokwu-success is attempting to deploy a commit to the David Dada's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Thanks for your contribution! ❤️ It looks like this PR isn't linked to an issue yet. Please add |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Follow-up to #917/#918/#919 (merged in #925) for the remaining part of #922. Two changes:
main, because most actions used mutable tags. Eachuses:line now has the commit SHA of its current release, with the version as a comment..github/workflows/socket.yml). It runssocket ci(CLI 1.10.0, pinned) to score added or changed packages for install scripts, obfuscated code, network access and new maintainers.Linked issue
Closes #922.
The Socket scan is merged but not yet active: it needs the
SOCKET_CLI_API_TOKENsecret, which a maintainer must add. Setup and the untested parts are listed below as follow-up for maintainers.Socket follow-up (maintainer)
SOCKET_CLI_API_TOKENrepository secret. The scan is skipped without it.SOCKET_ORGrepository variable to the Socket org slug. Without it, the CLI uses its own default org.Not verified
bun.lock. The CLI's dry run bails before manifest discovery without a token, so I couldn't confirm locally that it findsbun.lock. The first run with the secret is the test.Scorecard: what's still open (for follow-up issues)
Measured locally with Scorecard v5.6.0 against
mainbefore this PR: 5.8/10. Scorecard is run against the default branch, so the score after this PR is only known after the next scheduled run.auto-review.ymlruns underpull_request_targetand checks outbase.sha. Scorecard flags any checkout in that event as an untrusted checkout, so the flag may be a false positive. It is upstream's workflow, and changing it is a separate decision.braces,uuid,stream-jsonandelliptic. No fixes available in range (see Supply-chain hardening: patch advisories, install policy, dependency review #925).Test evidence
Local, bun 1.3.14 and Node 24:
typecheck,lint,format:check: passtest:coverage: 133 files, 1496 tests passed, 3 expected failsuses:ref is a 40-character SHAChecklist
Refs #922)typecheck,lint,format:check,testpass locallySupply-chain gate: this PR changes workflows, so the upstream
supply-chain review gatewill block it until a maintainer appliessupply-chain-reviewed.