Skip to content

ci: pin pnpm/action-setup to a commit SHA - #144

Merged
jamespsterling merged 1 commit into
mainfrom
devin/1793845556-pin-actions
Nov 5, 2026
Merged

jamespsterling merged 1 commit into
mainfrom
devin/1793845556-pin-actions

Conversation

@jamespsterling

@jamespsterling jamespsterling commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Summary

publish.yaml used pnpm/action-setup@v6 in the job that holds id-token: write for npm trusted publishing and RELEASE_BOT_PRIVATE_KEY. A moved v6 tag upstream would run with those credentials on the next release.

Every pnpm/action-setup reference (publish.yaml, ci.yaml, bump-openrouter-sdk.yaml) now points at 73e46992e05b19abd31f2f3c20e0e7370d762115 # v6.1.1, the commit v6 resolves to today, so Dependabot can keep it current. actions/* references are unchanged.

Found by running Sentry's gha-security-review skill over the organization's public repositories.

Link to Devin session: https://openrouter.devinenterprise.com/sessions/600df9efd8ca4fd28e3000389431436f
Open in Devin Desktop: https://openrouter.devinenterprise.com/desktop/session/600df9efd8ca4fd28e3000389431436f?variant=devin
Requested by: @jamespsterling


Devin Review

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access that start with 'Devin'.

  • Disable automatic comment, CI, and merge conflict monitoring

Original prompt from James

Run this on our GHA https://github.com/getsentry/skills/tree/main/skills/gha-security-review

@devin-ai-integration
devin-ai-integration Bot marked this pull request as ready for review November 5, 2026 02:27

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

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.

✅ Devin Review: No Issues Found

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

Devin Review

@jamespsterling
jamespsterling merged commit 644e89c into main Nov 5, 2026
8 checks passed
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