vibecodingmachine-cli 2026.3.10-1807 → 2026.3.14-1528

package/src/utils/provider-checker/provider-validator.js

@@ -4,7 +4,7 @@ const fs = require('fs');
 const path = require('path');
 
 const { CLI_AUTO_INSTALL, isCLIAvailable, installCLI } = require('./cli-installer');
-const { getResultFilePath, addTestRequirement, removeTestRequirement } = require('./requirements-manager');
+const { getResultFilePath } = require('./agents-manager');
 const { runAgentCheck } = require('./agent-runner');
 
 /**
@@ -28,21 +28,11 @@ async function checkProvider(providerId, config = {}, repoPath, onProgress = nul
     return { status: 'error', message: `Unknown provider: ${providerId}`, checkedAt: new Date().toISOString(), requirementLeftPending: false };
   }
 
-  // No platform restrictions needed anymore - all IDEs have cross-platform support
-
   if (!repoPath) {
     console.error(`[AGENT CHECK] No repository path available`);
     return { status: 'error', message: 'No repository path available', checkedAt: new Date().toISOString(), requirementLeftPending: false };
   }
 
-  const { getRequirementsPath } = require('vibecodingmachine-core');
-  let reqPath;
-  try {
-    reqPath = await getRequirementsPath(repoPath);
-  } catch {
-    reqPath = path.join(repoPath, '.vibecodingmachine', 'REQUIREMENTS.md');
-  }
-
   // Ensure .vibecodingmachine directory exists
   const vibeDir = path.join(repoPath, '.vibecodingmachine');
   const tempDir = path.join(vibeDir, 'temp');
@@ -55,26 +45,6 @@ async function checkProvider(providerId, config = {}, repoPath, onProgress = nul
     return { status: 'error', message: `Could not create .vibecodingmachine: ${err.message}`, checkedAt: new Date().toISOString(), requirementLeftPending: false };
   }
 
-  // Ensure REQUIREMENTS.md exists with proper structure
-  try {
-    if (!fs.existsSync(reqPath)) {
-      const initialRequirements = `# Requirements for this project
-
-## ⏳ Requirements not yet completed
-
-## ✅ Verified by AI
-
-`;
-      fs.writeFileSync(reqPath, initialRequirements, 'utf8');
-      console.log(`[AGENT CHECK] Created initial REQUIREMENTS.md`);
-    } else {
-      console.log(`[AGENT CHECK] REQUIREMENTS.md already exists`);
-    }
-  } catch (err) {
-    console.error(`[AGENT CHECK] Failed to ensure REQUIREMENTS.md: ${err.message}`);
-    return { status: 'error', message: `Could not create REQUIREMENTS.md: ${err.message}`, checkedAt: new Date().toISOString(), requirementLeftPending: false };
-  }
-
   const resultFile = getResultFilePath(repoPath);
 
   // Special handling: VS Code Copilot CLI is often installed but not authenticated.
@@ -235,11 +205,6 @@ async function checkProvider(providerId, config = {}, repoPath, onProgress = nul
     }
   }
 
-  // Add test requirement (idempotent — safe to call even if already present from a prior failed agent)
-  try { addTestRequirement(reqPath, resultFile); } catch (err) {
-    return { status: 'error', message: `Could not write to REQUIREMENTS.md: ${err.message}`, checkedAt: new Date().toISOString(), requirementLeftPending: false };
-  }
-
   // IDE providers generally don't need CLI system configuration to work
   // They can communicate directly with the IDE without requiring VCM CLI setup
 
@@ -248,20 +213,17 @@ async function checkProvider(providerId, config = {}, repoPath, onProgress = nul
   const result = await runAgentCheck(providerId, def, repoPath, resultFile, timeoutMs, signal, onProgress);
 
   if (result.status === 'success') {
-    // Agent completed — requirement has been moved to Verified; remove it and the result file
-    try { removeTestRequirement(reqPath); } catch { }
+    // Agent completed — clean up result file
     try { fs.unlinkSync(resultFile); } catch { }
     return { ...result, requirementLeftPending: false };
   }
 
-  // Agent failed — leave the requirement in pending for the next agent
-  return { ...result, requirementLeftPending: true };
+  // Agent failed
+  return { ...result, requirementLeftPending: false };
 }
 
 /**
  * Check all providers sequentially.
- * The test requirement is shared: it stays pending between agents until one
- * succeeds, then it's removed. If all fail, it's removed at the end.
  *
  * Returns { [providerId]: checkResult }
  */
