fix(review-enrichment): include changed paths in dead-export caller checks

Author: tmimmanuel

review-enrichment/src/analyzers/caller-impact.ts

@@ -310,6 +310,7 @@ async function resolveCallers(
   headSha: string,
   token: string,
   skipPaths: Set<string> | null,
+  additionalPaths: Iterable<string> = [],
   fetchImpl: typeof fetch,
   options: ScanOptions = {},
 ): Promise<string[]> {
@@ -321,8 +322,10 @@ async function resolveCallers(
     fetchImpl,
     options,
   );
+  const candidatePaths = new Set(hitPaths);
+  for (const path of additionalPaths) candidatePaths.add(path);
   const callers: string[] = [];
-  for (const path of hitPaths) {
+  for (const path of candidatePaths) {
     if (skipPaths?.has(path)) continue;
     if (callers.length >= MAX_CALLERS_PER_SYMBOL) break;
     if (
@@ -478,6 +481,7 @@ export async function scanCallerImpact(
       req.headSha,
       req.githubToken,
       candidate.kind === "dead" ? null : changedPaths,
+      candidate.kind === "dead" ? changedPaths : [],
       fetchImpl,
       options,
     );