@howlil/ez-agents 3.1.0 → 3.4.2

package/ez-agents/bin/lib/circuit-breaker.cjs

@@ -1,118 +1,118 @@
-#!/usr/bin/env node
-
-/**
- * GSD Circuit Breaker — Prevent cascading failures
- * 
- * Implements circuit breaker pattern:
- * - CLOSED: Normal operation
- * - OPEN: Failing, reject requests
- * - HALF_OPEN: Testing if service recovered
- * 
- * Usage:
- *   const CircuitBreaker = require('./circuit-breaker.cjs');
- *   const breaker = new CircuitBreaker({ failureThreshold: 5 });
- *   const result = await breaker.execute(() => riskyOperation());
- */
-
-const Logger = require('./logger.cjs');
-const logger = new Logger();
-
-class CircuitBreaker {
-  /**
-   * Create circuit breaker
-   * @param {Object} options - Configuration
-   */
-  constructor(options = {}) {
-    this.failureThreshold = options.failureThreshold || 5;
-    this.resetTimeout = options.resetTimeout || 60000;
-    this.state = 'CLOSED';
-    this.failures = 0;
-    this.lastFailureTime = null;
-    this.successes = 0;
-  }
-
-  /**
-   * Execute operation with circuit breaker protection
-   * @param {Function} operation - Async function to execute
-   * @returns {Promise<any>} - Result of operation
-   */
-  async execute(operation) {
-    // Check if circuit is OPEN
-    if (this.state === 'OPEN') {
-      const timeSinceFailure = Date.now() - this.lastFailureTime;
-      
-      if (timeSinceFailure > this.resetTimeout) {
-        // Try to recover
-        this.state = 'HALF_OPEN';
-        this.failures = 0;
-        logger.info('Circuit breaker HALF_OPEN - testing recovery');
-      } else {
-        const waitTime = Math.round((this.resetTimeout - timeSinceFailure) / 1000);
-        logger.warn('Circuit breaker OPEN - rejecting request', { waitTime });
-        throw new Error(`Circuit breaker is OPEN. Try again in ${waitTime}s`);
-      }
-    }
-
-    try {
-      const result = await operation();
-      
-      // Success - reset counters if in HALF_OPEN
-      if (this.state === 'HALF_OPEN') {
-        this.state = 'CLOSED';
-        this.failures = 0;
-        logger.info('Circuit breaker CLOSED - service recovered');
-      }
-      
-      this.successes++;
-      return result;
-    } catch (err) {
-      this.failures++;
-      this.lastFailureTime = Date.now();
-      
-      // Open circuit if threshold reached
-      if (this.failures >= this.failureThreshold) {
-        this.state = 'OPEN';
-        logger.error('Circuit breaker OPEN - failure threshold reached', { 
-          failures: this.failures 
-        });
-      }
-      
-      throw err;
-    }
-  }
-
-  /**
-   * Get current state
-   * @returns {string} - CLOSED, OPEN, or HALF_OPEN
-   */
-  getState() {
-    return this.state;
-  }
-
-  /**
-   * Get stats
-   * @returns {Object} - Statistics
-   */
-  getStats() {
-    return {
-      state: this.state,
-      failures: this.failures,
-      successes: this.successes,
-      failureThreshold: this.failureThreshold,
-      lastFailureTime: this.lastFailureTime
-    };
-  }
-
-  /**
-   * Reset circuit breaker
-   */
-  reset() {
-    this.state = 'CLOSED';
-    this.failures = 0;
-    this.successes = 0;
-    this.lastFailureTime = null;
-    logger.info('Circuit breaker reset');
-  }
-}
-
-module.exports = CircuitBreaker;
+#!/usr/bin/env node
+
+/**
+ * EZ Circuit Breaker — Prevent cascading failures
+ * 
+ * Implements circuit breaker pattern:
+ * - CLOSED: Normal operation
+ * - OPEN: Failing, reject requests
+ * - HALF_OPEN: Testing if service recovered
+ * 
+ * Usage:
+ *   const CircuitBreaker = require('./circuit-breaker.cjs');
+ *   const breaker = new CircuitBreaker({ failureThreshold: 5 });
+ *   const result = await breaker.execute(() => riskyOperation());
+ */
+
+const Logger = require('./logger.cjs');
+const logger = new Logger();
+
+class CircuitBreaker {
+  /**
+   * Create circuit breaker
+   * @param {Object} options - Configuration
+   */
+  constructor(options = {}) {
+    this.failureThreshold = options.failureThreshold || 5;
+    this.resetTimeout = options.resetTimeout || 60000;
+    this.state = 'CLOSED';
+    this.failures = 0;
+    this.lastFailureTime = null;
+    this.successes = 0;
+  }
+
+  /**
+   * Execute operation with circuit breaker protection
+   * @param {Function} operation - Async function to execute
+   * @returns {Promise<any>} - Result of operation
+   */
+  async execute(operation) {
+    // Check if circuit is OPEN
+    if (this.state === 'OPEN') {
+      const timeSinceFailure = Date.now() - this.lastFailureTime;
+      
+      if (timeSinceFailure > this.resetTimeout) {
+        // Try to recover
+        this.state = 'HALF_OPEN';
+        this.failures = 0;
+        logger.info('Circuit breaker HALF_OPEN - testing recovery');
+      } else {
+        const waitTime = Math.round((this.resetTimeout - timeSinceFailure) / 1000);
+        logger.warn('Circuit breaker OPEN - rejecting request', { waitTime });
+        throw new Error(`Circuit breaker is OPEN. Try again in ${waitTime}s`);
+      }
+    }
+
+    try {
+      const result = await operation();
+      
+      // Success - reset counters if in HALF_OPEN
+      if (this.state === 'HALF_OPEN') {
+        this.state = 'CLOSED';
+        this.failures = 0;
+        logger.info('Circuit breaker CLOSED - service recovered');
+      }
+      
+      this.successes++;
+      return result;
+    } catch (err) {
+      this.failures++;
+      this.lastFailureTime = Date.now();
+      
+      // Open circuit if threshold reached
+      if (this.failures >= this.failureThreshold) {
+        this.state = 'OPEN';
+        logger.error('Circuit breaker OPEN - failure threshold reached', { 
+          failures: this.failures 
+        });
+      }
+      
+      throw err;
+    }
+  }
+
+  /**
+   * Get current state
+   * @returns {string} - CLOSED, OPEN, or HALF_OPEN
+   */
+  getState() {
+    return this.state;
+  }
+
+  /**
+   * Get stats
+   * @returns {Object} - Statistics
+   */
+  getStats() {
+    return {
+      state: this.state,
+      failures: this.failures,
+      successes: this.successes,
+      failureThreshold: this.failureThreshold,
+      lastFailureTime: this.lastFailureTime
+    };
+  }
+
+  /**
+   * Reset circuit breaker
+   */
+  reset() {
+    this.state = 'CLOSED';
+    this.failures = 0;
+    this.successes = 0;
+    this.lastFailureTime = null;
+    logger.info('Circuit breaker reset');
+  }
+}
+
+module.exports = CircuitBreaker;

