Skip to content

fix(gitleaks): report scans blocked by tool setup failures - #51

Merged
posquit0 merged 1 commit into
mainfrom
fix/gitleaks-setup-failure-report
Nov 9, 2026
Merged

posquit0 merged 1 commit into
mainfrom
fix/gitleaks-setup-failure-report

Conversation

@posquit0

@posquit0 posquit0 commented Nov 9, 2026

Copy link
Copy Markdown
Member

👋 Background

When mise or Gitleaks installation fails, the secret scan never runs. The job fails, but its existing scan-result reporting steps are skipped, leaving no explanation in the job summary.

🔗 Related Issues

⚙️ Description

Add Report tool setup failure with id: setup-failure. On tool-setup failure, write a “Secret Scan Not Run” job summary and an error annotation explaining that no scan result is available and pointing to the setup logs and a rerun after resolving the installation error.

The setup step remains fatal. The report runs with failure(); subsequent normal scan steps are skipped by their implicit success() condition, and the job remains failed. Post-job cleanup can still run.

🚨 To Reviewers

  • Same 16-line change as datarize-infra/github-actions#30, including the step ID.
  • Validation: actionlint, yamllint, workflow schema validation and git diff --check passed.
  • Executed the actual report shell script with a temporary GITHUB_STEP_SUMMARY and confirmed the summary and error annotation.

@github-actions github-actions Bot added 💾 workflows This issue or pull request is related to workflows. size/S Small size issue or PR. labels Nov 9, 2026
@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown

GitHub Actions Integration

Repository jactionlint check-jsonschema (workflows) check-jsonschema (actions)
tedilabs/github-actions ✅ ✅ ✅

✅ All 1 passed.

@posquit0
posquit0 merged commit cf22656 into main Nov 9, 2026
7 checks passed
@posquit0
posquit0 deleted the fix/gitleaks-setup-failure-report branch November 9, 2026 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💾 workflows This issue or pull request is related to workflows. size/S Small size issue or PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant