Skip to content

ci: update SHA-pinned GitHub Actions via ratchet - #47

Closed
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/github-actions-ratchet-review-8aac
Closed

ci: update SHA-pinned GitHub Actions via ratchet#47
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/github-actions-ratchet-review-8aac

Conversation

@cursor

@cursor cursor Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Ran mise run //:ratchet:update to refresh the SHA pins of our GitHub Actions to the latest commits within their existing ratchet version constraints. No version constraints (# ratchet:owner/action@vN) were changed — these are patch-level SHA bumps only.

Three actions were updated across go.yml, docker.yml, eif-build.yml, and ratchet.yml:

Action Constraint Old New
actions/checkout @v7 v7.0.0 (9c091bb) v7.0.1 (3d3c42e)
aws-actions/configure-aws-credentials @v6 v6.2.2 (517a711) v6.2.3 (e6de054)
jdx/mise-action @v4 v4.2.1 (dad1bfd) v4.2.3 (9e7f763)

What changed in each update

actions/checkout v7.0.0 → v7.0.1

(compare)

  • Skip the unsafe-PR check when the ref input is the default.
  • Trim only ASCII whitespace when parsing branch names.
  • Escape values passed to git config --unset (hardening).
  • Various dependency updates.

aws-actions/configure-aws-credentials v6.2.2 → v6.2.3

(compare)

  • Bug fix: attach git credentials before the Tag-Major-Version push (relevant to the action's own release process).
  • Bug fix: PackedPolicyTooLarge detection in STS session tags — clearer error surfacing when session tags exceed the packed-policy size limit.

jdx/mise-action v4.2.1 → v4.2.3

(compare)

  • v4.2.2: Verify tar supports Zstd before selecting the .tar.zst mise archive; fall back to .tar.gz on older runners (e.g. RHEL8-compatible with GNU tar 1.30). README default corrected to mise-v1.
  • v4.2.3: Restore mise's PATH propagation to subsequent steps. v4.2.1 (our previous pin) had stopped exporting mise PATH entries via GITHUB_ENV; v4.2.3 re-forwards only the mise-added PATH prefix through GITHUB_PATH (without snapshotting the runner's full PATH). Adds an export_path input (default true).

Risk assessment

Overall: Low. All three are patch releases that stay within the pinned major, and every SHA was verified to resolve to the stated release tag via the GitHub API.

  • actions/checkout (Low): Bug fixes and git-config hardening only; no interface changes. Our usage is basic checkout (with ref/token inputs), unaffected by the changed edge cases.
  • aws-actions/configure-aws-credentials (Low): Both fixes are non-breaking. The STS PackedPolicyTooLarge improvement is only error-reporting; our OIDC role-assumption usage (no session tags) is unaffected.
  • jdx/mise-action (Low, net improvement): The Zstd fallback only affects older runners (we use ubuntu-latest). The PATH-propagation restoration is beneficial — it reverses a regression from v4.2.1. Our workflows invoke tools via mise run //:... (which resolves its own shims), so the previous PATH gap wasn't affecting us, and restoring it composes cleanly. Default export_path: true preserves prior intent.

No secrets, permissions, triggers, or workflow logic were modified.

Verification checklist

  • mise run //:ratchet:update produced only SHA changes; version constraints unchanged.
  • mise run //:ratchet:lint passes (all external refs remain pinned).
  • mise run //:ratchet:pin is idempotent — no additional diff (matches the CI git diff --exit-code gate).
  • mise run //:test (go test ./...) passes.
  • Each new SHA verified via gh api .../git/refs/tags to map to the release tag shown above.
  • CI green on this PR: Go (test + lint + tidy) and Ratchet (lint + pin verification) workflows.
  • Reviewer confirms no version-constraint (@vN) changes are intended in this update-only PR.
Open in Web View Automation 

Refresh pinned SHAs to latest within existing ratchet version constraints:
- actions/checkout: v7.0.0 -> v7.0.1
- aws-actions/configure-aws-credentials: v6.2.2 -> v6.2.3
- jdx/mise-action: v4.2.1 -> v4.2.3

Co-authored-by: Nick Pellegrino <nickpell@users.noreply.github.com>
@nickpell nickpell closed this Aug 5, 2026
nickpell added a commit that referenced this pull request Aug 5, 2026
## Summary

- Weekly automated refresh of SHA-pinned GitHub Actions via `mise run
//:ratchet:update`. Bumps pinned commit SHAs to the latest release
**within** existing major-version constraints (`#
ratchet:owner/action@vN` comments unchanged — no major upgrades).
- Supersedes the prior week's draft `#47` (now
closed): same two bumps plus a newer `jdx/mise-action` release (v4.2.1 →
v4.2.4).
- Pins: `actions/checkout` v7.0.0→v7.0.1,
`aws-actions/configure-aws-credentials` v6.2.2→v6.2.3, `jdx/mise-action`
v4.2.1→v4.2.4. Patch-level only; low risk; rollback is a trivial revert.

## Pre-merge checklist

- [x] Build / CI passes (Go `test` + `lint`, Ratchet Lint — all green)
- [x] Lint passes (Go lint + Ratchet Lint)
- [x] Tests pass (`mise run //:test` / Go workflow)
- [x] Diff contains no unintended changes (SHA pins only; `@vN`
constraints unchanged; verified SHAs map to stated release tags)
- [x] Docs / config updated if behavior or interface changed — N/A (CI
pin refresh only)
- [x] Cross-repo dependencies noted and ordered — N/A (openauction-only;
supersedes closed #47)

## Post-deploy/apply verification

- [ ] Deploy or apply completed successfully — N/A for merge itself;
next workflow runs on `main` exercise the new pins
- [ ] Target resource / workload reached the intended state — confirm
subsequent Go / Ratchet / docker / eif-build workflow runs on `main`
succeed with the new action SHAs
- [ ] Dependent service(s) healthy — N/A (no runtime deploy)
- [ ] Metrics / synthetics green after the change — N/A (CI tooling
only)

<div><a
href="https://cursor.com/agents/bc-b23ff061-5ae6-4023-b8d4-4f44d68aa720?cursor_ref=pr_footer&cursor_cta=open_in_web"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source
media="(prefers-color-scheme: light)"
srcset="https://cursor.com/assets/images/open-in-web-light.png"><img
alt="Open in Web" width="114" height="28"
src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a>&nbsp;<a
href="https://cursor.com/automations/65e8572f-8209-11f1-a7d1-d6b4613131ce"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://cursor.com/assets/images/view-automation-dark.png"><source
media="(prefers-color-scheme: light)"
srcset="https://cursor.com/assets/images/view-automation-light.png"><img
alt="View Automation" width="141" height="28"
src="https://cursor.com/assets/images/view-automation-dark.png"></picture></a>&nbsp;</div>

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Nick Pellegrino <nickpell@users.noreply.github.com>
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