@codedrifters/configulator 0.0.353 → 0.0.354

package/lib/index.js

@@ -18327,7 +18327,12 @@ var prReviewerSubAgent = {
     "   `review:human-required` label, set `mode = human-required`.",
     "3. **Labels that force human** \u2014 if the PR carries any label listed",
     "   under `human-required.labels-that-force-human` (e.g.",
-    "   `priority:critical`), set `mode = human-required`.",
+    "   `review:human-required`), set `mode = human-required`. Evaluate",
+    "   the PR's **own** labels only \u2014 rules 1\u20133 never traverse the",
+    "   `Closes #N` link to read the linked issue's labels. `priority:*`",
+    "   and `status:*` are issue-triage labels for the orchestrator's",
+    "   scheduling queue, not merge-risk signals, and are never listed",
+    "   here.",
     "4. **Path globs** \u2014 if any file in the PR diff matches any glob in",
     "   `human-required.paths`, set `mode = human-required`. Record the",
     "   first matching path + glob pair as the reason.",
@@ -18845,7 +18850,7 @@ var prReviewerSubAgent = {
     "when the mode is `human-required` and any of the following fired:",
     "",
     "- rule 2 (`review:human-required` label),",
-    "- rule 3 (any `labels-that-force-human` label such as `priority:critical`),",
+    "- rule 3 (any `labels-that-force-human` label),",
     "- rule 4 (`human-required.paths` glob match), or",
     "- rule 5 (`human-required.issue-types` match).",
     "",
@@ -19658,7 +19663,6 @@ function buildPrReviewBundle(policy = resolvePrReviewPolicy()) {
           "    insertions: 500",
           "  labels-that-force-human:",
           '    - "review:human-required"',
-          '    - "priority:critical"',
           "",
           "auto-merge:",
           "  labels-that-force-auto:",
@@ -19689,8 +19693,9 @@ function buildPrReviewBundle(policy = resolvePrReviewPolicy()) {
           "2. **`review:human-required` label** \u2014 reserved force-human label;",
           "   if present (and no force-auto label beat it in step 1), the mode",
           "   is `human-required`.",
-          "3. **`human-required.labels-that-force-human`** \u2014 any listed label on",
-          "   the PR (e.g. `priority:critical`) forces `human-required`.",
+          "3. **`human-required.labels-that-force-human`** \u2014 if the PR",
+          "   carries any label listed here (e.g. `review:human-required`),",
+          "   the mode is `human-required`.",
           "4. **`human-required.paths`** \u2014 if any file in the PR diff matches",
           "   any glob here, the mode is `human-required`. Matching uses",
           "   standard glob semantics (`**` for recursive directories,",
@@ -19711,6 +19716,24 @@ function buildPrReviewBundle(policy = resolvePrReviewPolicy()) {
           "7. **`default`** \u2014 applied only when no rule above matched",
           "   (normally `auto-merge`).",
           "",
+          "**Label-check scope.** All label checks in rules 1\u20133 evaluate",
+          "the PR's **own** labels only. Never traverse the `Closes #N`",
+          "link to read the linked issue's labels for a label rule. Rule 5",
+          "is the only rule that reads the linked issue, and it reads the",
+          "issue **type**, not its labels.",
+          "",
+          "**`priority:*` / `status:*` are not merge-risk signals.**",
+          "`priority:*` and `status:*` are issue-triage labels consumed by",
+          "the orchestrator for scheduling \u2014 `priority:*` sequences the",
+          "work queue and `status:*` tracks workflow position. They say",
+          "nothing about a PR's merge risk and must never appear in any",
+          "`labels-that-force-*` list. Gating merge on `priority:critical`",
+          "in particular inverts the intended behaviour: the most urgent",
+          "work would wait the longest for a human merge. Merge risk is",
+          "already covered by the path globs (rule 4), the",
+          "`release` / `hotfix` issue-types (rule 5), and the size",
+          "thresholds (rule 6).",
+          "",
           "The `auto-merge.paths-exempt-from-size` carve-out exists so",
           "**doc-only PRs** that routinely exceed the 500-insertion size",
           "threshold (large migrations, bulk additions, refresh passes)",
@@ -19742,8 +19765,8 @@ function buildPrReviewBundle(policy = resolvePrReviewPolicy()) {
           "The exception is keyed on the matched-rule index recorded in",
           "Phase 2.75. All other `human-required` triggers \u2014 rule 2",
           "(`review:human-required` label), rule 3 (any",
-          "`labels-that-force-human` label such as `priority:critical`),",
-          "rule 4 (`human-required.paths` glob match), and rule 5",
+          "`labels-that-force-human` label), rule 4",
+          "(`human-required.paths` glob match), and rule 5",
           "(`human-required.issue-types` match) \u2014 continue to block",
           "`update-branch` because each one signals a human reviewer who",
           "has explicit ownership of the branch's lifecycle.",