@defai.digital/automatosx 6.2.7 → 6.2.9

package/CHANGELOG.md

@@ -2,6 +2,81 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+## [6.2.9] - 2025-10-31
+
+### 🔧 Fixes
+
+**Timeline Visualization Division by Zero (Bug #36)**
+
+Through systematic heavy-thinking analysis of division operations in agent execution code:
+
+- **Bug #36 (MEDIUM)**: Division by zero in timeline visualization when all agents complete in same millisecond
+  - **File affected**: `src/agents/parallel-agent-executor.ts:616`
+  - **Problem**: `totalDuration = maxEndTime - minStartTime` can be **0** if all agents complete instantly (same ms)
+  - **Impact**: RangeError crash - `entry.duration / 0` = `Infinity`, then `'█'.repeat(Infinity)` throws RangeError
+  - **Root Cause**: No guard against zero duration when calculating progress bar lengths
+  - **Fix**: Added `safeTotalDuration = Math.max(1, totalDuration)` to ensure minimum 1ms duration
+  - **Scenario**: Fast-executing agents or test scenarios where all agents complete in < 1ms
+
+### 🔍 Analysis Methodology
+
+- **Heavy-thinking division analysis**: Systematically checked all division operations in `src/agents/`
+- **Math operations verified**: `Math.max()` does NOT protect against `Infinity` (only against negative/smaller values)
+- **Error chain traced**: `x/0` → `Infinity` → `Math.floor(Infinity)` → `Infinity` → `repeat(Infinity)` → RangeError
+- **Fixed vulnerable operation**: parallel-agent-executor.ts timeline bar length calculation
+
+### 📊 Impact
+
+- **Users affected**: Users running parallel agents that complete very quickly (< 1ms)
+- **Severity**: Medium (rare but causes complete crash when triggered)
+- **Breaking changes**: None
+- **Migration**: None required - automatic fix
+
+### ✅ Testing
+
+- All 2,281 unit tests passing
+- TypeScript compilation successful
+- Build successful
+
+## [6.2.8] - 2025-10-31
+
+### 🔧 Fixes
+
+**Provider Error Handler Timer Race Condition (Bug #35)**
+
+Through systematic heavy-thinking analysis of provider timer cleanup patterns:
+
+- **Bug #35 (MEDIUM)**: Missing cleanup() call in child process error handlers across all CLI providers
+  - **Files affected**:
+    - `src/providers/openai-provider.ts` (2 locations: lines 360, 760)
+    - `src/providers/claude-provider.ts` (line 445)
+    - `src/providers/gemini-provider.ts` (line 399)
+  - **Problem**: Error handlers call `cleanupAbortListener()` but NOT `cleanup()`, leaving timers running
+  - **Impact**: Race condition - if child.on('error') fires, mainTimeout continues and can create orphaned nestedKillTimeout
+  - **Root Cause**: Inconsistent cleanup pattern - close/timeout/abort handlers all call cleanup(), but error handler doesn't
+  - **Fix**: Added `cleanup()` call before `cleanupAbortListener()` in all error handlers (matching close handler pattern)
+  - **Scenario**: Between error event and close event, timers fire and create nested timers that aren't cleaned until close
+
+### 🔍 Analysis Methodology
+
+- **Heavy-thinking provider analysis**: Systematically checked all child.on('error') handlers in CLI providers
+- **Pattern comparison**: Verified cleanup() is called in close, timeout, and abort handlers
+- **Verified vulnerable operations**: All 3 CLI providers (OpenAI, Claude, Gemini) had same bug
+- **Fixed**: 4 total error handler locations across 3 provider files
+
+### 📊 Impact
+
+- **Users affected**: Users experiencing provider CLI process errors during execution
+- **Severity**: Medium (timers eventually cleaned in close handler, but race condition window exists)
+- **Breaking changes**: None
+- **Migration**: None required - automatic cleanup improvement
+
+### ✅ Testing
+
+- All 2,281 unit tests passing
+- TypeScript compilation successful
+- Build successful
+
 ## [6.2.7] - 2025-10-31
 
 ### 🔧 Fixes

package/dist/index.js

@@ -4363,6 +4363,7 @@ Details: ${errorMsg}`
             }
           });
           child.on("error", (error) => {
+            cleanup();
             cleanupAbortListener();
             if (hasTimedOut) {
               return;
@@ -5018,6 +5019,7 @@ This is a placeholder response. Set AUTOMATOSX_MOCK_PROVIDERS=false to use real
             }
           });
           child.on("error", (error) => {
+            cleanup();
             cleanupAbortListener();
             if (!hasTimedOut) {
               reject(ProviderError.executionError(this.config.name, error));
@@ -5510,6 +5512,7 @@ This is a placeholder response. Set AUTOMATOSX_MOCK_PROVIDERS=false to use real
             }
           });
           child.on("error", (error) => {
+            cleanup();
             cleanupAbortListener();
             if (!hasTimedOut) {
               reject(new Error(`Failed to spawn OpenAI CLI: ${error.message}`));
@@ -5779,6 +5782,7 @@ This is a placeholder streaming response.`;
             }
           });
           child.on("error", (error) => {
+            cleanup();
             cleanupAbortListener();
             if (!hasTimedOut) {
               reject(new Error(`Failed to spawn OpenAI CLI: ${error.message}`));
@@ -19611,6 +19615,7 @@ var ParallelAgentExecutor = class {
     const minStartTime = Math.min(...timeline.map((t) => t.startTime));
     const maxEndTime = Math.max(...timeline.map((t) => t.endTime));
     const totalDuration = maxEndTime - minStartTime;
+    const safeTotalDuration = Math.max(1, totalDuration);
     for (let level = 0; level <= maxLevel; level++) {
       const entriesAtLevel = timeline.filter((t) => t.level === level);
       if (entriesAtLevel.length === 0) {
@@ -19619,7 +19624,7 @@ var ParallelAgentExecutor = class {
       output += chalk29.gray(`Level ${level}:
 `);
       for (const entry of entriesAtLevel) {
-        const barLength = Math.max(1, Math.floor(entry.duration / totalDuration * 40));
+        const barLength = Math.max(1, Math.floor(entry.duration / safeTotalDuration * 40));
         const bar = "\u2588".repeat(barLength);
         let statusColor = chalk29.green;
         let statusIcon = "\u2713";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defai.digital/automatosx",
-  "version": "6.2.7",
+  "version": "6.2.9",
   "description": "AI Agent Orchestration Platform",
   "type": "module",
   "publishConfig": {