@kodus/kodus-graph 0.2.4 → 0.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kodus/kodus-graph",
-  "version": "0.2.4",
+  "version": "0.2.5",
   "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,10 +74,19 @@ export function computeStructuralDiff(
     } else {
       const newN = newNodesMap.get(qn)!;
       const changes: string[] = [];
-      // Only flag line_range when the file content actually changed (different file_hash).
-      // When file_hash matches, line shifts are just displacement from other edits — not real modifications.
-      const hashChanged = !n.file_hash || !newN.file_hash || n.file_hash !== newN.file_hash;
-      if (hashChanged && (n.line_start !== newN.line_start || n.line_end !== newN.line_end)) changes.push('line_range');
+      // 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');
+        }
+      }
       if ((n.params || '') !== (newN.params || '')) changes.push('params');
       if ((n.return_type || '') !== (newN.return_type || '')) changes.push('return_type');
       if (changes.length > 0) modified.push({ qualified_name: qn, changes });

package/src/graph/builder.ts

@@ -26,6 +26,7 @@ export function buildGraphData(
       return_type: f.returnType || undefined,
       is_test: false,
       file_hash: fileHashes.get(f.file) || '',
+      content_hash: f.content_hash,
     });
   }
 
@@ -41,6 +42,7 @@ export function buildGraphData(
       language: detectLang(c.file),
       is_test: false,
       file_hash: fileHashes.get(c.file) || '',
+      content_hash: c.content_hash,
     });
   }
 
@@ -56,6 +58,7 @@ export function buildGraphData(
       language: detectLang(i.file),
       is_test: false,
       file_hash: fileHashes.get(i.file) || '',
+      content_hash: i.content_hash,
     });
   }
 
@@ -71,6 +74,7 @@ export function buildGraphData(
       language: detectLang(e.file),
       is_test: false,
       file_hash: fileHashes.get(e.file) || '',
+      content_hash: e.content_hash,
     });
   }
 
@@ -86,6 +90,7 @@ export function buildGraphData(
       language: detectLang(t.file),
       is_test: true,
       file_hash: fileHashes.get(t.file) || '',
+      content_hash: t.content_hash,
     });
   }
 

package/src/graph/types.ts

@@ -19,6 +19,7 @@ export interface GraphNode {
   modifiers?: string;
   is_test: boolean;
   file_hash: string;
+  content_hash?: string;
 }
 
 // ── Graph edge (matches ast_edges table) ──
@@ -185,6 +186,7 @@ export interface RawFunction {
   kind: 'Function' | 'Method' | 'Constructor';
   className: string;
   qualified: string;
+  content_hash?: string;
 }
 
 export interface RawClass {
@@ -195,6 +197,7 @@ export interface RawClass {
   extends: string;
   implements: string;
   qualified: string;
+  content_hash?: string;
 }
 
 export interface RawInterface {
@@ -204,6 +207,7 @@ export interface RawInterface {
   line_end: number;
   methods: string[];
   qualified: string;
+  content_hash?: string;
 }
 
 export interface RawEnum {
@@ -212,6 +216,7 @@ export interface RawEnum {
   line_start: number;
   line_end: number;
   qualified: string;
+  content_hash?: string;
 }
 
 export interface RawTest {
@@ -220,6 +225,7 @@ export interface RawTest {
   line_start: number;
   line_end: number;
   qualified: string;
+  content_hash?: string;
 }
 
 export interface RawImport {

package/src/parser/extractors/generic.ts

@@ -1,5 +1,6 @@
 import type { SgNode, SgRoot } from '@ast-grep/napi';
 import type { RawCallSite, RawGraph } from '../../graph/types';
+import { computeContentHash } from '../../shared/file-hash';
 import { NOISE } from '../../shared/filters';
 import { log } from '../../shared/logger';
 import { LANG_KINDS } from '../languages';
@@ -24,6 +25,7 @@ export function extractGeneric(root: SgRoot, fp: string, lang: string, seen: Set
           extends: '',
           implements: '',
           qualified: `${fp}::${name}`,
+          content_hash: computeContentHash(node.text()),
         });
       }
     } catch (err) {
@@ -58,6 +60,7 @@ export function extractGeneric(root: SgRoot, fp: string, lang: string, seen: Set
           kind: className ? 'Method' : 'Function',
           className,
           qualified: className ? `${fp}::${className}.${name}` : `${fp}::${name}`,
+          content_hash: computeContentHash(node.text()),
         });
       }
     } catch (err) {

package/src/parser/extractors/python.ts

@@ -1,5 +1,6 @@
 import type { SgNode, SgRoot } from '@ast-grep/napi';
 import type { RawCallSite, RawGraph } from '../../graph/types';
+import { computeContentHash } from '../../shared/file-hash';
 import { NOISE } from '../../shared/filters';
 import { LANG_KINDS } from '../languages';
 
@@ -28,6 +29,7 @@ export function extractPython(root: SgRoot, fp: string, seen: Set<string>, graph
       extends: extendsName,
       implements: '',
       qualified: `${fp}::${name}`,
+      content_hash: computeContentHash(node.text()),
     });
   }
 
@@ -55,6 +57,7 @@ export function extractPython(root: SgRoot, fp: string, seen: Set<string>, graph
         line_start: line,
         line_end: node.range().end.line,
         qualified: `${fp}::test:${name}`,
+        content_hash: computeContentHash(node.text()),
       });
     }
 
