@kodus/kodus-graph 0.2.5 → 0.2.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kodus/kodus-graph",
-  "version": "0.2.5",
+  "version": "0.2.6",
   "description": "Code graph builder for Kodus code review — parses source code into structural graphs with nodes, edges, and analysis",
   "type": "module",
   "bin": {

package/src/analysis/diff.ts

@@ -74,18 +74,17 @@ export function computeStructuralDiff(
     } else {
       const newN = newNodesMap.get(qn)!;
       const changes: string[] = [];
-      // Detect real changes vs. pure displacement using content_hash.
-      // content_hash is per-node (SHA256 of the function/class source text).
-      // If both have content_hash: definitive comparison — same hash = identical content.
-      // If either is missing (legacy data): fallback to size heuristic.
-      if (n.line_start !== newN.line_start || n.line_end !== newN.line_end) {
-        if (n.content_hash && newN.content_hash) {
-          if (n.content_hash !== newN.content_hash) changes.push('line_range');
-        } else {
-          const oldSize = n.line_end - n.line_start;
-          const newSize = newN.line_end - newN.line_start;
-          if (oldSize !== newSize) changes.push('line_range');
-        }
+      // Detect real content changes vs. pure displacement.
+      // content_hash = SHA256 of the node's source text (position-independent).
+      if (n.content_hash && newN.content_hash) {
+        // Definitive: hash comparison catches ALL content changes,
+        // even same-line-count edits (e.g. `return 1` → `return 2`).
+        if (n.content_hash !== newN.content_hash) changes.push('body');
+      } else if (n.line_start !== newN.line_start || n.line_end !== newN.line_end) {
+        // Fallback (legacy data without content_hash): size heuristic.
+        const oldSize = n.line_end - n.line_start;
+        const newSize = newN.line_end - newN.line_start;
+        if (oldSize !== newSize) changes.push('line_range');
       }
       if ((n.params || '') !== (newN.params || '')) changes.push('params');
       if ((n.return_type || '') !== (newN.return_type || '')) changes.push('return_type');

package/src/analysis/prompt-formatter.ts

@@ -118,16 +118,56 @@ export function formatPrompt(output: ContextV2Output): string {
     lines.push('');
   }
 
-  // Structural diff summary
+  // Structural diff
   const diff = analysis.structural_diff;
-  if (diff.summary.added > 0 || diff.summary.removed > 0 || diff.summary.modified > 0) {
+  const hasNodeChanges = diff.summary.added > 0 || diff.summary.removed > 0 || diff.summary.modified > 0;
+  const hasEdgeChanges = diff.edges.added.length > 0 || diff.edges.removed.length > 0;
+
+  if (hasNodeChanges || hasEdgeChanges) {
     lines.push('## Structural Changes');
     lines.push('');
-    const parts: string[] = [];
-    if (diff.summary.added > 0) parts.push(`${diff.summary.added} added`);
-    if (diff.summary.removed > 0) parts.push(`${diff.summary.removed} removed`);
-    if (diff.summary.modified > 0) parts.push(`${diff.summary.modified} modified`);
-    lines.push(parts.join(', '));
+
+    if (hasNodeChanges) {
+      const parts: string[] = [];
+      if (diff.summary.added > 0) parts.push(`${diff.summary.added} added`);
+      if (diff.summary.removed > 0) parts.push(`${diff.summary.removed} removed`);
+      if (diff.summary.modified > 0) parts.push(`${diff.summary.modified} modified`);
+      lines.push(parts.join(', '));
+    }
+
+    if (diff.nodes.removed.length > 0) {
+      lines.push('');
+      lines.push('Removed:');
+      for (const n of diff.nodes.removed) {
+        const name = n.qualified_name.split('::').pop();
+        lines.push(`  - [${n.kind}] ${name}  [${n.file_path}:${n.line_start}]`);
+      }
+    }
+
+    if (diff.nodes.modified.length > 0) {
+      lines.push('');
+      lines.push('Modified:');
+      for (const m of diff.nodes.modified) {
+        const name = m.qualified_name.split('::').pop();
+        lines.push(`  - ${name} (${m.changes.join(', ')})`);
+      }
+    }
+
+    if (hasEdgeChanges) {
+      lines.push('');
+      lines.push('Dependency changes:');
+      for (const e of diff.edges.added) {
+        const src = e.source_qualified.split('::').pop();
+        const tgt = e.target_qualified.split('::').pop();
+        lines.push(`  + ${e.kind}: ${src} → ${tgt}`);
+      }
+      for (const e of diff.edges.removed) {
+        const src = e.source_qualified.split('::').pop();
+        const tgt = e.target_qualified.split('::').pop();
+        lines.push(`  - ${e.kind}: ${src} → ${tgt}`);
+      }
+    }
+
     lines.push('');
   }