Skip to content

chore(guard): sync vendored public-repo-guard to canonical - #61

Open
wave-av-release-bot[bot] wants to merge 1 commit into
mainfrom
chore/guard-canonical-sync
Open

wave-av-release-bot[bot] wants to merge 1 commit into
mainfrom
chore/guard-canonical-sync

Conversation

@wave-av-release-bot

@wave-av-release-bot wave-av-release-bot Bot commented Aug 4, 2026

Copy link
Copy Markdown

Syncs the vendored public-repo-guard trio to the canonical source in wave-foundation/scaffolder/public-repo-guard.

  • adds the internal-ip leak rule (Tailscale-CGNAT 100.64.0.0/10), lockstep with the pre-publish mirror gate
  • reconciles accumulated drift in the vendored copy

Each changed file is byte-for-byte identical to canonical (verified by git blob SHA). The repo's own Secrets + content policy gate re-scans this PR.

🤖 Generated with Claude Code


Open in Devin Review

Summary by cubic

Syncs the vendored public-repo-guard to the canonical wave-foundation/scaffolder/public-repo-guard to keep leak rules aligned and tighten the CI scan. Updates the guard workflow to use actions/checkout v7 and stop persisting the job token.

  • New Features

    • Add internal‑IP leak rule for Tailscale CGNAT 100.64.0.0/10 (mirrors the pre‑publish gate).
  • Dependencies

    • Upgrade actions/checkout to v7.0.0.
    • Set persist-credentials: false to drop the job token before running gitleaks over the working tree.

Written for commit 6b125d5. Summary will update on new commits.

Review in cubic

@greptile-apps

greptile-apps Bot commented Aug 4, 2026

Copy link
Copy Markdown

PR author is in the excluded authors list.

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

Open in Devin Review

runs-on: ubuntu-latest
steps:
- uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5.0.1
- uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔍 Verify the pinned commit SHA actually corresponds to v7.0.0

The commit SHA 9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 is annotated as v7.0.0. Since this workflow is the repo's security gate and the pin is the only integrity guarantee for the action, it is worth confirming the SHA resolves to the tagged release in actions/checkout (the rest of the job verifies gitleaks by SHA-256, so a mismatched action pin would be the weakest link).

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

@macroscopeapp

macroscopeapp Bot commented Aug 4, 2026

Copy link
Copy Markdown

Approvability

Verdict: Needs human review

Unable to check for correctness in 6b125d5. This CI workflow change (checkout version bump + security hardening) is low-risk, but the modified file is owned by wave-av/streaming-team and the bot author is not a designated owner. The CODEOWNERS team should review.

You can customize Macroscope's approvability policy. Learn more.

@wave-bugbot

wave-bugbot Bot commented Aug 12, 2026

Copy link
Copy Markdown

🟢 WAVE BugBot — clear

No confident findings on the changed lines.

Local review · $0 inference · wave-dispatch

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.

0 participants