ci: add secret scanning - #6
Conversation
PR SummaryLow Risk Overview Adds local pre-commit secret checks by introducing Reviewed by Cursor Bugbot for commit 462c315. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 462c315. Configure here.
| "format:check": "prettier --check .", | ||
| "format:fix": "prettier --write --ignore-unknown ." | ||
| "format:fix": "prettier --write --ignore-unknown .", | ||
| "prepare": "lefthook install --force || true" |
There was a problem hiding this comment.
Windows prepare ignore fails
Low Severity
The new prepare script uses || true so a failed lefthook install does not break dependency install. On Windows, npm/yarn run scripts via cmd.exe, where true is not a command, so when install fails the ignore path fails too and yarn install exits non-zero. This repo’s prerequisites steer Windows users to Command Prompt, not Git Bash.
Reviewed by Cursor Bugbot for commit 462c315. Configure here.


Adds the org secret scanning workflow, calling the reusable gitleaks scan in
pipelabs/tooling, per the secrets hygiene guideline. It scans commits introduced by PRs and pushes, not existing history. After merge,secret-scan / gitleaksbecomes a required status check onmain, which also means direct pushes to that branch will be blocked in favour of PRs.Also adds
lefthookwith apreparescript so pre-commit secret scanning installs on the next dependency install (each dev needsbrew install gitleaksonce).