Skip to content

Repo hygiene: CI leak-canary + drop dead code/author-probes + macOS guard#16

Merged
0xadvait merged 1 commit into
mainfrom
repo-hygiene-ci
Jun 23, 2026
Merged

Repo hygiene: CI leak-canary + drop dead code/author-probes + macOS guard#16
0xadvait merged 1 commit into
mainfrom
repo-hygiene-ci

Conversation

@0xadvait

Copy link
Copy Markdown
Collaborator
  • CI (.github/workflows/test.yml): runs the scrubber suite incl. the golden leak canary on every PR + push to main, so a future PII/secret-leak regression fails in CI.
  • Dead code: remove privacy/gen_searxng_settings.py (never invoked; the search path is ddgs, not SearXNG) and the unused mkdir searxng in install.sh.
  • Author-specific probe: remove the NordVPN status check in bin/ghost (it grepped a literal NordVPN DNS subnet -- every other user just saw a meaningless ⚠️ no NordVPN).
  • macOS guard: install.sh fails fast on non-macOS (it uses launchd) instead of part-installing then erroring confusingly.
  • README file-map updated (drop the removed module; list scrub_patterns.py + presidio_scrub.py).

…, macOS guard

- Add .github/workflows/test.yml: runs the scrubber suite (incl. the golden leak
  canary) on every PR + push to main, so a future leak regression is caught in CI.
- Remove dead code: privacy/gen_searxng_settings.py (never invoked; SearXNG isn't
  the search path -- ddgs is) and the unused 'mkdir searxng' in install.sh.
- Remove the author-specific NordVPN status probe in bin/ghost (grepped a literal
  NordVPN DNS subnet -- meaningless '⚠️ no NordVPN' noise for every other user).
- install.sh now fails fast on non-macOS (it uses launchd) instead of part-
  installing and erroring confusingly later.
- README: fix the file map (drop the removed module, list scrub_patterns + presidio).
@0xadvait
0xadvait merged commit c049d33 into main Jun 23, 2026
@0xadvait
0xadvait deleted the repo-hygiene-ci branch June 23, 2026 19:59
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