ci: CodeQL buildless (build-mode none) for C# + web TypeScript - #411
Merged
Conversation
Rework codeql.yml before it first runs: switch both languages to buildless extraction (build-mode: none) and matrix the two shipped codebases — csharp (the .NET product) and javascript-typescript (web/packetnet-ui). Buildless C# drops the full Release compile, so the workflow no longer needs the exclusive flock — much lighter on the shared self-hosted runner. Keep [self-hosted, Linux, X64]; add packages: read for private query packs; bump the action to v4. Declined GitHub's generated "Advanced setup" template: it targets ubuntu-latest/macos-latest (violates the no-hosted-runners rule) and would have overwritten this file. The spec dirs (python/rust/c) stay out of scope. https://claude.ai/code/session_01GSRLZXowGYeRieqaevEtBb
This was referenced Jun 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refines the
codeql.ymlshipped in #409 before it first runs, after comparing against GitHub's auto-generated "Advanced setup" template.Changes
build-mode: none. C# buildless skips the full Release compile entirely, so the workflow no longer needs the exclusiveflock -x /tmp/pdn-ci-heavy.lock— much lighter on the contended shared self-hosted runner. Small completeness cost vs a traced build, acceptable for a first cut.csharp(the .NET product) andjavascript-typescript(web/packetnet-ui, the served control-panel SPA). Thepython-spec/rust-spec/c-specdirs stay out of scope (reference artifacts, not product).runs-on: [self-hosted, Linux, X64]; addedpackages: read(private query packs); bumped the action to@v4.Why not GitHub's generated template
GitHub's "Advanced setup" suggestion targets
ubuntu-latest/macos-latest, which violates this project's hard rule (no GitHub-hosted runners — zero minutes budget), and it writes to the samecodeql.ymlpath, so it would have overwritten the merged workflow. Declined; folded in only its good ideas (buildless C#,packages: read,@v4).Caveat (unchanged from #409)
SARIF upload still requires code scanning enabled on the repo (GitHub Advanced Security on a private repo). Until that's on, the
analyzestep's upload fails. Workflow is correct regardless.YAML validated; matrix =
[csharp, javascript-typescript]. Workflow-only change.https://claude.ai/code/session_01GSRLZXowGYeRieqaevEtBb
Generated by Claude Code