package/ez-agents/bin/lib/commands.cjs

@@ -3,9 +3,9 @@
  */
 const fs = require('fs');
 const path = require('path');
-const { execSync } = require('child_process');
 const { safeReadFile, loadConfig, isGitIgnored, execGit, normalizePhaseName, comparePhaseNum, getArchivedPhaseDirs, generateSlugInternal, getMilestoneInfo, resolveModelInternal, MODEL_PROFILES, toPosixPath, output, error, findPhaseInternal } = require('./core.cjs');
 const { extractFrontmatter } = require('./frontmatter.cjs');
+const { defaultLogger: logger } = require('./logger.cjs');
 
 function cmdGenerateSlug(text, raw) {
   if (!text) {
@@ -70,9 +70,13 @@ function cmdListTodos(cwd, area, raw) {
           area: todoArea,
           path: toPosixPath(path.join('.planning', 'todos', 'pending', file)),
         });
-      } catch {}
+      } catch (err) {
+        logger.warn('Failed to parse todo file in cmdListTodos', { file, error: err.message });
+      }
     }
-  } catch {}
+  } catch (err) {
+    logger.warn('Failed to list pending todos in cmdListTodos', { pendingDir, error: err.message });
+  }
 
   const result = { count, todos };
   output(result, raw, count.toString());
