fixed explorer

Author: Omri Hopson

dashboard/src/pages/Explorer/Explorer.tsx

@@ -67,6 +67,7 @@ export default function Explorer() {
   const [refreshTrigger, setRefreshTrigger] = useState(0);
   const [rawData, setRawData] = useState<GraphData | null>(null);
   const [highlightedNode, setHighlightedNode] = useState<string | null>(null);
+  const [nodeError, setNodeError] = useState<string | null>(null);
 
   useEffect(() => {
     fetch('/api/repositories')
@@ -96,14 +97,20 @@ export default function Explorer() {
 
   const loadNodeDetails = useCallback(async (nodeId: string) => {
     if (!selectedRepo) return;
+    setNodeError(null);
     try {
       const response = await fetch(`/api/graph/${selectedRepo}/node/${nodeId}`);
-      if (!response.ok) return;
+      if (!response.ok) {
+        const errorData = await response.json().catch(() => null);
+        setNodeError(errorData?.error || `Failed to load node (${response.status})`);
+        return;
+      }
       const data: NodeDetails = await response.json();
       setSelectedNode(data);
       setHighlightedNode(nodeId);
     } catch (error) {
       console.error('Failed to load node details:', error);
+      setNodeError('Network error loading node details');
     }
   }, [selectedRepo]);
 
@@ -502,7 +509,18 @@ export default function Explorer() {
           )}
 
           {/* Details Inspector (both modes) */}
-          <div className="absolute right-6 top-6 bottom-6 w-80 glass-panel rounded-xl border border-outline-variant/20 p-6 flex flex-col gap-6 shadow-2xl details-panel overflow-y-auto">
+          <div className="absolute right-6 top-6 bottom-6 w-80 glass-panel rounded-xl border border-outline-variant/20 p-6 flex flex-col gap-6 shadow-2xl details-panel overflow-y-auto z-20">
+            {nodeError && (
+              <div className="px-3 py-2 bg-error/10 border border-error/20 rounded-lg">
+                <p className="text-sm text-error font-medium">{nodeError}</p>
+                <button
+                  onClick={() => setNodeError(null)}
+                  className="text-xs text-error/70 mt-1 underline"
+                >
+                  Dismiss
+                </button>
+              </div>
+            )}
             {selectedNode ? (
               <>
                 <div>

src/core/indexer.ts

@@ -62,6 +62,13 @@ export interface IndexerEvents {
 
 export class Indexer extends EventEmitter {
   private contextIgnore: ContextIgnore;
+  private pendingReferences: Array<{
+    calls: Array<{ callerNodeId: string; calleeName: string }>;
+    imports: import('./parser.js').ParsedImport[];
+    inheritance: import('./parser.js').ParsedInheritance[];
+    repositoryId: string;
+    filePath: string;
+  }> = [];
 
   constructor(
     public storage: StorageProvider,
@@ -116,6 +123,8 @@ export class Indexer extends EventEmitter {
       const files = await this.discoverFiles(absolutePath, options.respectIgnore);
       job.filesTotal = files.length;
 
+      this.pendingReferences = [];
+
       const BATCH_SIZE = 50;
       for (let i = 0; i < files.length; i += BATCH_SIZE) {
         const batch = files.slice(i, i + BATCH_SIZE);
@@ -134,6 +143,12 @@ export class Indexer extends EventEmitter {
         }
       }
 
+      const forwardEdges = this.resolveForwardEdges(repositoryId);
+      if (forwardEdges > 0) {
+        console.log(`Resolved ${forwardEdges} forward-reference edges`);
+      }
+      this.pendingReferences = [];
+
       const stats = this.graph.getStats();
       job.nodesCreated = stats.nodeCount;
       job.edgesCreated = stats.edgeCount;
@@ -217,6 +232,14 @@ export class Indexer extends EventEmitter {
       }
       this.storage.upsertEdges(edges);
 
+      this.pendingReferences.push({
+        calls: parsed.calls,
+        imports: parsed.imports,
+        inheritance: parsed.inheritance,
+        repositoryId,
+        filePath: parsed.filePath,
+      });
+
       fileMetadata.hash = parsed.hash;
       fileMetadata.language = parsed.language;
       fileMetadata.nodeCount = parsed.nodes.length;
@@ -278,6 +301,110 @@ export class Indexer extends EventEmitter {
     }
   }
 
+  private resolveForwardEdges(repositoryId: string): number {
+    let created = 0;
+    const now = new Date();
+
+    for (const ref of this.pendingReferences) {
+      if (ref.repositoryId !== repositoryId) continue;
+
+      for (const call of ref.calls) {
+        const targets = this.graph.findByName(call.calleeName);
+        for (const target of targets) {
+          if (target.repositoryId !== repositoryId) continue;
+          const edgeId = this.generateEdgeId(call.callerNodeId, target.id, 'calls');
+          try {
+            if (!this.graph.getNode(call.callerNodeId)) continue;
+            this.graph.addEdge({
+              id: edgeId,
+              sourceId: call.callerNodeId,
+              targetId: target.id,
+              kind: 'calls',
+              confidence: 0.9,
+              repositoryId,
+              createdAt: now,
+              updatedAt: now,
+            });
+            created++;
+          } catch {
+            // Already exists or source/target missing
+          }
+        }
+      }
+
+      for (const inh of ref.inheritance) {
+        const parents = this.graph.findByName(inh.parentName);
+        for (const parent of parents) {
+          if (parent.repositoryId !== repositoryId) continue;
+          const edgeId = this.generateEdgeId(inh.childNodeId, parent.id, inh.kind);
+          try {
+            if (!this.graph.getNode(inh.childNodeId)) continue;
+            this.graph.addEdge({
+              id: edgeId,
+              sourceId: inh.childNodeId,
+              targetId: parent.id,
+              kind: inh.kind,
+              confidence: 1.0,
+              repositoryId,
+              createdAt: now,
+              updatedAt: now,
+            });
+            created++;
+          } catch {
+            // Already exists or source/target missing
+          }
+        }
+      }
+
+      for (const imp of ref.imports) {
+        if (!imp.source || !imp.source.startsWith('.')) continue;
+        const importDir = dirname(ref.filePath);
+        const resolvedBase = join(importDir, imp.source).replace(/\\/g, '/');
+        const extensions = ['', '.ts', '.tsx', '.js', '.jsx', '/index.ts', '/index.tsx', '/index.js'];
+
+        for (const importedName of imp.imported) {
+          const targets = this.graph.findByName(importedName);
+          for (const target of targets) {
+            if (target.repositoryId !== repositoryId) continue;
+            if (target.filePath === ref.filePath) continue;
+
+            const targetFileBase = target.filePath.replace(/\.(ts|tsx|js|jsx)$/, '').replace(/\/index$/, '');
+            const matchesPath = extensions.some(ext => {
+              const candidate = resolvedBase + ext;
+              return target.filePath === candidate || targetFileBase === resolvedBase;
+            });
+            if (!matchesPath) continue;
+
+            const nodesInFile = this.graph.getNodesInFile(ref.filePath);
+            const sourceNode = nodesInFile.find(n =>
+              n.kind === 'function' || n.kind === 'class' || n.kind === 'method'
+            ) || nodesInFile[0];
+            if (!sourceNode) continue;
+
+            const edgeId = this.generateEdgeId(sourceNode.id, target.id, 'imports');
+            try {
+              this.graph.addEdge({
+                id: edgeId,
+                sourceId: sourceNode.id,
+                targetId: target.id,
+                kind: 'imports',
+                confidence: 1.0,
+                repositoryId,
+                createdAt: now,
+                updatedAt: now,
+              });
+              created++;
+            } catch {
+              // Already exists
+            }
+          }
+        }
+      }
+    }
+
+    return created;
+  }
+
   private static readonly SKIP_DIRS = new Set([
     'node_modules', '.git', '.svn', '.hg', 'vendor', '__pycache__',
     '.venv', 'venv', 'dist', 'build', 'out', 'target', '.next',