agileflow 2.80.0 → 2.82.0

package/scripts/agileflow-configure.js

@@ -29,6 +29,7 @@
 const fs = require('fs');
 const path = require('path');
 const { execSync } = require('child_process');
+const { isValidProfileName, isValidFeatureName, parseIntBounded } = require('../lib/validate');
 
 // ============================================================================
 // CONFIGURATION
@@ -1623,7 +1624,8 @@ function main() {
         .split('=')[1]
         .split(',')
         .map(s => s.trim().toLowerCase());
-    else if (arg.startsWith('--archival-days=')) archivalDays = parseInt(arg.split('=')[1]) || 30;
+    else if (arg.startsWith('--archival-days='))
+      archivalDays = parseIntBounded(arg.split('=')[1], 30, 1, 365);
     else if (arg === '--migrate') migrate = true;
     else if (arg === '--detect' || arg === '--validate') detect = true;
     else if (arg === '--upgrade') upgrade = true;

package/scripts/agileflow-welcome.js

@@ -207,6 +207,12 @@ function checkParallelSessions(rootDir) {
     otherActive: 0,
     currentId: null,
     cleaned: 0,
+    // Extended session info for non-main sessions
+    isMain: true,
+    nickname: null,
+    branch: null,
+    sessionPath: null,
+    mainPath: rootDir,
   };
 
   try {
@@ -247,6 +253,24 @@ function checkParallelSessions(rootDir) {
     } catch (e) {
       // Count failed
     }
+
+    // Get detailed status for current session (for banner display)
+    try {
+      const statusOutput = execSync(`node "${scriptPath}" status`, {
+        cwd: rootDir,
+        encoding: 'utf8',
+        stdio: ['pipe', 'pipe', 'pipe'],
+      });
+      const statusData = JSON.parse(statusOutput);
+      if (statusData.current) {
+        result.isMain = statusData.current.is_main === true;
+        result.nickname = statusData.current.nickname;
+        result.branch = statusData.current.branch;
+        result.sessionPath = statusData.current.path;
+      }
+    } catch (e) {
+      // Status failed
+    }
   } catch (e) {
     // Session system not available
   }
@@ -849,6 +873,41 @@ function formatTable(
   return lines.join('\n');
 }
 
+// Format session banner for non-main sessions
+function formatSessionBanner(parallelSessions) {
+  if (!parallelSessions.available || parallelSessions.isMain) {
+    return null;
+  }
+
+  const W = 62; // banner width
+  const lines = [];
+
+  // Get display name
+  const sessionName = parallelSessions.nickname
+    ? `SESSION ${parallelSessions.currentId} "${parallelSessions.nickname}"`
+    : `SESSION ${parallelSessions.currentId}`;
+
+  lines.push(`${c.dim}${box.tl}${box.h.repeat(W)}${box.tr}${c.reset}`);
+  lines.push(
+    `${c.dim}${box.v}${c.reset} ${c.teal}${c.bold}${pad(sessionName, W - 2)}${c.reset} ${c.dim}${box.v}${c.reset}`
+  );
+  lines.push(
+    `${c.dim}${box.v}${c.reset}    ${c.slate}Branch:${c.reset} ${pad(parallelSessions.branch || 'unknown', W - 13)} ${c.dim}${box.v}${c.reset}`
+  );
+
+  // Show relative path to main
+  if (parallelSessions.sessionPath) {
+    const relPath = path.relative(parallelSessions.sessionPath, parallelSessions.mainPath) || '.';
+    lines.push(
+      `${c.dim}${box.v}${c.reset}    ${c.slate}Main at:${c.reset} ${pad(relPath, W - 14)} ${c.dim}${box.v}${c.reset}`
+    );
+  }
+
+  lines.push(`${c.dim}${box.bl}${box.h.repeat(W)}${box.br}${c.reset}`);
+
+  return lines.join('\n');
+}
+
 // Main
 async function main() {
   const rootDir = getProjectRoot();
@@ -882,6 +941,12 @@ async function main() {
     // Update check failed - continue without it
   }
 
+  // Show session banner FIRST if in a non-main session
+  const sessionBanner = formatSessionBanner(parallelSessions);
+  if (sessionBanner) {
+    console.log(sessionBanner);
+  }
+
   console.log(
     formatTable(
       info,

package/scripts/damage-control-bash.js

@@ -15,23 +15,13 @@
  * Usage: Configured as PreToolUse hook in .claude/settings.json
  */
 
-const fs = require('fs');
-const path = require('path');
-const { c } = require('../lib/colors');
-
-/**
- * Find project root by looking for .agileflow directory
- */
-function findProjectRoot() {
-  let dir = process.cwd();
-  while (dir !== '/') {
-    if (fs.existsSync(path.join(dir, '.agileflow'))) {
-      return dir;
-    }
-    dir = path.dirname(dir);
-  }
-  return process.cwd();
-}
+const {
+  findProjectRoot,
+  loadPatterns,
+  outputBlocked,
+  runDamageControlHook,
+  c,
+} = require('./lib/damage-control-utils');
 
 /**
  * Parse simplified YAML for damage control patterns
@@ -91,31 +81,6 @@ function parseSimpleYAML(content) {
   return config;
 }
 
-/**
- * Load patterns configuration from YAML file
- */
-function loadPatterns(projectRoot) {
-  const configPaths = [
-    path.join(projectRoot, '.agileflow/config/damage-control-patterns.yaml'),
-    path.join(projectRoot, '.agileflow/config/damage-control-patterns.yml'),
-    path.join(projectRoot, '.agileflow/templates/damage-control-patterns.yaml'),
-  ];
-
-  for (const configPath of configPaths) {
-    if (fs.existsSync(configPath)) {
-      try {
-        const content = fs.readFileSync(configPath, 'utf8');
-        return parseSimpleYAML(content);
-      } catch (e) {
-        // Continue to next path
-      }
-    }
-  }
-
-  // Return empty config if no file found (fail-open)
-  return { bashToolPatterns: [], askPatterns: [], agileflowProtections: [] };
-}
-
 /**
  * Test command against a single pattern rule
  */
@@ -163,76 +128,18 @@ function validateCommand(command, config) {
   return { action: 'allow' };
 }
 
-/**
- * Main function - read input and validate
- */
-function main() {
-  const projectRoot = findProjectRoot();
-  let inputData = '';
-
-  process.stdin.setEncoding('utf8');
-
-  process.stdin.on('data', chunk => {
-    inputData += chunk;
-  });
-
-  process.stdin.on('end', () => {
-    try {
-      // Parse tool input from Claude Code
-      const input = JSON.parse(inputData);
-      const command = input.command || input.tool_input?.command || '';
-
-      if (!command) {
-        // No command to validate - allow
-        process.exit(0);
-      }
-
-      // Load patterns and validate
-      const config = loadPatterns(projectRoot);
-      const result = validateCommand(command, config);
-
-      switch (result.action) {
-        case 'block':
-          // Output error message and block
-          console.error(`${c.coral}[BLOCKED]${c.reset} ${result.reason}`);
-          console.error(
-            `${c.dim}Command: ${command.substring(0, 100)}${command.length > 100 ? '...' : ''}${c.reset}`
-          );
-          process.exit(2);
-          break;
-
-        case 'ask':
-          // Output JSON to trigger user confirmation
-          console.log(
-            JSON.stringify({
-              result: 'ask',
-              message: result.reason,
-            })
-          );
-          process.exit(0);
-          break;
-
-        case 'allow':
-        default:
-          // Allow command to proceed
-          process.exit(0);
-      }
-    } catch (e) {
-      // Parse error or other issue - fail open
-      // This ensures broken config doesn't block all commands
-      process.exit(0);
-    }
-  });
-
-  // Handle no stdin (direct invocation)
-  process.stdin.on('error', () => {
-    process.exit(0);
-  });
-
-  // Set timeout to prevent hanging
-  setTimeout(() => {
-    process.exit(0);
-  }, 4000);
-}
-
-main();
+// Run the hook
+const projectRoot = findProjectRoot();
+const defaultConfig = { bashToolPatterns: [], askPatterns: [], agileflowProtections: [] };
+
+runDamageControlHook({
+  getInputValue: input => input.command || input.tool_input?.command,
+  loadConfig: () => loadPatterns(projectRoot, parseSimpleYAML, defaultConfig),
+  validate: validateCommand,
+  onBlock: (result, command) => {
+    outputBlocked(
+      result.reason,
+      `Command: ${command.substring(0, 100)}${command.length > 100 ? '...' : ''}`
+    );
+  },
+});

package/scripts/damage-control-edit.js

@@ -12,34 +12,13 @@
  * Usage: Configured as PreToolUse hook in .claude/settings.json
  */
 
-const fs = require('fs');
-const path = require('path');
-const os = require('os');
-const { c } = require('../lib/colors');
-
-/**
- * Find project root by looking for .agileflow directory
- */
-function findProjectRoot() {
-  let dir = process.cwd();
-  while (dir !== '/') {
-    if (fs.existsSync(path.join(dir, '.agileflow'))) {
-      return dir;
-    }
-    dir = path.dirname(dir);
-  }
-  return process.cwd();
-}
-
-/**
- * Expand ~ to home directory
- */
-function expandPath(p) {
-  if (p.startsWith('~/')) {
-    return path.join(os.homedir(), p.slice(2));
-  }
-  return p;
-}
+const {
+  findProjectRoot,
+  loadPatterns,
+  pathMatches,
+  outputBlocked,
+  runDamageControlHook,
+} = require('./lib/damage-control-utils');
 
 /**
  * Parse simplified YAML for path patterns
@@ -79,79 +58,6 @@ function parseSimpleYAML(content) {
   return config;
 }
 
-/**
- * Load patterns configuration from YAML file
- */
-function loadPatterns(projectRoot) {
-  const configPaths = [
-    path.join(projectRoot, '.agileflow/config/damage-control-patterns.yaml'),
-    path.join(projectRoot, '.agileflow/config/damage-control-patterns.yml'),
-    path.join(projectRoot, '.agileflow/templates/damage-control-patterns.yaml'),
-  ];
-
-  for (const configPath of configPaths) {
-    if (fs.existsSync(configPath)) {
-      try {
-        const content = fs.readFileSync(configPath, 'utf8');
-        return parseSimpleYAML(content);
-      } catch (e) {
-        // Continue to next path
-      }
-    }
-  }
-
-  // Return empty config if no file found (fail-open)
-  return { zeroAccessPaths: [], readOnlyPaths: [], noDeletePaths: [] };
-}
-
-/**
- * Check if a file path matches any of the protected patterns
- */
-function pathMatches(filePath, patterns) {
-  if (!filePath) return null;
-
-  const normalizedPath = path.resolve(filePath);
-  const relativePath = path.relative(process.cwd(), normalizedPath);
-
-  for (const pattern of patterns) {
-    const expandedPattern = expandPath(pattern);
-
-    // Check if pattern is a directory prefix
-    if (pattern.endsWith('/')) {
-      const patternDir = expandedPattern.slice(0, -1);
-      if (normalizedPath.startsWith(patternDir)) {
-        return pattern;
-      }
-    }
-
-    // Check exact match
-    if (normalizedPath === expandedPattern) {
-      return pattern;
-    }
-
-    // Check if normalized path ends with pattern (for filenames like "id_rsa")
-    if (normalizedPath.endsWith(pattern) || relativePath.endsWith(pattern)) {
-      return pattern;
-    }
-
-    // Check if pattern appears in path (for patterns like "*.pem")
-    if (pattern.startsWith('*')) {
-      const ext = pattern.slice(1);
-      if (normalizedPath.endsWith(ext) || relativePath.endsWith(ext)) {
-        return pattern;
-      }
-    }
-
-    // Check if path contains pattern (for things like ".env.production")
-    const patternBase = path.basename(pattern);
-    if (path.basename(normalizedPath) === patternBase) {
-      return pattern;
-    }
-  }
-
-  return null;
-}
-
 /**
  * Validate file path for edit operation
  */
@@ -180,58 +86,15 @@ function validatePath(filePath, config) {
   return { action: 'allow' };
 }
 
-/**
- * Main function - read input and validate
- */
-function main() {
-  const projectRoot = findProjectRoot();
-  let inputData = '';
-
-  process.stdin.setEncoding('utf8');
-
-  process.stdin.on('data', chunk => {
-    inputData += chunk;
-  });
-
-  process.stdin.on('end', () => {
-    try {
-      // Parse tool input from Claude Code
-      const input = JSON.parse(inputData);
-      const filePath = input.file_path || input.tool_input?.file_path || '';
-
-      if (!filePath) {
-        // No path to validate - allow
-        process.exit(0);
-      }
-
-      // Load patterns and validate
-      const config = loadPatterns(projectRoot);
-      const result = validatePath(filePath, config);
-
-      if (result.action === 'block') {
-        console.error(`${c.coral}[BLOCKED]${c.reset} ${result.reason}`);
-        console.error(`${c.dim}${result.detail}${c.reset}`);
-        console.error(`${c.dim}File: ${filePath}${c.reset}`);
-        process.exit(2);
-      }
-
-      // Allow
-      process.exit(0);
-    } catch (e) {
-      // Parse error or other issue - fail open
-      process.exit(0);
-    }
-  });
-
-  // Handle no stdin
-  process.stdin.on('error', () => {
-    process.exit(0);
-  });
-
-  // Set timeout to prevent hanging
-  setTimeout(() => {
-    process.exit(0);
-  }, 4000);
-}
-
-main();
+// Run the hook
+const projectRoot = findProjectRoot();
+const defaultConfig = { zeroAccessPaths: [], readOnlyPaths: [], noDeletePaths: [] };
+
+runDamageControlHook({
+  getInputValue: input => input.file_path || input.tool_input?.file_path,
+  loadConfig: () => loadPatterns(projectRoot, parseSimpleYAML, defaultConfig),
+  validate: validatePath,
+  onBlock: (result, filePath) => {
+    outputBlocked(result.reason, result.detail, `File: ${filePath}`);
+  },
+});

package/scripts/damage-control-write.js

@@ -12,34 +12,13 @@
  * Usage: Configured as PreToolUse hook in .claude/settings.json
  */
 
-const fs = require('fs');
-const path = require('path');
-const os = require('os');
-const { c } = require('../lib/colors');
-
-/**
- * Find project root by looking for .agileflow directory
- */
-function findProjectRoot() {
-  let dir = process.cwd();
-  while (dir !== '/') {
-    if (fs.existsSync(path.join(dir, '.agileflow'))) {
-      return dir;
-    }
-    dir = path.dirname(dir);
-  }
-  return process.cwd();
-}
-
-/**
- * Expand ~ to home directory
- */
-function expandPath(p) {
-  if (p.startsWith('~/')) {
-    return path.join(os.homedir(), p.slice(2));
-  }
-  return p;
-}
+const {
+  findProjectRoot,
+  loadPatterns,
+  pathMatches,
+  outputBlocked,
+  runDamageControlHook,
+} = require('./lib/damage-control-utils');
 
 /**
  * Parse simplified YAML for path patterns
@@ -79,79 +58,6 @@ function parseSimpleYAML(content) {
   return config;
 }
 
-/**
- * Load patterns configuration from YAML file
- */
-function loadPatterns(projectRoot) {
-  const configPaths = [
-    path.join(projectRoot, '.agileflow/config/damage-control-patterns.yaml'),
-    path.join(projectRoot, '.agileflow/config/damage-control-patterns.yml'),
-    path.join(projectRoot, '.agileflow/templates/damage-control-patterns.yaml'),
-  ];
-
-  for (const configPath of configPaths) {
-    if (fs.existsSync(configPath)) {
-      try {
-        const content = fs.readFileSync(configPath, 'utf8');
-        return parseSimpleYAML(content);
-      } catch (e) {
-        // Continue to next path
-      }
-    }
-  }
-
-  // Return empty config if no file found (fail-open)
-  return { zeroAccessPaths: [], readOnlyPaths: [], noDeletePaths: [] };
-}
-
-/**
- * Check if a file path matches any of the protected patterns
- */
-function pathMatches(filePath, patterns) {
-  if (!filePath) return null;
-
-  const normalizedPath = path.resolve(filePath);
-  const relativePath = path.relative(process.cwd(), normalizedPath);
-
-  for (const pattern of patterns) {
-    const expandedPattern = expandPath(pattern);
-
-    // Check if pattern is a directory prefix
-    if (pattern.endsWith('/')) {
-      const patternDir = expandedPattern.slice(0, -1);
-      if (normalizedPath.startsWith(patternDir)) {
-        return pattern;
-      }
-    }
-
-    // Check exact match
-    if (normalizedPath === expandedPattern) {
-      return pattern;
-    }
-
-    // Check if normalized path ends with pattern (for filenames like "id_rsa")
-    if (normalizedPath.endsWith(pattern) || relativePath.endsWith(pattern)) {
-      return pattern;
-    }
-
-    // Check if pattern appears in path (for patterns like "*.pem")
-    if (pattern.startsWith('*')) {
-      const ext = pattern.slice(1);
-      if (normalizedPath.endsWith(ext) || relativePath.endsWith(ext)) {
-        return pattern;
-      }
-    }
-
-    // Check if path contains pattern (for things like ".env.production")
-    const patternBase = path.basename(pattern);
-    if (path.basename(normalizedPath) === patternBase) {
-      return pattern;
-    }
-  }
-
-  return null;
-}
-
 /**
  * Validate file path for write operation
  */
@@ -180,58 +86,15 @@ function validatePath(filePath, config) {
   return { action: 'allow' };
 }
 
-/**
- * Main function - read input and validate
- */
-function main() {
-  const projectRoot = findProjectRoot();
-  let inputData = '';
-
-  process.stdin.setEncoding('utf8');
-
-  process.stdin.on('data', chunk => {
-    inputData += chunk;
-  });
-
-  process.stdin.on('end', () => {
-    try {
-      // Parse tool input from Claude Code
-      const input = JSON.parse(inputData);
-      const filePath = input.file_path || input.tool_input?.file_path || '';
-
-      if (!filePath) {
-        // No path to validate - allow
-        process.exit(0);
-      }
-
-      // Load patterns and validate
-      const config = loadPatterns(projectRoot);
-      const result = validatePath(filePath, config);
-
-      if (result.action === 'block') {
-        console.error(`${c.coral}[BLOCKED]${c.reset} ${result.reason}`);
-        console.error(`${c.dim}${result.detail}${c.reset}`);
-        console.error(`${c.dim}File: ${filePath}${c.reset}`);
-        process.exit(2);
-      }
-
-      // Allow
-      process.exit(0);
-    } catch (e) {
-      // Parse error or other issue - fail open
-      process.exit(0);
-    }
-  });
-
-  // Handle no stdin
-  process.stdin.on('error', () => {
-    process.exit(0);
-  });
-
-  // Set timeout to prevent hanging
-  setTimeout(() => {
-    process.exit(0);
-  }, 4000);
-}
-
-main();
+// Run the hook
+const projectRoot = findProjectRoot();
+const defaultConfig = { zeroAccessPaths: [], readOnlyPaths: [], noDeletePaths: [] };
+
+runDamageControlHook({
+  getInputValue: input => input.file_path || input.tool_input?.file_path,
+  loadConfig: () => loadPatterns(projectRoot, parseSimpleYAML, defaultConfig),
+  validate: validatePath,
+  onBlock: (result, filePath) => {
+    outputBlocked(result.reason, result.detail, `File: ${filePath}`);
+  },
+});