@@ -90,7 +94,8 @@ function cmdVerifyPathExists(cwd, targetPath, raw) {
     const type = stats.isDirectory() ? 'directory' : stats.isFile() ? 'file' : 'other';
     const result = { exists: true, type };
     output(result, raw, 'true');
-  } catch {
+  } catch (err) {
+    logger.warn('Path verification failed in cmdVerifyPathExists', { fullPath, error: err.message });
     const result = { exists: false, type: null };
     output(result, raw, 'false');
   }
@@ -119,7 +124,9 @@ function cmdHistoryDigest(cwd, raw) {
       for (const dir of currentDirs) {
         allPhaseDirs.push({ name: dir, fullPath: path.join(phasesDir, dir), milestone: null });
       }
-    } catch {}
+    } catch (err) {
+      logger.warn('Failed to enumerate current phase directories in cmdHistoryDigest', { phasesDir, error: err.message });
+    }
   }
 
   if (allPhaseDirs.length === 0) {
@@ -177,8 +184,8 @@ function cmdHistoryDigest(cwd, raw) {
             fm['tech-stack'].added.forEach(t => digest.tech_stack.add(typeof t === 'string' ? t : t.name));
           }
 
-        } catch (e) {
-          // Skip malformed summaries
+        } catch (err) {
+          logger.warn('Skipping malformed summary in cmdHistoryDigest', { summary, dirPath, error: err.message });
         }
       }
     }
@@ -192,8 +199,9 @@ function cmdHistoryDigest(cwd, raw) {
     digest.tech_stack = [...digest.tech_stack];
 
     output(digest, raw);
-  } catch (e) {
-    error('Failed to generate history digest: ' + e.message);
+  } catch (err) {
+    logger.error('Failed to generate history digest', { error: err.message });
+    error('Failed to generate history digest: ' + err.message);
   }
 }
 
@@ -213,7 +221,7 @@ function cmdResolveModel(cwd, agentType, raw) {
   output(result, raw, model);
 }
 
