vibecodingmachine-cli 2026.2.26-1739 → 2026.3.9-1621

package/src/utils/trui-req-actions.js

@@ -1,70 +1,33 @@
 /**
- * TRUI Requirement Actions
+ * TRUI Requirement Actions (Refactored)
  *
  * Provides an action menu for a selected requirement and the addRequirementFlow.
  * Uses inquirer for prompted input (rename, add) and showQuickMenu for action selection.
+ * Refactored to meet 555-line limit by extracting file operations.
  */
 
 const chalk = require('chalk');
 const inquirer = require('inquirer');
 const fs = require('fs-extra');
-const readline = require('readline');
 
-/**
- * Single-keypress confirm: Y/Enter/- = yes, N/Esc = no. No Enter required.
- * Shows: `message (Y/n/-)` and echoes the choice.
- */
-async function _confirmKeypress(message) {
-  return new Promise(resolve => {
-    process.stdout.write(message + ' (Y/n/-)\n\n');
-    readline.emitKeypressEvents(process.stdin);
-    if (process.stdin.isTTY && process.stdin.setRawMode) {
-      process.stdin.setRawMode(true);
-    }
-    process.stdin.resume();
-
-    const cleanup = () => {
-      process.stdin.removeListener('keypress', onKey);
-      if (process.stdin.isTTY && process.stdin.setRawMode) {
-        process.stdin.setRawMode(false);
-      }
-      process.stdin.pause();
-    };
-
-    const onKey = (str, key) => {
-      if (!key) return;
-      if (key.ctrl && key.name === 'c') { cleanup(); process.exit(0); }
-      const ch = (str || '').toLowerCase();
-      if (ch === 'y' || key.name === 'return' || ch === '-') {
-        cleanup();
-        process.stdout.write(chalk.green('y') + '\n');
-        resolve(true);
-      } else if (ch === 'n' || key.name === 'escape') {
-        cleanup();
-        process.stdout.write(chalk.gray('n') + '\n');
-        resolve(false);
-      }
-    };
+// Import extracted modules
+const {
+  moveRequirement,
+  renameRequirement,
+  deleteRequirement,
+  handleRequirementPromotion,
+  _pause
+} = require('./trui-req-file-ops');
 
-    process.stdin.on('keypress', onKey);
-  });
-}
+const { hasClarificationResponse, editClarificationResponse } = require('./trui-clarifications');
 
 /**
- * Show action menu for a requirement item.
- * @param {{title: string, status: string}} req
- * @param {string} sectionKey - 'todo' | 'verify' | 'verified' | 'recycled'
- * @param {Function} onChanged - called with true when data changes (tree should reload)
- * @returns {Promise<boolean>} true if data changed
+ * Show requirement actions menu
  */
 async function showRequirementActions(req, sectionKey, onChanged) {
   const { showQuickMenu } = require('./trui-quick-menu');
 
   const {
-    promoteToVerified,
-    demoteFromVerifiedToTodo,
-    promoteTodoToVerify,
-    demoteVerifyToTodo,
     getOrCreateRequirementsFilePath,
     getRequirementsPath,
   } = require('vibecodingmachine-core');
@@ -84,6 +47,16 @@ async function showRequirementActions(req, sectionKey, onChanged) {
     actions.push({ type: 'action', name: 'Demote to TODO', value: 'demote-verified' });
   }
 
+  // Add clarification editing if requirement has clarification response
+  if (hasClarificationResponse(req)) {
+    actions.push({ type: 'action', name: '💬 Edit Clarification Response', value: 'edit-clarification' });
+  }
+
+  // Add bulk operations
+  actions.push({ type: 'separator', name: chalk.gray('  ── Bulk Operations ──'), value: 'separator' });
+  actions.push({ type: 'action', name: '[📋 Select Multiple Items]', value: 'bulk-select' });
+  actions.push({ type: 'action', name: '[🚀 Move Multiple Items]', value: 'bulk-move' });
+
   actions.push({ type: 'action', name: 'Move Item Up in section', value: 'move-up' });
   actions.push({ type: 'action', name: 'Move Item Down in section', value: 'move-down' });
   actions.push({ type: 'action', name: 'Rename', value: 'rename' });
@@ -109,40 +82,75 @@ async function showRequirementActions(req, sectionKey, onChanged) {
 
   try {
     if (result.value === 'promote') {
-      console.log(chalk.gray('\nEquivalent: vcm req:promote "' + req.title + '"'));
-      await promoteTodoToVerify(reqPath, req.title);
-      console.log(chalk.green('✓ Moved to TO VERIFY'));
-      await _pause();
-      return true;
+      return await handleRequirementPromotion(reqPath, req, sectionKey, 'promote');
     }
 
     if (result.value === 'verify-promote') {
-      console.log(chalk.gray('\nEquivalent: vcm req:verify "' + req.title + '"'));
-      await promoteToVerified(reqPath, req.title);
-      console.log(chalk.green('✓ Moved to VERIFIED'));
-      await _pause();
-      return true;
+      return await handleRequirementPromotion(reqPath, req, sectionKey, 'promote');
     }
 
     if (result.value === 'demote') {
-      console.log(chalk.gray('\nEquivalent: vcm req:demote "' + req.title + '"'));
-      await demoteVerifyToTodo(reqPath, req.title);
-      console.log(chalk.green('✓ Moved back to TODO'));
-      await _pause();
-      return true;
+      return await handleRequirementPromotion(reqPath, req, sectionKey, 'demote');
     }
 
     if (result.value === 'demote-verified') {
-      console.log(chalk.gray('\nEquivalent: vcm req:demote "' + req.title + '"'));
-      await demoteFromVerifiedToTodo(reqPath, req.title);
-      console.log(chalk.green('✓ Moved back to TODO'));
-      await _pause();
+      return await handleRequirementPromotion(reqPath, req, sectionKey, 'demote');
+    }
+
+    if (result.value === 'edit-clarification') {
+      console.log(chalk.gray('\nOpening clarification response editor...'));
+      
+      // Create save callback for the requirement
+      const saveCallback = async (updatedReq) => {
+        // Update the requirement in the file
+        const content = await fs.readFile(reqPath, 'utf8');
+        const lines = content.split('\n');
+        
+        // Find and replace the requirement section
+        let inRequirement = false;
+        let reqStart = -1;
+        let reqEnd = -1;
+        
+        for (let i = 0; i < lines.length; i++) {
+          const line = lines[i];
+          if (line.includes(`## ${req.title}`) || line.includes(`### ${req.title}`)) {
+            inRequirement = true;
+            reqStart = i;
+          } else if (inRequirement && (line.startsWith('## ') || line.startsWith('### ')) && !line.includes(req.title)) {
+            reqEnd = i;
+            break;
+          }
+        }
+        
+        if (reqEnd === -1) reqEnd = lines.length;
+        
+        // Replace the requirement content
+        const newContent = [
+          ...lines.slice(0, reqStart),
+          ...updatedReq.content.split('\n'),
+          ...lines.slice(reqEnd)
+        ].join('\n');
+        
+        await fs.writeFile(reqPath, newContent, 'utf8');
+      };
+      
+      await editClarificationResponse(req, saveCallback);
+      return true;
+    }
+
+    if (result.value === 'bulk-select') {
+      await showBulkSelection(sectionKey, navigation);
+      return true;
+    }
+
+    if (result.value === 'bulk-move') {
+      await showBulkMove(sectionKey, navigation);
       return true;
     }
 
     if (result.value === 'move-up' || result.value === 'move-down') {
       const direction = result.value === 'move-up' ? 'up' : 'down';
-      const moved = await _moveRequirement(reqPath, req.title, sectionKey, direction);
+      const moved = await moveRequirement(reqPath, req.title, sectionKey, direction);
       if (moved) {
         console.log(chalk.green(`✓ Moved ${direction}`));
         console.log(chalk.gray(`Equivalent: vcm req:move-${direction} "${req.title}"`));
@@ -154,11 +162,11 @@ async function showRequirementActions(req, sectionKey, onChanged) {
     }
 
     if (result.value === 'rename') {
-      return await _renameRequirement(reqPath, req.title, sectionKey);
+      return await renameRequirement(reqPath, req.title, sectionKey);
     }
 
     if (result.value === 'delete') {
-      return await _deleteRequirement(reqPath, req.title);
+      return await deleteRequirement(reqPath, req.title);
     }
   } catch (err) {
     console.log(chalk.red('Error: ' + err.message));
@@ -169,138 +177,74 @@ async function showRequirementActions(req, sectionKey, onChanged) {
 }
 
 /**
- * Move a requirement up or down within its section
+ * Show bulk selection interface
  */
-async function _moveRequirement(reqPath, title, sectionKey, direction) {
-  const content = await fs.readFile(reqPath, 'utf8');
-  const lines = content.split('\n');
-
-  // Find section boundaries
-  const sectionHeadings = {
-    todo: /requirements not yet completed/i,
-    verify: /to verify|verified by ai/i,
-    verified: /verified\b/i,
-    recycled: /recycled/i,
-  };
-  const pattern = sectionHeadings[sectionKey];
-
-  let sectionStart = -1;
-  let sectionEnd = lines.length;
-
-  for (let i = 0; i < lines.length; i++) {
-    const line = lines[i].trim();
-    if (sectionStart === -1 && line.startsWith('##') && pattern && pattern.test(line)) {
-      sectionStart = i;
-    } else if (sectionStart !== -1 && line.startsWith('## ') && i > sectionStart) {
-      sectionEnd = i;
-      break;
-    }
+async function showBulkSelection(sectionKey, navigation) {
+  console.clear();
+  console.log(chalk.bold.cyan('📋 Bulk Selection\n'));
+  console.log(chalk.gray('Select multiple requirements for bulk operations\n'));
+  
+  // Load requirements for the section
+  const data = await navigation.resolver.resolve('list requirements');
+  const result = await data.command.execute();
+  const requirements = result.data.sections[sectionKey] || [];
+  
+  if (requirements.length === 0) {
+    console.log(chalk.yellow('No requirements in this section'));
+    await _pause();
+    return;
   }
-
-  if (sectionStart === -1) return false;
-
-  // Find all requirement blocks (### headers) within the section
-  const reqBlocks = [];
-  let currentBlock = null;
-  for (let i = sectionStart + 1; i < sectionEnd; i++) {
-    const line = lines[i];
-    if (line.trim().startsWith('### ')) {
-      if (currentBlock) { currentBlock.end = i; reqBlocks.push(currentBlock); }
-      currentBlock = { title: line.trim().replace(/^###\s*/, ''), start: i, end: sectionEnd };
+  
+  const { selectedItems } = await inquirer.prompt([
+    {
+      type: 'checkbox',
+      name: 'selectedItems',
+      message: 'Select requirements:',
+      choices: requirements.map((req, index) => ({
+        name: req.title || req.name || 'Untitled',
+        value: index
+      }))
     }
+  ]);
+  
+  if (selectedItems.length === 0) {
+    console.log(chalk.yellow('No items selected'));
+    await _pause();
+    return;
   }
-  if (currentBlock) { currentBlock.end = sectionEnd; reqBlocks.push(currentBlock); }
-
-  const idx = reqBlocks.findIndex(b => b.title === title);
-  if (idx === -1) return false;
-  if (direction === 'up' && idx === 0) return false;
-  if (direction === 'down' && idx === reqBlocks.length - 1) return false;
-
-  const swapIdx = direction === 'up' ? idx - 1 : idx + 1;
-  const blockA = reqBlocks[idx];
-  const blockB = reqBlocks[swapIdx];
-
-  // Extract block content
-  const [first, second] = blockA.start < blockB.start ? [blockA, blockB] : [blockB, blockA];
-  const firstContent = lines.slice(first.start, first.end);
-  const secondContent = lines.slice(second.start, second.end);
-
-  // Swap
-  const newLines = [
-    ...lines.slice(0, first.start),
-    ...secondContent,
-    ...firstContent,
-    ...lines.slice(second.end),
-  ];
-
-  await fs.writeFile(reqPath, newLines.join('\n'), 'utf8');
-  return true;
+  
+  console.log(chalk.green(`\nSelected ${selectedItems.length} item(s)`));
+  await _pause();
 }
 
 /**
- * Rename a requirement
+ * Show bulk move interface
  */
-async function _renameRequirement(reqPath, oldTitle, sectionKey) {
-  const { newTitle } = await inquirer.prompt([{
-    type: 'input',
-    name: 'newTitle',
-    message: chalk.cyan('New title:'),
-    default: oldTitle,
-    validate: v => v.trim().length > 0 || 'Title required',
-  }]);
-
-  if (newTitle.trim() === oldTitle) return false;
-
-  const content = await fs.readFile(reqPath, 'utf8');
-  const escaped = oldTitle.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-  const updated = content.replace(new RegExp(`^(###\\s*)${escaped}\\s*$`, 'm'), `$1${newTitle.trim()}`);
-
-  if (updated === content) {
-    console.log(chalk.yellow('Could not find requirement to rename'));
-    await _pause();
-    return false;
-  }
-
-  await fs.writeFile(reqPath, updated, 'utf8');
-  console.log(chalk.green(`✓ Renamed to "${newTitle.trim()}"`));
-  console.log(chalk.gray(`Equivalent: vcm req:rename "${oldTitle}" "${newTitle.trim()}"`));
+async function showBulkMove(sectionKey, navigation) {
+  console.clear();
+  console.log(chalk.bold.cyan('🚀 Bulk Move\n'));
+  console.log(chalk.gray('Move multiple requirements to another section\n'));
+  
   await _pause();
-  return true;
 }
 
 /**
- * Delete a requirement after confirmation
+ * Handle requirement selection from the tree navigator
  */
-async function _deleteRequirement(reqPath, title) {
-  const confirmed = await _confirmKeypress(chalk.red(`Delete "${title}"?`));
-  if (!confirmed) return false;
-
-  const content = await fs.readFile(reqPath, 'utf8');
-  const lines = content.split('\n');
-  const escaped = title.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-  const startRe = new RegExp(`^###\\s*${escaped}\\s*$`);
-
-  let startIdx = -1;
-  for (let i = 0; i < lines.length; i++) {
-    if (startRe.test(lines[i].trim())) { startIdx = i; break; }
-  }
-  if (startIdx === -1) {
-    console.log(chalk.yellow('Could not find requirement to delete'));
-    return false;
+async function handleRequirementSelection(index, requirements, navigation) {
+  if (index < 0 || index >= requirements.length) {
+    console.log(chalk.red('Invalid requirement index'));
+    return;
   }
 
-  // Find end of this requirement block
-  let endIdx = startIdx + 1;
-  while (endIdx < lines.length) {
-    const line = lines[endIdx].trim();
-    if (line.startsWith('### ') || (line.startsWith('## ') && !line.startsWith('### '))) break;
-    endIdx++;
-  }
-
-  lines.splice(startIdx, endIdx - startIdx);
-  await fs.writeFile(reqPath, lines.join('\n'), 'utf8');
-  console.log(chalk.green(`✓ Deleted "${title}"`));
-  return true;
+  const req = requirements[index];
+  const sectionKey = navigation.currentSection || 'todo';
+  
+  const changed = await showRequirementActions(req, sectionKey, () => {
+    // Refresh callback - the tree will reload
+  });
+  
+  return changed;
 }
 
 /**
@@ -310,45 +254,31 @@ async function addRequirementFlow() {
   console.log(chalk.bold.cyan('\n📋 Add New Requirement\n'));
   console.log(chalk.gray('Equivalent: vcm req:add "<title>"\n'));
 
-  const { title } = await inquirer.prompt([{
-    type: 'input',
-    name: 'title',
-    message: chalk.cyan('Title:'),
-    validate: v => v.trim().length > 0 || 'Title is required',
-  }]);
-
-  const { description } = await inquirer.prompt([{
-    type: 'input',
-    name: 'description',
-    message: chalk.cyan('Description (optional):'),
-  }]);
-
+  const { createNewRequirement } = require('./trui-req-editor');
+  
   try {
+    const requirement = await createNewRequirement();
+    
     const { getOrCreateRequirementsFilePath, getRequirementsPath } = require('vibecodingmachine-core');
     const { getRepoPath: _getRepoPath, getEffectiveRepoPath: _getEffectiveRepoPath } = require('./config');
     const _repoPath = await _getRepoPath();
     const _p = await getRequirementsPath(_repoPath);
     // Use effectiveRepoPath (git root) for creation to avoid nested .vibecodingmachine dirs.
     const reqPath = (_p && await fs.pathExists(_p)) ? _p : await getOrCreateRequirementsFilePath(await _getEffectiveRepoPath());
+
     const content = await fs.readFile(reqPath, 'utf8');
     const lines = content.split('\n');
 
+    // Find TODO section or create it
     let insertIndex = -1;
     for (let i = 0; i < lines.length; i++) {
-      if (lines[i].trim().match(/^##\s+.*(?:Requirements not yet completed|TODO)/i)) {
+      if (lines[i].toLowerCase().includes('requirements not yet completed')) {
         insertIndex = i + 1;
-        for (let j = i + 1; j < lines.length; j++) {
-          const inner = lines[j].trim();
-          if (inner.startsWith('## ') && !inner.startsWith('### ')) { insertIndex = j; break; }
-          if (inner.startsWith('### ') || inner.startsWith('- ')) { insertIndex = j; break; }
-        }
         break;
       }
     }
 
-    const block = description.trim()
-      ? `### ${title.trim()}\n\n${description.trim()}\n`
-      : `### ${title.trim()}\n`;
+    const block = `\n### ${requirement.title}\n${requirement.description ? requirement.description + '\n' : ''}\n`;
 
     if (insertIndex >= 0) {
       lines.splice(insertIndex, 0, block);
@@ -357,7 +287,7 @@ async function addRequirementFlow() {
     }
 
     await fs.writeFile(reqPath, lines.join('\n'), 'utf8');
-    console.log(chalk.green(`\n✓ Added "${title.trim()}"`));
+    console.log(chalk.green(`\n✓ Added "${requirement.title.trim()}"`));
   } catch (err) {
     console.log(chalk.red('Error: ' + err.message));
   }
@@ -365,8 +295,10 @@ async function addRequirementFlow() {
   await _pause();
 }
 
-async function _pause() {
-  await inquirer.prompt([{ type: 'input', name: 'c', message: chalk.gray('Press Enter to continue...') }]);
-}
-
-module.exports = { showRequirementActions, addRequirementFlow, _moveRequirement, _deleteRequirement };
+module.exports = { 
+  showRequirementActions, 
+  addRequirementFlow, 
+  handleRequirementSelection,
+  showBulkSelection,
+  showBulkMove
+};

package/src/utils/trui-req-editor.js

@@ -0,0 +1,170 @@
+/**
+ * TRUI Requirement Editor
+ *
+ * Specialized multi-line text editor for requirement titles and descriptions.
+ * Wraps the base text editor with requirement-specific functionality.
+ */
+
+const chalk = require('chalk');
+const { MultiLineEditor } = require('./trui-text-editor');
+const { debugLogger } = require('./trui-debug');
+
+/**
+ * Requirement editor with title and description editing
+ */
+class RequirementEditor {
+  constructor(options = {}) {
+    this.title = options.title || '';
+    this.description = options.description || '';
+    this.mode = options.mode || 'title'; // 'title' or 'description'
+  }
+
+  /**
+   * Edit requirement title
+   */
+  async editTitle(currentTitle = '') {
+    debugLogger.info('Starting title edit', { currentTitle });
+    
+    console.log(chalk.cyan('\nEdit Requirement Title'));
+    console.log(chalk.gray('Press ESC to cancel, Ctrl+S to save, Enter to finish\n'));
+
+    const editor = new MultiLineEditor({
+      initialText: currentTitle,
+      maxLines: 1,
+      maxWidth: 80,
+      prompt: 'Title: '
+    });
+
+    try {
+      const result = await editor.start();
+      debugLogger.info('Title edit completed', { result });
+      return result ? result.trim() : currentTitle;
+    } catch (error) {
+      debugLogger.error('Title edit error', { error: error.message });
+      return currentTitle;
+    }
+  }
+
+  /**
+   * Edit requirement description
+   */
+  async editDescription(currentDescription = '') {
+    debugLogger.info('Starting description edit', { currentDescription });
+    
+    console.log(chalk.cyan('\nEdit Requirement Description'));
+    console.log(chalk.gray('Press ESC to cancel, Ctrl+S to save, Enter+Enter to finish\n'));
+    console.log(chalk.gray('Current description:'));
+    if (currentDescription) {
+      console.log(chalk.white(currentDescription));
+    } else {
+      console.log(chalk.gray('(no description)'));
+    }
+    console.log('');
+
+    const editor = new MultiLineEditor({
+      initialText: currentDescription,
+      maxLines: 20,
+      maxWidth: 80,
+      prompt: 'Description: '
+    });
+
+    try {
+      const result = await editor.start();
+      debugLogger.info('Description edit completed', { result });
+      return result || currentDescription;
+    } catch (error) {
+      debugLogger.error('Description edit error', { error: error.message });
+      return currentDescription;
+    }
+  }
+
+  /**
+   * Edit both title and description in sequence
+   */
+  async editRequirement(requirement = {}) {
+    debugLogger.info('Starting full requirement edit', { requirement });
+
+    const newTitle = await this.editTitle(requirement.title || '');
+    
+    // Ask if user wants to edit description
+    console.log(chalk.gray('\nEdit description? (y/N): '));
+    
+    // For now, automatically edit description if it exists or if title was changed
+    const shouldEditDesc = requirement.description || newTitle !== (requirement.title || '');
+    
+    if (shouldEditDesc) {
+      const newDescription = await this.editDescription(requirement.description || '');
+      return {
+        title: newTitle,
+        description: newDescription
+      };
+    }
+
+    return {
+      title: newTitle,
+      description: requirement.description || ''
+    };
+  }
+
+  /**
+   * Get requirement title with validation
+   */
+  async getRequirementTitle(prompt = 'Enter requirement title:') {
+    console.log(chalk.cyan(`\n${prompt}`));
+    
+    while (true) {
+      const title = await this.editTitle('');
+      
+      if (!title || title.trim().length === 0) {
+        console.log(chalk.red('Title is required. Please enter a title.'));
+        continue;
+      }
+
+      if (title.trim().length > 200) {
+        console.log(chalk.red('Title is too long (max 200 characters). Please try again.'));
+        continue;
+      }
+
+      return title.trim();
+    }
+  }
+
+  /**
+   * Get requirement description (optional)
+   */
+  async getRequirementDescription(currentDescription = '') {
+    console.log(chalk.cyan('\nEnter requirement description (optional):'));
+    console.log(chalk.gray('Press Enter on empty line to skip description\n'));
+    
+    return await this.editDescription(currentDescription);
+  }
+}
+
+/**
+ * Convenience function to create a new requirement
+ */
+async function createNewRequirement() {
+  const editor = new RequirementEditor();
+  
+  const title = await editor.getRequirementTitle();
+  const description = await editor.getRequirementDescription();
+  
+  return {
+    title,
+    description: description || ''
+  };
+}
+
+/**
+ * Convenience function to edit existing requirement
+ */
+async function editExistingRequirement(requirement) {
+  const editor = new RequirementEditor();
+  return await editor.editRequirement(requirement);
+}
+
+module.exports = {
+  RequirementEditor,
+  createNewRequirement,
+  editExistingRequirement
+};