[PR #254] [MERGED] feat(pr-risk): allow human risk overrides #256

New issue

Closed

opened 2026-10-26 22:37:50 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-26 22:37:50 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/github-workflows/pull/254
Author: @huang47
Created: 9/2/2026
Status: ✅ Merged
Merged: 9/2/2026
Merged by: @huang47

Base: main ← Head: cx/risk-dispute-label-override

📝 Commits (1)

1567edf feat(pr-risk): allow human risk overrides

📊 Changes

5 files changed (+85 additions, -47 deletions)

View changed files

📝 .github/workflows/pr-risk.yml (+17 -17)
📝 docs/callers/pr-risk.md (+20 -16)
📝 scripts/pr-risk/README.md (+10 -9)
📝 scripts/pr-risk/apply-risk-label.sh (+17 -5)
📝 scripts/pr-risk/tests/test_apply_risk_label.sh (+21 -0)

📄 Description

Summary

preserve the existing risk-dispute label behavior
add risk-dispute:low through risk-dispute:xhigh to override the visible risk label

Example: computed R2 with risk-dispute:medium displays risk:medium; removing the dispute restores risk:high.

Consumer callers must add labeled and unlabeled pull-request events, filtered to risk-dispute:*.

Not included: comment commands, legacy tier aliases, audit comments, automatic expiry, or label provisioning.

Validation

ShellCheck
PR risk grader, label, target, pin-contract, publish-surface, and enablement suites
git diff --check

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/github-workflows/pull/254 **Author:** [@huang47](https://github.com/huang47) **Created:** 9/2/2026 **Status:** ✅ Merged **Merged:** 9/2/2026 **Merged by:** [@huang47](https://github.com/huang47) **Base:** `main` ← **Head:** `cx/risk-dispute-label-override` --- ### 📝 Commits (1) - [`1567edf`](https://github.com/Comfy-Org/github-workflows/commit/1567edfb1e984539d8c3bde8da841ec0990165b3) feat(pr-risk): allow human risk overrides ### 📊 Changes **5 files changed** (+85 additions, -47 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/pr-risk.yml` (+17 -17) 📝 `docs/callers/pr-risk.md` (+20 -16) 📝 `scripts/pr-risk/README.md` (+10 -9) 📝 `scripts/pr-risk/apply-risk-label.sh` (+17 -5) 📝 `scripts/pr-risk/tests/test_apply_risk_label.sh` (+21 -0) </details> ### 📄 Description ## Summary - preserve the existing `risk-dispute` label behavior - add `risk-dispute:low` through `risk-dispute:xhigh` to override the visible risk label Example: computed `R2` with `risk-dispute:medium` displays `risk:medium`; removing the dispute restores `risk:high`. Consumer callers must add `labeled` and `unlabeled` pull-request events, filtered to `risk-dispute:*`. Not included: comment commands, legacy tier aliases, audit comments, automatic expiry, or label provisioning. ## Validation - ShellCheck - PR risk grader, label, target, pin-contract, publish-surface, and enablement suites - `git diff --check` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-26 22:37:50 +00:00