Skip to content

Commit 9eebda4

Browse files
Merge branch 'main' into dependabot/pip/myst-parser-gte-4.0.1-and-lt-6
2 parents ed421ae + 9ce367c commit 9eebda4

4 files changed

Lines changed: 59 additions & 5 deletions

File tree

.github/labels.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,14 @@
117117
description: Trigger Claude Code automation
118118
color: "b41d8f"
119119

120+
- name: claude:review:passed
121+
description: Automated Claude PR review found no blocking issues on the current head commit
122+
color: "0e8a16"
123+
124+
- name: claude:review:failed
125+
description: Automated Claude PR review found blocking issues on the current head commit
126+
color: "b60205"
127+
120128
- name: copilot
121129
description: GitHub Copilot related
122130
color: "e6dac6"

.github/workflows/claude-code-automation-pr-review.yml

Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -214,6 +214,49 @@ jobs:
214214
215215
Use `gh pr comment` with your Bash tool to leave your comprehensive review as a comment on the PR.
216216
217+
## Machine-Readable Verdict (MANDATORY)
218+
219+
After posting your review comment, you MUST emit a single-label verdict on the PR. This label is consumed by branch-protection rules to gate auto-merge — it is the only deterministic signal of your review outcome.
220+
221+
**Verdict criteria** (all must hold for PASS):
222+
223+
- No blocking findings under "CRITICAL CHECKS" — i.e. no missing test markers, no coverage drop below 85%, no `make lint` failures, no conventional-commit violations.
224+
- No blocking architecture or security violations under "Repository-Specific Review Areas".
225+
- Suggestions / nice-to-haves do NOT block the verdict.
226+
227+
If any blocking finding remains: verdict is **FAIL**.
228+
Otherwise: verdict is **PASS**.
229+
230+
**Apply the label** (the two labels are mutually exclusive — always remove the opposite one):
231+
232+
```bash
233+
# PASS:
234+
gh pr edit ${{ github.event.pull_request.number }} \
235+
--add-label "claude:review:passed" \
236+
--remove-label "claude:review:failed"
237+
238+
# FAIL:
239+
gh pr edit ${{ github.event.pull_request.number }} \
240+
--add-label "claude:review:failed" \
241+
--remove-label "claude:review:passed"
242+
```
243+
244+
Note: `--remove-label` is a no-op if the label is not present, so it is safe to always include it.
245+
246+
Also include the verdict as the final line of your sticky review comment, formatted exactly as:
247+
248+
```
249+
**Verdict**: ✅ claude:review:passed
250+
```
251+
252+
or
253+
254+
```
255+
**Verdict**: ❌ claude:review:failed
256+
```
257+
258+
This makes the verdict visible to humans without scrolling through all findings.
259+
217260
---
218261
219262
**Remember**: This is medical device software. Insist on highest standards. Be thorough, actionable, and kind.

pyproject.toml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ dependencies = [
9999
"highdicom>=0.26.1,<1; python_version < '3.14'", # transitive dependency pyjpegls not yet supporting Python 3.14
100100
"html-sanitizer>=2.6.0,<3",
101101
"httpx>=0.28.1,<1",
102-
"idc-index-data==23.8.1",
102+
"idc-index-data==23.10.1",
103103
"ijson>=3.4.0.post0,<4",
104104
"jsf>=0.11.2,<1",
105105
"jsonschema[format-nongpl]>=4.25.1,<5",
@@ -138,6 +138,7 @@ dependencies = [
138138
"lxml>=6.1.0", # CVE-2026-41066 (Renovate #556); also required for python 3.14 pre-built wheels
139139
"filelock>=3.20.3", # CVE-2025-68146 (>=3.20.1); CVE-2026-22701 (>=3.20.3, Renovate #387)
140140
"marshmallow>=3.26.2", # CVE-2025-68480
141+
"pygments>=2.20.0", # CVE-2026-4539 (>=2.20.0); transitive via rich
141142
"cryptography>=46.0.7", # CVE-2026-39892 (>=46.0.7); transitive via pyjwt[crypto]
142143
"pydicom>=3.0.2", # CVE-2026-32711 (>=3.0.2); transitive via dicomweb-client/wsidicom/highdicom
143144
"pyasn1>=0.6.3", # CVE-2026-30922 (>=0.6.3); transitive via cryptography

uv.lock

Lines changed: 6 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)