Skip to content

[repo] fix unpublished CodeQL alert-suppression pack reference - #226

Merged
sethwv merged 1 commit into
Dispatcharr:mainfrom
sv-dispatcharr:main
Aug 12, 2026
Merged

[repo] fix unpublished CodeQL alert-suppression pack reference#226
sethwv merged 1 commit into
Dispatcharr:mainfrom
sv-dispatcharr:main

Conversation

@sethwv

@sethwv sethwv commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Follow-up to [repo] honor inline CodeQL suppression comments, dedupe repeated finding text #225: advanced-security/<lang>-alert-suppression isn't published to the CodeQL registry, so codeql database init failed with "not found in the registry" on the first live run.
  • Point at codeql/<lang>-queries:AlertSuppression.ql instead - that's the built-in suppression query already shipped inside the same codeql/<lang>-queries pack security-extended resolves against, so no extra registry fetch is needed.

…location

advanced-security/*-alert-suppression isn't published to the CodeQL
registry, so codeql database init failed with "not found in the registry"
on the first live run. AlertSuppression.ql already ships inside each
language's standard codeql/<lang>-queries pack (the same pack
security-extended resolves against), so point at it there instead - no
extra registry fetch required.
@dispatcharr-plugins-bot dispatcharr-plugins-bot Bot added the Repo Update Scripts or other repository infrastructure. label Aug 12, 2026
@sethwv
sethwv merged commit 86f0c36 into Dispatcharr:main Aug 12, 2026
10 checks passed
sethwv added a commit to swvn-dispatch/PluginsRepoV2 that referenced this pull request Aug 12, 2026
…tcharr#226) to pluginctl

Rebasing onto upstream/main pulled in PR Dispatcharr#225's AlertSuppression fix and
PR Dispatcharr#226's follow-up, but both only touched the old inline CodeQL block in
validate-plugin.yml, which the pluginctl rewrite (4973a50) had already
replaced with _codeql-scan.yml. Port the same fix into that architecture:
point CodeQL init at codeql-config.yml (sourced from the trusted base-branch
checkout, not the untrusted PR merge tree) and dedupe repeated data-flow
sentences in pluginctl's process_message, matching PR Dispatcharr#225's jq logic.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Repo Update Scripts or other repository infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant