gitnexus 1.4.5 → 1.4.7

package/dist/cli/eval-server.js

@@ -24,6 +24,7 @@
  *   POST /shutdown     — Graceful shutdown.
  */
 import http from 'http';
+import { writeSync } from 'node:fs';
 import { LocalBackend } from '../mcp/local/local-backend.js';
 // ─── Text Formatters ──────────────────────────────────────────────────
 // Convert structured JSON results into compact, LLM-friendly text.
@@ -121,8 +122,10 @@ export function formatContextResult(result) {
     return lines.join('\n').trim();
 }
 export function formatImpactResult(result) {
-    if (result.error)
-        return `Error: ${result.error}`;
+    if (result.error) {
+        const suggestion = result.suggestion ? `\nSuggestion: ${result.suggestion}` : '';
+        return `Error: ${result.error}${suggestion}`;
+    }
     const target = result.target;
     const direction = result.direction;
     const byDepth = result.byDepth || {};
@@ -132,7 +135,11 @@ export function formatImpactResult(result) {
     }
     const lines = [];
     const dirLabel = direction === 'upstream' ? 'depends on this (will break if changed)' : 'this depends on';
-    lines.push(`Blast radius for ${target?.kind || ''} ${target?.name} (${direction}): ${total} symbol(s) ${dirLabel}\n`);
+    lines.push(`Blast radius for ${target?.kind || ''} ${target?.name} (${direction}): ${total} symbol(s) ${dirLabel}`);
+    if (result.partial) {
+        lines.push('⚠️  Partial results — graph traversal was interrupted. Deeper impacts may exist.');
+    }
+    lines.push('');
     const depthLabels = {
         1: 'WILL BREAK (direct)',
         2: 'LIKELY AFFECTED (indirect)',
@@ -346,10 +353,11 @@ export async function evalServerCommand(options) {
             console.error(`  Auto-shutdown after ${idleTimeoutSec}s idle`);
         }
         try {
-            process.stdout.write(`GITNEXUS_EVAL_SERVER_READY:${port}\n`);
+            // Use fd 1 directly — LadybugDB captures process.stdout (#324)
+            writeSync(1, `GITNEXUS_EVAL_SERVER_READY:${port}\n`);
         }
         catch {
-            // stdout may not be available
+            // stdout may not be available (e.g., broken pipe)
         }
     });
     resetIdleTimer();

package/dist/cli/tool.d.ts

@@ -10,8 +10,9 @@
  *   gitnexus impact --target "AuthService" --direction upstream
  *   gitnexus cypher "MATCH (n:Function) RETURN n.name LIMIT 10"
  *