-function cmdCommit(cwd, message, files, raw, amend) {
+async function cmdCommit(cwd, message, files, raw, amend) {
   if (!message && !amend) {
     error('commit message required');
   }
@@ -228,7 +236,7 @@ function cmdCommit(cwd, message, files, raw, amend) {
   }
 
   // Check if .planning is gitignored
-  if (isGitIgnored(cwd, '.planning')) {
+  if (await isGitIgnored(cwd, '.planning')) {
     const result = { committed: false, hash: null, reason: 'skipped_gitignored' };
     output(result, raw, 'skipped');
     return;
@@ -237,12 +245,12 @@ function cmdCommit(cwd, message, files, raw, amend) {
   // Stage files
   const filesToStage = files && files.length > 0 ? files : ['.planning/'];
   for (const file of filesToStage) {
-    execGit(cwd, ['add', file]);
+    await execGit(cwd, ['add', file]);
   }
 
   // Commit
   const commitArgs = amend ? ['commit', '--amend', '--no-edit'] : ['commit', '-m', message];
-  const commitResult = execGit(cwd, commitArgs);
+  const commitResult = await execGit(cwd, commitArgs);
   if (commitResult.exitCode !== 0) {
     if (commitResult.stdout.includes('nothing to commit') || commitResult.stderr.includes('nothing to commit')) {
       const result = { committed: false, hash: null, reason: 'nothing_to_commit' };
@@ -255,7 +263,7 @@ function cmdCommit(cwd, message, files, raw, amend) {
   }
 
   // Get short hash
-  const hashResult = execGit(cwd, ['rev-parse', '--short', 'HEAD']);
+  const hashResult = await execGit(cwd, ['rev-parse', '--short', 'HEAD']);
   const hash = hashResult.exitCode === 0 ? hashResult.stdout : null;
   const result = { committed: true, hash, reason: 'committed' };
   output(result, raw, hash || 'committed');
@@ -375,6 +383,7 @@ async function cmdWebsearch(query, options, raw) {
       results
     }, raw, results.map(r => `${r.title}\n${r.url}\n${r.description}`).join('\n\n'));
   } catch (err) {
+    logger.warn('Websearch request failed in cmdWebsearch', { query, error: err.message });
     output({ available: false, error: err.message }, raw, '');
   }
 }
@@ -411,7 +420,9 @@ function cmdProgressRender(cwd, format, raw) {
 
       phases.push({ number: phaseNum, name: phaseName, plans, summaries, status });
     }
-  } catch {}
+  } catch (err) {
+    logger.warn('Failed to enumerate phase directories in cmdProgressRender', { phasesDir, error: err.message });
+  }
 
   const percent = totalPlans > 0 ? Math.min(100, Math.round((totalSummaries / totalPlans) * 100)) : 0;
 
@@ -492,7 +503,7 @@ function cmdScaffold(cwd, type, options, raw) {
   switch (type) {
     case 'context': {
       filePath = path.join(phaseDir, `${padded}-CONTEXT.md`);
-      content = `---\nphase: "${padded}"\nname: "${name || phaseInfo?.phase_name || 'Unnamed'}"\ncreated: ${today}\n---\n\n# Phase ${phase}: ${name || phaseInfo?.phase_name || 'Unnamed'} — Context\n\n## Decisions\n\n_Decisions will be captured during /gsd:discuss-phase ${phase}_\n\n## Discretion Areas\n\n_Areas where the executor can use judgment_\n\n## Deferred Ideas\n\n_Ideas to consider later_\n`;
+      content = `---\nphase: "${padded}"\nname: "${name || phaseInfo?.phase_name || 'Unnamed'}"\ncreated: ${today}\n---\n\n# Phase ${phase}: ${name || phaseInfo?.phase_name || 'Unnamed'} — Context\n\n## Decisions\n\n_Decisions will be captured during /ez-discuss-phase ${phase}_\n\n## Discretion Areas\n\n_Areas where the executor can use judgment_\n\n## Deferred Ideas\n\n_Ideas to consider later_\n`;
       break;
     }
     case 'uat': {
@@ -532,7 +543,7 @@ function cmdScaffold(cwd, type, options, raw) {
   output({ created: true, path: relPath }, raw, relPath);
 }
 
-function cmdStats(cwd, format, raw) {
+async function cmdStats(cwd, format, raw) {
   const phasesDir = path.join(cwd, '.planning', 'phases');
   const reqPath = path.join(cwd, '.planning', 'REQUIREMENTS.md');
   const statePath = path.join(cwd, '.planning', 'STATE.md');
@@ -566,7 +577,9 @@ function cmdStats(cwd, format, raw) {
 
       phases.push({ number: phaseNum, name: phaseName, plans, summaries, status });
     }
-  } catch {}
+  } catch (err) {
+    logger.warn('Failed to enumerate phase directories in cmdStats', { phasesDir, error: err.message });
+  }
 
   const percent = totalPlans > 0 ? Math.min(100, Math.round((totalSummaries / totalPlans) * 100)) : 0;
 
@@ -581,7 +594,9 @@ function cmdStats(cwd, format, raw) {
       requirementsComplete = checked ? checked.length : 0;
       requirementsTotal = requirementsComplete + (unchecked ? unchecked.length : 0);
     }
-  } catch {}
+  } catch (err) {
+    logger.warn('Failed to parse REQUIREMENTS.md in cmdStats', { reqPath, error: err.message });
+  }
 
   // Last activity from STATE.md
   let lastActivity = null;
@@ -591,17 +606,21 @@ function cmdStats(cwd, format, raw) {
       const activityMatch = stateContent.match(/\*\*Last Activity:\*\*\s*(.+)/);
       if (activityMatch) lastActivity = activityMatch[1].trim();
     }
-  } catch {}
+  } catch (err) {
+    logger.warn('Failed to read STATE.md in cmdStats', { statePath, error: err.message });
+  }
 
   // Git stats
   let gitCommits = 0;
   let gitFirstCommitDate = null;
   try {
-    const commitCount = execGit(cwd, ['rev-list', '--count', 'HEAD']);
+    const commitCount = await execGit(cwd, ['rev-list', '--count', 'HEAD']);
     gitCommits = parseInt(commitCount.trim(), 10) || 0;
-    const firstDate = execGit(cwd, ['log', '--reverse', '--format=%as', '--max-count=1']);
+    const firstDate = await execGit(cwd, ['log', '--reverse', '--format=%as', '--max-count=1']);
     gitFirstCommitDate = firstDate.trim() || null;
-  } catch {}
+  } catch (err) {
+    logger.warn('Failed to compute git stats in cmdStats', { cwd, error: err.message });
+  }
 
   const completedPhases = phases.filter(p => p.status === 'Complete').length;