@liendev/lien 0.29.0 → 0.29.1

package/dist/index.js

@@ -8803,6 +8803,25 @@ function deduplicateResults(results) {
     return true;
   });
 }
+function cleanMetadataValue(key, value) {
+  if (value === void 0 || value === "") return null;
+  if (Array.isArray(value)) {
+    const filtered = value.filter((v) => v !== "");
+    return filtered.length > 0 ? filtered : null;
+  }
+  if (key === "symbols" && typeof value === "object" && value !== null) {
+    const symbols = value;
+    const filterArr = (arr) => Array.isArray(arr) ? arr.filter((s) => s !== "") : [];
+    const filtered = {
+      functions: filterArr(symbols.functions),
+      classes: filterArr(symbols.classes),
+      interfaces: filterArr(symbols.interfaces)
+    };
+    const hasAny = filtered.functions.length > 0 || filtered.classes.length > 0 || filtered.interfaces.length > 0;
+    return hasAny ? filtered : null;
+  }
+  return value;
+}
 function pickMetadata(metadata, allowlist) {
   const result = {
     file: metadata.file,
@@ -8812,8 +8831,9 @@ function pickMetadata(metadata, allowlist) {
   const out = result;
   for (const key of allowlist) {
     if (key === "file" || key === "startLine" || key === "endLine") continue;
-    if (metadata[key] !== void 0) {
-      out[key] = metadata[key];
+    const cleaned = cleanMetadataValue(key, metadata[key]);
+    if (cleaned !== null) {
+      out[key] = cleaned;
     }
   }
   return result;
@@ -8881,17 +8901,12 @@ async function handleSemanticSearch(args, ctx) {
       }
       log(`Returning ${results.length} results`);
       const shaped = shapeResults(results, "semantic_search");
-      const response = {
+      return {
         indexInfo: getIndexMetadata(),
-        results: shaped
+        results: shaped,
+        ...crossRepo && vectorDB instanceof QdrantDB && { groupedByRepo: groupResultsByRepo(shaped) },
+        ...notes.length > 0 && { note: notes.join(" ") }
       };
-      if (crossRepo && vectorDB instanceof QdrantDB) {
-        response.groupedByRepo = groupResultsByRepo(shaped);
-      }
-      if (notes.length > 0) {
-        response.note = notes.join(" ");
-      }
-      return response;
     }
   )(args);
 }
@@ -9086,7 +9101,9 @@ async function findRelatedChunks(filepaths, fileChunksMap, ctx) {
     const targetCanonical = getCanonicalPath(filepath, workspaceRoot);
     relatedChunksMap[index] = related.filter((r) => {
       const chunkCanonical = getCanonicalPath(r.metadata.file, workspaceRoot);
-      return chunkCanonical !== targetCanonical;
+      if (chunkCanonical === targetCanonical) return false;
+      if (r.metadata.language === "markdown") return false;
+      return true;
     });
   });
   return relatedChunksMap;
@@ -9122,23 +9139,15 @@ function findTestAssociations(filepaths, allChunks, ctx) {
     return Array.from(testFiles);
   });
 }
-function deduplicateChunks(fileChunks, relatedChunks, workspaceRoot) {
-  const seenChunks = /* @__PURE__ */ new Set();
-  return [...fileChunks, ...relatedChunks].filter((chunk) => {
-    const canonicalFile = getCanonicalPath(chunk.metadata.file, workspaceRoot);
-    const chunkId = `${canonicalFile}:${chunk.metadata.startLine}-${chunk.metadata.endLine}`;
-    if (seenChunks.has(chunkId)) return false;
-    seenChunks.add(chunkId);
-    return true;
-  });
+function deduplicateChunks(fileChunks, relatedChunks) {
+  return deduplicateResults([...fileChunks, ...relatedChunks]);
 }
-function buildFilesData(filepaths, fileChunksMap, relatedChunksMap, testAssociationsMap, workspaceRoot) {
+function buildFilesData(filepaths, fileChunksMap, relatedChunksMap, testAssociationsMap) {
   const filesData = {};
   filepaths.forEach((filepath, i) => {
     const dedupedChunks = deduplicateChunks(
       fileChunksMap[i],
-      relatedChunksMap[i] || [],
-      workspaceRoot
+      relatedChunksMap[i] || []
     );
     filesData[filepath] = {
       chunks: dedupedChunks,
@@ -9216,8 +9225,7 @@ async function handleGetFilesContext(args, ctx) {
         filepaths,
         fileChunksMap,
         relatedChunksMap,
-        testAssociationsMap,
-        workspaceRoot
+        testAssociationsMap
       );
       const totalChunks = Object.values(filesData).reduce(
         (sum, f) => sum + f.chunks.length,
@@ -9796,7 +9804,11 @@ async function handleGetComplexity(args, ctx) {
         threshold,
         top ?? 10
       );
-      const response = {
+      const note = buildCrossRepoFallbackNote(fallback);
+      if (note) {
+        log("Warning: crossRepo=true requires Qdrant backend. Falling back to single-repo analysis.", "warning");
+      }
+      return {
         indexInfo: getIndexMetadata(),
         summary: {
           filesAnalyzed: report.summary.filesAnalyzed,
@@ -9805,17 +9817,12 @@ async function handleGetComplexity(args, ctx) {
           violationCount: violations.length,
           bySeverity
         },
-        violations: topViolations
+        violations: topViolations,
+        ...crossRepo && !fallback && allChunks.length > 0 && {
+          groupedByRepo: groupViolationsByRepo(topViolations, allChunks)
+        },
+        ...note && { note }
       };
-      if (crossRepo && !fallback && allChunks.length > 0) {
-        response.groupedByRepo = groupViolationsByRepo(topViolations, allChunks);
-      }
-      const note = buildCrossRepoFallbackNote(fallback);
-      if (note) {
-        log("Warning: crossRepo=true requires Qdrant backend. Falling back to single-repo analysis.", "warning");
-        response.note = note;
-      }
-      return response;
     }
   )(args);
 }