@@ -275,14 +237,8 @@ async function checkAllProviders(providerIds, config = {}, repoPath, onProgress
     if (onProgress) onProgress(id, 'done', results[id].status === 'error' ? results[id].message : null);
   }
 
-  // Final cleanup — remove requirement if still pending (all agents failed or skipped)
+  // Final cleanup — remove result file if it exists
   if (repoPath) {
-    const { getRequirementsPath } = require('vibecodingmachine-core');
-    let reqPath;
-    try { reqPath = await getRequirementsPath(repoPath); } catch {
-      reqPath = path.join(repoPath, '.vibecodingmachine', 'REQUIREMENTS.md');
-    }
-    try { removeTestRequirement(reqPath); } catch { }
     try { fs.unlinkSync(getResultFilePath(repoPath)); } catch { }
   }
 

package/src/utils/provider-checker/requirements-manager.js

@@ -3,10 +3,7 @@
 const fs = require('fs');
 const path = require('path');
 
-const TEST_REQ_TITLE = 'VCM agent connectivity check';
-const PENDING_HEADER = '## ⏳ Requirements not yet completed';
-const IN_PROGRESS_HEADER = '## 🔨 Current In Progress Requirement';
-const VERIFIED_HEADER = '## ✅ Verified by AI';
+const TEST_REQ_TITLE = '✅ ACTION REQUIRED - VCM Connectivity Test';
 
 /**
  * Return the absolute path to the result file used by the check.
@@ -16,94 +13,118 @@ function getResultFilePath(repoPath) {
 }
 
 /**
- * Add the test requirement as the FIRST item in the pending section.
- * The first item in the "Requirements not yet completed" section is the current in-progress item.
- * Idempotent — if it's already there, do nothing.
+ * Add test requirement to requirements file
  */
-function addTestRequirement(reqPath, resultFile) {
-  let content = '';
-  try { content = fs.readFileSync(reqPath, 'utf8'); } catch {}
+function addTestRequirement(reqPath, resultFilePath) {
+  if (!fs.existsSync(reqPath)) {
+    // Create the file with basic structure if it doesn't exist
+    const basicContent = `## Requirements
 
-  // Already present?
-  if (content.includes(TEST_REQ_TITLE)) return;
+`;
+    fs.writeFileSync(reqPath, basicContent, 'utf8');
+  }
 
-  const testBlock = `- ${TEST_REQ_TITLE}
-🚦 Current Status: [PREPARE]
+  const content = fs.readFileSync(reqPath, 'utf8');
+  
+  // Check if test requirement already exists
+  if (content.includes(TEST_REQ_TITLE)) {
+    return;
+  }
 
-**AUTOMATED CONNECTIVITY TEST - Please read and acknowledge:**
+  const testRequirement = `- ${TEST_REQ_TITLE}
+🚦 Status: [PREPARE]
 
-This is a simple test to verify VCM can communicate with your IDE.
+**✅ ACTION REQUIRED - VCM Connectivity Test:**
 
-**To complete this test, update the status above to [DONE]:**
-- Change \`🚦 Current Status: [PREPARE]\` to \`🚦 Current Status: [DONE]\`
+VCM is testing if it can communicate with your IDE. To complete this test:
 
-That's it! VCM will detect the status change and mark the connectivity test as successful.
+**Change the status above from [PREPARE] to [DONE] and save the file.**
 
-**Alternative (if you want to create a file):**
-1. Create file: \`${resultFile}\`
-2. Write: \`VCM_CHECK_OK\`
-3. Save
+Example:
+  Before: 🚦 Status: [PREPARE]
+  After:  🚦 Status: [DONE]
 
-Either method works - status update is simplest.
+That's it! VCM will detect the status change within seconds.
 
 `;
 
-  const idx = content.indexOf(PENDING_HEADER);
-  if (idx !== -1) {
-    const afterHeader = idx + PENDING_HEADER.length;
-    // Insert after the header, then existing content follows
-    const before = content.slice(0, afterHeader);
-    const after = content.slice(afterHeader).replace(/^\n+/, '');
-    content = before + '\n\n' + testBlock + after;
+  // Find the Requirements section and add the test requirement
+  const requirementsMatch = content.match(/## Requirements\s*\n/);
+  if (requirementsMatch) {
+    const insertIndex = requirementsMatch.index + requirementsMatch[0].length;
+    const updatedContent = content.slice(0, insertIndex) + testRequirement + content.slice(insertIndex);
+    fs.writeFileSync(reqPath, updatedContent, 'utf8');
   } else {
-    // No pending section — prepend one
-    content = PENDING_HEADER + '\n\n' + testBlock + '\n' + (content || VERIFIED_HEADER + '\n');
+    // If no Requirements section found, append to end
+    const updatedContent = content + testRequirement;
+    fs.writeFileSync(reqPath, updatedContent, 'utf8');
   }
-
-  fs.writeFileSync(reqPath, content, 'utf8');
 }
 
 /**
- * Remove the test requirement from REQUIREMENTS.md regardless of which section it's in.
- * Also cleans up triple-blank lines left behind.
+ * Remove test requirement from requirements file
  */
 function removeTestRequirement(reqPath) {
-  let content = '';
-  try { content = fs.readFileSync(reqPath, 'utf8'); } catch { return; }
-
-  if (!content.includes(TEST_REQ_TITLE)) return;
-
-  const lines = content.split('\n');
-
-  // Find the ### header line for this requirement
-  let blockStart = -1;
-  for (let i = 0; i < lines.length; i++) {
-    if (lines[i].includes(TEST_REQ_TITLE) || (lines[i].startsWith('###') && i + 1 < lines.length && lines[i + 1].includes(TEST_REQ_TITLE))) {
-      // Walk back to find the ### line
-      blockStart = lines[i].startsWith('###') ? i : i - 1;
-      if (blockStart < 0) blockStart = i;
-      break;
-    }
+  if (!fs.existsSync(reqPath)) {
+    return;
+  }
+
+  const content = fs.readFileSync(reqPath, 'utf8');
+  
+  // Find the test requirement section
+  const startIndex = content.indexOf(`- ${TEST_REQ_TITLE}`);
+  if (startIndex === -1) {
+    return;
+  }
+
+  // Find the end of the test requirement (next requirement or end of file)
+  let endIndex = content.indexOf('\n- ', startIndex + 1);
+  if (endIndex === -1) {
+    endIndex = content.length;
   }
 
-  if (blockStart === -1) return;
+  // Remove the test requirement and clean up extra blank lines
+  const beforeTest = content.slice(0, startIndex);
+  const afterTest = content.slice(endIndex);
+  
+  // Clean up triple blank lines
+  const cleanedContent = (beforeTest + afterTest).replace(/\n{3,}/g, '\n\n');
+  
+  fs.writeFileSync(reqPath, cleanedContent, 'utf8');
+}
+
+/**
+ * Extract test requirement from requirements file
+ */
+function extractTestRequirement(reqPath) {
+  if (!fs.existsSync(reqPath)) {
+    return null;
+  }
+
+  const content = fs.readFileSync(reqPath, 'utf8');
+  
+  // Find the test requirement section
+  const startIndex = content.indexOf(`- ${TEST_REQ_TITLE}`);
+  if (startIndex === -1) {
+    return null;
+  }
 
-  // Find end of block (next ### or ##, or EOF)
-  let blockEnd = blockStart + 1;
-  while (blockEnd < lines.length && !lines[blockEnd].startsWith('###') && !lines[blockEnd].startsWith('## ')) {
-    blockEnd++;
+  // Find the end of the test requirement (next requirement or end of file)
+  let endIndex = content.indexOf('\n- ', startIndex + 1);
+  if (endIndex === -1) {
+    endIndex = content.length;
   }
 
-  lines.splice(blockStart, blockEnd - blockStart);
-  const cleaned = lines.join('\n').replace(/\n{3,}/g, '\n\n').trimEnd() + '\n';
-  fs.writeFileSync(reqPath, cleaned, 'utf8');
+  // Extract the test requirement content including the title line
+  const testContent = content.slice(startIndex, endIndex);
+  
+  return testContent.trim();
 }
 
 module.exports = {
-  TEST_REQ_TITLE,
-  PENDING_HEADER,
-  VERIFIED_HEADER,
   getResultFilePath,
   addTestRequirement,
-  removeTestRequirement
+  removeTestRequirement,
+  extractTestRequirement,
+  TEST_REQ_TITLE
 };

package/src/utils/provider-checker/test-requirements.js

@@ -8,8 +8,7 @@ const fs = require('fs');
 const path = require('path');
 
 const TEST_REQ_TITLE = 'VCM agent connectivity check';
-const PENDING_HEADER = '## ⏳ Requirements not yet completed';
-const VERIFIED_HEADER = '## ✅ Verified by AI';
+const REQUIREMENTS_HEADER = '## Requirements';
 
 /**
  * Return the absolute path to the result file used by the check.
@@ -29,27 +28,36 @@ function addTestRequirement(reqPath, resultFile) {
   // Check if test requirement already exists
   if (content.includes(TEST_REQ_TITLE)) return;
 
-  // Find the start of the pending section
-  const pendingStart = content.indexOf(PENDING_HEADER);
-  if (pendingStart === -1) return;
+  // Clean up old header format if it exists
+  content = content.replace(/## ⏳ Requirements not yet completed\s*\n/g, '');
 
-  // Find the end of the pending section (next ## header or end of file)
-  const afterPending = content.indexOf('\n##', pendingStart + 1);
-  const insertPos = afterPending === -1 ? content.length : afterPending;
+  // Find the Requirements section
+  const requirementsStart = content.indexOf(REQUIREMENTS_HEADER);
+  if (requirementsStart === -1) {
+    // If no Requirements section, create one at the beginning
+    content = REQUIREMENTS_HEADER + '\n\n' + content;
+  }
 
-  // Build the test requirement content
-  const testReq = `\n### ${TEST_REQ_TITLE}
+  // Find where to insert (after the Requirements header)
+  const headerEnd = content.indexOf(REQUIREMENTS_HEADER) + REQUIREMENTS_HEADER.length;
+  const insertPos = headerEnd;
 
-**Purpose:** Verify agent connectivity and functionality
+  // Build the test requirement content with status tracking
+  const testReq = `
+- ${TEST_REQ_TITLE}
+🚦 Status: [PREPARE]
 
-**Expected Results:**
-- Agent responds to test prompt
-- No errors or timeouts
-- Proper formatting and structure
+**✅ ACTION REQUIRED - VCM Connectivity Test:**
 
-**Test Command:** \`vibecodingmachine agent test\`
+VCM is testing if it can communicate with your IDE. To complete this test:
 
-**Verification:** Check TEMP_agent_check.txt for results
+**Change the status above from [PREPARE] to [DONE] and save the file.**
+
+Example:
+  Before: 🚦 Status: [PREPARE]
+  After:  🚦 Status: [DONE]
+
+That's it! VCM will detect the status change within seconds.
 
 `;
 

package/src/utils/status-card.js

@@ -1,121 +1,121 @@
-const chalk = require('chalk');
-const boxen = require('boxen');
-
-/**
- * Render a status card showing current requirement progress
- * Similar to the purple card in the GUI
- * @param {object} status - Current status object
- * @param {string} status.requirement - Current requirement being worked on
- * @param {string} status.step - Current step (PREPARE, ACT, CLEAN UP, VERIFY, DONE)
- * @param {number} status.chatCount - Current chat count
- * @param {number|null} status.maxChats - Maximum chats or null for unlimited
- * @param {number} status.progress - Progress percentage (0-100)
- * @returns {string} Formatted status card
- */
-function renderStatusCard(status) {
-  const {
-    requirement = 'No requirement loaded',
-    step = 'UNKNOWN',
-    chatCount = 0,
-    maxChats = null,
-    progress = 0
-  } = status;
-
-  // Step color mapping
-  const stepColors = {
-    'PREPARE': chalk.cyan,
-    'ACT': chalk.yellow,
-    'CLEAN UP': chalk.magenta,
-    'VERIFY': chalk.blue,
-    'DONE': chalk.green,
-    'UNKNOWN': chalk.gray
-  };
-
-  const stepColor = stepColors[step] || chalk.gray;
-
-  // Progress bar
-  const barWidth = 30;
-  const filledWidth = Math.round((progress / 100) * barWidth);
-  const emptyWidth = barWidth - filledWidth;
-  const progressBar = chalk.green('█'.repeat(filledWidth)) + chalk.gray('░'.repeat(emptyWidth));
-
-  // Chat counter
-  const chatDisplay = maxChats
-    ? `Chat ${chatCount}/${maxChats}`
-    : `Chat ${chatCount} (unlimited)`;
-
-  // Build card content
-  const content = [
-    chalk.bold('📋 Current Requirement'),
-    '',
-    chalk.white(requirement.length > 60 ? requirement.substring(0, 57) + '...' : requirement),
-    '',
-    chalk.bold('🚦 Status: ') + stepColor.bold(step),
-    '',
-    `${progressBar} ${progress}%`,
-    '',
-    chalk.gray(chatDisplay)
-  ].join('\n');
-
-  // Render with boxen (purple/magenta border like the GUI)
-  return boxen(content, {
-    padding: 1,
-    margin: { top: 0, right: 0, bottom: 1, left: 0 },
-    borderStyle: 'round',
-    borderColor: 'magenta',
-    title: 'Auto Mode Status',
-    titleAlignment: 'center',
-    width: 80
-  });
-}
-
-/**
- * Clear the terminal and move cursor to top
- */
-function clearAndMoveToTop() {
-  // ANSI escape codes
-  process.stdout.write('\x1B[2J'); // Clear entire screen
-  process.stdout.write('\x1B[H');  // Move cursor to home (top-left)
-}
-
-/**
- * Move cursor up N lines
- * @param {number} lines - Number of lines to move up
- */
-function moveCursorUp(lines) {
-  process.stdout.write(`\x1B[${lines}A`);
-}
-
-/**
- * Save cursor position
- */
-function saveCursor() {
-  process.stdout.write('\x1B[s');
-}
-
-/**
- * Restore cursor position
- */
-function restoreCursor() {
-  process.stdout.write('\x1B[u');
-}
-
-/**
- * Render the menu header and status card together
- * @param {string} menuContent - The menu content to display
- * @param {object} status - Status object for the status card
- */
-function renderHeaderWithStatus(menuContent, status) {
-  clearAndMoveToTop();
-  console.log(menuContent);
-  console.log(renderStatusCard(status));
-}
-
-module.exports = {
-  renderStatusCard,
-  clearAndMoveToTop,
-  moveCursorUp,
-  saveCursor,
-  restoreCursor,
-  renderHeaderWithStatus
-};
+const chalk = require('chalk');
+const boxen = require('boxen');
+
+/**
+ * Render a status card showing current requirement progress
+ * Similar to the purple card in the GUI
+ * @param {object} status - Current status object
+ * @param {string} status.requirement - Current requirement being worked on
+ * @param {string} status.step - Current step (PREPARE, ACT, CLEAN UP, VERIFY, DONE)
+ * @param {number} status.chatCount - Current chat count
+ * @param {number|null} status.maxChats - Maximum chats or null for unlimited
+ * @param {number} status.progress - Progress percentage (0-100)
+ * @returns {string} Formatted status card
+ */
+function renderStatusCard(status) {
+  const {
+    requirement = 'No requirement loaded',
+    step = 'UNKNOWN',
+    chatCount = 0,
+    maxChats = null,
+    progress = 0
+  } = status;
+
+  // Step color mapping
+  const stepColors = {
+    'PREPARE': chalk.cyan,
+    'ACT': chalk.yellow,
+    'CLEAN UP': chalk.magenta,
+    'VERIFY': chalk.blue,
+    'DONE': chalk.green,
+    'UNKNOWN': chalk.gray
+  };
+
+  const stepColor = stepColors[step] || chalk.gray;
+
+  // Progress bar
+  const barWidth = 30;
+  const filledWidth = Math.round((progress / 100) * barWidth);
+  const emptyWidth = barWidth - filledWidth;
+  const progressBar = chalk.green('█'.repeat(filledWidth)) + chalk.gray('░'.repeat(emptyWidth));
+
+  // Chat counter
+  const chatDisplay = maxChats
+    ? `Chat ${chatCount}/${maxChats}`
+    : `Chat ${chatCount} (unlimited)`;
+
+  // Build card content
+  const content = [
+    chalk.bold('📋 Current Requirement'),
+    '',
+    chalk.white(requirement.length > 60 ? requirement.substring(0, 57) + '...' : requirement),
+    '',
+    chalk.bold('🚦 Status: ') + stepColor.bold(step),
+    '',
+    `${progressBar} ${progress}%`,
+    '',
+    chalk.gray(chatDisplay)
+  ].join('\n');
+
+  // Render with boxen (purple/magenta border like the GUI)
+  return boxen(content, {
+    padding: 1,
+    margin: { top: 0, right: 0, bottom: 1, left: 0 },
+    borderStyle: 'round',
+    borderColor: 'magenta',
+    title: 'Auto Mode Status',
+    titleAlignment: 'center',
+    width: 80
+  });
+}
+
+/**
+ * Clear the terminal and move cursor to top
+ */
+function clearAndMoveToTop() {
+  // ANSI escape codes
+  process.stdout.write('\x1B[2J'); // Clear entire screen
+  process.stdout.write('\x1B[H');  // Move cursor to home (top-left)
+}
+
+/**
+ * Move cursor up N lines
+ * @param {number} lines - Number of lines to move up
+ */
+function moveCursorUp(lines) {
+  process.stdout.write(`\x1B[${lines}A`);
+}
+
+/**
+ * Save cursor position
+ */
+function saveCursor() {
+  process.stdout.write('\x1B[s');
+}
+
+/**
+ * Restore cursor position
+ */
+function restoreCursor() {
+  process.stdout.write('\x1B[u');
+}
+
+/**
+ * Render the menu header and status card together
+ * @param {string} menuContent - The menu content to display
+ * @param {object} status - Status object for the status card
+ */
+function renderHeaderWithStatus(menuContent, status) {
+  clearAndMoveToTop();
+  console.log(menuContent);
+  console.log(renderStatusCard(status));
+}
+
+module.exports = {
+  renderStatusCard,
+  clearAndMoveToTop,
+  moveCursorUp,
+  saveCursor,
+  restoreCursor,
+  renderHeaderWithStatus
+};