Skip to content

Add a leak check to CI - #1

Merged
mgilardi merged 1 commit into
mainfrom
add-leak-check
Nov 8, 2026
Merged

mgilardi merged 1 commit into
mainfrom
add-leak-check

Conversation

@mgilardi

@mgilardi mgilardi commented Nov 8, 2026

Copy link
Copy Markdown
Contributor

Adds scripts/leak-check.sh and runs it in CI right after checkout.

  • CI: fails on email addresses other than noreply, example.com and the reserved .example/.test domains, and on committed binaries or build output (jars, zips, dist/, build/, node_modules/ and so on).
  • Locally: also checks for private terms from a file outside the repo, named by LEAK_PRIVATE_TERMS (default ~/.leak-private-terms). It reports file and line only, never the term. No terms are in the repo or in CI.
  • scripts/leak-allow.txt, if added later, skips lines containing a listed public phrase.

Tested: passes on the current tree without a terms file. The terms path was exercised with a stand-in file only.

🤖 Generated with Claude Code

Every public repo should run a leak check. In CI it fails on email addresses that aren't noreply or placeholders and on committed binaries or build output. Run locally, it also checks for private terms read from a file outside the repo (LEAK_PRIVATE_TERMS, default ~/.leak-private-terms), and reports only file and line, never the term.

It passes on the current tree without a terms file. The private-terms path was exercised with a stand-in file only; no terms file at the default path exists yet.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mgilardi
mgilardi merged commit 8bdc3f2 into main Nov 8, 2026
2 checks passed
@mgilardi
mgilardi deleted the add-leak-check branch November 8, 2026 05:16
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