From 3343c49e5fd1fb4bf5ca685b588abfc2555c0495 Mon Sep 17 00:00:00 2001 From: Lourince Daging Date: Wed, 22 Jul 2026 15:50:19 +0200 Subject: [PATCH] docs(signals): move orphaned review.enrichment doc comment to its function focus-manifest.ts had two doc blocks stacked in front of resolveReviewAutoReviewConfig: its own (#2060) plus the #2050 review.enrichment-analyzer-toggles block, which actually documents resolveEnrichmentAnalyzerToggles (defined 24 lines later with no comment of its own -- it lost its comment in a reorder). Move the #2050 block down to sit directly above resolveEnrichmentAnalyzerToggles, leaving resolveReviewAutoReviewConfig with only its own #2060 comment. Comment-only; no code or behavior change. Closes #8021 --- src/signals/focus-manifest.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/signals/focus-manifest.ts b/src/signals/focus-manifest.ts index ea9b998145..c59343c270 100644 --- a/src/signals/focus-manifest.ts +++ b/src/signals/focus-manifest.ts @@ -347,9 +347,6 @@ export function resolveReviewPreMergeChecks(manifest: FocusManifest | null): Pre return manifest?.review.preMergeChecks ?? []; } -/** Resolve `review.enrichment` analyzer toggles from a possibly-null manifest (null = load failure ⇒ no toggles ⇒ - * the operator's default analyzer set runs unchanged). Centralized so the enrichment caller threads them in one - * place with the null-manifest branch covered here (unit-tested) rather than inline in the processor. (#2050) */ /** Resolve `review.auto_review` from a possibly-null manifest (null = load failure => no ignored authors). The * runtime eligibility check then fails open instead of suppressing review output on an ambiguous manifest read. * (#2060) */ @@ -373,6 +370,9 @@ export function resolveReviewVisualConfig(manifest: FocusManifest | null): Visua return manifest?.review.visual ?? { ...EMPTY_VISUAL_CONFIG }; } +/** Resolve `review.enrichment` analyzer toggles from a possibly-null manifest (null = load failure ⇒ no toggles ⇒ + * the operator's default analyzer set runs unchanged). Centralized so the enrichment caller threads them in one + * place with the null-manifest branch covered here (unit-tested) rather than inline in the processor. (#2050) */ export function resolveEnrichmentAnalyzerToggles(manifest: FocusManifest | null): Partial> { return manifest?.review.enrichmentAnalyzers ?? {}; }