wogiflow 2.17.0 → 2.18.0

package/scripts/flow-request-log.js

@@ -0,0 +1,133 @@
+#!/usr/bin/env node
+
+/**
+ * Wogi Flow - Request Log Operations
+ *
+ * Read/count/append operations for request-log.md. Extracted from
+ * flow-utils.js (wf-94cc3b72 epic — flow-utils decomposition).
+ *
+ * Uses appendFileSync for atomic append (avoids read-modify-write race).
+ */
+
+'use strict';
+
+const fs = require('node:fs');
+const { PATHS } = require('./flow-paths');
+const { readFile } = require('./flow-io');
+const { error } = require('./flow-output');
+
+/**
+ * Get the current AI CLI session ID (kept local to avoid circular flow-utils dep).
+ * @returns {string|null}
+ */
+function getSessionId() {
+  return process.env.CLAUDE_SESSION_ID
+      || process.env.AI_SESSION_ID
+      || null;
+}
+
+/**
+ * Count entries in request-log.md
+ * @returns {number}
+ */
+function countRequestLogEntries() {
+  try {
+    const content = readFile(PATHS.requestLog, '');
+    const matches = content.match(/^### R-/gm);
+    return matches ? matches.length : 0;
+  } catch (_err) {
+    return 0;
+  }
+}
+
+/**
+ * Get the last request log entry (header line only)
+ * @returns {string|null}
+ */
+function getLastRequestLogEntry() {
+  try {
+    const content = readFile(PATHS.requestLog, '');
+    const matches = content.match(/^### R-.*$/gm);
+    return matches ? matches[matches.length - 1] : null;
+  } catch (_err) {
+    return null;
+  }
+}
+
+/**
+ * Get the highest request ID number from request-log.md.
+ * More robust than counting — handles gaps and deleted entries.
+ * @returns {number}
+ */
+function getHighestRequestId() {
+  try {
+    const content = readFile(PATHS.requestLog, '');
+    const matches = content.match(/### R-(\d{3,})/g);
+    if (!matches || matches.length === 0) return 0;
+
+    const numbers = matches.map(m => {
+      const num = m.match(/R-(\d+)/);
+      return num ? parseInt(num[1], 10) : 0;
+    });
+    return Math.max(...numbers);
+  } catch (_err) {
+    return 0;
+  }
+}
+
+/**
+ * Get next request ID (highest existing + 1, zero-padded to 3 digits).
+ * @returns {string} e.g., 'R-042'
+ */
+function getNextRequestId() {
+  const highestId = getHighestRequestId();
+  return `R-${String(highestId + 1).padStart(3, '0')}`;
+}
+
+/**
+ * Append an entry to request-log.md.
+ * @param {Object} entry
+ * @param {string} entry.type - new | fix | change | refactor
+ * @param {string[]} entry.tags - e.g., ['#figma', '#component:Button']
+ * @param {string} entry.request - What was requested
+ * @param {string} entry.result - What was done
+ * @param {string[]} [entry.files] - Files changed
+ * @param {string} [entry.sessionId] - CLI session ID (auto-detected if absent)
+ * @returns {string|null} Assigned R-ID, or null on failure.
+ */
+function addRequestLogEntry(entry) {
+  const { type, tags, request, result, files = [], sessionId } = entry;
+  const id = getNextRequestId();
+  const now = new Date();
+  const timestamp = now.toISOString().replace('T', ' ').substring(0, 16);
+
+  const session = sessionId || getSessionId();
+  const sessionLine = session ? `\n**Session**: ${session}` : '';
+
+  const filesLine = files.length > 0 ? `\n**Files**: ${files.join(', ')}` : '';
+  const tagsStr = tags.join(' ');
+
+  const logEntry = `
+### ${id} | ${timestamp}
+**Type**: ${type}
+**Tags**: ${tagsStr}${sessionLine}
+**Request**: "${request}"
+**Result**: ${result}${filesLine}
+`;
+
+  try {
+    fs.appendFileSync(PATHS.requestLog, logEntry);
+    return id;
+  } catch (err) {
+    error(`Failed to add request log entry: ${err.message}`);
+    return null;
+  }
+}
+
+module.exports = {
+  countRequestLogEntries,
+  getLastRequestLogEntry,
+  getHighestRequestId,
+  getNextRequestId,
+  addRequestLogEntry,
+};

package/scripts/flow-research-protocol.js

@@ -149,7 +149,6 @@ function getResearchConfig() {
       [DEPTHS.DEEP]: 50000,
       [DEPTHS.EXHAUSTIVE]: 100000
     },
-    requireCitations: research.requireCitations !== false,
     cacheVerifications: research.cacheVerifications !== false,
     cacheExpiryHours: research.cacheExpiryHours || 24,
     triggers: research.triggers || {

package/scripts/flow-revision-tracker.js

@@ -18,7 +18,8 @@ const {
   getConfig,
   PATHS,
   readJson,
-  fileExists
+  fileExists,
+  getTodayDate
 } = require('./flow-utils');
 const { recordRevision, loadStats } = require('./flow-stats-collector');
 

package/scripts/flow-roadmap.js

@@ -25,7 +25,8 @@ const {
   parseFlags,
   getConfig,
   isPathWithinProject,
-  escapeRegex
+  escapeRegex,
+  getTodayDate
 } = require('./flow-utils');
 const { success, warn, error, info, printHeader } = require('./flow-output');
 

package/scripts/flow-rules-sync.js

@@ -141,13 +141,9 @@ function parseMarkdownSections(content) {
  * @param {string} title - Section title
  * @returns {string} - Slugified filename
  */
-function slugify(title) {
-  return title
-    .toLowerCase()
-    .replace(/[^a-z0-9]+/g, '-')
-    .replace(/^-+|-+$/g, '')
-    .substring(0, 50); // Limit length
-}
+// Local slugify removed in favor of canonical flow-output slugify (wf-7072d3ac).
+const { slugify: _slugify } = require('./flow-output');
+const slugify = (title) => _slugify(title, { maxLength: 50 });
 
 /**
  * Determine path scope for a section based on title keywords

package/scripts/flow-session-end.js

@@ -12,6 +12,7 @@ const {
   spawnSync } = require('node:child_process');
 const readline = require('node:readline/promises');
 const path = require('node:path');
+const fs = require('node:fs');
 const {
   PATHS,
   PROJECT_ROOT,
@@ -23,7 +24,8 @@ const {
   writeFile,
   isGitRepo,
   getGitStatus,
-  safeJsonParse
+  safeJsonParse,
+  getTodayDate
 } = require('./flow-utils')
 const { color, printSection, success, warn, error } = require('./flow-output');;
 

package/scripts/flow-session-learning.js

@@ -14,6 +14,8 @@
 
 const fs = require('node:fs');
 const path = require('node:path');
+const { slugify: _slugify } = require('./flow-output');
+const slugify = (s) => _slugify(s, { maxLength: 40 });
 const {
   PATHS,
   fileExists,
@@ -23,7 +25,8 @@ const {
   success,
   info,
   warn,
-  safeJsonParse
+  safeJsonParse,
+  getTodayDate
 } = require('./flow-utils');
 
 // Import shared parsing functions from log manager (DRY - avoid duplication)
@@ -700,18 +703,8 @@ ${learning.description}
 // Utility Functions
 // ============================================================
 
-/**
- * Convert string to kebab-case slug
- */
-function slugify(str) {
-  return str
-    .toLowerCase()
-    .replace(/[^a-z0-9\s-]/g, '')
-    .replace(/\s+/g, '-')
-    .replace(/-+/g, '-')
-    .replace(/^-|-$/g, '')
-    .slice(0, 40);
-}
+// Local slugify removed in favor of canonical flow-output slugify (wf-7072d3ac).
+// Callers below now use `slugify(str, { maxLength: 40 })` imported at top.
 
 /**
  * Calculate confidence based on occurrences

package/scripts/flow-session-state.js

@@ -117,10 +117,10 @@ function loadSessionState() {
       ...getDefaultState(),
       ...state
     };
-  } catch (parseError) {
+  } catch (err) {
     // Log in debug mode to help diagnose issues
     if (process.env.DEBUG) {
-      console.warn(`[DEBUG] Could not parse session state: ${parseError.message}`);
+      console.warn(`[DEBUG] Could not parse session state: ${err.message}`);
     }
     return getDefaultState();
   }

package/scripts/flow-setup-hooks.js

@@ -17,7 +17,8 @@ const fs = require('node:fs');
 const path = require('node:path');
 const {
   getProjectRoot,
-  getConfig, PATHS
+  getConfig, PATHS,
+  getTodayDate
 } = require('./flow-utils')
 const { color, success, warn, error } = require('./flow-output');;
 

package/scripts/flow-skill-create.js

@@ -14,7 +14,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const readline = require('node:readline/promises');
-const { getProjectRoot, colors, PATHS } = require('./flow-utils');
+const { getProjectRoot, colors, PATHS, getTodayDate } = require('./flow-utils');
 const { success, warn, error: errorMsg, info, print } = require('./flow-output');
 const { getAllSkills, getSkillDir } = require('./flow-skill-matcher');
 

package/scripts/flow-skill-freshness.js

@@ -8,6 +8,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const { getConfig } = require('./flow-config-loader');
+const { getTodayDate } = require('./flow-output');
 
 // ============================================
 // CONFIGURATION
@@ -15,8 +16,8 @@ const { getConfig } = require('./flow-config-loader');
 
 const DEFAULT_FRESHNESS_THRESHOLD_DAYS = 90;
 
-// Dangerous keys that could cause prototype pollution
-const DANGEROUS_KEYS = new Set(['__proto__', 'constructor', 'prototype']);
+// Local DANGEROUS_KEYS consolidated to flow-io canonical (wf-2f6fbb12 / dup-002).
+const { DANGEROUS_KEYS } = require('./flow-io');
 
 // ============================================
 // FRESHNESS CHECKING
@@ -51,7 +52,7 @@ function parseFrontmatter(content) {
  * @returns {Array<{name: string, source: string, lastDocCheck: string|null, daysSinceCheck: number|null, isStale: boolean, prebuiltVersion: string|null}>}
  */
 function getSkillFreshnessReport(projectRoot) {
-  const config = loadConfig(projectRoot);
+  const config = getConfig();
   const threshold = config?.skills?.freshnessThreshold || DEFAULT_FRESHNESS_THRESHOLD_DAYS;
   const skillsDir = path.join(projectRoot, '.claude', 'skills');
 
@@ -160,9 +161,7 @@ function updateLastDocCheck(skillDir, date) {
 // CONFIG LOADING
 // ============================================
 
-function loadConfig(_projectRoot) {
-  return getConfig();
-}
+// loadConfig(_projectRoot) trivial wrapper removed — callers now use getConfig() directly (wf-7072d3ac / dup-001).
 
 // ============================================
 // CLI
@@ -197,7 +196,7 @@ function printFreshnessReport(report, threshold) {
 
 if (require.main === module) {
   const projectRoot = process.cwd();
-  const config = loadConfig(projectRoot);
+  const config = getConfig();
   const threshold = config?.skills?.freshnessThreshold || DEFAULT_FRESHNESS_THRESHOLD_DAYS;
 
   const args = process.argv.slice(2);

package/scripts/flow-skill-learn.js

@@ -20,7 +20,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const { execSync } = require('node:child_process');
-const { getProjectRoot, getConfig, colors, PATHS } = require('./flow-utils');
+const { getProjectRoot, getConfig, colors, PATHS, getTodayDate } = require('./flow-utils');
 const { getAllSkills, getSkillDir } = require('./flow-skill-matcher');
 
 const SKILLS_DIR = path.join(PATHS.root, '.claude', 'skills');

package/scripts/flow-step-coverage.js

@@ -10,7 +10,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const { execSync } = require('node:child_process');
-const { getProjectRoot, colors, getConfig, safeJsonParse, readJson, error } = require('./flow-utils');
+const { getProjectRoot, colors, getConfig, safeJsonParse, readJson, error, PATHS } = require('./flow-utils');
 const { getCommand, getExec } = require('./flow-script-resolver');
 
 // Common coverage output locations

package/scripts/flow-step-security.js

@@ -115,7 +115,7 @@ class SecurityStep extends BaseWorkflowStep {
                 });
               }
             }
-          } catch (_parseError) {
+          } catch (_err) {
             // Ignore parse errors
           }
         }

package/scripts/flow-story.js

@@ -283,16 +283,10 @@ function analyzeForDecomposition(title) {
  * @param {string} title - Title to slugify
  * @returns {string} Slugified title
  */
-function slugify(title) {
-  return title
-    .toLowerCase()
-    .trim()
-    .replace(/[^\w\s-]/g, '')     // Remove non-word chars (except spaces and hyphens)
-    .replace(/[\s_]+/g, '-')       // Replace spaces and underscores with hyphens
-    .replace(/-+/g, '-')           // Replace multiple hyphens with single
-    .replace(/^-+|-+$/g, '')       // Trim hyphens from start/end
-    .substring(0, 50);             // Limit length
-}
+// Local slugify removed in favor of canonical flow-output slugify (wf-7072d3ac).
+// 'word' mode preserves underscores (old behavior).
+const { slugify: _slugify } = require('./flow-output');
+const slugify = (title) => _slugify(title, { mode: 'word', maxLength: 50 });
 
 /**
  * Create story with optional deep decomposition
@@ -375,6 +369,46 @@ async function createStory(title, options = {}) {
     result.patterns = analysis.patterns;
   }
 
+  // Propagate simple (non-decomposed) stories to ready.json.
+  // wf-b5cff650: previously only decomposed stories reached the task queue;
+  // simple stories sat in .workflow/changes/ and were invisible to /wogi-start.
+  if (!shouldDecompose && !dryRun && fs.existsSync(READY_PATH)) {
+    try {
+      await withLock(READY_PATH, async () => {
+        const ready = safeJsonParse(READY_PATH, { ready: [] });
+        ready.ready = ready.ready || [];
+        // Idempotent: skip if this task ID is already present anywhere in the queue.
+        const allIds = [
+          ...(ready.ready || []),
+          ...(ready.inProgress || []),
+          ...(ready.blocked || []),
+          ...(ready.recentlyCompleted || []),
+          ...(ready.backlog || []),
+        ].map(t => t && t.id).filter(Boolean);
+        if (!allIds.includes(taskId)) {
+          ready.ready.push({
+            id: taskId,
+            title,
+            type: 'story',
+            level: 'L1',
+            status: 'ready',
+            priority,
+            created: new Date().toISOString(),
+            specPath: path.relative(PATHS.root, storyFile),
+          });
+          ready.lastUpdated = new Date().toISOString();
+          fs.writeFileSync(READY_PATH, JSON.stringify(ready, null, 2));
+        }
+      });
+      result.addedToReady = true;
+    } catch (err) {
+      result.addedToReady = false;
+      result.readyError = err.message;
+    }
+  } else if (!shouldDecompose && dryRun) {
+    result.wouldAddToReady = true;
+  }
+
   if (shouldDecompose && analysis.suggestedSubTasks.length > 0) {
     // Create sub-task files in feature folder
     let subNum = 1;
@@ -612,6 +646,20 @@ Examples:
     log('dim', `   Run: flow story "${title}" --deep`);
   }
 
+  // Simple (non-decomposed) story ready.json status — wf-b5cff650 fix
+  if (!result.decomposed) {
+    if (result.addedToReady) {
+      console.log('');
+      success('Added to ready.json (run: flow ready)');
+    } else if (result.wouldAddToReady) {
+      console.log('');
+      warn('[DRY RUN] Would add to ready.json');
+    } else if (result.readyError) {
+      console.log('');
+      warn(`Could not add to ready.json: ${result.readyError}`);
+    }
+  }
+
   console.log('');
   if (result.dryRun) {
     log('yellow', '━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');

package/scripts/flow-sys.js

@@ -0,0 +1,204 @@
+#!/usr/bin/env node
+
+/**
+ * Wogi Flow - System-Level Helpers
+ *
+ * CLI tool detection (fd, semver compare) + git repo utilities + app-map
+ * state-file operations. Extracted from flow-utils.js (wf-94cc3b72 epic
+ * — flow-utils decomposition).
+ */
+
+'use strict';
+
+const fs = require('node:fs');
+const path = require('node:path');
+const { execSync, execFileSync } = require('node:child_process');
+const { PATHS, PROJECT_ROOT } = require('./flow-paths');
+const { readFile, writeFile } = require('./flow-io');
+const { warn, error } = require('./flow-output');
+
+// ============================================================
+// CLI Tool Detection (Claude Code 2.1.72+ compatibility)
+// ============================================================
+
+/**
+ * Compare a parsed semver against a required minimum.
+ * @param {number} major - Parsed major
+ * @param {number} minor - Parsed minor
+ * @param {number} patch - Parsed patch
+ * @param {number} rMajor - Required major
+ * @param {number} rMinor - Required minor
+ * @param {number} rPatch - Required patch
+ * @returns {boolean}
+ */
+function meetsVersion(major, minor, patch, rMajor, rMinor, rPatch) {
+  return major > rMajor ||
+    (major === rMajor && minor > rMinor) ||
+    (major === rMajor && minor === rMinor && patch >= rPatch);
+}
+
+/**
+ * Detect if fd or fdfind is available on the system.
+ * fd/fdfind is auto-approved in Claude Code 2.1.72+ bash allowlist,
+ * making it a better choice than find for reduced permission prompts.
+ *
+ * Result is memoized for the process lifetime.
+ * @returns {string|false} The fd command name ('fd' or 'fdfind'), or false
+ */
+let _fdCommand = null;
+function getFdCommand() {
+  if (_fdCommand !== null) return _fdCommand;
+  for (const cmd of ['fd', 'fdfind']) {
+    try {
+      execFileSync(cmd, ['--version'], { stdio: 'pipe', timeout: 3000 });
+      _fdCommand = cmd;
+      return cmd;
+    } catch (_err) {
+      // Not available
+    }
+  }
+  _fdCommand = false;
+  return false;
+}
+
+// ============================================================
+// Git Operations
+// ============================================================
+
+/**
+ * Check if current directory is a git repo.
+ * Note: .git can be a directory (normal repo) or file (worktree).
+ */
+function isGitRepo() {
+  const gitPath = path.join(PROJECT_ROOT, '.git');
+  return fs.existsSync(gitPath);
+}
+
+/**
+ * Get git status info.
+ * @returns {{ isRepo: boolean, branch?: string, uncommitted?: number, clean?: boolean, error?: string }}
+ */
+function getGitStatus() {
+  if (!isGitRepo()) {
+    return { isRepo: false };
+  }
+
+  try {
+    const branch = execSync('git branch --show-current', {
+      encoding: 'utf-8',
+      stdio: ['pipe', 'pipe', 'pipe']
+    }).trim();
+
+    const status = execSync('git status --porcelain', {
+      encoding: 'utf-8',
+      stdio: ['pipe', 'pipe', 'pipe']
+    });
+
+    const uncommitted = status.split('\n').filter(Boolean).length;
+
+    return {
+      isRepo: true,
+      branch,
+      uncommitted,
+      clean: uncommitted === 0
+    };
+  } catch (err) {
+    return { isRepo: true, error: err.message };
+  }
+}
+
+// ============================================================
+// App Map Operations
+// ============================================================
+
+/**
+ * Count components in app-map.md (data rows only — excludes headers).
+ * @returns {number}
+ */
+function countAppMapComponents() {
+  try {
+    const content = readFile(PATHS.appMap, '');
+    const dataRows = content.match(/^\|[^-|][^|]*\|/gm);
+    const headerCount = (content.match(/^## /gm) || []).length * 1;
+    return dataRows ? Math.max(0, dataRows.length - headerCount) : 0;
+  } catch (_err) {
+    return 0;
+  }
+}
+
+/**
+ * Add a component row to app-map.md in the appropriate section.
+ * @param {Object} component
+ * @param {string} component.name
+ * @param {string} component.type - 'screen' | 'modal' | 'component' | 'layout'
+ * @param {string} component.path
+ * @param {string[]} [component.variants]
+ * @param {string} [component.description]
+ * @returns {boolean}
+ */
+function addAppMapComponent(component) {
+  const { name, type, path: filePath, variants = [], description = '' } = component;
+
+  try {
+    let content = readFile(PATHS.appMap, '');
+
+    const sectionMap = {
+      screen: '## Screens',
+      modal: '## Modals',
+      component: '## Components',
+      layout: '## Layouts'
+    };
+
+    const section = sectionMap[type] || '## Components';
+    const variantsStr = variants.length > 0 ? variants.join(', ') : '-';
+    const descStr = description || '-';
+
+    const newRow = `| ${name} | ${filePath} | ${variantsStr} | ${descStr} |`;
+
+    const sectionIndex = content.indexOf(section);
+    if (sectionIndex === -1) {
+      warn(`Section "${section}" not found in app-map.md`);
+      return false;
+    }
+
+    const nextSectionMatch = content.substring(sectionIndex + section.length).match(/\n## /);
+    const endIndex = nextSectionMatch
+      ? sectionIndex + section.length + nextSectionMatch.index
+      : content.length;
+
+    const sectionContent = content.substring(sectionIndex, endIndex);
+    const lastPipeIndex = sectionContent.lastIndexOf('\n|');
+
+    if (lastPipeIndex !== -1) {
+      const afterPipe = sectionContent.substring(lastPipeIndex);
+      const newlineOffset = afterPipe.indexOf('\n', 1);
+      const insertOffset = newlineOffset !== -1 ? newlineOffset : afterPipe.length;
+      const insertIndex = sectionIndex + lastPipeIndex + insertOffset;
+      content = content.substring(0, insertIndex) + '\n' + newRow + content.substring(insertIndex);
+    } else {
+      const headerEnd = sectionContent.indexOf('\n\n');
+      if (headerEnd !== -1) {
+        const insertIndex = sectionIndex + headerEnd;
+        content = content.substring(0, insertIndex) + '\n' + newRow + content.substring(insertIndex);
+      } else {
+        warn(`Could not find proper insertion point in section "${section}"`);
+        return false;
+      }
+    }
+
+    writeFile(PATHS.appMap, content);
+    return true;
+  } catch (err) {
+    error(`Failed to add component to app-map: ${err.message}`);
+    return false;
+  }
+}
+
+module.exports = {
+  meetsVersion,
+  getFdCommand,
+  isGitRepo,
+  getGitStatus,
+  countAppMapComponents,
+  addAppMapComponent,
+};