Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions .github/codeql/codeql-config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,3 +31,11 @@ paths-ignore:
- '**/dist/**'
- '**/.next/**'
- 'apps/docs/content/**'

# Do NOT add `queries:`, `packs:`, `query-filters:`, or `disable-default-queries`
# here to try to speed the scan up. Under the code-scanning feature flag the
# action's checkOverlayAnalysisFeatureEnabled treats any of those as
# OverlayDisabledReason.NonDefaultQueries and permanently turns off overlay
# (incremental) analysis. Extraction is ~53% of a run and is exactly what overlay
# skips, so scoping the queries trades a documented up-to-10x win for a few
# percent off the 27% query phase.
20 changes: 18 additions & 2 deletions .github/workflows/codeql.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,16 @@ on:
# created, prevents developers from introducing new vulnerabilities."
push:
branches: [main]
# main only, not staging. Feature PRs land on staging and are ~90% of PR scan
# volume, and every one of them is scanned again — against the exact tree being
# promoted — when the staging->main PR opens. Scanning at the promotion
# boundary defers the signal rather than dropping it.
#
# Deliberately a branch cut and not an activity-type cut: dropping
# `synchronize` would have scanned each PR's first commit and never its final
# state, which is backwards, since review fixups land in later pushes.
pull_request:
branches: [main, staging]
branches: [main]
# `ready_for_review` is not a default activity type, so it has to be listed
# alongside the defaults it replaces. Without it, a PR opened as a draft and
# then marked ready is skipped by the job-level draft guard and never
Expand All @@ -41,7 +49,15 @@ on:
# Safety net behind the push trigger, and the thing that keeps the
# default-branch alert view fresh when main is quiet. Only fires once this
# file is on the default branch — schedule events ignore other branches.
- cron: '17 8 * * 1'
#
# Daily rather than weekly. Pushes to main are rare, and with PR scans now
# limited to main the alert view leans on this more than it used to; a week
# is too long to leave it stale. It also reseeds the overlay-base database
# that PR runs restore from — that cache key embeds the CodeQL bundle
# version, so a bundle bump invalidates it, and an unused Actions cache is
# evicted after 7 days. One 8 vCPU default-branch scan a day is a few
# dollars a month against a PR scan that halves when the base is warm.
- cron: '17 8 * * *'
Comment thread
waleedlatif1 marked this conversation as resolved.
workflow_dispatch:

concurrency:
Expand Down
Loading