Skip to content

ci(scorecard): code-scanning alerts vermeiden#60

Merged
tomtastisch merged 2 commits intomainfrom
codex/fix/scorecard-no-sarif-code-scanning
Feb 14, 2026
Merged

ci(scorecard): code-scanning alerts vermeiden#60
tomtastisch merged 2 commits intomainfrom
codex/fix/scorecard-no-sarif-code-scanning

Conversation

@tomtastisch
Copy link
Owner

@tomtastisch tomtastisch commented Feb 14, 2026

Ziel & Scope

  • Main-CI wieder gruen bekommen, ohne Copilot abzuschalten.
  • Ursache: OpenSSF Scorecard SARIF Upload erzeugt Code-Scanning-Alerts (TokenPermissionsID) und blockiert Preflight-Gate.

Umgesetzte Aufgaben (abhaken)

  • Entfernt: Upload von artifacts/ci/scorecard/results.sarif nach Code Scanning
  • Entfernt: security-events: write im Scorecard-Workflow (nicht mehr benoetigt)
  • Beibehalten: deterministic Scorecard-Run + Upload von ci-scorecard Artefakt
  • Erwartung: keine neuen Scorecard Code-Scanning-Alerts

Nachbesserungen aus Review (iterativ)

  • PR-Body auf Governance-Pflichtsektionen angepasst
  • Falls CI weitere Findings meldet: sequenziell nachziehen

Security- und Merge-Gates

  • Pflichtaussage: security/code-scanning/tools muss bei Merge 0 offene Alerts haben.

Evidence (auditierbar)

  • Datei: .github/workflows/scorecard.yml
  • Checks:
    • gh api /repos/tomtastisch/FileClassifier/code-scanning/alerts?state=open --paginate | jq length

DoD (mindestens 2 pro Punkt)

  • Keine neuen Scorecard TokenPermissions Code-Scanning-Alerts auf main
  • Preflight (CI) laeuft ohne Deadlock durch Scorecard Alerts

Copilot AI review requested due to automatic review settings February 14, 2026 16:36
@github-actions github-actions bot added area:pipeline ci CI/workflow change impl:config versioning:patch Fix/Refactor/Docs/CI/Tooling; requires PATCH bump labels Feb 14, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Scorecard GitHub Actions workflow to avoid generating GitHub Code Scanning alerts from the OpenSSF Scorecard SARIF upload, unblocking the preflight gate while keeping the Scorecard run and artifact output.

Changes:

  • Removed the github/codeql-action/upload-sarif step for Scorecard SARIF results.
  • Dropped security-events: write permission from the Scorecard job.
  • Kept the deterministic Scorecard run and upload of the ci-scorecard artifact.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tomtastisch tomtastisch merged commit 6fcfa09 into main Feb 14, 2026
29 checks passed
@tomtastisch tomtastisch deleted the codex/fix/scorecard-no-sarif-code-scanning branch February 14, 2026 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:pipeline ci CI/workflow change impl:config versioning:patch Fix/Refactor/Docs/CI/Tooling; requires PATCH bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant