From cf597c27e25153f1e611829ea016e32964d3c72a Mon Sep 17 00:00:00 2001 From: hurryup52 Date: Mon, 27 Jul 2026 16:55:08 +0200 Subject: [PATCH] fix(queue): correct maybeApplyManifestPolicyGate's "three"->"two" enforceable-findings comments policyCodes only has two entries (manifest_linked_issue_required, manifest_missing_tests) since manifest_blocked_path was retired in 329af5a9 (#5304), which removed the dead code but missed updating these two doc comments. resolveConfiguredGateMode in src/rules/advisory.ts confirms two is the current, correct count. Comment-only change; no logic modified. --- src/queue/processors.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/queue/processors.ts b/src/queue/processors.ts index 897e0f7a15..7c14efa488 100644 --- a/src/queue/processors.ts +++ b/src/queue/processors.ts @@ -9315,7 +9315,7 @@ export function reviewDurationMsSince(startedAt: string | null, nowMs: number): /** * Focus-manifest policy gate (#555, opt-in via `manifestPolicyGateMode`). Reloads the CACHED manifest (the * settings resolver discards the raw manifest, but loadRepoFocusManifest is cached so this is cheap), - * recomputes the guidance over the PR's changed files, and pushes ONLY the three enforceable policy findings + * recomputes the guidance over the PR's changed files, and pushes ONLY the two enforceable policy findings * onto the advisory so isConfiguredGateBlocker can block under `manifestPolicy: block`. Also runs the E2E * test-generation auto-trigger (#4196, part of the #4189 epic) — see the inline comments below for the full * rationale of each step. `manifestPolicyGateMode: "off"` (the default) is a no-op, so the advisory/gate @@ -9338,7 +9338,7 @@ async function maybeApplyManifestPolicyGate( ): Promise { // Focus-manifest policy (#555, opt-in via manifestPolicyGateMode). Reload the CACHED manifest (the // settings resolver discards the raw manifest, but loadRepoFocusManifest is cached so this is cheap), - // recompute the guidance over the PR's changed files, and push ONLY the three enforceable policy + // recompute the guidance over the PR's changed files, and push ONLY the two enforceable policy // findings into the advisory so isConfiguredGateBlocker can block under manifestPolicy: block. if (args.settings.manifestPolicyGateMode !== "off") { // `gateFiles` is threaded in by the ONLY caller (maybePublishPrPublicSurface) already resolved via