Skip to content

ci(security): add org secret scan and pre-publish guard - #504

Merged
Pigbibi merged 1 commit into
mainfrom
ci/org-secret-scan-20261101
Nov 1, 2026
Merged

Pigbibi merged 1 commit into
mainfrom
ci/org-secret-scan-20261101

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

AI assistance

Grok Bot / Codex coordinated change by 云端运维对账 on behalf of Li SiYi.

What changed

Adds the org shared secret scan (QuantStrategyLab/.github #29, squash 0da4da68, pinned by full SHA):

  • required-gitleaks.yml: PR + push to main; check name Secret scan / Gitleaks.
  • weekly-gitleaks.yml: Mon 04:23 UTC, all branches and full history (covers logs/state).
  • Adds callers only; artifact uploads are not changed here (follow-up). The local Gitleaks job is kept so the dependabot gate list stays valid. The fee-schedule bump only rewrites date pins and goes through a PR that runs the PR scan.
    The existing Gitleaks job is untouched. Making the new check required is a separate settings step, done after it is green on main.

How verified

actionlint on all workflows: clean. No secrets read or printed. CI results below once run.

Why

A secret reached a public logs branch because no scan guarded non-PR publishing. See the org leak root-cause work.

Risk

Tier: workflow change (needs owner-approved label per risk-gate). Publishing path now fails closed if the scan finds a secret; weekly scan only reads.

Pin the shared QuantStrategyLab/.github scan at 0da4da68 and scan data-branch content before push.
@coderabbitai

coderabbitai Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d3400384-cf4a-4534-bf3b-17ab06dc2414
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@Pigbibi
Pigbibi marked this pull request as ready for review November 1, 2026 16:22
@Pigbibi Pigbibi added the owner-approved Owner approval for high-risk PRs. AI agents must never apply this label. label Nov 1, 2026
@Pigbibi
Pigbibi merged commit a3fc00c into main Nov 1, 2026
10 of 12 checks passed
@Pigbibi
Pigbibi deleted the ci/org-secret-scan-20261101 branch November 1, 2026 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

owner-approved Owner approval for high-risk PRs. AI agents must never apply this label.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant