Skip to content

chore: Pin third-party GitHub Actions to full commit SHAs#71

Merged
willporter-workos merged 1 commit into
mainfrom
devin/1777478642-pin-github-actions
Apr 29, 2026
Merged

chore: Pin third-party GitHub Actions to full commit SHAs#71
willporter-workos merged 1 commit into
mainfrom
devin/1777478642-pin-github-actions

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot commented Apr 29, 2026

file:///home/ubuntu/pin-actions/authkit-react-router_pr_body.md

Link to Devin session: https://app.devin.ai/sessions/add87be2227046f198fbac38a32e5358


Open in Devin Review

@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

Original prompt from will.porter

'Pin all third-party Github Actions for Public SDKs' (SECENG-294)

User instruction: @devin can you look at the workos organization in github, and report back all of the public repositories that are not archived, and whether or not if they use any github workflows?

@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration devin-ai-integration Bot changed the title Pin third-party GitHub Actions to full commit SHAs chore: Pin third-party GitHub Actions to full commit SHAs Apr 29, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 29, 2026

Greptile Summary

This PR replaces mutable version tags (@v6) with full commit SHA pins for actions/checkout and actions/setup-node across both workflow files, following supply-chain security best practices. Both SHAs are verified: de0fac2echeckout@v6.0.2 and 48b55a01setup-node@v6.4.0.

Confidence Score: 5/5

Safe to merge — SHA pins are correct and verified against upstream releases.

Only change is pinning GitHub Actions to verified commit SHAs. Both SHAs resolve to the expected versions. The sole finding is a P2 style nit about the version comment being # v6 instead of # v6.4.0.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/ci.yml All action references pinned to full commit SHAs; checkout SHA verified as v6.0.2, setup-node SHA verified as v6.4.0. Version comment for setup-node is abbreviated to # v6 instead of the specific # v6.4.0.
.github/workflows/release.yml Same SHA pinning applied consistently; same imprecise # v6 comment pattern for setup-node as in ci.yml, but otherwise correct and safe.

Reviews (1): Last reviewed commit: "Pin third-party GitHub Actions to full c..." | Re-trigger Greptile

Comment thread .github/workflows/ci.yml
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
- name: Setup Node
uses: actions/setup-node@v6
uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Imprecise version comment for pinned SHA

The SHA 48b55a0... resolves to actions/setup-node@v6.4.0, but the comment only says # v6. Using the full version tag in the comment (e.g., # v6.4.0) makes it much easier to audit which exact release is pinned and to know when a newer patch/minor is available. The same applies to every other setup-node reference in this file and in release.yml.

Suggested change
uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6
uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0

@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

Third-Party Action SHA Age Report

Action Pinned Version Full SHA Commit Date Age (days) Status
actions/checkout v6 de0fac2e4500dabe0009e67214ff5f5447ce83dd 2026-01-09 109 ✅ OK
actions/setup-node v6 48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e 2026-04-20 9 ✅ OK

Copy link
Copy Markdown
Contributor Author

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@willporter-workos willporter-workos merged commit 164d9f0 into main Apr 29, 2026
9 checks passed
@willporter-workos willporter-workos deleted the devin/1777478642-pin-github-actions branch April 29, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants