Skip to content

docs(claude): replace the required-context count with the measured list #1965

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
seonghobae wants to merge 1 commit into
base: main
Choose a base branch
from
+13 −4
Open

docs(claude): replace the required-context count with the measured list #1965

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
17 changes: 13 additions & 4 deletions CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -160,10 +160,19 @@ repeatable compile command.
in each target repository's context and discards its `paths`, `paths-ignore`, `branches`, and
`types` there (confirmed live: `bandscope` has no local `codeql-pr.yml`/`strix.yml`/
`security-scan.yml`, yet ruleset-injected runs of all three exist). `.github` is excluded from
that ruleset and instead uses classic branch protection with 14 named required contexts, where a
path-filtered workflow leaves its context Pending forever. Never add a trigger-level filter to a
required workflow; skip at job level via a `changed-scope` gate job instead, and always keep one
job with no output-dependent `if:` so the run concludes `success` rather than `skipped`. See
that ruleset and instead uses classic branch protection with a fixed list of named required
contexts, where a path-filtered workflow leaves its context Pending forever. Measured from
`branches/main/protection` on 2026-09-06, that list is these 12: `CodeQL compatibility analysis
(actions)`, `CodeQL compatibility analysis (python)`, `Detect CodeQL languages`,
`coverage-evidence`, `dependency-review`, `noema-review`, `opencode-review`, `osv-scan`,
`required-workflow-bootstrap`, `scan-pr-queue`, `scorecard`, `trivy-fs`. Re-read the endpoint
rather than trusting this list: branch-protection edits leave no trace in git, so a stale list
here cannot be detected by reading the repository. This sentence previously said "14 named
required contexts" and had never been revised since `7d093881` (#1766) introduced it; that is a
correction of an unverifiable count, **not** evidence that two contexts were removed. Never add a
trigger-level filter to a required workflow; skip at job level via a `changed-scope` gate job
instead, and always keep one job with no output-dependent `if:` so the run concludes `success`
rather than `skipped`. See
`docs/doctoring/required-workflow-path-filter-boundary.md`.
- **Narrowing a PR does not carry its delta automatically.** When a large PR is split into
successors, diff the union of the successors against the original before treating the supersession
Expand Down
Loading