@link-assistant/hive-mind 2.0.14 → 2.0.15

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @link-assistant/hive-mind
 
+## 2.0.15
+
+### Patch Changes
+
+- 0d2f2bb: Fix "Cannot read properties of null (reading 'type')" crash that aborted Codex (and other agent) runs when the tool echoed a stream line that parsed to a bare `null` or non-object JSON primitive. All NDJSON stream parsers (Codex, Claude, Agent, OpenCode) now ignore non-object lines instead of dereferencing them.
+
 ## 2.0.14
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/hive-mind",
-  "version": "2.0.14",
+  "version": "2.0.15",
   "description": "AI-powered issue solver and hive mind for collaborative problem solving",
   "main": "src/hive.mjs",
   "type": "module",

package/src/agent.lib.mjs

@@ -542,6 +542,9 @@ export const executeAgentCommand = async params => {
             if (!line.trim()) continue;
             try {
               const data = sanitizeObjectStrings(JSON.parse(line));
+              // Issue #1968: a bare `null`/primitive NDJSON line must not abort
+              // event processing (any data.X access would throw on null).
+              if (data === null || typeof data !== 'object') continue;
               // Output formatted JSON
               await log(JSON.stringify(data, null, 2));
               // Capture session ID from the first message
@@ -616,6 +619,8 @@ export const executeAgentCommand = async params => {
               if (!stderrLine.trim()) continue;
               try {
                 const stderrData = sanitizeObjectStrings(JSON.parse(stderrLine));
+                // Issue #1968: skip bare `null`/primitive lines (see stdout handler above).
+                if (stderrData === null || typeof stderrData !== 'object') continue;
                 // Output formatted JSON (same formatting as stdout)
                 await log(JSON.stringify(stderrData, null, 2));
                 // Capture session ID from stderr too (agent sends it via stderr)
@@ -695,6 +700,9 @@ export const executeAgentCommand = async params => {
           try {
             const msg = sanitizeObjectStrings(JSON.parse(line));
 
+            // Issue #1968: ignore bare `null`/primitive lines (msg.type would throw on null).
+            if (msg === null || typeof msg !== 'object') continue;
+
             // Check for explicit error message types from agent
             if (msg.type === 'error' || msg.type === 'step_error') {
               return { detected: true, type: 'AgentError', match: msg.message || msg.error || line.substring(0, 100) };

package/src/claude.lib.mjs

@@ -872,6 +872,7 @@ export const executeClaudeCommand = async params => {
             if (!line.trim()) continue;
             try {
               const data = sanitizeObjectStrings(JSON.parse(line));
+              if (data === null || typeof data !== 'object') continue; // Issue #1968: skip bare null/primitive NDJSON lines
               // Issue #1510: Track last event time for all modes (not just interactive)
               // so activity timeout can report accurate idle duration
               lastEventTime = Date.now();
@@ -1110,11 +1111,10 @@ export const executeClaudeCommand = async params => {
         try {
           const data = sanitizeObjectStrings(JSON.parse(stdoutLineBuffer));
           await log(JSON.stringify(data, null, 2));
-          if (data.type === 'result' && data.subtype === 'success' && data.total_cost_usd != null) {
+          if (data?.type === 'result' && data.subtype === 'success' && data.total_cost_usd != null) {
             anthropicTotalCostUSD = data.total_cost_usd;
-          } else if (data.type === 'result' && data.total_cost_usd != null) {
-            // Issue #1886: keep a non-success terminal result's cost as a fallback
-            // for accumulation (see the streaming branch above).
+          } else if (data?.type === 'result' && data.total_cost_usd != null) {
+            // Issue #1886: keep a non-success terminal result's cost as a fallback (see streaming branch above).
             anthropicCostFromAnyResult = data.total_cost_usd;
           }
           // Issue #1472: Forward remaining buffer event to interactive handler (was previously missed)

package/src/codex.lib.mjs

@@ -470,6 +470,16 @@ export const parseCodexExecJsonOutput = (output, state = {}, requestedModelId =
       continue;
     }
 
+    // Issue #1968: a stream line that parses to a bare `null` (or any non-object
+    // JSON primitive such as a number/string/boolean) must not crash the parser.
+    // Codex echoes the stdout of every command it runs back into its own NDJSON
+    // stream (see issue #1955), so a target repo that prints a standalone `null`
+    // line surfaces here as `JSON.parse('null') === null`. Accessing `data.type`
+    // on that null threw "Cannot read properties of null (reading 'type')" and
+    // aborted the entire solve. Real Codex events are always JSON objects, so any
+    // non-object line is safely ignored.
+    if (data === null || typeof data !== 'object') continue;
+
     const eventType = typeof data.type === 'string' ? data.type : 'unknown';
     nextState.eventCounts[eventType] = (nextState.eventCounts[eventType] || 0) + 1;
 
@@ -1044,6 +1054,9 @@ export const executeCodexCommand = async params => {
               if (!line) continue;
               try {
                 const data = sanitizeObjectStrings(JSON.parse(line));
+                // Issue #1968: skip bare `null`/primitive lines so the handlers
+                // below never receive a non-object event (see parseCodexExecJsonOutput).
+                if (data === null || typeof data !== 'object') continue;
                 if (interactiveHandler) await interactiveHandler.processEvent(data);
                 if (progressMonitor) await progressMonitor.processStreamEvent(data);
               } catch {

package/src/opencode.lib.mjs

@@ -342,6 +342,9 @@ export const executeOpenCodeCommand = async params => {
             for (const line of lines) {
               if (!line.trim()) continue;
               const data = sanitizeObjectStrings(JSON.parse(line));
+              // Issue #1968: a bare `null`/primitive NDJSON line must not abort the
+              // rest of the chunk (data.type access would throw on null).
+              if (data === null || typeof data !== 'object') continue;
               accumulateAgentStepFinishUsage(streamingTokenUsage, data);
               // Track text content for result summary
               // OpenCode outputs text via 'text', 'assistant', 'message', or 'result' type events
@@ -385,6 +388,8 @@ export const executeOpenCodeCommand = async params => {
               for (const line of lines) {
                 if (!line.trim()) continue;
                 const data = sanitizeObjectStrings(JSON.parse(line));
+                // Issue #1968: skip bare `null`/primitive lines (see stdout handler above).
+                if (data === null || typeof data !== 'object') continue;
                 accumulateAgentStepFinishUsage(streamingTokenUsage, data);
                 if (data.type === 'text' && data.text) {
                   lastTextContent = data.text;