@howlil/ez-agents 3.4.1 → 3.4.2

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

@@ -1,118 +1,118 @@
-#!/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;
+#!/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/config.cjs

@@ -1,190 +1,190 @@
-/**
- * Config — Planning config CRUD operations
- */
-
-const fs = require('fs');
-const path = require('path');
-const { output, error } = require('./core.cjs');
-const { safePlanningWriteSync } = require('./planning-write.cjs');
-
-const VALID_CONFIG_KEYS = new Set([
-  'mode', 'granularity', 'parallelization', 'commit_docs', 'model_profile',
-  'search_gitignored', 'brave_search',
-  'workflow.research', 'workflow.plan_check', 'workflow.verifier',
-  'workflow.nyquist_validation', 'workflow.ui_phase', 'workflow.ui_safety_gate',
-  'workflow._auto_chain_active',
-  'git.branching_strategy', 'git.phase_branch_template', 'git.milestone_branch_template',
-  'planning.commit_docs', 'planning.search_gitignored',
-]);
-
-function cmdConfigEnsureSection(cwd, raw) {
-  const configPath = path.join(cwd, '.planning', 'config.json');
-  const planningDir = path.join(cwd, '.planning');
-
-  // Ensure .planning directory exists
-  try {
-    if (!fs.existsSync(planningDir)) {
-      fs.mkdirSync(planningDir, { recursive: true });
-    }
-  } catch (err) {
-    error('Failed to create .planning directory: ' + err.message);
-  }
-
-  // Check if config already exists
-  if (fs.existsSync(configPath)) {
-    const result = { created: false, reason: 'already_exists' };
-    output(result, raw, 'exists');
-    return;
-  }
-
-  // Detect Brave Search API key availability (prefer ~/.ez)
-  const homedir = require('os').homedir();
-  const braveKeyCandidates = [
-    path.join(homedir, '.ez', 'brave_api_key'),
-  ];
-  const hasBraveSearch = !!(process.env.BRAVE_API_KEY || braveKeyCandidates.some(p => fs.existsSync(p)));
-
-  // Load user-level defaults from ~/.ez/defaults.json
-  const defaultsCandidates = [
-    path.join(homedir, '.ez', 'defaults.json'),
-  ];
-  const existingDefaultsPath = defaultsCandidates.find(p => fs.existsSync(p));
-  const globalDefaultsPath = existingDefaultsPath || defaultsCandidates[0];
-  let userDefaults = {};
-  try {
-    if (existingDefaultsPath) {
-      userDefaults = JSON.parse(fs.readFileSync(globalDefaultsPath, 'utf-8'));
-      // Migrate deprecated "depth" key to "granularity"
-      if ('depth' in userDefaults && !('granularity' in userDefaults)) {
-        const depthToGranularity = { quick: 'coarse', standard: 'standard', comprehensive: 'fine' };
-        userDefaults.granularity = depthToGranularity[userDefaults.depth] || userDefaults.depth;
-        delete userDefaults.depth;
-        try { fs.writeFileSync(globalDefaultsPath, JSON.stringify(userDefaults, null, 2), 'utf-8'); } catch {}
-      }
-    }
-  } catch (err) {
-    // Ignore malformed global defaults, fall back to hardcoded
-  }
-
-  // Create default config (user-level defaults override hardcoded defaults)
-  const hardcoded = {
-    model_profile: 'balanced',
-    commit_docs: true,
-    search_gitignored: false,
-    branching_strategy: 'none',
-    phase_branch_template: 'ez/phase-{phase}-{slug}',
-    milestone_branch_template: 'ez/{milestone}-{slug}',
-    workflow: {
-      research: true,
-      plan_check: true,
-      verifier: true,
-      nyquist_validation: true,
-    },
-    parallelization: true,
-    brave_search: hasBraveSearch,
-  };
-  const defaults = {
-    ...hardcoded,
-    ...userDefaults,
-    workflow: { ...hardcoded.workflow, ...(userDefaults.workflow || {}) },
-  };
-
-  try {
-    safePlanningWriteSync(configPath, JSON.stringify(defaults, null, 2));
-    const result = { created: true, path: '.planning/config.json' };
-    output(result, raw, 'created');
-  } catch (err) {
-    error('Failed to create config.json: ' + err.message);
-  }
-}
-
-function cmdConfigSet(cwd, keyPath, value, raw) {
-  const configPath = path.join(cwd, '.planning', 'config.json');
-
-  if (!keyPath) {
-    error('Usage: config-set <key.path> <value>');
-  }
-
-  if (!VALID_CONFIG_KEYS.has(keyPath)) {
-    error(`Unknown config key: "${keyPath}". Valid keys: ${[...VALID_CONFIG_KEYS].sort().join(', ')}`);
-  }
-
-  // Parse value (handle booleans and numbers)
-  let parsedValue = value;
-  if (value === 'true') parsedValue = true;
-  else if (value === 'false') parsedValue = false;
-  else if (!isNaN(value) && value !== '') parsedValue = Number(value);
-
-  // Load existing config or start with empty object
-  let config = {};
-  try {
-    if (fs.existsSync(configPath)) {
-      config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
-    }
-  } catch (err) {
-    error('Failed to read config.json: ' + err.message);
-  }
-
-  // Set nested value using dot notation (e.g., "workflow.research")
-  const keys = keyPath.split('.');
-  let current = config;
-  for (let i = 0; i < keys.length - 1; i++) {
-    const key = keys[i];
-    if (current[key] === undefined || typeof current[key] !== 'object') {
-      current[key] = {};
-    }
-    current = current[key];
-  }
-  current[keys[keys.length - 1]] = parsedValue;
-
-  // Write back
-  try {
-    safePlanningWriteSync(configPath, JSON.stringify(config, null, 2));
-    const result = { updated: true, key: keyPath, value: parsedValue };
-    output(result, raw, `${keyPath}=${parsedValue}`);
-  } catch (err) {
-    error('Failed to write config.json: ' + err.message);
-  }
-}
-
-function cmdConfigGet(cwd, keyPath, raw) {
-  const configPath = path.join(cwd, '.planning', 'config.json');
-
-  if (!keyPath) {
-    error('Usage: config-get <key.path>');
-  }
-
-  let config = {};
-  try {
-    if (fs.existsSync(configPath)) {
-      config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
-    } else {
-      error('No config.json found at ' + configPath);
-    }
-  } catch (err) {
-    if (err.message.startsWith('No config.json')) throw err;
-    error('Failed to read config.json: ' + err.message);
-  }
-
-  // Traverse dot-notation path (e.g., "workflow.auto_advance")
-  const keys = keyPath.split('.');
-  let current = config;
-  for (const key of keys) {
-    if (current === undefined || current === null || typeof current !== 'object') {
-      error(`Key not found: ${keyPath}`);
-    }
-    current = current[key];
-  }
-
-  if (current === undefined) {
-    error(`Key not found: ${keyPath}`);
-  }
-
-  output(current, raw, String(current));
-}
-
-module.exports = {
-  cmdConfigEnsureSection,
-  cmdConfigSet,
-  cmdConfigGet,
-};
+/**
+ * Config — Planning config CRUD operations
+ */
+
+const fs = require('fs');
+const path = require('path');
+const { output, error } = require('./core.cjs');
+const { safePlanningWriteSync } = require('./planning-write.cjs');
+
+const VALID_CONFIG_KEYS = new Set([
+  'mode', 'granularity', 'parallelization', 'commit_docs', 'model_profile',
+  'search_gitignored', 'brave_search',
+  'workflow.research', 'workflow.plan_check', 'workflow.verifier',
+  'workflow.nyquist_validation', 'workflow.ui_phase', 'workflow.ui_safety_gate',
+  'workflow._auto_chain_active',
+  'git.branching_strategy', 'git.phase_branch_template', 'git.milestone_branch_template',
+  'planning.commit_docs', 'planning.search_gitignored',
+]);
+
+function cmdConfigEnsureSection(cwd, raw) {
+  const configPath = path.join(cwd, '.planning', 'config.json');
+  const planningDir = path.join(cwd, '.planning');
+
+  // Ensure .planning directory exists
+  try {
+    if (!fs.existsSync(planningDir)) {
+      fs.mkdirSync(planningDir, { recursive: true });
+    }
+  } catch (err) {
+    error('Failed to create .planning directory: ' + err.message);
+  }
+
+  // Check if config already exists
+  if (fs.existsSync(configPath)) {
+    const result = { created: false, reason: 'already_exists' };
+    output(result, raw, 'exists');
+    return;
+  }
+
+  // Detect Brave Search API key availability (prefer ~/.ez)
+  const homedir = require('os').homedir();
+  const braveKeyCandidates = [
+    path.join(homedir, '.ez', 'brave_api_key'),
+  ];
+  const hasBraveSearch = !!(process.env.BRAVE_API_KEY || braveKeyCandidates.some(p => fs.existsSync(p)));
+
+  // Load user-level defaults from ~/.ez/defaults.json
+  const defaultsCandidates = [
+    path.join(homedir, '.ez', 'defaults.json'),
+  ];
+  const existingDefaultsPath = defaultsCandidates.find(p => fs.existsSync(p));
+  const globalDefaultsPath = existingDefaultsPath || defaultsCandidates[0];
+  let userDefaults = {};
+  try {
+    if (existingDefaultsPath) {
+      userDefaults = JSON.parse(fs.readFileSync(globalDefaultsPath, 'utf-8'));
+      // Migrate deprecated "depth" key to "granularity"
+      if ('depth' in userDefaults && !('granularity' in userDefaults)) {
+        const depthToGranularity = { quick: 'coarse', standard: 'standard', comprehensive: 'fine' };
+        userDefaults.granularity = depthToGranularity[userDefaults.depth] || userDefaults.depth;
+        delete userDefaults.depth;
+        try { fs.writeFileSync(globalDefaultsPath, JSON.stringify(userDefaults, null, 2), 'utf-8'); } catch {}
+      }
+    }
+  } catch (err) {
+    // Ignore malformed global defaults, fall back to hardcoded
+  }
+
+  // Create default config (user-level defaults override hardcoded defaults)
+  const hardcoded = {
+    model_profile: 'balanced',
+    commit_docs: true,
+    search_gitignored: false,
+    branching_strategy: 'none',
+    phase_branch_template: 'ez/phase-{phase}-{slug}',
+    milestone_branch_template: 'ez/{milestone}-{slug}',
+    workflow: {
+      research: true,
+      plan_check: true,
+      verifier: true,
+      nyquist_validation: true,
+    },
+    parallelization: true,
+    brave_search: hasBraveSearch,
+  };
+  const defaults = {
+    ...hardcoded,
+    ...userDefaults,
+    workflow: { ...hardcoded.workflow, ...(userDefaults.workflow || {}) },
+  };
+
+  try {
+    safePlanningWriteSync(configPath, JSON.stringify(defaults, null, 2));
+    const result = { created: true, path: '.planning/config.json' };
+    output(result, raw, 'created');
+  } catch (err) {
+    error('Failed to create config.json: ' + err.message);
+  }
+}
+
+function cmdConfigSet(cwd, keyPath, value, raw) {
+  const configPath = path.join(cwd, '.planning', 'config.json');
+
+  if (!keyPath) {
+    error('Usage: config-set <key.path> <value>');
+  }
+
+  if (!VALID_CONFIG_KEYS.has(keyPath)) {
+    error(`Unknown config key: "${keyPath}". Valid keys: ${[...VALID_CONFIG_KEYS].sort().join(', ')}`);
+  }
+
+  // Parse value (handle booleans and numbers)
+  let parsedValue = value;
+  if (value === 'true') parsedValue = true;
+  else if (value === 'false') parsedValue = false;
+  else if (!isNaN(value) && value !== '') parsedValue = Number(value);
+
+  // Load existing config or start with empty object
+  let config = {};
+  try {
+    if (fs.existsSync(configPath)) {
+      config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+    }
+  } catch (err) {
+    error('Failed to read config.json: ' + err.message);
+  }
+
+  // Set nested value using dot notation (e.g., "workflow.research")
+  const keys = keyPath.split('.');
+  let current = config;
+  for (let i = 0; i < keys.length - 1; i++) {
+    const key = keys[i];
+    if (current[key] === undefined || typeof current[key] !== 'object') {
+      current[key] = {};
+    }
+    current = current[key];
+  }
+  current[keys[keys.length - 1]] = parsedValue;
+
+  // Write back
+  try {
+    safePlanningWriteSync(configPath, JSON.stringify(config, null, 2));
+    const result = { updated: true, key: keyPath, value: parsedValue };
+    output(result, raw, `${keyPath}=${parsedValue}`);
+  } catch (err) {
+    error('Failed to write config.json: ' + err.message);
+  }
+}
+
+function cmdConfigGet(cwd, keyPath, raw) {
+  const configPath = path.join(cwd, '.planning', 'config.json');
+
+  if (!keyPath) {
+    error('Usage: config-get <key.path>');
+  }
+
+  let config = {};
+  try {
+    if (fs.existsSync(configPath)) {
+      config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+    } else {
+      error('No config.json found at ' + configPath);
+    }
+  } catch (err) {
+    if (err.message.startsWith('No config.json')) throw err;
+    error('Failed to read config.json: ' + err.message);
+  }
+
+  // Traverse dot-notation path (e.g., "workflow.auto_advance")
+  const keys = keyPath.split('.');
+  let current = config;
+  for (const key of keys) {
+    if (current === undefined || current === null || typeof current !== 'object') {
+      error(`Key not found: ${keyPath}`);
+    }
+    current = current[key];
+  }
+
+  if (current === undefined) {
+    error(`Key not found: ${keyPath}`);
+  }
+
+  output(current, raw, String(current));
+}
+
+module.exports = {
+  cmdConfigEnsureSection,
+  cmdConfigSet,
+  cmdConfigGet,
+};