Context
src/signals/focus-manifest.ts:352-360 has two doc comments stacked in front of one function. Lines 352-354 document review.enrichment analyzer toggles (referencing #2050) but sit directly above resolveReviewAutoReviewConfig's own doc comment (lines 355-357, referencing #2060) and function (lines 358-360).
The function the first block (352-354) actually describes, resolveEnrichmentAnalyzerToggles, is defined 24 lines later at line 378, with no doc comment of its own — it lost its comment somewhere along the way (likely a reorder), and resolveReviewAutoReviewConfig ended up with two doc blocks in front of it instead of one.
This is a unique occurrence in src/signals/**/src/github/**/src/mcp/**/src/services/** — a full grep for stacked */→/** doc blocks across those directories found only this one instance, so it's an isolated drift from a reorder, not a systemic pattern needing a broader sweep.
Requirements
- Move the
review.enrichment doc comment (lines 352-354) down to sit directly above resolveEnrichmentAnalyzerToggles (line 378), where it actually belongs.
- Leave
resolveReviewAutoReviewConfig's own doc comment (lines 355-357) as its sole doc comment.
Deliverables
Test Coverage Requirements
Docs-only change — no test coverage obligation (no logic changes), consistent with this repo's convention for comment-only fixes.
Expected Outcome
Each function in focus-manifest.ts has its own accurate doc comment directly above it, with no orphaned block describing the wrong function.
Links & Resources
src/signals/focus-manifest.ts (lines 352-360 for the misplaced comment, line 378 for its real home)
Context
src/signals/focus-manifest.ts:352-360has two doc comments stacked in front of one function. Lines 352-354 documentreview.enrichmentanalyzer toggles (referencing #2050) but sit directly aboveresolveReviewAutoReviewConfig's own doc comment (lines 355-357, referencing #2060) and function (lines 358-360).The function the first block (352-354) actually describes,
resolveEnrichmentAnalyzerToggles, is defined 24 lines later at line 378, with no doc comment of its own — it lost its comment somewhere along the way (likely a reorder), andresolveReviewAutoReviewConfigended up with two doc blocks in front of it instead of one.This is a unique occurrence in
src/signals/**/src/github/**/src/mcp/**/src/services/**— a full grep for stacked*/→/**doc blocks across those directories found only this one instance, so it's an isolated drift from a reorder, not a systemic pattern needing a broader sweep.Requirements
review.enrichmentdoc comment (lines 352-354) down to sit directly aboveresolveEnrichmentAnalyzerToggles(line 378), where it actually belongs.resolveReviewAutoReviewConfig's own doc comment (lines 355-357) as its sole doc comment.Deliverables
review.enrichment/feat(config): add review.enrichment analyzer per-lane toggles #2050 doc comment sits aboveresolveEnrichmentAnalyzerToggles.resolveReviewAutoReviewConfighas exactly one doc comment (its own, feat(review): skip auto-review for bot/ignored authors via review.auto_review.ignore_authors globs #2060).Test Coverage Requirements
Docs-only change — no test coverage obligation (no logic changes), consistent with this repo's convention for comment-only fixes.
Expected Outcome
Each function in
focus-manifest.tshas its own accurate doc comment directly above it, with no orphaned block describing the wrong function.Links & Resources
src/signals/focus-manifest.ts(lines 352-360 for the misplaced comment, line 378 for its real home)