- * Note: Output goes to stderr because LadybugDB's native module captures stdout
- * at the OS level during init. This is consistent with augment.ts.
+ * Note: Output goes to stdout via fs.writeSync(fd 1), bypassing LadybugDB's
+ * native module which captures the Node.js process.stdout stream during init.
+ * See the output() function for details (#324).
  */
 export declare function queryCommand(queryText: string, options?: {
     repo?: string;

package/dist/cli/tool.js

@@ -10,9 +10,11 @@
  *   gitnexus impact --target "AuthService" --direction upstream
  *   gitnexus cypher "MATCH (n:Function) RETURN n.name LIMIT 10"
  *
- * Note: Output goes to stderr because LadybugDB's native module captures stdout
- * at the OS level during init. This is consistent with augment.ts.
+ * Note: Output goes to stdout via fs.writeSync(fd 1), bypassing LadybugDB's
+ * native module which captures the Node.js process.stdout stream during init.
+ * See the output() function for details (#324).
  */
+import { writeSync } from 'node:fs';
 import { LocalBackend } from '../mcp/local/local-backend.js';
 let _backend = null;
 async function getBackend() {
@@ -26,10 +28,30 @@ async function getBackend() {
     }
     return _backend;
 }
+/**
+ * Write tool output to stdout using low-level fd write.
+ *
+ * LadybugDB's native module captures Node.js process.stdout during init,
+ * but the underlying OS file descriptor 1 (stdout) remains intact.
+ * By using fs.writeSync(1, ...) we bypass the Node.js stream layer
+ * and write directly to the real stdout fd (#324).
+ *
+ * Falls back to stderr if the fd write fails (e.g., broken pipe).
+ */
 function output(data) {
     const text = typeof data === 'string' ? data : JSON.stringify(data, null, 2);
-    // stderr because LadybugDB captures stdout at OS level
-    process.stderr.write(text + '\n');
+    try {
+        writeSync(1, text + '\n');
+    }
+    catch (err) {
+        if (err?.code === 'EPIPE') {
+            // Consumer closed the pipe (e.g., `gitnexus cypher ... | head -1`)
+            // Exit cleanly per Unix convention
+            process.exit(0);
+        }
+        // Fallback: stderr (previous behavior, works on all platforms)
+        process.stderr.write(text + '\n');
+    }
 }
 export async function queryCommand(queryText, options) {
     if (!queryText?.trim()) {
@@ -67,15 +89,28 @@ export async function impactCommand(target, options) {
         console.error('Usage: gitnexus impact <symbol_name> [--direction upstream|downstream]');
         process.exit(1);
     }
-    const backend = await getBackend();
-    const result = await backend.callTool('impact', {
-        target,
-        direction: options?.direction || 'upstream',
-        maxDepth: options?.depth ? parseInt(options.depth) : undefined,
-        includeTests: options?.includeTests ?? false,
-        repo: options?.repo,
-    });
-    output(result);
+    try {
+        const backend = await getBackend();
+        const result = await backend.callTool('impact', {
+            target,
+            direction: options?.direction || 'upstream',
+            maxDepth: options?.depth ? parseInt(options.depth, 10) : undefined,
+            includeTests: options?.includeTests ?? false,
+            repo: options?.repo,
+        });
+        output(result);
+    }
+    catch (err) {
+        // Belt-and-suspenders: catch infrastructure failures (getBackend, callTool transport)
+        // The backend's impact() already returns structured errors for graph query failures
+        output({
+            error: (err instanceof Error ? err.message : String(err)) || 'Impact analysis failed unexpectedly',
+            target: { name: target },
+            direction: options?.direction || 'upstream',
+            suggestion: 'Try reducing --depth or using gitnexus context <symbol> as a fallback',
+        });
+        process.exit(1);
+    }
 }
 export async function cypherCommand(query, options) {
     if (!query?.trim()) {

package/dist/core/graph/types.d.ts

@@ -25,7 +25,7 @@ export type NodeProperties = {
     parameterCount?: number;
     returnType?: string;
 };
-export type RelationshipType = 'CONTAINS' | 'CALLS' | 'INHERITS' | 'OVERRIDES' | 'IMPORTS' | 'USES' | 'DEFINES' | 'DECORATES' | 'IMPLEMENTS' | 'EXTENDS' | 'HAS_METHOD' | 'MEMBER_OF' | 'STEP_IN_PROCESS';
+export type RelationshipType = 'CONTAINS' | 'CALLS' | 'INHERITS' | 'OVERRIDES' | 'IMPORTS' | 'USES' | 'DEFINES' | 'DECORATES' | 'IMPLEMENTS' | 'EXTENDS' | 'HAS_METHOD' | 'HAS_PROPERTY' | 'ACCESSES' | 'MEMBER_OF' | 'STEP_IN_PROCESS';
 export interface GraphNode {
     id: string;
     label: NodeLabel;
@@ -38,7 +38,7 @@ export interface GraphRelationship {
     type: RelationshipType;
     /** Confidence score 0-1 (1.0 = certain, lower = uncertain resolution) */
     confidence: number;
-    /** Resolution reason: 'import-resolved', 'same-file', 'fuzzy-global', or empty for non-CALLS */
+    /** Semantics are edge-type-dependent: CALLS uses resolution tier, ACCESSES uses 'read'/'write', OVERRIDES uses MRO reason */
     reason: string;
     /** Step number for STEP_IN_PROCESS relationships (1-indexed) */
     step?: number;

package/dist/core/ingestion/call-processor.d.ts

@@ -1,17 +1,22 @@
 import { KnowledgeGraph } from '../graph/types.js';
 import { ASTCache } from './ast-cache.js';
 import type { ResolutionContext } from './resolution-context.js';
-import type { ExtractedCall, ExtractedHeritage, ExtractedRoute, FileConstructorBindings } from './workers/parse-worker.js';
+import type { ExtractedCall, ExtractedAssignment, ExtractedHeritage, ExtractedRoute, FileConstructorBindings } from './workers/parse-worker.js';
 export declare const processCalls: (graph: KnowledgeGraph, files: {
     path: string;
     content: string;
 }[], astCache: ASTCache, ctx: ResolutionContext, onProgress?: (current: number, total: number) => void) => Promise<ExtractedHeritage[]>;
-export declare const extractReturnTypeName: (raw: string, depth?: number) => string | undefined;
 /**
  * Fast path: resolve pre-extracted call sites from workers.
  * No AST parsing — workers already extracted calledName + sourceId.
  */
 export declare const processCallsFromExtracted: (graph: KnowledgeGraph, extractedCalls: ExtractedCall[], ctx: ResolutionContext, onProgress?: (current: number, total: number) => void, constructorBindings?: FileConstructorBindings[]) => Promise<void>;
+/**
+ * Resolve pre-extracted field write assignments to ACCESSES {reason: 'write'} edges.
+ * Accepts optional constructorBindings for return-type-aware receiver inference,
+ * mirroring processCallsFromExtracted's verified binding lookup.
+ */
+export declare const processAssignmentsFromExtracted: (graph: KnowledgeGraph, assignments: ExtractedAssignment[], ctx: ResolutionContext, constructorBindings?: FileConstructorBindings[]) => void;
 /**
  * Resolve pre-extracted Laravel routes to CALLS edges from route files to controller methods.
  */