Skip to content

ci: fleet hygiene checks (gitleaks, PII/host, artifacts, shellcheck) - #451

Merged
shuv1337 merged 4 commits into
integration-v2from
fleet-hygiene
Oct 16, 2026
Merged

shuv1337 merged 4 commits into
integration-v2from
fleet-hygiene

Conversation

@shuv1337

@shuv1337 shuv1337 commented Oct 16, 2026 •

Copy link
Copy Markdown
Owner

FM-PYRAMID-2, fix 2: one shared fleet hygiene workflow.

  • Reusable workflow: shuv1337/shark/.github/workflows/fleet-hygiene.yml (public, so private repos can call it). One ubuntu-latest job, 10 min timeout, concurrency cancel.
  • Checks: gitleaks 8.21.2 on the tracked tree (checksum-verified), PII and host details (*.ts.net, 100.64/10 Tailscale IPs, personal-domain emails other than an allowlist, /home/<user> and /Users/<user>), spike/harness/scratch files, and shellcheck -S warning on *.sh.
  • Exceptions: .github/hygiene-allowlist.txt and .gitleaks.toml. Existing hits were added as a baseline.
  • Known follow-ups: the gitleaks baseline allows whole files, callers are pinned to a commit on shark's fleet-hygiene branch (repin to main once the shark PR merges), and the allowlist header text is stale.

This repo

Internal host details in two docs were replaced with placeholders and removed from the allowlist. The remaining baseline files are upstream fixtures and docs. All gitleaks matches were reviewed: upstream test tokens, a public OAuth client secret, i18n false positives, and a signed media URL in a recording fixture.

@shuv1337
shuv1337 merged commit ec19acb into integration-v2 Oct 16, 2026
4 checks passed
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