agileflow 2.99.7 → 3.0.0

package/scripts/lib/configure-detect.js

@@ -8,6 +8,7 @@ const fs = require('fs');
 const path = require('path');
 const { execFileSync } = require('child_process');
 const { c, log, header, readJSON } = require('./configure-utils');
+const { tryOptional } = require('../../lib/errors');
 
 // ============================================================================
 // DETECTION
@@ -64,12 +65,10 @@ function detectConfig(version) {
   // Git detection
   if (fs.existsSync('.git')) {
     status.git.initialized = true;
-    try {
-      status.git.remote = execFileSync('git', ['remote', 'get-url', 'origin'], {
-        encoding: 'utf8',
-        stdio: ['pipe', 'pipe', 'pipe'],
-      }).trim();
-    } catch {}
+    status.git.remote = tryOptional(() => execFileSync('git', ['remote', 'get-url', 'origin'], {
+      encoding: 'utf8',
+      stdio: ['pipe', 'pipe', 'pipe'],
+    }).trim(), 'git remote') ?? null;
   }
 
   // Settings file detection

package/scripts/lib/configure-features.js

@@ -56,6 +56,10 @@ const FEATURES = {
     metadataOnly: true,
     description: 'Default flags for Claude CLI (e.g., --dangerously-skip-permissions)',
   },
+  agentteams: {
+    metadataOnly: true,
+    description: 'Enable Claude Code native Agent Teams (experimental multi-agent orchestration)',
+  },
 };
 
 const PROFILES = {
@@ -324,6 +328,27 @@ function enableFeature(feature, options = {}, version) {
     return true;
   }
 
+  // Handle agent teams (metadata only)
+  if (feature === 'agentteams') {
+    updateMetadata(
+      {
+        features: {
+          agentTeams: {
+            enabled: true,
+            version,
+            at: new Date().toISOString(),
+          },
+        },
+      },
+      version
+    );
+    success('Native Agent Teams enabled');
+    info('Claude Code will use native TeamCreate/SendMessage tools when available');
+    info('Requires: CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS=1 (set by Claude Code)');
+    info('Fallback: subagent mode (Task/TaskOutput) when native is unavailable');
+    return true;
+  }
+
   // Handle shell aliases
   if (feature === 'shellaliases') {
     const result = enableShellAliases();
@@ -730,6 +755,25 @@ function disableFeature(feature, version) {
     return true;
   }
 
+  // Disable agent teams
+  if (feature === 'agentteams') {
+    updateMetadata(
+      {
+        features: {
+          agentTeams: {
+            enabled: false,
+            version,
+            at: new Date().toISOString(),
+          },
+        },
+      },
+      version
+    );
+    success('Native Agent Teams disabled');
+    info('AgileFlow will use subagent mode (Task/TaskOutput) for multi-agent orchestration');
+    return true;
+  }
+
   // Disable shell aliases
   if (feature === 'shellaliases') {
     const result = disableShellAliases();

package/scripts/lib/configure-repair.js

@@ -9,6 +9,7 @@ const path = require('path');
 const crypto = require('crypto');
 const { execFileSync } = require('child_process');
 const { feedback } = require('../../lib/feedback');
+const { tryOptional } = require('../../lib/errors');
 const {
   c,
   log,
@@ -115,11 +116,11 @@ function listScripts() {
     // Check if modified
     let isModified = false;
     if (exists && fileIndex?.files?.[`scripts/${script}`]) {
-      try {
+      isModified = tryOptional(() => {
         const currentHash = sha256(fs.readFileSync(scriptPath));
         const indexHash = fileIndex.files[`scripts/${script}`].sha256;
-        isModified = currentHash !== indexHash;
-      } catch {}
+        return currentHash !== indexHash;
+      }, 'hash check') ?? false;
     }
 
     // Print status
@@ -279,9 +280,7 @@ function repairScripts(targetFeature = null) {
       if (fs.existsSync(srcPath)) {
         try {
           fs.copyFileSync(srcPath, destPath);
-          try {
-            fs.chmodSync(destPath, 0o755);
-          } catch {}
+          tryOptional(() => fs.chmodSync(destPath, 0o755), 'chmod');
           success(`Restored ${script}`);
           repaired++;
         } catch (err) {

package/scripts/lib/configure-utils.js

@@ -6,6 +6,7 @@
 
 const fs = require('fs');
 const path = require('path');
+const { tryOptional } = require('../../lib/errors');
 
 // ============================================================================
 // COLORS & LOGGING
@@ -58,9 +59,7 @@ const copyTemplate = (templateName, destPath) => {
   for (const src of sources) {
     if (fs.existsSync(src)) {
       fs.copyFileSync(src, destPath);
-      try {
-        fs.chmodSync(destPath, '755');
-      } catch {}
+      tryOptional(() => fs.chmodSync(destPath, '755'), 'chmod');
       return true;
     }
   }

package/scripts/lib/context-formatter.js

@@ -321,6 +321,23 @@ function generateSummary(prefetched = null, options = {}) {
 
   summary += divider();
 
+  // Scale indicator (EP-0033)
+  let scaleDetectorSummary;
+  try { scaleDetectorSummary = require('./scale-detector'); } catch { /* not available */ }
+  if (scaleDetectorSummary) {
+    try {
+      const scaleResult = scaleDetectorSummary.detectScale({
+        rootDir: process.cwd(),
+        statusJson: prefetched?.json?.statusJson,
+        sessionState: prefetched?.json?.sessionState,
+      });
+      const label = scaleDetectorSummary.getScaleLabel(scaleResult.scale);
+      summary += row('Scale', label, C.lavender, C.cyan);
+    } catch {
+      // Silently ignore
+    }
+  }
+
   // Key files
   const keyFileChecks = [
     { path: 'CLAUDE.md', label: 'CLAUDE' },
@@ -378,7 +395,7 @@ function generateSummary(prefetched = null, options = {}) {
  * @returns {string} Full content
  */
 function generateFullContent(prefetched = null, options = {}) {
-  const { commandName = null, activeSections = [] } = options;
+  const { commandName = null, activeSections = [], smartDetectResults = null } = options;
 
   let content = '';
 
@@ -437,12 +454,17 @@ function generateFullContent(prefetched = null, options = {}) {
 
   if (askUserQuestionConfig?.enabled) {
     content += `${C.coral}${C.bold}┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓${C.reset}\n`;
-    content += `${C.coral}${C.bold}┃ 🔔 MANDATORY: AskUserQuestion After EVERY Response      ┃${C.reset}\n`;
+    content += `${C.coral}${C.bold}┃ 🔔 SMART AskUserQuestion After EVERY Response           ┃${C.reset}\n`;
     content += `${C.coral}${C.bold}┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛${C.reset}\n`;
     content += `${C.bold}After completing ANY task${C.reset} (implementation, fix, etc.):\n`;
-    content += `${C.mintGreen}→ ALWAYS${C.reset} call ${C.skyBlue}AskUserQuestion${C.reset} tool to offer next steps\n`;
-    content += `${C.coral}→ NEVER${C.reset} end with text like "Done!" or "What's next?"\n\n`;
-    content += `${C.dim}Balance: Use at natural pause points. Don't ask permission for routine work.${C.reset}\n\n`;
+    content += `${C.mintGreen}→ ALWAYS${C.reset} call ${C.skyBlue}AskUserQuestion${C.reset} tool with ${C.bold}SMART, contextual${C.reset} options\n`;
+    content += `${C.coral}→ NEVER${C.reset} generic options like "Continue" or "What next?"\n\n`;
+    content += `${C.bold}Smart Suggestion Principles:${C.reset}\n`;
+    content += `${C.mintGreen}→${C.reset} Always mark one option ${C.bold}"(Recommended)"${C.reset} based on logical next step\n`;
+    content += `${C.mintGreen}→${C.reset} Be specific: ${C.dim}"Run npm test for auth changes"${C.reset} not ${C.dim}"Run tests"${C.reset}\n`;
+    content += `${C.mintGreen}→${C.reset} Include context: ${C.dim}"3 files changed"${C.reset}, story IDs, test results\n`;
+    content += `${C.mintGreen}→${C.reset} Suggest workflow progression: plan → implement → test → commit\n`;
+    content += `${C.mintGreen}→${C.reset} After errors: suggest specific alternative, not ${C.dim}"fix it"${C.reset}\n\n`;
   }
 
   // CONTEXT BUDGET WARNING
@@ -479,6 +501,27 @@ function generateFullContent(prefetched = null, options = {}) {
     content += '\n';
   }
 
+  // SCALE DETECTION (EP-0033)
+  let scaleDetector;
+  try { scaleDetector = require('./scale-detector'); } catch { /* not available */ }
+  if (scaleDetector) {
+    try {
+      const scaleResult = scaleDetector.detectScale({
+        rootDir: process.cwd(),
+        statusJson: prefetched?.json?.statusJson,
+        sessionState: prefetched?.json?.sessionState,
+      });
+      const recs = scaleDetector.getScaleRecommendations(scaleResult.scale);
+      const label = scaleDetector.getScaleLabel(scaleResult.scale);
+      content += `\n${C.cyan}${C.bold}═══ Project Scale: ${label} ═══${C.reset}\n`;
+      content += `${C.dim}Detected: ${scaleResult.metrics.files} files, ${scaleResult.metrics.stories} stories, ${scaleResult.metrics.commits} commits (6mo)${C.reset}\n`;
+      content += `Planning depth: ${C.mintGreen}${recs.planningDepth}${C.reset} | Experts: ${C.mintGreen}${recs.expertCount}${C.reset}\n`;
+      content += `${C.dim}${recs.description}${C.reset}\n`;
+    } catch {
+      // Silently ignore scale detection errors
+    }
+  }
+
   // GIT STATUS
   content += `\n${C.skyBlue}${C.bold}═══ Git Status ═══${C.reset}\n`;
   const branch = prefetched?.git?.branch ?? safeExec('git branch --show-current') ?? 'unknown';
@@ -533,7 +576,7 @@ function generateFullContent(prefetched = null, options = {}) {
     if (Array.isArray(sessionState.active_commands) && sessionState.active_commands.length > 0) {
       const cmdNames = sessionState.active_commands.map(c => c.name).join(', ');
       content += `Active commands: ${C.skyBlue}${cmdNames}${C.reset}\n`;
-    } else if (sessionState.active_command) {
+    } else if (sessionState.active_command && sessionState.active_command.name) {
       content += `Active command: ${C.skyBlue}${sessionState.active_command.name}${C.reset}\n`;
     }
 
@@ -592,7 +635,7 @@ function generateRemainingContent(prefetched, options = {}) {
             const sessionDir = story.claimedBy?.path
               ? path.basename(story.claimedBy.path)
               : 'unknown';
-            content += `  ${C.coral}🔒${C.reset} ${C.lavender}${story.id}${C.reset} "${story.title}" ${C.dim}→ Session ${story.claimedBy?.session_id || '?'} (${sessionDir})${C.reset}\n`;
+            content += `  ${C.coral}🔒${C.reset} ${C.lavender}${story.id ?? '?'}${C.reset} "${story.title ?? 'untitled'}" ${C.dim}→ Session ${story.claimedBy?.session_id ?? '?'} (${sessionDir})${C.reset}\n`;
           });
           content += '\n';
         }
@@ -601,7 +644,7 @@ function generateRemainingContent(prefetched, options = {}) {
         if (myResult.ok && myResult.stories && myResult.stories.length > 0) {
           content += `\n${C.mintGreen}${C.bold}═══ ✓ Your Claimed Stories ═══${C.reset}\n`;
           myResult.stories.forEach(story => {
-            content += `  ${C.mintGreen}✓${C.reset} ${C.lavender}${story.id}${C.reset} "${story.title}"\n`;
+            content += `  ${C.mintGreen}✓${C.reset} ${C.lavender}${story.id ?? '?'}${C.reset} "${story.title ?? 'untitled'}"\n`;
           });
           content += '\n';
         }
@@ -813,6 +856,42 @@ function generateRemainingContent(prefetched, options = {}) {
     }
   }
 
+  // SMART RECOMMENDATIONS (Contextual Feature Router)
+  const smartDetectResults = options.smartDetectResults;
+  if (smartDetectResults && !smartDetectResults.disabled) {
+    const { immediate, available, auto_enabled } = smartDetectResults.recommendations;
+    const hasRecommendations = immediate.length > 0 || available.length > 0;
+
+    if (hasRecommendations) {
+      content += `\n${C.brand}${C.bold}═══ Smart Recommendations ═══${C.reset}\n`;
+      content += `${C.dim}Phase: ${smartDetectResults.lifecycle_phase} (${smartDetectResults.phase_reason})${C.reset}\n`;
+
+      if (immediate.length > 0) {
+        content += `\n${C.coral}${C.bold}Immediate:${C.reset}\n`;
+        immediate.forEach(r => {
+          content += `  ${C.coral}!${C.reset} ${C.bold}${r.feature}${C.reset}: ${r.trigger} ${C.dim}(${r.command})${C.reset}\n`;
+        });
+      }
+
+      if (available.length > 0) {
+        content += `\n${C.skyBlue}Available:${C.reset}\n`;
+        available.slice(0, 5).forEach(r => {
+          content += `  ${C.skyBlue}>${C.reset} ${r.feature}: ${r.trigger} ${C.dim}(${r.command})${C.reset}\n`;
+        });
+        if (available.length > 5) {
+          content += `  ${C.dim}... and ${available.length - 5} more${C.reset}\n`;
+        }
+      }
+    }
+
+    // Show auto-enabled modes
+    const modes = auto_enabled || {};
+    const enabledModes = Object.entries(modes).filter(([, v]) => v).map(([k]) => k.replace('_', ' '));
+    if (enabledModes.length > 0) {
+      content += `\n${C.mintGreen}Auto-enabled:${C.reset} ${enabledModes.join(', ')}\n`;
+    }
+  }
+
   // FOOTER
   content += `\n${C.dim}─────────────────────────────────────────${C.reset}\n`;
   content += `${C.dim}Context gathered in single execution. Claude has full context.${C.reset}\n`;

package/scripts/lib/damage-control-utils.js

@@ -33,6 +33,14 @@ const c = {
   reset: '\x1b[0m',
 };
 
+// Pattern cache: avoids re-reading and re-parsing YAML on every hook invocation.
+// Invalidated when the config file's mtime changes.
+const _patternCache = {
+  /** @type {string|null} */ filePath: null,
+  /** @type {number} */ mtime: 0,
+  /** @type {object|null} */ config: null,
+};
+
 // Shared constants
 const CONFIG_PATHS = [
   '.agileflow/config/damage-control-patterns.yaml',
@@ -70,8 +78,9 @@ function expandPath(p) {
 }
 
 /**
- * Load patterns configuration from YAML file
- * Returns empty config if not found (fail-open)
+ * Load patterns configuration from YAML file with caching.
+ * Returns cached config when the file hasn't changed (mtime check).
+ * Returns empty config if not found (fail-open).
  *
  * @param {string} projectRoot - Project root directory
  * @param {function} parseYAML - Function to parse YAML content
@@ -83,8 +92,23 @@ function loadPatterns(projectRoot, parseYAML, defaultConfig = {}) {
     const fullPath = path.join(projectRoot, configPath);
     if (fs.existsSync(fullPath)) {
       try {
+        // Check mtime for cache invalidation
+        const stat = fs.statSync(fullPath);
+        const mtime = stat.mtimeMs;
+
+        if (_patternCache.filePath === fullPath && _patternCache.mtime === mtime && _patternCache.config) {
+          return _patternCache.config;
+        }
+
         const content = fs.readFileSync(fullPath, 'utf8');
-        return parseYAML(content);
+        const config = parseYAML(content);
+
+        // Store in cache
+        _patternCache.filePath = fullPath;
+        _patternCache.mtime = mtime;
+        _patternCache.config = config;
+
+        return config;
       } catch (e) {
         // Continue to next path
       }
@@ -95,6 +119,15 @@ function loadPatterns(projectRoot, parseYAML, defaultConfig = {}) {
   return defaultConfig;
 }
 
+/**
+ * Clear the pattern cache (for testing or forced reload).
+ */
+function clearPatternCache() {
+  _patternCache.filePath = null;
+  _patternCache.mtime = 0;
+  _patternCache.config = null;
+}
+
 /**
  * Check if a file path matches any of the protected patterns
  *
@@ -559,6 +592,7 @@ module.exports = {
   findProjectRoot,
   expandPath,
   loadPatterns,
+  clearPatternCache,
   pathMatches,
   outputBlocked,
   runDamageControlHook,