Skip to content

docs: record the CodeQL advanced-setup decision for apps - #47

Merged
jdwillmsen merged 2 commits into
mainfrom
docs/JDWLABS-450-codeql-decision
Aug 29, 2026
Merged

docs: record the CodeQL advanced-setup decision for apps#47
jdwillmsen merged 2 commits into
mainfrom
docs/JDWLABS-450-codeql-decision

Conversation

@jdwillmsen

Copy link
Copy Markdown
Member

What

  • docs/code-scanning-strategy.md: new "Decision: CodeQL advanced setup, not default setup" section with the measured numbers, and a since-then note on the baseline bullet.
  • docs/ci-standards.md: apps inventory row swaps prune-actions-cache.yml for codeql.yml.

Numbers behind the decision (measured 2026-08-29)

  • prune workflow: 447 runs / 28 days = 112 runs/week, ~119 billed runner-min/week.
  • Actions cache: 25 codeql-* entries = 3.88 GiB of 5.83 GiB total (67%).
  • Open CodeQL alerts lost by switching: 0 (2 open alerts, both Trivy). Closed CodeQL history (2) is retained.
  • Per-push CodeQL cost unchanged (same four extractors).

Companion change: jdwlabs/apps#209 adds codeql.yml and deletes the prune workflow. Merge that one first; this doc describes the post-merge state.

Refs JDWLABS-450.

🤖 Generated with Claude Code

https://claude.ai/code/session_016XRddumw4ZNvFqHSd9KSf6

jdwillmsen and others added 2 commits August 29, 2026 04:55
Default-setup CodeQL wrote a never-repeating dependency cache entry per
analysis, and the hourly prune workflow in apps existed only to clean up
after it. Record why apps moved to a checked-in advanced-setup workflow
(dependency caching off at the source, zero open CodeQL alerts to lose,
same per-push cost) so later audits stop re-flagging the prune workflow,
and swap the inventory row from prune-actions-cache.yml to codeql.yml.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016XRddumw4ZNvFqHSd9KSf6
The checked-in workflow is code-scanning only; the action rejects
analysis-kinds in custom workflows.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016XRddumw4ZNvFqHSd9KSf6
@jdwillmsen
jdwillmsen merged commit b8f6068 into main Aug 29, 2026
5 checks passed
@jdwillmsen
jdwillmsen deleted the docs/JDWLABS-450-codeql-decision branch August 29, 2026 05:06
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