Skip to content

Commit f33d007

Browse files
cliffhallclaude
andcommitted
fix: record the private-fork advisory PR as an AGENTS.md exception
AGENTS.md said every PR links a public issue and that no PR but Dependabot's is exempt, which contradicted the security-advisory skill's private-fork fix. Add a narrow exception beside the Dependabot one: a PR in an advisory's private fork links no public issue, carries no labels and is tracked by the [GHSA-...] draft card, while still targeting v2/main. The skill points at it. Refs #4870 Signed-off-by: cliffhall <[email protected]> Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
1 parent ef5a764 commit f33d007

2 files changed

Lines changed: 12 additions & 2 deletions

File tree

‎.claude/skills/security-advisory/SKILL.md‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,9 @@ publication**, never merely after the release (step 6).
3232

3333
⚠️ **The PR flow does not apply to the fix itself.** It needs a public issue
3434
and a public PR against `v2/main`, which is the disclosure this flow exists to
35-
delay. The fix is reviewed inside the private fork (step 4).
35+
delay. The fix is reviewed inside the private fork (step 4), and `AGENTS.md`
36+
records that fork PR as an exception to its PR rules (no public issue link, no
37+
labels, tracked by the draft card).
3638

3739
## The flow
3840

‎AGENTS.md‎

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,15 @@ holds for a maintainer's own one-line fix as much as for a feature.
168168
linked issue, no `v2` label and no checklist. They are exempt from the PR rules
169169
in this file (issue link, base branch, labels, checklist, board) until the
170170
change that replaces them with issue-filing sweeps lands and removes this
171-
exception. No other PR is exempt.
171+
exception.
172+
- **Exception: a security advisory's fix, in its private fork.** A PR in the
173+
advisory's temporary private fork (`servers-<ghsa-id>`) links no public
174+
issue, carries no labels, and is tracked by the advisory's `[GHSA-…]` draft
175+
card rather than an issue's card, because each of those would disclose the
176+
vulnerability before a fix ships. It still targets `v2/main`, and it is
177+
reviewed inside the fork. The flow is the `security-advisory` skill.
178+
179+
No other PR is exempt.
172180

173181
Every PR answers the checklist in
174182
[`.github/pull_request_template.md`](./.github/pull_request_template.md):

0 commit comments

Comments
 (0)