fix(review): add missing duplicationDelta to REES_ANALYZER_NAMES (both copies)

packages/loopover-engine/src/review/enrichment-analyzer-names.ts

@@ -23,9 +23,10 @@
   "commitSignature",
   "iacMisconfig",
   "nativeBuild",
   "history",
   "docCommentDrift",
   "duplication",
+  "duplicationDelta",
   "churnHotspot",
   "blameLink",
   "approvalIntegrity",

src/review/enrichment-analyzer-names.ts

@@ -23,9 +23,10 @@
   "commitSignature",
   "iacMisconfig",
   "nativeBuild",
   "history",
   "docCommentDrift",
   "duplication",
+  "duplicationDelta",
   "churnHotspot",
   "blameLink",
   "approvalIntegrity",

test/unit/enrichment-analyzers-taxonomy.test.ts

@@ -2,9 +2,10 @@
 import { join } from "node:path";
 import { describe, expect, it } from "vitest";
 import {
   buildEnrichmentAnalyzersTaxonomyDocument,
   ENRICHMENT_ANALYZERS_URI,
 } from "../../src/review/enrichment-analyzers-taxonomy";
+import { REES_ANALYZER_NAMES } from "../../src/review/enrichment-analyzer-names";
 
 const metadataPath = join(process.cwd(), "review-enrichment/analyzer-metadata.json");
 
@@ -45,3 +46,26 @@
     expect(ENRICHMENT_ANALYZERS_URI).toBe("gittensory://enrichment-analyzers");
   });
 });
+
+describe("REES_ANALYZER_NAMES stays in sync with analyzer-metadata.json", () => {
+  const metadataNames = (
+    JSON.parse(readFileSync(metadataPath, "utf8")) as { analyzers: Array<{ name: string }> }
+  ).analyzers.map((a) => a.name);
+
+  it("covers exactly the analyzers the metadata registry defines (no missing, no extra)", () => {
+    // The canonical name list validates every operator `REES_ANALYZERS` env entry and per-repo
+    // `.loopover.yml review.enrichment` toggle, so an analyzer present in the metadata registry but absent
+    // here is silently un-toggleable/un-selectable. Compared as sets against the registry (the source of
+    // truth) rather than a hardcoded count so a newly-added analyzer can't drift the two apart unnoticed.
+    expect(new Set(REES_ANALYZER_NAMES)).toEqual(new Set(metadataNames));
+  });
+
+  it("includes duplicationDelta alongside duplication (regression for the one-entry gap)", () => {
+    expect(REES_ANALYZER_NAMES).toContain("duplication");
+    expect(REES_ANALYZER_NAMES).toContain("duplicationDelta");
+  });
+
+  it("has no duplicate entries", () => {
+    expect(new Set(REES_ANALYZER_NAMES).size).toBe(REES_ANALYZER_NAMES.length);
+  });
+});