@opensip-cli/graph 0.1.10 → 0.1.12

package/dist/rules/define-rule.js

@@ -34,19 +34,16 @@ export function defineRule(config) {
     if (typeof config.slug !== 'string' ||
         config.slug.length === 0 ||
         !config.slug.startsWith('graph:')) {
-        // @fitness-ignore-next-line result-pattern-consistency -- authoring-time programmer error: invalid rule metadata
         throw new ValidationError(`defineRule: 'slug' must be a non-empty string starting with 'graph:' (got ${JSON.stringify(config.slug)})`, {
             code: VALIDATION_CODE,
         });
     }
     if (config.defaultSeverity !== 'error' && config.defaultSeverity !== 'warning') {
-        // @fitness-ignore-next-line result-pattern-consistency -- authoring-time programmer error: invalid rule metadata
         throw new ValidationError(`defineRule: 'defaultSeverity' must be 'error' or 'warning' (got ${JSON.stringify(config.defaultSeverity)})`, {
             code: VALIDATION_CODE,
         });
     }
     if (typeof config.evaluate !== 'function') {
-        // @fitness-ignore-next-line result-pattern-consistency -- authoring-time programmer error: invalid rule metadata
         throw new ValidationError(`defineRule: 'evaluate' must be a function for rule '${config.slug}'`, {
             code: VALIDATION_CODE,
         });

package/dist/rules/define-rule.js.map

@@ -1 +1 @@
-{"version":3,"file":"define-rule.js","sourceRoot":"","sources":["../../src/rules/define-rule.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAC;AAiDpD,MAAM,eAAe,GAAG,uBAAuB,CAAC;AAEhD;;;;;GAKG;AACH,MAAM,UAAU,UAAU,CAAC,MAAwB;IACjD,IACE,OAAO,MAAM,CAAC,IAAI,KAAK,QAAQ;QAC/B,MAAM,CAAC,IAAI,CAAC,MAAM,KAAK,CAAC;QACxB,CAAC,MAAM,CAAC,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,EACjC,CAAC;QACD,iHAAiH;QACjH,MAAM,IAAI,eAAe,CACvB,6EAA6E,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,EAC3G;YACE,IAAI,EAAE,eAAe;SACtB,CACF,CAAC;IACJ,CAAC;IACD,IAAI,MAAM,CAAC,eAAe,KAAK,OAAO,IAAI,MAAM,CAAC,eAAe,KAAK,SAAS,EAAE,CAAC;QAC/E,iHAAiH;QACjH,MAAM,IAAI,eAAe,CACvB,mEAAmE,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,eAAe,CAAC,GAAG,EAC5G;YACE,IAAI,EAAE,eAAe;SACtB,CACF,CAAC;IACJ,CAAC;IACD,IAAI,OAAO,MAAM,CAAC,QAAQ,KAAK,UAAU,EAAE,CAAC;QAC1C,iHAAiH;QACjH,MAAM,IAAI,eAAe,CACvB,uDAAuD,MAAM,CAAC,IAAI,GAAG,EACrE;YACE,IAAI,EAAE,eAAe;SACtB,CACF,CAAC;IACJ,CAAC;IAED,MAAM,IAAI,GAAS;QACjB,IAAI,EAAE,MAAM,CAAC,IAAI;QACjB,eAAe,EAAE,MAAM,CAAC,eAAe;QACvC,GAAG,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,MAAM,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAClE,QAAQ,CACN,OAAgB,EAChB,OAAgB,EAChB,UAAuB,EACvB,KAAiB,EACjB,QAAuB;YAEvB,uEAAuE;YACvE,0EAA0E;YAC1E,OAAO,MAAM,CAAC,QAAQ,CAAC,EAAE,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,CAAC;QACpF,CAAC;KACF,CAAC;IACF,OAAO,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;AAC7B,CAAC"}
+{"version":3,"file":"define-rule.js","sourceRoot":"","sources":["../../src/rules/define-rule.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAC;AAiDpD,MAAM,eAAe,GAAG,uBAAuB,CAAC;AAEhD;;;;;GAKG;AACH,MAAM,UAAU,UAAU,CAAC,MAAwB;IACjD,IACE,OAAO,MAAM,CAAC,IAAI,KAAK,QAAQ;QAC/B,MAAM,CAAC,IAAI,CAAC,MAAM,KAAK,CAAC;QACxB,CAAC,MAAM,CAAC,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,EACjC,CAAC;QACD,MAAM,IAAI,eAAe,CACvB,6EAA6E,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,EAC3G;YACE,IAAI,EAAE,eAAe;SACtB,CACF,CAAC;IACJ,CAAC;IACD,IAAI,MAAM,CAAC,eAAe,KAAK,OAAO,IAAI,MAAM,CAAC,eAAe,KAAK,SAAS,EAAE,CAAC;QAC/E,MAAM,IAAI,eAAe,CACvB,mEAAmE,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,eAAe,CAAC,GAAG,EAC5G;YACE,IAAI,EAAE,eAAe;SACtB,CACF,CAAC;IACJ,CAAC;IACD,IAAI,OAAO,MAAM,CAAC,QAAQ,KAAK,UAAU,EAAE,CAAC;QAC1C,MAAM,IAAI,eAAe,CACvB,uDAAuD,MAAM,CAAC,IAAI,GAAG,EACrE;YACE,IAAI,EAAE,eAAe;SACtB,CACF,CAAC;IACJ,CAAC;IAED,MAAM,IAAI,GAAS;QACjB,IAAI,EAAE,MAAM,CAAC,IAAI;QACjB,eAAe,EAAE,MAAM,CAAC,eAAe;QACvC,GAAG,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,MAAM,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAClE,QAAQ,CACN,OAAgB,EAChB,OAAgB,EAChB,UAAuB,EACvB,KAAiB,EACjB,QAAuB;YAEvB,uEAAuE;YACvE,0EAA0E;YAC1E,OAAO,MAAM,CAAC,QAAQ,CAAC,EAAE,OAAO,EAAE,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,KAAK,EAAE,QAAQ,EAAE,CAAC,CAAC;QACpF,CAAC;KACF,CAAC;IACF,OAAO,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;AAC7B,CAAC"}

package/dist/rules/duplicated-function-body.d.ts

@@ -1,40 +1,18 @@
 /**
- * graph:duplicated-function-body — group catalog entries whose body
- * normalizes to the same hash and report duplicate bodies. The rule has
- * two complementary code paths under one slug:
+ * graph:duplicated-function-body — group catalog entries whose body normalizes to the
+ * same hash and report duplicate bodies. Two complementary paths under one slug:
  *
- *   1. **Per-instance (size-gated).** A "duplicate" is two or more
- *      functions whose normalized body is byte-identical
- *      (whitespace/comment normalization handled by hashFunctionBody).
- *      Two thresholds filter out non-actionable hits:
+ *   1. Per-instance (size-gated): N-1 signals per group of ≥2 byte-identical normalized
+ *      bodies passing a line floor (default 5) AND a normalized-char floor (default 200,
+ *      which drops thin `defineCheck`/pass-through wrappers).
+ *   2. Aggregate (cross-package): one signal per body hash in ≥ 3 distinct packages,
+ *      suppressing that hash's per-instance signals (lighter body-size-only floor, 80).
  *
- *        - minDuplicateBodyLines (default 5): the source span must be at
- *          least this many lines.
- *        - minDuplicateBodySize  (default 200): the *normalized* body
- *          (post comment-strip + whitespace-collapse) must be at least
- *          this many characters. This catches the common case of a
- *          `defineCheck({ ... analyze(content) { return analyzeFile(...) } })`
- *          wrapper where the source spans 8+ lines but the executable
- *          body is only ~80 characters once normalized — structurally
- *          identical to every other thin wrapper, but never an actionable
- *          refactor target.
- *
- *      Emits N-1 signals per group (one per non-primary occurrence).
- *
- *   2. **Aggregate (cross-package).** For each body hash present in
- *      ≥ minCrossPackageDuplicatePackages (default 3) DISTINCT packages
- *      (via `pkgOf`), the rule emits ONE aggregate signal naming the
- *      packages and SUPPRESSES the per-instance signals for that same hash
- *      (no double-reporting). Bodies that don't reach N packages flow
- *      through path (1) unchanged.
- *
- *      This path applies a LIGHTER, body-size-only floor
- *      (`minCrossPackageDuplicateBodySize`, default 80 chars — no line floor)
- *      rather than the per-instance floor: a *small* body copied across
- *      packages is exactly what this path exists to catch, so the floor is
- *      tuned only to drop TRIVIAL bodies (empty DI-constructor shims,
- *      one-line getters, thin delegators) that are never consolidation
- *      targets, while keeping genuinely-small shared utilities visible.
+ * The detection algorithm + curation policy now live in `@opensip-cli/clone-detection`
+ * (ADR-0064) so graph and yagni single-source them. This rule is a thin adapter: it maps
+ * the catalog to `CloneCandidate[]` (pre-resolving `bodyLines` from the feature table and
+ * `package` from `pkgOf`), calls `findDuplicateBodies`, and wraps each finding into a
+ * graph `Signal` with the unchanged message/severity/metadata — so output is byte-stable.
  */
 export declare const duplicatedFunctionBodyRule: import("../types.js").Rule;
 //# sourceMappingURL=duplicated-function-body.d.ts.map

package/dist/rules/duplicated-function-body.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"duplicated-function-body.d.ts","sourceRoot":"","sources":["../../src/rules/duplicated-function-body.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAqCG;AAoBH,eAAO,MAAM,0BAA0B,4BA0DrC,CAAC"}
+{"version":3,"file":"duplicated-function-body.d.ts","sourceRoot":"","sources":["../../src/rules/duplicated-function-body.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAeH,eAAO,MAAM,0BAA0B,4BAoBrC,CAAC"}

package/dist/rules/duplicated-function-body.js

@@ -1,262 +1,112 @@
 /**
- * graph:duplicated-function-body — group catalog entries whose body
- * normalizes to the same hash and report duplicate bodies. The rule has
- * two complementary code paths under one slug:
+ * graph:duplicated-function-body — group catalog entries whose body normalizes to the
+ * same hash and report duplicate bodies. Two complementary paths under one slug:
  *
- *   1. **Per-instance (size-gated).** A "duplicate" is two or more
- *      functions whose normalized body is byte-identical
- *      (whitespace/comment normalization handled by hashFunctionBody).
- *      Two thresholds filter out non-actionable hits:
+ *   1. Per-instance (size-gated): N-1 signals per group of ≥2 byte-identical normalized
+ *      bodies passing a line floor (default 5) AND a normalized-char floor (default 200,
+ *      which drops thin `defineCheck`/pass-through wrappers).
+ *   2. Aggregate (cross-package): one signal per body hash in ≥ 3 distinct packages,
+ *      suppressing that hash's per-instance signals (lighter body-size-only floor, 80).
  *
- *        - minDuplicateBodyLines (default 5): the source span must be at
- *          least this many lines.
- *        - minDuplicateBodySize  (default 200): the *normalized* body
- *          (post comment-strip + whitespace-collapse) must be at least
- *          this many characters. This catches the common case of a
- *          `defineCheck({ ... analyze(content) { return analyzeFile(...) } })`
- *          wrapper where the source spans 8+ lines but the executable
- *          body is only ~80 characters once normalized — structurally
- *          identical to every other thin wrapper, but never an actionable
- *          refactor target.
- *
- *      Emits N-1 signals per group (one per non-primary occurrence).
- *
- *   2. **Aggregate (cross-package).** For each body hash present in
- *      ≥ minCrossPackageDuplicatePackages (default 3) DISTINCT packages
- *      (via `pkgOf`), the rule emits ONE aggregate signal naming the
- *      packages and SUPPRESSES the per-instance signals for that same hash
- *      (no double-reporting). Bodies that don't reach N packages flow
- *      through path (1) unchanged.
- *
- *      This path applies a LIGHTER, body-size-only floor
- *      (`minCrossPackageDuplicateBodySize`, default 80 chars — no line floor)
- *      rather than the per-instance floor: a *small* body copied across
- *      packages is exactly what this path exists to catch, so the floor is
- *      tuned only to drop TRIVIAL bodies (empty DI-constructor shims,
- *      one-line getters, thin delegators) that are never consolidation
- *      targets, while keeping genuinely-small shared utilities visible.
+ * The detection algorithm + curation policy now live in `@opensip-cli/clone-detection`
+ * (ADR-0064) so graph and yagni single-source them. This rule is a thin adapter: it maps
+ * the catalog to `CloneCandidate[]` (pre-resolving `bodyLines` from the feature table and
+ * `package` from `pkgOf`), calls `findDuplicateBodies`, and wraps each finding into a
+ * graph `Signal` with the unchanged message/severity/metadata — so output is byte-stable.
  */
+import { findDuplicateBodies } from '@opensip-cli/clone-detection';
 import { pkgOf } from '../resolve-callee.js';
 import { createGraphSignal } from './create-graph-signal.js';
 import { defineRule } from './define-rule.js';
-const DEFAULT_MIN_LINES = 5;
-const DEFAULT_MIN_BODY_SIZE = 200;
-const DEFAULT_MIN_CROSS_PACKAGE_PACKAGES = 3;
-// Lighter than DEFAULT_MIN_BODY_SIZE (the per-instance floor) and applied with
-// NO line floor: the aggregate path is meant to catch genuinely-small shared
-// utilities copied across packages, so its floor only drops trivial bodies
-// (empty DI shims, one-line getters/delegators).
-const DEFAULT_MIN_CROSS_PACKAGE_BODY_SIZE = 80;
 const SLUG = 'graph:duplicated-function-body';
 export const duplicatedFunctionBodyRule = defineRule({
     slug: SLUG,
     defaultSeverity: 'warning',
     featureDeps: ['bodyLines'],
     evaluate({ catalog, config, features }) {
-        const minLines = config.minDuplicateBodyLines ?? DEFAULT_MIN_LINES;
-        const minBodySize = config.minDuplicateBodySize ?? DEFAULT_MIN_BODY_SIZE;
-        const minPackages = config.minCrossPackageDuplicatePackages ?? DEFAULT_MIN_CROSS_PACKAGE_PACKAGES;
-        const minCrossPackageBodySize = config.minCrossPackageDuplicateBodySize ?? DEFAULT_MIN_CROSS_PACKAGE_BODY_SIZE;
-        const signals = [];
-        // Aggregate path first: group every kind/test-eligible occurrence by
-        // body hash (no line floor) so we can detect cross-package spread and
-        // decide which hashes to suppress on the per-instance path below.
-        const aggregateBuckets = groupByHashUnfloored(catalog);
-        const suppressedHashes = new Set();
-        for (const [bodyHash, occs] of aggregateBuckets) {
-            const packages = [...new Set(occs.map((o) => pkgOf(o)))].sort();
-            if (packages.length < minPackages)
-                continue;
-            const anchor = lowestByQualifiedName(occs);
-            // Lighter, body-size-only floor (NO line floor): keeps the aggregate
-            // path catching genuinely-small shared utilities copied across packages,
-            // while dropping trivial bodies — empty DI-constructor shims, one-line
-            // getters, thin delegators — that are not consolidation targets. A body
-            // that fails this won't surface on the per-instance path either (its
-            // 200-char floor is stricter), so there is nothing to suppress.
-            if (anchor.bodySize !== undefined && anchor.bodySize < minCrossPackageBodySize)
-                continue;
-            // This hash is owned by the aggregate path; suppress its per-instance
-            // signals so a single duplicate group never double-reports.
-            suppressedHashes.add(bodyHash);
-            signals.push(createGraphSignal(SLUG, config, {
-                severity: 'low',
-                category: 'quality',
-                message: `This body is duplicated across ${String(packages.length)} packages (${packages.join(', ')}) in ${String(occs.length)} occurrences — hoist it into a shared package.`,
-                code: { file: anchor.filePath, line: anchor.line, column: anchor.column },
-                suggestion: 'Hoist the shared body into a single shared package and have every copy import it.',
-                metadata: {
-                    packages,
-                    packageCount: packages.length,
-                    occurrenceCount: occs.length,
-                    bodyHash,
-                },
-            }));
-        }
-        // Per-instance path: size-gated groups, skipping any hash already
-        // claimed by an aggregate signal so a group never double-reports.
-        const groups = groupByHash(catalog, minLines, minBodySize, features);
-        signals.push(...emitPerInstanceSignals(groups, suppressedHashes, config));
-        return signals;
+        const candidates = toCandidates(catalog, features);
+        const { aggregates, groups } = findDuplicateBodies(candidates, {
+            minLines: config.minDuplicateBodyLines,
+            minBodySize: config.minDuplicateBodySize,
+            minCrossPackagePackages: config.minCrossPackageDuplicatePackages,
+            minCrossPackageBodySize: config.minCrossPackageDuplicateBodySize,
+        });
+        // Aggregate (cross-package) signals first, then per-instance signals — same order
+        // as the prior loops, but built as bounded in-memory projections over rule findings.
+        return [
+            ...aggregates.map((agg) => aggregateSignal(agg, config)),
+            ...groups.flatMap((group) => groupSignals(group, config)),
+        ];
     },
 });
-/**
- * Per-instance duplicate signals: N-1 per size-gated group, skipping any hash
- * already claimed by an aggregate signal (no double-reporting). Extracted from
- * evaluate() to keep its cognitive complexity within budget.
- */
-function emitPerInstanceSignals(groups, suppressedHashes, config) {
-    const signals = [];
-    for (const group of groups) {
-        if (group.length < 2)
-            continue;
-        const primary = group[0];
-        /* v8 ignore next */
-        if (!primary)
-            continue;
-        if (suppressedHashes.has(primary.bodyHash))
-            continue;
-        for (let i = 1; i < group.length; i++) {
-            const occ = group[i];
-            /* v8 ignore next */
-            if (!occ)
-                continue;
-            signals.push(createGraphSignal(SLUG, config, {
-                severity: 'low',
-                category: 'quality',
-                message: `${occ.simpleName} has the same body as ${primary.qualifiedName} (${primary.filePath}:${String(primary.line)}).`,
-                code: { file: occ.filePath, line: occ.line, column: occ.column },
-                suggestion: 'Extract the shared body to a single function and have both call sites import it.',
-                metadata: {
-                    primary: primary.qualifiedName,
-                    duplicate: occ.qualifiedName,
-                    groupSize: group.length,
-                },
-            }));
-        }
-    }
-    return signals;
-}
-/**
- * A function's stable physical identity: where it is declared. The catalog's
- * `functions` record is keyed by simple name, and the same physical function
- * can surface more than once across that index (e.g. indexed under both its
- * simple and qualified name, or a recursive function whose self-reference is
- * walked as an extra occurrence). Two entries that share this identity are the
- * SAME function — they must never count as a duplicate of each other. Keying on
- * file + declaration position + simple name keeps genuinely-distinct functions
- * (same body, different file/line) in separate identities while collapsing
- * self-matches.
- */
-function identityOf(occ) {
-    return `${occ.filePath}:${String(occ.line)}:${String(occ.column)}:${occ.simpleName}`;
+function aggregateSignal(agg, config) {
+    return createGraphSignal(SLUG, config, {
+        severity: 'low',
+        category: 'quality',
+        message: `This body is duplicated across ${String(agg.packages.length)} packages (${agg.packages.join(', ')}) in ${String(agg.occurrenceCount)} occurrences — hoist it into a shared package.`,
+        code: { file: agg.anchor.filePath, line: agg.anchor.line, column: agg.anchor.column },
+        suggestion: 'Hoist the shared body into a single shared package and have every copy import it.',
+        metadata: {
+            packages: agg.packages,
+            packageCount: agg.packages.length,
+            occurrenceCount: agg.occurrenceCount,
+            bodyHash: agg.bodyHash,
+        },
+    });
 }
-/**
- * Add an occurrence to its body-hash bucket, deduping by physical identity so a
- * function never appears twice in its own bucket. Without this, a function that
- * surfaces more than once in the catalog index (self-reference, dual name
- * indexing) would form a phantom 2-member group and self-report as a duplicate.
- */
-function pushDeduped(buckets, seenByHash, occ) {
-    let bucket = buckets.get(occ.bodyHash);
-    let seen = seenByHash.get(occ.bodyHash);
-    if (!bucket) {
-        bucket = [];
-        buckets.set(occ.bodyHash, bucket);
-        seen = new Set();
-        seenByHash.set(occ.bodyHash, seen);
-    }
+function groupSignals(group, config) {
+    const primary = group.members[0];
     /* v8 ignore next */
-    if (!seen)
-        return;
-    const id = identityOf(occ);
-    if (seen.has(id))
-        return;
-    seen.add(id);
-    bucket.push(occ);
-}
-function groupByHash(catalog, minLines, minBodySize, features) {
-    // Walk the catalog directly; the byBodyHash index dedupes by hash,
-    // which is exactly what we need to NOT do here.
-    const buckets = new Map();
-    const seenByHash = new Map();
-    for (const name of Object.keys(catalog.functions)) {
-        const occs = catalog.functions[name];
-        /* v8 ignore next */
-        if (!occs)
-            continue;
-        for (const occ of occs) {
-            if (!isInterestingForDup(occ, minLines, minBodySize, features))
-                continue;
-            pushDeduped(buckets, seenByHash, occ);
-        }
-    }
-    return [...buckets.values()];
+    if (!primary)
+        return [];
+    return group.members.slice(1).map((occ) => createGraphSignal(SLUG, config, {
+        severity: 'low',
+        category: 'quality',
+        message: `${occ.simpleName} has the same body as ${primary.qualifiedName} (${primary.filePath}:${String(primary.line)}).`,
+        code: { file: occ.filePath, line: occ.line, column: occ.column },
+        suggestion: 'Extract the shared body to a single function and have both call sites import it.',
+        metadata: {
+            primary: primary.qualifiedName,
+            duplicate: occ.qualifiedName,
+            groupSize: group.members.length,
+        },
+    }));
 }
 /**
- * Group occurrences by body hash applying ONLY the kind/test-file
- * exclusions (no size/line floor) — the grouping the aggregate
- * cross-package path consumes. Returns a Map so callers keep the body
- * hash key for suppression bookkeeping.
+ * Map the catalog to `CloneCandidate[]` in the SAME iteration order the prior in-rule
+ * grouping used (`Object.keys(catalog.functions)` → each name's occurrences), so the
+ * substrate's bucket/primary selection and emission order are byte-identical.
+ * `bodyLines` is pre-resolved from the feature table (canonical span) with the inline
+ * `endLine − line + 1` fallback handled by the substrate; `package` is `pkgOf` (the
+ * nearest-package.json resolution the aggregate path keys on).
  */
-function groupByHashUnfloored(catalog) {
-    const buckets = new Map();
-    const seenByHash = new Map();
+function toCandidates(catalog, features) {
+    const candidates = [];
     for (const name of Object.keys(catalog.functions)) {
         const occs = catalog.functions[name];
         /* v8 ignore next */
         if (!occs)
             continue;
-        for (const occ of occs) {
-            if (!isEligibleKind(occ))
-                continue;
-            pushDeduped(buckets, seenByHash, occ);
-        }
+        for (const occ of occs)
+            candidates.push(toCandidate(occ, features));
     }
-    return buckets;
-}
-function lowestByQualifiedName(occs) {
-    return occs.reduce((lo, c) => (c.qualifiedName < lo.qualifiedName ? c : lo));
+    return candidates;
 }
-/**
- * The kind/test-file exclusions shared by both code paths. Inline arrows
- * / function expressions / module-init occurrences are never an
- * extract/hoist target:
- *  - test-suite arrows with identical 5-line bodies are common boilerplate
- *  - module-inits hash whatever is at top level; identical hashes mean
- *    twin files, not a dup-extraction opportunity
- *  - constructor/getter/setter/method dups are interesting; keep those
- *
- * Test-file occurrences are excluded on both paths.
- */
-function isEligibleKind(occ) {
-    if (occ.kind === 'arrow' || occ.kind === 'function-expression' || occ.kind === 'module-init') {
-        return false;
-    }
-    if (occ.inTestFile)
-        return false;
-    return true;
-}
-/**
- * The per-instance dup-body filter: the shared kind/test exclusions plus
- * the size/line floor. The bodySize threshold drops trivial wrappers — a
- * `defineCheck` or pass-through analyze that contains an `if`-guard plus a
- * delegating call. These have identical normalized bodies across many
- * files but are never the right refactor target. Catalogs from older runs
- * that lack `bodySize` skip the size check (treated as "passes").
- */
-function isInterestingForDup(occ, minLines, minBodySize, features) {
-    if (!isEligibleKind(occ))
-        return false;
-    // The bodyLines feature column is the canonical span (computed once in
-    // pipeline/features.ts). The inline `endLine − line + 1` here is the single
-    // sanctioned graceful-degrade fallback for features-absent calls (3/4-arg
-    // test evaluate), not a duplicate of the engine derivation.
-    const span = features?.function.get(occ.bodyHash)?.bodyLines ?? occ.endLine - occ.line + 1;
-    if (span < minLines)
-        return false;
-    if (occ.bodySize !== undefined && occ.bodySize < minBodySize)
-        return false;
-    return true;
+function toCandidate(occ, features) {
+    const bodyLines = features?.function.get(occ.bodyHash)?.bodyLines;
+    return {
+        bodyHash: occ.bodyHash,
+        kind: occ.kind,
+        inTestFile: occ.inTestFile,
+        filePath: occ.filePath,
+        line: occ.line,
+        column: occ.column,
+        endLine: occ.endLine,
+        simpleName: occ.simpleName,
+        qualifiedName: occ.qualifiedName,
+        package: pkgOf(occ),
+        ...(occ.bodySize === undefined ? {} : { bodySize: occ.bodySize }),
+        ...(bodyLines === undefined ? {} : { bodyLines }),
+    };
 }
 //# sourceMappingURL=duplicated-function-body.js.map

package/dist/rules/duplicated-function-body.js.map

@@ -1 +1 @@
-{"version":3,"file":"duplicated-function-body.js","sourceRoot":"","sources":["../../src/rules/duplicated-function-body.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAqCG;AAEH,OAAO,EAAE,KAAK,EAAE,MAAM,sBAAsB,CAAC;AAE7C,OAAO,EAAE,iBAAiB,EAAE,MAAM,0BAA0B,CAAC;AAC7D,OAAO,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAC;AAK9C,MAAM,iBAAiB,GAAG,CAAC,CAAC;AAC5B,MAAM,qBAAqB,GAAG,GAAG,CAAC;AAClC,MAAM,kCAAkC,GAAG,CAAC,CAAC;AAC7C,+EAA+E;AAC/E,6EAA6E;AAC7E,2EAA2E;AAC3E,iDAAiD;AACjD,MAAM,mCAAmC,GAAG,EAAE,CAAC;AAC/C,MAAM,IAAI,GAAG,gCAAgC,CAAC;AAE9C,MAAM,CAAC,MAAM,0BAA0B,GAAG,UAAU,CAAC;IACnD,IAAI,EAAE,IAAI;IACV,eAAe,EAAE,SAAS;IAC1B,WAAW,EAAE,CAAC,WAAW,CAAC;IAC1B,QAAQ,CAAC,EAAE,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE;QACpC,MAAM,QAAQ,GAAG,MAAM,CAAC,qBAAqB,IAAI,iBAAiB,CAAC;QACnE,MAAM,WAAW,GAAG,MAAM,CAAC,oBAAoB,IAAI,qBAAqB,CAAC;QACzE,MAAM,WAAW,GACf,MAAM,CAAC,gCAAgC,IAAI,kCAAkC,CAAC;QAChF,MAAM,uBAAuB,GAC3B,MAAM,CAAC,gCAAgC,IAAI,mCAAmC,CAAC;QAEjF,MAAM,OAAO,GAAa,EAAE,CAAC;QAE7B,qEAAqE;QACrE,sEAAsE;QACtE,kEAAkE;QAClE,MAAM,gBAAgB,GAAG,oBAAoB,CAAC,OAAO,CAAC,CAAC;QACvD,MAAM,gBAAgB,GAAG,IAAI,GAAG,EAAU,CAAC;QAE3C,KAAK,MAAM,CAAC,QAAQ,EAAE,IAAI,CAAC,IAAI,gBAAgB,EAAE,CAAC;YAChD,MAAM,QAAQ,GAAG,CAAC,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;YAChE,IAAI,QAAQ,CAAC,MAAM,GAAG,WAAW;gBAAE,SAAS;YAC5C,MAAM,MAAM,GAAG,qBAAqB,CAAC,IAAI,CAAC,CAAC;YAC3C,qEAAqE;YACrE,yEAAyE;YACzE,uEAAuE;YACvE,wEAAwE;YACxE,qEAAqE;YACrE,gEAAgE;YAChE,IAAI,MAAM,CAAC,QAAQ,KAAK,SAAS,IAAI,MAAM,CAAC,QAAQ,GAAG,uBAAuB;gBAAE,SAAS;YACzF,sEAAsE;YACtE,4DAA4D;YAC5D,gBAAgB,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;YAC/B,OAAO,CAAC,IAAI,CACV,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE;gBAC9B,QAAQ,EAAE,KAAK;gBACf,QAAQ,EAAE,SAAS;gBACnB,OAAO,EAAE,kCAAkC,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,cAAc,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,gDAAgD;gBAC9K,IAAI,EAAE,EAAE,IAAI,EAAE,MAAM,CAAC,QAAQ,EAAE,IAAI,EAAE,MAAM,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE;gBACzE,UAAU,EACR,mFAAmF;gBACrF,QAAQ,EAAE;oBACR,QAAQ;oBACR,YAAY,EAAE,QAAQ,CAAC,MAAM;oBAC7B,eAAe,EAAE,IAAI,CAAC,MAAM;oBAC5B,QAAQ;iBACT;aACF,CAAC,CACH,CAAC;QACJ,CAAC;QAED,kEAAkE;QAClE,kEAAkE;QAClE,MAAM,MAAM,GAAG,WAAW,CAAC,OAAO,EAAE,QAAQ,EAAE,WAAW,EAAE,QAAQ,CAAC,CAAC;QACrE,OAAO,CAAC,IAAI,CAAC,GAAG,sBAAsB,CAAC,MAAM,EAAE,gBAAgB,EAAE,MAAM,CAAC,CAAC,CAAC;QAC1E,OAAO,OAAO,CAAC;IACjB,CAAC;CACF,CAAC,CAAC;AAEH;;;;GAIG;AACH,SAAS,sBAAsB,CAC7B,MAAkD,EAClD,gBAAqC,EACrC,MAAmB;IAEnB,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,IAAI,KAAK,CAAC,MAAM,GAAG,CAAC;YAAE,SAAS;QAC/B,MAAM,OAAO,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;QACzB,oBAAoB;QACpB,IAAI,CAAC,OAAO;YAAE,SAAS;QACvB,IAAI,gBAAgB,CAAC,GAAG,CAAC,OAAO,CAAC,QAAQ,CAAC;YAAE,SAAS;QACrD,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YACtC,MAAM,GAAG,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;YACrB,oBAAoB;YACpB,IAAI,CAAC,GAAG;gBAAE,SAAS;YACnB,OAAO,CAAC,IAAI,CACV,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE;gBAC9B,QAAQ,EAAE,KAAK;gBACf,QAAQ,EAAE,SAAS;gBACnB,OAAO,EAAE,GAAG,GAAG,CAAC,UAAU,yBAAyB,OAAO,CAAC,aAAa,KAAK,OAAO,CAAC,QAAQ,IAAI,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI;gBACzH,IAAI,EAAE,EAAE,IAAI,EAAE,GAAG,CAAC,QAAQ,EAAE,IAAI,EAAE,GAAG,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,CAAC,MAAM,EAAE;gBAChE,UAAU,EACR,kFAAkF;gBACpF,QAAQ,EAAE;oBACR,OAAO,EAAE,OAAO,CAAC,aAAa;oBAC9B,SAAS,EAAE,GAAG,CAAC,aAAa;oBAC5B,SAAS,EAAE,KAAK,CAAC,MAAM;iBACxB;aACF,CAAC,CACH,CAAC;QACJ,CAAC;IACH,CAAC;IACD,OAAO,OAAO,CAAC;AACjB,CAAC;AAED;;;;;;;;;;GAUG;AACH,SAAS,UAAU,CAAC,GAAuB;IACzC,OAAO,GAAG,GAAG,CAAC,QAAQ,IAAI,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,IAAI,GAAG,CAAC,UAAU,EAAE,CAAC;AACvF,CAAC;AAED;;;;;GAKG;AACH,SAAS,WAAW,CAClB,OAA0C,EAC1C,UAAoC,EACpC,GAAuB;IAEvB,IAAI,MAAM,GAAG,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;IACvC,IAAI,IAAI,GAAG,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;IACxC,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,MAAM,GAAG,EAAE,CAAC;QACZ,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QAClC,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;QACzB,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC;IACrC,CAAC;IACD,oBAAoB;IACpB,IAAI,CAAC,IAAI;QAAE,OAAO;IAClB,MAAM,EAAE,GAAG,UAAU,CAAC,GAAG,CAAC,CAAC;IAC3B,IAAI,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC;QAAE,OAAO;IACzB,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;IACb,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACnB,CAAC;AAED,SAAS,WAAW,CAClB,OAAgB,EAChB,QAAgB,EAChB,WAAmB,EACnB,QAAkC;IAElC,mEAAmE;IACnE,gDAAgD;IAChD,MAAM,OAAO,GAAG,IAAI,GAAG,EAAgC,CAAC;IACxD,MAAM,UAAU,GAAG,IAAI,GAAG,EAAuB,CAAC;IAClD,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,SAAS,CAAC,EAAE,CAAC;QAClD,MAAM,IAAI,GAA8C,OAAO,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QAChF,oBAAoB;QACpB,IAAI,CAAC,IAAI;YAAE,SAAS;QACpB,KAAK,MAAM,GAAG,IAAI,IAAI,EAAE,CAAC;YACvB,IAAI,CAAC,mBAAmB,CAAC,GAAG,EAAE,QAAQ,EAAE,WAAW,EAAE,QAAQ,CAAC;gBAAE,SAAS;YACzE,WAAW,CAAC,OAAO,EAAE,UAAU,EAAE,GAAG,CAAC,CAAC;QACxC,CAAC;IACH,CAAC;IACD,OAAO,CAAC,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAAC;AAC/B,CAAC;AAED;;;;;GAKG;AACH,SAAS,oBAAoB,CAAC,OAAgB;IAC5C,MAAM,OAAO,GAAG,IAAI,GAAG,EAAgC,CAAC;IACxD,MAAM,UAAU,GAAG,IAAI,GAAG,EAAuB,CAAC;IAClD,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,SAAS,CAAC,EAAE,CAAC;QAClD,MAAM,IAAI,GAA8C,OAAO,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QAChF,oBAAoB;QACpB,IAAI,CAAC,IAAI;YAAE,SAAS;QACpB,KAAK,MAAM,GAAG,IAAI,IAAI,EAAE,CAAC;YACvB,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC;gBAAE,SAAS;YACnC,WAAW,CAAC,OAAO,EAAE,UAAU,EAAE,GAAG,CAAC,CAAC;QACxC,CAAC;IACH,CAAC;IACD,OAAO,OAAO,CAAC;AACjB,CAAC;AAED,SAAS,qBAAqB,CAAC,IAAmC;IAChE,OAAO,IAAI,CAAC,MAAM,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,aAAa,GAAG,EAAE,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AAC/E,CAAC;AAED;;;;;;;;;;GAUG;AACH,SAAS,cAAc,CAAC,GAAuB;IAC7C,IAAI,GAAG,CAAC,IAAI,KAAK,OAAO,IAAI,GAAG,CAAC,IAAI,KAAK,qBAAqB,IAAI,GAAG,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;QAC7F,OAAO,KAAK,CAAC;IACf,CAAC;IACD,IAAI,GAAG,CAAC,UAAU;QAAE,OAAO,KAAK,CAAC;IACjC,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,mBAAmB,CAC1B,GAAuB,EACvB,QAAgB,EAChB,WAAmB,EACnB,QAAkC;IAElC,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC;QAAE,OAAO,KAAK,CAAC;IACvC,uEAAuE;IACvE,4EAA4E;IAC5E,0EAA0E;IAC1E,4DAA4D;IAC5D,MAAM,IAAI,GAAG,QAAQ,EAAE,QAAQ,CAAC,GAAG,CAAC,GAAG,CAAC,QAAQ,CAAC,EAAE,SAAS,IAAI,GAAG,CAAC,OAAO,GAAG,GAAG,CAAC,IAAI,GAAG,CAAC,CAAC;IAC3F,IAAI,IAAI,GAAG,QAAQ;QAAE,OAAO,KAAK,CAAC;IAClC,IAAI,GAAG,CAAC,QAAQ,KAAK,SAAS,IAAI,GAAG,CAAC,QAAQ,GAAG,WAAW;QAAE,OAAO,KAAK,CAAC;IAC3E,OAAO,IAAI,CAAC;AACd,CAAC"}
+{"version":3,"file":"duplicated-function-body.js","sourceRoot":"","sources":["../../src/rules/duplicated-function-body.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,OAAO,EAAE,mBAAmB,EAAE,MAAM,8BAA8B,CAAC;AAEnE,OAAO,EAAE,KAAK,EAAE,MAAM,sBAAsB,CAAC;AAE7C,OAAO,EAAE,iBAAiB,EAAE,MAAM,0BAA0B,CAAC;AAC7D,OAAO,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAC;AAM9C,MAAM,IAAI,GAAG,gCAAgC,CAAC;AAE9C,MAAM,CAAC,MAAM,0BAA0B,GAAG,UAAU,CAAC;IACnD,IAAI,EAAE,IAAI;IACV,eAAe,EAAE,SAAS;IAC1B,WAAW,EAAE,CAAC,WAAW,CAAC;IAC1B,QAAQ,CAAC,EAAE,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE;QACpC,MAAM,UAAU,GAAG,YAAY,CAAC,OAAO,EAAE,QAAQ,CAAC,CAAC;QACnD,MAAM,EAAE,UAAU,EAAE,MAAM,EAAE,GAAG,mBAAmB,CAAC,UAAU,EAAE;YAC7D,QAAQ,EAAE,MAAM,CAAC,qBAAqB;YACtC,WAAW,EAAE,MAAM,CAAC,oBAAoB;YACxC,uBAAuB,EAAE,MAAM,CAAC,gCAAgC;YAChE,uBAAuB,EAAE,MAAM,CAAC,gCAAgC;SACjE,CAAC,CAAC;QAEH,kFAAkF;QAClF,qFAAqF;QACrF,OAAO;YACL,GAAG,UAAU,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,eAAe,CAAC,GAAG,EAAE,MAAM,CAAC,CAAC;YACxD,GAAG,MAAM,CAAC,OAAO,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,YAAY,CAAC,KAAK,EAAE,MAAM,CAAC,CAAC;SAC1D,CAAC;IACJ,CAAC;CACF,CAAC,CAAC;AAEH,SAAS,eAAe,CACtB,GAAiE,EACjE,MAAmB;IAEnB,OAAO,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE;QACrC,QAAQ,EAAE,KAAK;QACf,QAAQ,EAAE,SAAS;QACnB,OAAO,EAAE,kCAAkC,MAAM,CAAC,GAAG,CAAC,QAAQ,CAAC,MAAM,CAAC,cAAc,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,MAAM,CAAC,GAAG,CAAC,eAAe,CAAC,gDAAgD;QAC9L,IAAI,EAAE,EAAE,IAAI,EAAE,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,IAAI,EAAE,GAAG,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,CAAC,MAAM,CAAC,MAAM,EAAE;QACrF,UAAU,EAAE,mFAAmF;QAC/F,QAAQ,EAAE;YACR,QAAQ,EAAE,GAAG,CAAC,QAAQ;YACtB,YAAY,EAAE,GAAG,CAAC,QAAQ,CAAC,MAAM;YACjC,eAAe,EAAE,GAAG,CAAC,eAAe;YACpC,QAAQ,EAAE,GAAG,CAAC,QAAQ;SACvB;KACF,CAAC,CAAC;AACL,CAAC;AAED,SAAS,YAAY,CACnB,KAA+D,EAC/D,MAAmB;IAEnB,MAAM,OAAO,GAAG,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC;IACjC,oBAAoB;IACpB,IAAI,CAAC,OAAO;QAAE,OAAO,EAAE,CAAC;IACxB,OAAO,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,CACxC,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE;QAC9B,QAAQ,EAAE,KAAK;QACf,QAAQ,EAAE,SAAS;QACnB,OAAO,EAAE,GAAG,GAAG,CAAC,UAAU,yBAAyB,OAAO,CAAC,aAAa,KAAK,OAAO,CAAC,QAAQ,IAAI,MAAM,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI;QACzH,IAAI,EAAE,EAAE,IAAI,EAAE,GAAG,CAAC,QAAQ,EAAE,IAAI,EAAE,GAAG,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,CAAC,MAAM,EAAE;QAChE,UAAU,EACR,kFAAkF;QACpF,QAAQ,EAAE;YACR,OAAO,EAAE,OAAO,CAAC,aAAa;YAC9B,SAAS,EAAE,GAAG,CAAC,aAAa;YAC5B,SAAS,EAAE,KAAK,CAAC,OAAO,CAAC,MAAM;SAChC;KACF,CAAC,CACH,CAAC;AACJ,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,YAAY,CAAC,OAAgB,EAAE,QAAkC;IACxE,MAAM,UAAU,GAAqB,EAAE,CAAC;IACxC,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,SAAS,CAAC,EAAE,CAAC;QAClD,MAAM,IAAI,GAAG,OAAO,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACrC,oBAAoB;QACpB,IAAI,CAAC,IAAI;YAAE,SAAS;QACpB,KAAK,MAAM,GAAG,IAAI,IAAI;YAAE,UAAU,CAAC,IAAI,CAAC,WAAW,CAAC,GAAG,EAAE,QAAQ,CAAC,CAAC,CAAC;IACtE,CAAC;IACD,OAAO,UAAU,CAAC;AACpB,CAAC;AAED,SAAS,WAAW,CAAC,GAAuB,EAAE,QAAkC;IAC9E,MAAM,SAAS,GAAG,QAAQ,EAAE,QAAQ,CAAC,GAAG,CAAC,GAAG,CAAC,QAAQ,CAAC,EAAE,SAAS,CAAC;IAClE,OAAO;QACL,QAAQ,EAAE,GAAG,CAAC,QAAQ;QACtB,IAAI,EAAE,GAAG,CAAC,IAAI;QACd,UAAU,EAAE,GAAG,CAAC,UAAU;QAC1B,QAAQ,EAAE,GAAG,CAAC,QAAQ;QACtB,IAAI,EAAE,GAAG,CAAC,IAAI;QACd,MAAM,EAAE,GAAG,CAAC,MAAM;QAClB,OAAO,EAAE,GAAG,CAAC,OAAO;QACpB,UAAU,EAAE,GAAG,CAAC,UAAU;QAC1B,aAAa,EAAE,GAAG,CAAC,aAAa;QAChC,OAAO,EAAE,KAAK,CAAC,GAAG,CAAC;QACnB,GAAG,CAAC,GAAG,CAAC,QAAQ,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,QAAQ,EAAE,GAAG,CAAC,QAAQ,EAAE,CAAC;QACjE,GAAG,CAAC,SAAS,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,SAAS,EAAE,CAAC;KAClD,CAAC;AACJ,CAAC"}

package/dist/rules/near-duplicate-function-body.d.ts

@@ -0,0 +1,11 @@
+/**
+ * graph:near-duplicate-function-body — LSH-banded MinHash near-clone detection.
+ *
+ * Complements `graph:duplicated-function-body` (exact hash). The detection algorithm now
+ * lives in `@opensip-cli/clone-detection` (ADR-0064); this rule is a thin adapter that
+ * maps the catalog to `CloneCandidate[]` (stamping `language` from `languageOfFile` for
+ * the same-language gate, which stays in graph), calls `findNearDuplicates`, and wraps
+ * each cluster into a graph `Signal` with the unchanged message/severity/metadata.
+ */
+export declare const nearDuplicateFunctionBodyRule: import("../types.js").Rule;
+//# sourceMappingURL=near-duplicate-function-body.d.ts.map

package/dist/rules/near-duplicate-function-body.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"near-duplicate-function-body.d.ts","sourceRoot":"","sources":["../../src/rules/near-duplicate-function-body.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAeH,eAAO,MAAM,6BAA6B,4BA4BxC,CAAC"}

package/dist/rules/near-duplicate-function-body.js

@@ -0,0 +1,74 @@
+/**
+ * graph:near-duplicate-function-body — LSH-banded MinHash near-clone detection.
+ *
+ * Complements `graph:duplicated-function-body` (exact hash). The detection algorithm now
+ * lives in `@opensip-cli/clone-detection` (ADR-0064); this rule is a thin adapter that
+ * maps the catalog to `CloneCandidate[]` (stamping `language` from `languageOfFile` for
+ * the same-language gate, which stays in graph), calls `findNearDuplicates`, and wraps
+ * each cluster into a graph `Signal` with the unchanged message/severity/metadata.
+ */
+import { findNearDuplicates } from '@opensip-cli/clone-detection';
+import { languageOfFile } from '../lang-adapter/language-of-file.js';
+import { createGraphSignal } from './create-graph-signal.js';
+import { defineRule } from './define-rule.js';
+const SLUG = 'graph:near-duplicate-function-body';
+export const nearDuplicateFunctionBodyRule = defineRule({
+    slug: SLUG,
+    defaultSeverity: 'warning',
+    evaluate({ catalog, config }) {
+        const candidates = toCandidates(catalog);
+        const clusters = findNearDuplicates(candidates, {
+            minSimilarity: config.minNearDuplicateSimilarity,
+            minBodySize: config.minNearDuplicateBodySize,
+            lshBands: config.nearDuplicateLshBands,
+        });
+        return clusters.map((c) => createGraphSignal(SLUG, config, {
+            severity: 'low',
+            category: 'quality',
+            message: `${String(c.nearMembers.length)} near-duplicate function bodies cluster around ${c.anchor.qualifiedName} (estimated Jaccard ≥ ${c.estimatedSimilarity.toFixed(2)}).`,
+            code: { file: c.anchor.filePath, line: c.anchor.line, column: c.anchor.column },
+            suggestion: 'Extract the shared logic into one function and replace the near-clone copies with calls to it.',
+            metadata: {
+                nearMembers: c.nearMembers,
+                exactMembers: c.exactMembers,
+                estimatedSimilarity: c.estimatedSimilarity,
+                clusterSize: c.clusterSize,
+            },
+        }));
+    },
+});
+/**
+ * Map the catalog to `CloneCandidate[]` in catalog iteration order (so the substrate's
+ * `collectEligible` index identity — which the union-find clustering depends on — matches
+ * the prior in-rule behavior). `language` is the caller-resolved same-language gate
+ * (`languageOfFile`, which stays in graph by the rule of three).
+ */
+function toCandidates(catalog) {
+    const candidates = [];
+    for (const name of Object.keys(catalog.functions)) {
+        const occs = catalog.functions[name];
+        if (!occs)
+            continue;
+        for (const occ of occs)
+            candidates.push(toCandidate(occ));
+    }
+    return candidates;
+}
+function toCandidate(occ) {
+    const language = languageOfFile(occ.filePath);
+    return {
+        bodyHash: occ.bodyHash,
+        kind: occ.kind,
+        inTestFile: occ.inTestFile,
+        filePath: occ.filePath,
+        line: occ.line,
+        column: occ.column,
+        endLine: occ.endLine,
+        simpleName: occ.simpleName,
+        qualifiedName: occ.qualifiedName,
+        ...(occ.bodySignature === undefined ? {} : { bodySignature: occ.bodySignature }),
+        ...(occ.bodySize === undefined ? {} : { bodySize: occ.bodySize }),
+        ...(language === undefined ? {} : { language }),
+    };
+}
+//# sourceMappingURL=near-duplicate-function-body.js.map

package/dist/rules/near-duplicate-function-body.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"near-duplicate-function-body.js","sourceRoot":"","sources":["../../src/rules/near-duplicate-function-body.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,kBAAkB,EAAE,MAAM,8BAA8B,CAAC;AAElE,OAAO,EAAE,cAAc,EAAE,MAAM,qCAAqC,CAAC;AAErE,OAAO,EAAE,iBAAiB,EAAE,MAAM,0BAA0B,CAAC;AAC7D,OAAO,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAC;AAM9C,MAAM,IAAI,GAAG,oCAAoC,CAAC;AAElD,MAAM,CAAC,MAAM,6BAA6B,GAAG,UAAU,CAAC;IACtD,IAAI,EAAE,IAAI;IACV,eAAe,EAAE,SAAS;IAC1B,QAAQ,CAAC,EAAE,OAAO,EAAE,MAAM,EAAE;QAC1B,MAAM,UAAU,GAAG,YAAY,CAAC,OAAO,CAAC,CAAC;QACzC,MAAM,QAAQ,GAAG,kBAAkB,CAAC,UAAU,EAAE;YAC9C,aAAa,EAAE,MAAM,CAAC,0BAA0B;YAChD,WAAW,EAAE,MAAM,CAAC,wBAAwB;YAC5C,QAAQ,EAAE,MAAM,CAAC,qBAAqB;SACvC,CAAC,CAAC;QAEH,OAAO,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CACxB,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE;YAC9B,QAAQ,EAAE,KAAK;YACf,QAAQ,EAAE,SAAS;YACnB,OAAO,EAAE,GAAG,MAAM,CAAC,CAAC,CAAC,WAAW,CAAC,MAAM,CAAC,kDAAkD,CAAC,CAAC,MAAM,CAAC,aAAa,yBAAyB,CAAC,CAAC,mBAAmB,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI;YAC7K,IAAI,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,EAAE,CAAC,CAAC,MAAM,CAAC,MAAM,EAAE;YAC/E,UAAU,EACR,gGAAgG;YAClG,QAAQ,EAAE;gBACR,WAAW,EAAE,CAAC,CAAC,WAAW;gBAC1B,YAAY,EAAE,CAAC,CAAC,YAAY;gBAC5B,mBAAmB,EAAE,CAAC,CAAC,mBAAmB;gBAC1C,WAAW,EAAE,CAAC,CAAC,WAAW;aAC3B;SACF,CAAC,CACH,CAAC;IACJ,CAAC;CACF,CAAC,CAAC;AAEH;;;;;GAKG;AACH,SAAS,YAAY,CAAC,OAAgB;IACpC,MAAM,UAAU,GAAqB,EAAE,CAAC;IACxC,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,SAAS,CAAC,EAAE,CAAC;QAClD,MAAM,IAAI,GAAG,OAAO,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QACrC,IAAI,CAAC,IAAI;YAAE,SAAS;QACpB,KAAK,MAAM,GAAG,IAAI,IAAI;YAAE,UAAU,CAAC,IAAI,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC,CAAC;IAC5D,CAAC;IACD,OAAO,UAAU,CAAC;AACpB,CAAC;AAED,SAAS,WAAW,CAAC,GAAuB;IAC1C,MAAM,QAAQ,GAAG,cAAc,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;IAC9C,OAAO;QACL,QAAQ,EAAE,GAAG,CAAC,QAAQ;QACtB,IAAI,EAAE,GAAG,CAAC,IAAI;QACd,UAAU,EAAE,GAAG,CAAC,UAAU;QAC1B,QAAQ,EAAE,GAAG,CAAC,QAAQ;QACtB,IAAI,EAAE,GAAG,CAAC,IAAI;QACd,MAAM,EAAE,GAAG,CAAC,MAAM;QAClB,OAAO,EAAE,GAAG,CAAC,OAAO;QACpB,UAAU,EAAE,GAAG,CAAC,UAAU;QAC1B,aAAa,EAAE,GAAG,CAAC,aAAa;QAChC,GAAG,CAAC,GAAG,CAAC,aAAa,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,aAAa,EAAE,GAAG,CAAC,aAAa,EAAE,CAAC;QAChF,GAAG,CAAC,GAAG,CAAC,QAAQ,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,QAAQ,EAAE,GAAG,CAAC,QAAQ,EAAE,CAAC;QACjE,GAAG,CAAC,QAAQ,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,QAAQ,EAAE,CAAC;KAChD,CAAC;AACJ,CAAC"}

package/dist/rules/registry.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"registry.d.ts","sourceRoot":"","sources":["../../src/rules/registry.ts"],"names":[],"mappings":"AACA;;;;;;;;;;;;;;;;GAgBG;AAeH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,aAAa,CAAC;AAyBxC;;;GAGG;AACH,qBAAa,kBAAkB;IAC7B,OAAO,CAAC,QAAQ,CAAC,KAAK,CAInB;;IAQH,MAAM,IAAI,SAAS,IAAI,EAAE;CAG1B;AAED,+DAA+D;AAC/D,wBAAgB,mBAAmB,IAAI,kBAAkB,CAExD;AA2BD;;;GAGG;AACH,wBAAgB,YAAY,IAAI,SAAS,IAAI,EAAE,CAE9C"}
+{"version":3,"file":"registry.d.ts","sourceRoot":"","sources":["../../src/rules/registry.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAgBH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,aAAa,CAAC;AA0BxC;;;GAGG;AACH,qBAAa,kBAAkB;IAC7B,OAAO,CAAC,QAAQ,CAAC,KAAK,CAInB;;IAQH,MAAM,IAAI,SAAS,IAAI,EAAE;CAG1B;AAED,+DAA+D;AAC/D,wBAAgB,mBAAmB,IAAI,kBAAkB,CAExD;AA2BD;;;GAGG;AACH,wBAAgB,YAAY,IAAI,SAAS,IAAI,EAAE,CAE9C"}

package/dist/rules/registry.js

@@ -1,4 +1,3 @@
-// @fitness-ignore-file batch-operation-limits -- iterates bounded collection (registered rules + getAll() returning the rule list per RunScope)
 /**
  * Rule registry — per-RunScope.
  *
@@ -22,6 +21,7 @@ import { cycleRule } from './cycle.js';
 import { duplicatedFunctionBodyRule } from './duplicated-function-body.js';
 import { highBlastUntestedRule } from './high-blast-untested.js';
 import { largeFunctionRule } from './large-function.js';
+import { nearDuplicateFunctionBodyRule } from './near-duplicate-function-body.js';
 import { noSideEffectPathRule } from './no-side-effect-path.js';
 import { orphanSubtreeRule } from './orphan-subtree.js';
 import { testOnlyReachableRule } from './test-only-reachable.js';
@@ -33,6 +33,7 @@ import { wideFunctionRule } from './wide-function.js';
 const BUILT_IN_RULES = [
     orphanSubtreeRule,
     duplicatedFunctionBodyRule,
+    nearDuplicateFunctionBodyRule,
     noSideEffectPathRule,
     testOnlyReachableRule,
     alwaysThrowsBranchRule,

package/dist/rules/registry.js.map

@@ -1 +1 @@
-{"version":3,"file":"registry.js","sourceRoot":"","sources":["../../src/rules/registry.ts"],"names":[],"mappings":"AAAA,gJAAgJ;AAChJ;;;;;;;;;;;;;;;;GAgBG;AAEH,OAAO,EAAE,QAAQ,EAAE,YAAY,EAAqB,MAAM,mBAAmB,CAAC;AAE9E,OAAO,EAAE,sBAAsB,EAAE,MAAM,2BAA2B,CAAC;AACnE,OAAO,EAAE,SAAS,EAAE,MAAM,YAAY,CAAC;AACvC,OAAO,EAAE,0BAA0B,EAAE,MAAM,+BAA+B,CAAC;AAC3E,OAAO,EAAE,qBAAqB,EAAE,MAAM,0BAA0B,CAAC;AACjE,OAAO,EAAE,iBAAiB,EAAE,MAAM,qBAAqB,CAAC;AACxD,OAAO,EAAE,oBAAoB,EAAE,MAAM,0BAA0B,CAAC;AAChE,OAAO,EAAE,iBAAiB,EAAE,MAAM,qBAAqB,CAAC;AACxD,OAAO,EAAE,qBAAqB,EAAE,MAAM,0BAA0B,CAAC;AACjE,OAAO,EAAE,sBAAsB,EAAE,MAAM,0BAA0B,CAAC;AAClE,OAAO,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAC;AAUtD;;uBAEuB;AACvB,MAAM,cAAc,GAAoB;IACtC,iBAAiB;IACjB,0BAA0B;IAC1B,oBAAoB;IACpB,qBAAqB;IACrB,sBAAsB;IACtB,4CAA4C;IAC5C,iBAAiB;IACjB,gBAAgB;IAChB,qBAAqB;IACrB,SAAS;IACT,sBAAsB;CACvB,CAAC;AAEF;;;GAGG;AACH,MAAM,OAAO,kBAAkB;IACZ,KAAK,GAAG,IAAI,QAAQ,CAAmB;QACtD,MAAM,EAAE,aAAa;QACrB,eAAe,EAAE,iBAAiB;QAClC,SAAS,EAAE,qBAAqB;KACjC,CAAC,CAAC;IAEH;QACE,KAAK,MAAM,IAAI,IAAI,cAAc,EAAE,CAAC;YAClC,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,EAAE,EAAE,EAAE,IAAI,CAAC,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE,IAAI,EAAE,CAAC,CAAC;QAChE,CAAC;IACH,CAAC;IAED,MAAM;QACJ,OAAO,IAAI,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;IAChD,CAAC;CACF;AAED,+DAA+D;AAC/D,MAAM,UAAU,mBAAmB;IACjC,OAAO,IAAI,kBAAkB,EAAE,CAAC;AAClC,CAAC;AAED;;;;;;GAMG;AACH,SAAS,oBAAoB;IAC3B,MAAM,KAAK,GAAG,YAAY,EAAE,CAAC;IAC7B,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CACb,2DAA2D;YACzD,0EAA0E;YAC1E,8DAA8D,CACjE,CAAC;IACJ,CAAC;IACD,IAAI,CAAC,KAAK,CAAC,KAAK,EAAE,CAAC;QACjB,MAAM,IAAI,KAAK,CACb,uEAAuE;YACrE,sDAAsD,CACzD,CAAC;IACJ,CAAC;IACD,OAAO,KAAK,CAAC,KAAK,CAAC,KAAK,CAAC;AAC3B,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY;IAC1B,OAAO,oBAAoB,EAAE,CAAC,MAAM,EAAE,CAAC;AACzC,CAAC"}
+{"version":3,"file":"registry.js","sourceRoot":"","sources":["../../src/rules/registry.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,OAAO,EAAE,QAAQ,EAAE,YAAY,EAAqB,MAAM,mBAAmB,CAAC;AAE9E,OAAO,EAAE,sBAAsB,EAAE,MAAM,2BAA2B,CAAC;AACnE,OAAO,EAAE,SAAS,EAAE,MAAM,YAAY,CAAC;AACvC,OAAO,EAAE,0BAA0B,EAAE,MAAM,+BAA+B,CAAC;AAC3E,OAAO,EAAE,qBAAqB,EAAE,MAAM,0BAA0B,CAAC;AACjE,OAAO,EAAE,iBAAiB,EAAE,MAAM,qBAAqB,CAAC;AACxD,OAAO,EAAE,6BAA6B,EAAE,MAAM,mCAAmC,CAAC;AAClF,OAAO,EAAE,oBAAoB,EAAE,MAAM,0BAA0B,CAAC;AAChE,OAAO,EAAE,iBAAiB,EAAE,MAAM,qBAAqB,CAAC;AACxD,OAAO,EAAE,qBAAqB,EAAE,MAAM,0BAA0B,CAAC;AACjE,OAAO,EAAE,sBAAsB,EAAE,MAAM,0BAA0B,CAAC;AAClE,OAAO,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAC;AAUtD;;uBAEuB;AACvB,MAAM,cAAc,GAAoB;IACtC,iBAAiB;IACjB,0BAA0B;IAC1B,6BAA6B;IAC7B,oBAAoB;IACpB,qBAAqB;IACrB,sBAAsB;IACtB,4CAA4C;IAC5C,iBAAiB;IACjB,gBAAgB;IAChB,qBAAqB;IACrB,SAAS;IACT,sBAAsB;CACvB,CAAC;AAEF;;;GAGG;AACH,MAAM,OAAO,kBAAkB;IACZ,KAAK,GAAG,IAAI,QAAQ,CAAmB;QACtD,MAAM,EAAE,aAAa;QACrB,eAAe,EAAE,iBAAiB;QAClC,SAAS,EAAE,qBAAqB;KACjC,CAAC,CAAC;IAEH;QACE,KAAK,MAAM,IAAI,IAAI,cAAc,EAAE,CAAC;YAClC,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,EAAE,EAAE,EAAE,IAAI,CAAC,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE,IAAI,EAAE,CAAC,CAAC;QAChE,CAAC;IACH,CAAC;IAED,MAAM;QACJ,OAAO,IAAI,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;IAChD,CAAC;CACF;AAED,+DAA+D;AAC/D,MAAM,UAAU,mBAAmB;IACjC,OAAO,IAAI,kBAAkB,EAAE,CAAC;AAClC,CAAC;AAED;;;;;;GAMG;AACH,SAAS,oBAAoB;IAC3B,MAAM,KAAK,GAAG,YAAY,EAAE,CAAC;IAC7B,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CACb,2DAA2D;YACzD,0EAA0E;YAC1E,8DAA8D,CACjE,CAAC;IACJ,CAAC;IACD,IAAI,CAAC,KAAK,CAAC,KAAK,EAAE,CAAC;QACjB,MAAM,IAAI,KAAK,CACb,uEAAuE;YACrE,sDAAsD,CACzD,CAAC;IACJ,CAAC;IACD,OAAO,KAAK,CAAC,KAAK,CAAC,KAAK,CAAC;AAC3B,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY;IAC1B,OAAO,oBAAoB,EAAE,CAAC,MAAM,EAAE,CAAC;AACzC,CAAC"}

package/dist/tool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tool.d.ts","sourceRoot":"","sources":["../src/tool.ts"],"names":[],"mappings":"AACA;;;;;;;;;;;;;;;;;;;GAmBG;AA8BH,OAAO,yBAAyB,CAAC;AAGjC,OAAO,KAAK,EAGV,IAAI,EAGL,MAAM,mBAAmB,CAAC;AAmG3B;;GAEG;AACH,eAAO,MAAM,sBAAsB,UAAU,CAAC;AAE9C,eAAO,MAAM,eAAe,yCAAyC,CAAC;AAEtE,eAAO,MAAM,SAAS,EAAE,IAoBtB,CAAC"}
+{"version":3,"file":"tool.d.ts","sourceRoot":"","sources":["../src/tool.ts"],"names":[],"mappings":"AACA;;;;;;;;;;;;;;;;;;;GAmBG;AA+BH,OAAO,yBAAyB,CAAC;AAGjC,OAAO,KAAK,EAAuB,IAAI,EAAa,MAAM,mBAAmB,CAAC;AAmG9E;;GAEG;AACH,eAAO,MAAM,sBAAsB,UAAU,CAAC;AAE9C,eAAO,MAAM,eAAe,yCAAyC,CAAC;AAEtE,eAAO,MAAM,SAAS,EAAE,IAuBtB,CAAC"}