Skip to content

chore: Pin GitHub Actions#1501

Merged
gjtorikian merged 2 commits intomainfrom
chore/pin-github-actions
Feb 27, 2026
Merged

chore: Pin GitHub Actions#1501
gjtorikian merged 2 commits intomainfrom
chore/pin-github-actions

Conversation

@gjtorikian
Copy link
Contributor

@gjtorikian gjtorikian commented Feb 26, 2026

Summary

Pin all third-party GitHub Actions to immutable commit SHAs.

Why

Action tags (like v3, v4, main) can be moved or retagged, which means a future workflow run could execute different code than what we reviewed today. Pinning to SHAs makes the workflow supply chain deterministic and auditable, reducing the risk of action-level compromise or accidental breaking changes. We can still update intentionally by bumping the SHA.

@gjtorikian gjtorikian requested a review from a team as a code owner February 26, 2026 19:38
@gjtorikian gjtorikian requested a review from awolfden February 26, 2026 19:38
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 26, 2026

Greptile Summary

This PR pins all third-party GitHub Actions to immutable commit SHAs across 4 workflow files, improving supply chain security by preventing tag movement. However, the PR also deletes 2 Coana workflow files (coana-analysis.yml and coana-guardrail.yml) which is not mentioned in the PR description.

Key changes:

  • Pinned actions/checkout@v6 to de0fac2e... (v6.0.2)
  • Pinned actions/setup-node@v6 to 6044e13b... (v6.2.0)
  • Pinned denoland/setup-deno@v2 to e95548e5... (v2.0.3)
  • Pinned oven-sh/setup-bun@v2 to 3d267786... (v2.1.2)
  • Deleted Coana vulnerability analysis and guardrail workflows

The SHA pinning follows security best practices with clear version comments. The unexpected workflow deletions should be verified and documented in the PR description.

Confidence Score: 3/5

  • This PR is safe to merge but requires clarification on workflow deletions
  • The SHA pinning implementation is correct and improves security, but the deletion of two Coana workflows without explanation in the PR description is a documentation concern that should be addressed
  • Pay close attention to coana-analysis.yml and coana-guardrail.yml - verify these deletions are intentional

Important Files Changed

Filename Overview
.github/workflows/ci.yml Pinned actions/checkout and actions/setup-node to commit SHAs with version comments
.github/workflows/runtime-tests.yml Pinned all four actions (actions/checkout, actions/setup-node, denoland/setup-deno, oven-sh/setup-bun) to commit SHAs with version comments
.github/workflows/coana-analysis.yml File deleted - vulnerability analysis workflow removed without explanation in PR description
.github/workflows/coana-guardrail.yml File deleted - PR guardrail workflow removed without explanation in PR description

Last reviewed commit: 72f2b0e

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

6 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 26, 2026

Additional Comments (2)

.github/workflows/coana-analysis.yml
PR description says "Pin all third-party GitHub Actions" but deletes this entire Coana vulnerability analysis workflow. Check if this is intentional and update PR description.


.github/workflows/coana-guardrail.yml
PR description doesn't mention removing the Coana guardrail workflow. Verify deletion is intentional and update description accordingly.

@gjtorikian gjtorikian changed the title Pin GitHub Actions chore: Pin GitHub Actions Feb 26, 2026
@gjtorikian gjtorikian merged commit 4302c62 into main Feb 27, 2026
8 checks passed
@gjtorikian gjtorikian deleted the chore/pin-github-actions branch February 27, 2026 21:02
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