Skip to content

chore(ci): remove one-off CodeQL dismissal workflow - #50

Merged
JOY (JOY) merged 1 commit into
devfrom
chore/remove-oneoff-workflow
Sep 21, 2026
Merged

JOY (JOY) merged 1 commit into
devfrom
chore/remove-oneoff-workflow

Conversation

@JOY

Copy link
Copy Markdown

What kind of change does this PR introduce?

CI/CD. Removes .github/workflows/oneoff-dismiss-alert.yml (added in #48/#49). Nothing else changes.

Why was this change needed?

The one-off workflow proved the point it was created for: its debug run showed GITHUB_TOKEN can read code scanning alerts but the dismissal endpoint rejects it with 404 even under security-events: write - GitHub restricts alert dismissal to user tokens for the dismissed_by audit trail. The dismissal therefore needs the operator user token with security_events scope; the temporary workflow has no further use and is removed to keep .github/workflows clean. Alert 1567 stays open with the containment guard from PR #43 in place until the operator-scope dismissal happens.

Technical Details & Scope

  • Deletes the workflow file. No application code, no other workflows.

Verification & Testing

The run https://github.com/DOS/Crove-Post/actions/runs/35622844568 contains the debug evidence (GET succeeds, PUT 404). CI on this PR confirms nothing else breaks.

QA

  1. Merge this PR - .github/workflows/oneoff-dismiss-alert.yml disappears from dev
  2. CI: build.yml and branding-guard.yml green on this PR

Checklist:

  • My code follows the project's code style and architectural conventions.
  • Local verification n/a (file deletion; CI gates on this PR).
  • Branding guard - CI gate on this PR.
  • Tests unaffected.
  • Documentation n/a.
  • No secrets included.
  • QA section filled with real steps.

GITHUB_TOKEN can read code scanning alerts but cannot dismiss them: the
dismissal endpoint is restricted to user tokens for the dismissed_by audit
trail and returns 404 even with security-events: write. The dismissal needs
the operator user token with security_events scope instead. Alert 1567 stays
open until then; the containment guard from PR #43 remains in place.
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@JOY
JOY (JOY) merged commit 555b7b8 into dev Sep 21, 2026
5 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