Skip to content

chore: replace runs-on/cache with the StepSecurity-maintained fork - #29

Open
rehan-fr wants to merge 2 commits into
masterfrom
chore/lingering-3rd-party-actions-cleanup
Open

chore: replace runs-on/cache with the StepSecurity-maintained fork#29
rehan-fr wants to merge 2 commits into
masterfrom
chore/lingering-3rd-party-actions-cleanup

Conversation

@rehan-fr

Copy link
Copy Markdown

What

Swaps all six runs-on/cache* references in this repo to the StepSecurity-maintained fork, pinned to the
sha the rest of the org already uses (c5b0cba15d05488ebc630ad8d1e95b3d9b35ac73 # v5.0.7):

File Line(s) Old New
.github/workflows/ci.yml 65, 80, 134, 149 runs-on/cache@a5f51d6f step-security/runs-on-cache@c5b0cba1
.github/workflows/ci.yml 210 runs-on/cache/save@a5f51d6f step-security/runs-on-cache/save@c5b0cba1
.github/workflows/dev-publish.yaml 45 runs-on/cache/restore@a5f51d6f step-security/runs-on-cache/restore@c5b0cba1

Why

Part of the org-wide effort to move third-party GitHub Actions onto StepSecurity-maintained forks.

This repo is a fork, and GitHub code search does not index forks — so it was invisible to the earlier
sweeps. It is the last place in freshaengineering still calling runs-on/cache, runs-on/cache/save or
runs-on/cache/restore from a workflow.

Why this is safe

  • The action contract is unchanged. Diffing action.yml, save/action.yml and restore/action.yml
    between runs-on/cache@a5f51d6f and step-security/runs-on-cache@c5b0cba1, the only differing line in
    all three files is author:. Identical inputs, outputs and runs.using: 'node24' — no node jump. The
    # v4# v5.0.7 comment change is a label correction, not a behaviour change.
  • The RunsOn S3 cache backend is preserved. Every job here runs on RunsOn runners
    (runs-on,runner={1,2,4}cpu-linux-x64), so the S3-backed cache path is load-bearing. The fork keeps
    RUNS_ON_S3_BUCKET_CACHE detection in src/restoreImpl.ts / src/saveImpl.ts and in the shipped
    dist/ bundles, including the automatic fallback to GitHub's cache service.
  • Proven on the identical template. Our sister Elixir fork freshaengineering/libcluster runs the same
    CI template and has been on step-security/runs-on-cache@c5b0cba1 # v5.0.7 (root, /save and
    /restore) with green builds.
  • No subscription gate. This repo is public, so the fork reports "Free for public repositories" rather
    than requiring a StepSecurity subscription.

Note we deliberately did not use the fork's v4 tag: its save/ and restore/ are node20, which
would be a regression from the node24 currently pinned here.

Notes

  • .github/actions.lock.yaml is generated and deliberately untouched; it refreshes when its generator
    next runs, so it will keep naming runs-on/cache for a while. That is expected, not a failed swap — and
    it's why StepSecurity's dashboard lags behind.
  • Open bot PR [StepSecurity] Apply security best practices #28 ("[StepSecurity] Apply security best practices") touches the same two files but only
    inserts harden-runner steps — different hunks, no overlap with these lines. Whichever merges second may
    need a trivial rebase.

rehan-fr added 2 commits July 28, 2026 17:25
5 refs in ci.yml (4x root cache, 1x cache/save) move to
step-security/runs-on-cache@c5b0cba1 # v5.0.7 - the sha the rest of the
org already pins. action.yml is byte-identical apart from author:, and
the RunsOn S3 backend (RUNS_ON_S3_BUCKET_CACHE) is preserved.
@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedstep-security/​runs-on-cache@​c5b0cba15d05488ebc630ad8d1e95b3d9b35ac7366100100100100

View full report

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