Skip to content

fix: bump suite to reusable-security-suite@v1.22.0 (activate warn-only pinact + config fallback) - #79

Merged
dkastl merged 1 commit into
mainfrom
fix/suite-bump-reusable-ref
Jul 1, 2026
Merged

fix: bump suite to reusable-security-suite@v1.22.0 (activate warn-only pinact + config fallback)#79
dkastl merged 1 commit into
mainfrom
fix/suite-bump-reusable-ref

Conversation

@dkastl

@dkastl dkastl commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

What

Bumps the security-suite.yml thin caller from reusable-security-suite@v1.20.0 to @v1.22.0.

Why (latent bug)

The ruleset (security-suite.yml@v1) was running a stale chain:

security-suite.yml@v1 → reusable-security-suite@v1.20.0 (pre-PR-Y)
                      → reusable-pinact-check@v1.18.0   (gating, no config fallback)

So #75 (centralized .pinact.yml fallback) and #76 (warn-only pinact) were inert — the ruleset kept running the old gating pinact with no fallback. This bump repoints to @v1.22.0, which resolves to reusable-pinact-check@v1.21.0 with fail-on-findings: false, so the suite finally runs pinact warn-only + centralized-config.

Note: internal pin maintenance

This is an internal same-repo reusable pin. Dependabot's github-actions ecosystem generally does not bump same-repo reusable-workflow refs, which is why it went stale. Options to avoid recurrence: keep bumping it manually when reusable-security-suite changes, or track it another way. Flagging for awareness.

Part of geolonia-operations#196.

Validation

  • actionlint clean; confirmed the resolved chain reaches pinact@v1.21.0 + fail-on-findings: false.
  • This PR's own CI runs the new chain on .github (first live exercise of warn-only pinact + fallback).

Summary by CodeRabbit

  • Chores
    • Updated the project’s security workflow to use a newer approved version of the reusable security suite.

The thin caller still pinned @v1.20.0 (pre-PR-Y), so the ruleset was
running the old suite: pinact@v1.18.0, gating, no centralized-config
fallback. #75 (central config) and #76 (warn-only pinact) were therefore
inert. Bump to @v1.22.0 so the ruleset actually runs pinact with
fail-on-findings: false + the .pinact.yml fallback.
@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 04ad7a7d-cf67-4ccc-9d6e-9125d013708c

📥 Commits

Reviewing files that changed from the base of the PR and between d32aef4 and d4bb9d7.

📒 Files selected for processing (1)
  • .github/workflows/security-suite.yml

Walkthrough

This PR updates a single line in a GitHub Actions workflow file, bumping the pinned commit reference for the reusable security suite workflow from v1.20.0 to v1.22.0.

Changes

Reusable Workflow Version Bump

Layer / File(s) Summary
Pinned version update
.github/workflows/security-suite.yml
The suite job's reference to reusable-security-suite.yml is updated from the v1.20.0 pinned commit to the v1.22.0 pinned commit.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • geolonia/.github#71: Introduced the reusable-security-suite.yml wiring that this PR's version bump references.
  • geolonia/.github#73: Introduced the reusable-security-suite.yml workflow that this PR updates the pinned version for.
  • geolonia/.github#74: Also modifies security-suite.yml to reference the same reusable workflow with a different version bump.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly states the main change: bumping the reusable security suite to v1.22.0, with useful context about the behavior change.
Description check ✅ Passed The description is mostly complete: it explains the change, rationale, validation, and related issue, though it doesn't use the template headings exactly.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/suite-bump-reusable-ref

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Secret Leak Check

OK No secrets detected in this PR's diff.

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

🛡️ Security suite

Check Result
✅ Supply chain · bumblebee No exposure matches
✅ Secrets · betterleaks No secrets in diff
✅ Action pinning · pinact All actions pinned
⚠️ Actions audit · zizmor Findings (warn-only; see annotations)

Warning

Warnings only. These do not block the PR, but please review them.

Updated for d4bb9d7 · workflow run

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.

1 participant