@@ -68,6 +71,7 @@ export function extractPython(root: SgRoot, fp: string, seen: Set<string>, graph
       kind: name === '__init__' ? 'Constructor' : className ? 'Method' : 'Function',
       className,
       qualified: className ? `${fp}::${className}.${name}` : `${fp}::${name}`,
+      content_hash: computeContentHash(node.text()),
     });
   }
 

package/src/parser/extractors/ruby.ts

@@ -1,5 +1,6 @@
 import type { SgNode, SgRoot } from '@ast-grep/napi';
 import type { RawCallSite, RawGraph } from '../../graph/types';
+import { computeContentHash } from '../../shared/file-hash';
 import { NOISE } from '../../shared/filters';
 import { log } from '../../shared/logger';
 import { LANG_KINDS } from '../languages';
@@ -23,6 +24,7 @@ export function extractRuby(root: SgRoot, fp: string, seen: Set<string>, graph:
       extends: superclass,
       implements: '',
       qualified: `${fp}::${name}`,
+      content_hash: computeContentHash(node.text()),
     });
   }
 
@@ -39,6 +41,7 @@ export function extractRuby(root: SgRoot, fp: string, seen: Set<string>, graph:
       extends: '',
       implements: '',
       qualified: `${fp}::${name}`,
+      content_hash: computeContentHash(node.text()),
     });
   }
 
@@ -63,6 +66,7 @@ export function extractRuby(root: SgRoot, fp: string, seen: Set<string>, graph:
       kind: className ? 'Method' : 'Function',
       className,
       qualified: className ? `${fp}::${className}.${name}` : `${fp}::${name}`,
+      content_hash: computeContentHash(node.text()),
     });
   }
 
@@ -88,6 +92,7 @@ export function extractRuby(root: SgRoot, fp: string, seen: Set<string>, graph:
           line_start: m.range().start.line,
           line_end: m.range().end.line,
           qualified: `${fp}::test:${name}`,
+          content_hash: computeContentHash(m.text()),
         });
       }
     } catch (err) {

package/src/parser/extractors/typescript.ts

@@ -1,6 +1,7 @@
 import type { SgNode, SgRoot } from '@ast-grep/napi';
 import { Lang } from '@ast-grep/napi';
 import type { RawCallSite, RawGraph } from '../../graph/types';
+import { computeContentHash } from '../../shared/file-hash';
 import { NOISE } from '../../shared/filters';
 import { LANG_KINDS } from '../languages';
 
@@ -52,6 +53,7 @@ export function extractTypeScript(
         extends: extendsName,
         implements: implementsName,
         qualified: `${fp}::${name}`,
+        content_hash: computeContentHash(node.text()),
       });
     }
   }
@@ -112,6 +114,7 @@ export function extractTypeScript(
         kind: 'Constructor',
         className,
         qualified: `${fp}::${className}.constructor`,
+        content_hash: computeContentHash(node.text()),
       });
     } else {
       graph.functions.push({
@@ -124,6 +127,7 @@ export function extractTypeScript(
         kind: className ? 'Method' : 'Function',
         className,
         qualified: className ? `${fp}::${className}.${name}` : `${fp}::${name}`,
+        content_hash: computeContentHash(node.text()),
       });
     }
   }
@@ -148,6 +152,7 @@ export function extractTypeScript(
       kind: 'Function',
       className: '',
       qualified: `${fp}::${name}`,
+      content_hash: computeContentHash(node.text()),
     });
   }
 
@@ -172,6 +177,7 @@ export function extractTypeScript(
       kind: 'Function',
       className: '',
       qualified: `${fp}::${name}`,
+      content_hash: computeContentHash(node.text()),
     });
   }
 
@@ -198,6 +204,7 @@ export function extractTypeScript(
         line_end: node.range().end.line,
         methods,
         qualified: `${fp}::${name}`,
+        content_hash: computeContentHash(node.text()),
       });
     }
 
@@ -213,6 +220,7 @@ export function extractTypeScript(
         line_start: node.range().start.line,
         line_end: node.range().end.line,
         qualified: `${fp}::${name}`,
+        content_hash: computeContentHash(node.text()),
       });
     }
 
@@ -288,6 +296,7 @@ export function extractTypeScript(
         line_start: m.range().start.line,
         line_end: m.range().end.line,
         qualified: `${fp}::test:${name}`,
+        content_hash: computeContentHash(m.text()),
       });
     }
   }

package/src/shared/file-hash.ts

@@ -5,3 +5,8 @@ export function computeFileHash(filePath: string): string {
   const content = readFileSync(filePath);
   return createHash('sha256').update(content).digest('hex');
 }
+
+/** Hash a node's source text (function body, class body, etc.) */
+export function computeContentHash(sourceText: string): string {
+  return createHash('sha256').update(sourceText).digest('hex');
+}

package/src/shared/schemas.ts

@@ -10,6 +10,7 @@ const GraphNodeSchema = z.object({
   language: z.string(),
   is_test: z.boolean(),
   file_hash: z.string(),
+  content_hash: z.string().optional(),
   parent_name: z.string().optional(),
   params: z.string().optional(),
   return_type: z.string().optional(),