vibecodingmachine-cli 2025.11.2-9.855 → 2025.12.6-1702

package/src/commands/requirements.js

@@ -2,16 +2,98 @@ const path = require('path');
 const fs = require('fs-extra');
 const chalk = require('chalk');
 const chokidar = require('chokidar');
-const { getRepoPath } = require('../utils/config');
+const { getRepoPath, setRepoPath } = require('../utils/config');
 const { getRequirementsPath } = require('vibecodingmachine-core');
 
 async function getReqPathOrExit() {
-  const repoPath = await getRepoPath();
+  let repoPath = await getRepoPath();
+
+  // Auto-detect and initialize if we're in a git repository
+  if (!repoPath) {
+    const cwd = process.cwd();
+    const cwdResolved = path.resolve(cwd);
+
+    // Find the git root by walking up the directory tree
+    let currentDir = cwdResolved;
+    let foundGitRoot = false;
+    const maxDepth = 50; // Prevent infinite loops
+    let depth = 0;
+
+    // Walk up the directory tree looking for .git directory
+    while (depth < maxDepth && currentDir !== path.dirname(currentDir)) {
+      const gitPath = path.join(currentDir, '.git');
+      try {
+        if (await fs.pathExists(gitPath)) {
+          repoPath = currentDir;
+          foundGitRoot = true;
+          break;
+        }
+      } catch (err) {
+        // If we can't check, continue to parent
+      }
+      const parentDir = path.dirname(currentDir);
+      if (parentDir === currentDir) {
+        // Reached filesystem root
+        break;
+      }
+      currentDir = parentDir;
+      depth++;
+    }
+
+    // Fallback: if no .git found but .vibecodingmachine exists, use current dir
+    if (!foundGitRoot) {
+      try {
+        const vibecodingmachineDir = path.join(cwdResolved, '.vibecodingmachine');
+        if (await fs.pathExists(vibecodingmachineDir)) {
+          repoPath = cwdResolved;
+          foundGitRoot = true;
+        }
+      } catch (err) {
+        // Ignore errors
+      }
+    }
+
+    // Final fallback: use current directory if we have package.json
+    if (!foundGitRoot) {
+      try {
+        const packageJsonPath = path.join(cwdResolved, 'package.json');
+        if (await fs.pathExists(packageJsonPath)) {
+          repoPath = cwdResolved;
+          foundGitRoot = true;
+        }
+      } catch (err) {
+        // Ignore errors
+      }
+    }
+
+    // If we found a repo path, save it
+    if (repoPath) {
+      try {
+        await setRepoPath(repoPath);
+      } catch (err) {
+        // If setting fails, we still have repoPath set, so continue
+      }
+    }
+  }
+
+  // Final validation - ensure we have a valid path
+  // If we still don't have a repoPath, use current directory as absolute last resort
+  if (!repoPath) {
+    repoPath = path.resolve(process.cwd());
+    try {
+      await setRepoPath(repoPath);
+    } catch (err) {
+      // Even if we can't save it, we'll use it for this session
+    }
+  }
+
+  // At this point, repoPath MUST be set (we've tried everything including current directory)
   if (!repoPath) {
     console.log(chalk.yellow('No repository path configured'));
     console.log(chalk.gray('Use'), chalk.cyan('vcm repo:init'), chalk.gray('or'), chalk.cyan('vcm repo:set <path>'));
     process.exit(1);
   }
+
   // Use getRequirementsPath which handles hostname-specific files
   const reqPath = await getRequirementsPath(repoPath);
   return { repoPath, reqPath };
@@ -40,7 +122,7 @@ async function list(options) {
   }
 }
 
-async function add(name) {
+async function add(name, pkg, description) {
   try {
     const { reqPath } = await getReqPathOrExit();
     await fs.ensureFile(reqPath);
@@ -64,6 +146,34 @@ async function add(name) {
       if (!inserted && lines[i].includes(todoSectionHeader)) {
         // Add requirement header
         newLines.push(`### ${name}`);
+
+        // Add package if provided (and not 'all')
+        if (pkg && Array.isArray(pkg) && pkg.length > 0) {
+          const packageValue = pkg.length === 1 ? pkg[0] : pkg.join(', ');
+          if (packageValue !== 'all') {
+            newLines.push(`PACKAGE: ${packageValue}`);
+          }
+        } else if (pkg && typeof pkg === 'string' && pkg !== 'all') {
+          newLines.push(`PACKAGE: ${pkg}`);
+        }
+
+        // Add description if provided
+        if (description && typeof description === 'string' && description.trim()) {
+          const descLines = description.split('\n');
+          descLines.forEach(line => {
+            if (line.trim()) {
+              newLines.push(line);
+            }
+          });
+        } else if (description && Array.isArray(description)) {
+          // Handle array of description lines
+          description.forEach(line => {
+            if (line && typeof line === 'string' && line.trim()) {
+              newLines.push(line);
+            }
+          });
+        }
+
         // Add blank line after requirement
         newLines.push('');
         inserted = true;
@@ -120,15 +230,10 @@ async function next() {
 async function edit() {
   try {
     const { reqPath } = await getReqPathOrExit();
-    const opener = process.env.EDITOR ? null : 'open';
-    if (process.env.EDITOR) {
-      const { spawn } = require('child_process');
-      const child = spawn(process.env.EDITOR, [reqPath], { stdio: 'inherit' });
-      child.on('exit', (code) => process.exit(code || 0));
-    } else {
-      const { execSync } = require('child_process');
-      execSync(`${opener} ${JSON.stringify(reqPath)}`);
-    }
+    // Always use system default opener to avoid taking over the terminal
+    const { execSync } = require('child_process');
+    // Use 'open' for macOS (user's OS), could be expanded for others if needed
+    execSync(`open ${JSON.stringify(reqPath)}`);
   } catch (error) {
     console.error(chalk.red('Error opening file:'), error.message);
     process.exit(1);
@@ -148,13 +253,99 @@ async function watch() {
   }
 }
 
+async function rename(oldTitle, newTitle, description) {
+  try {
+    const { reqPath } = await getReqPathOrExit();
+    if (!await fs.pathExists(reqPath)) {
+      console.log(chalk.yellow('No REQUIREMENTS.md found.'));
+      return;
+    }
+
+    // Read the requirements file
+    const content = await fs.readFile(reqPath, 'utf8');
+    const lines = content.split('\n');
+
+    // Find the requirement block
+    let requirementStartIndex = -1;
+    let requirementEndIndex = -1;
+
+    for (let i = 0; i < lines.length; i++) {
+      const line = lines[i].trim();
+      if (line.startsWith('###')) {
+        const title = line.replace(/^###\s*/, '').trim();
+        if (title === oldTitle) {
+          requirementStartIndex = i;
+          // Find the end of this requirement (next ### or ## header)
+          for (let j = i + 1; j < lines.length; j++) {
+            const nextLine = lines[j].trim();
+            if (nextLine.startsWith('###') || (nextLine.startsWith('##') && !nextLine.startsWith('###'))) {
+              requirementEndIndex = j;
+              break;
+            }
+          }
+          if (requirementEndIndex === -1) {
+            requirementEndIndex = lines.length;
+          }
+          break;
+        }
+      }
+    }
+
+    if (requirementStartIndex === -1) {
+      console.log(chalk.yellow(`⚠️  Could not find requirement: "${oldTitle}"`));
+      return;
+    }
+
+    // Build new requirement block
+    const newBlock = [`### ${newTitle}`];
+
+    // If description provided, use it
+    if (description) {
+      if (Array.isArray(description)) {
+        description.forEach(line => {
+          if (line && line.trim()) {
+            newBlock.push(line);
+          }
+        });
+      } else if (typeof description === 'string') {
+        description.split('\n').forEach(line => {
+          if (line.trim()) {
+            newBlock.push(line);
+          }
+        });
+      }
+    } else {
+      // Keep existing details (skip the ### header line)
+      for (let i = requirementStartIndex + 1; i < requirementEndIndex; i++) {
+        const line = lines[i];
+        if (line.trim()) {
+          newBlock.push(line);
+        }
+      }
+    }
+
+    newBlock.push(''); // Blank line after requirement
+
+    // Replace the old block with the new one
+    lines.splice(requirementStartIndex, requirementEndIndex - requirementStartIndex, ...newBlock);
+
+    // Save
+    await fs.writeFile(reqPath, lines.join('\n'));
+    console.log(chalk.green('✓ Requirement renamed/updated'));
+  } catch (error) {
+    console.error(chalk.red('Error renaming requirement:'), error.message);
+    process.exit(1);
+  }
+}
+
 module.exports = {
   list,
   add,
   current,
   next,
   edit,
-  watch
+  watch,
+  rename
 };
 
 

package/src/commands/sync.js

@@ -0,0 +1,280 @@
+const chalk = require('chalk');
+const Table = require('cli-table3');
+const SyncEngine = require('vibecodingmachine-core/src/sync/sync-engine');
+
+/**
+ * Trigger immediate sync
+ */
+async function syncNow(options = {}) {
+  const syncEngine = new SyncEngine();
+  
+  try {
+    console.log(chalk.cyan('\n🔄 Starting sync...\n'));
+    
+    await syncEngine.initialize();
+    
+    // Listen for sync completion
+    const syncPromise = new Promise((resolve, reject) => {
+      syncEngine.once('sync-complete', (result) => {
+        if (result.status === 'error') {
+          reject(new Error(result.error || 'Sync failed'));
+        } else {
+          resolve(result);
+        }
+      });
+    });
+    
+    // Start sync
+    await syncEngine.sync();
+    
+    // Wait for completion
+    const result = await syncPromise;
+    
+    const status = syncEngine.getStatus();
+    
+    console.log(chalk.green('✓ Sync complete!\n'));
+    console.log(chalk.white('Last Sync:       ') + new Date(status.lastSyncTime).toLocaleString());
+    console.log(chalk.white('Remote Changes:  ') + (result.remoteChanges || 0));
+    console.log(chalk.white('Local Changes:   ') + (result.localChanges || 0));
+    console.log(chalk.white('Conflicts:       ') + (result.conflicts || 0));
+    console.log(chalk.white('Queued Changes:  ') + status.queuedChanges);
+    console.log('');
+    
+  } catch (error) {
+    console.error(chalk.red('\n✗ Sync failed:'), error.message);
+    console.log(chalk.gray('\nTip: Check AWS credentials and DynamoDB table configuration.\n'));
+    // Don't throw - just log the error
+  } finally {
+    syncEngine.stop();
+  }
+}
+
+/**
+ * Show sync status and statistics
+ */
+async function syncStatus(options = {}) {
+  const syncEngine = new SyncEngine();
+  
+  try {
+    await syncEngine.initialize();
+    
+    const status = syncEngine.getStatus();
+    
+    console.log('\n' + chalk.bold.cyan('Sync Status'));
+    console.log(chalk.gray('─'.repeat(60)));
+    
+    // Connection status
+    const onlineIcon = status.isOnline ? chalk.green('●') : chalk.red('●');
+    const onlineText = status.isOnline ? 'Online' : 'Offline';
+    console.log(chalk.white('Connection:      ') + onlineIcon + ' ' + onlineText);
+    
+    // Sync status
+    const syncingIcon = status.isSyncing ? chalk.yellow('⟳') : chalk.gray('○');
+    const syncingText = status.isSyncing ? 'Syncing...' : 'Idle';
+    console.log(chalk.white('Status:          ') + syncingIcon + ' ' + syncingText);
+    
+    // Last sync
+    const lastSync = status.lastSyncTime 
+      ? new Date(status.lastSyncTime).toLocaleString()
+      : 'Never';
+    console.log(chalk.white('Last Sync:       ') + lastSync);
+    
+    // Queued changes
+    const queueColor = status.queuedChanges > 0 ? chalk.yellow : chalk.gray;
+    console.log(chalk.white('Queued Changes:  ') + queueColor(status.queuedChanges));
+    
+    // Conflict strategy
+    console.log(chalk.white('Conflict Mode:   ') + status.conflictStrategy);
+    
+    // Computer ID
+    console.log(chalk.white('Computer ID:     ') + status.computerId);
+    
+    console.log('');
+    
+    // Show recent sync events
+    const history = syncEngine.getHistory(5);
+    if (history.length > 0) {
+      console.log(chalk.bold.cyan('Recent Sync Events'));
+      console.log(chalk.gray('─'.repeat(60)));
+      
+      for (const event of history.reverse()) {
+        const time = new Date(event.timestamp).toLocaleTimeString();
+        const icon = event.type === 'conflict-resolution' ? chalk.yellow('⚠') : chalk.gray('•');
+        console.log(`${icon} ${chalk.gray(time)} ${event.type}`);
+      }
+      
+      console.log('');
+    }
+    
+  } catch (error) {
+    console.error(chalk.red('\n✗ Failed to get sync status:'), error.message);
+    throw error;
+  } finally {
+    syncEngine.stop();
+  }
+}
+
+/**
+ * View pending changes in offline queue
+ */
+async function viewQueue(options = {}) {
+  const syncEngine = new SyncEngine();
+  
+  try {
+    await syncEngine.initialize();
+    
+    const status = syncEngine.getStatus();
+    
+    if (status.queuedChanges === 0) {
+      console.log(chalk.gray('\nNo pending changes in queue.\n'));
+      return;
+    }
+    
+    console.log(chalk.cyan(`\n📋 Offline Queue (${status.queuedChanges} changes)\n`));
+    
+    // Create table
+    const table = new Table({
+      head: [
+        chalk.cyan('#'),
+        chalk.cyan('Type'),
+        chalk.cyan('Requirement'),
+        chalk.cyan('Timestamp')
+      ],
+      colWidths: [5, 15, 40, 20]
+    });
+    
+    // Add rows (accessing internal queue - in production, add a public getter)
+    syncEngine.offlineQueue.forEach((change, index) => {
+      table.push([
+        index + 1,
+        change.type || 'update',
+        truncate(change.requirementName || change.requirementId || '-', 38),
+        new Date(change.timestamp).toLocaleString()
+      ]);
+    });
+    
+    console.log(table.toString() + '\n');
+    
+    console.log(chalk.gray('These changes will be synced when connection is restored.\n'));
+    console.log(chalk.white('Commands:'));
+    console.log(chalk.gray('  vcm sync:force  ') + '- Force sync now');
+    console.log(chalk.gray('  vcm sync:now    ') + '- Sync when online');
+    console.log('');
+    
+  } catch (error) {
+    console.error(chalk.red('\n✗ Failed to view queue:'), error.message);
+    throw error;
+  } finally {
+    syncEngine.stop();
+  }
+}
+
+/**
+ * Force sync even if offline
+ */
+async function forceSync(options = {}) {
+  const syncEngine = new SyncEngine();
+  
+  try {
+    console.log(chalk.yellow('\n⚠ Forcing sync (ignoring offline status)...\n'));
+    
+    await syncEngine.initialize();
+    
+    // Temporarily set online
+    const wasOnline = syncEngine.isOnline;
+    syncEngine.setOnlineMode(true);
+    
+    try {
+      await syncEngine.sync();
+      console.log(chalk.green('✓ Force sync complete!\n'));
+    } finally {
+      // Restore original online status
+      syncEngine.setOnlineMode(wasOnline);
+    }
+    
+  } catch (error) {
+    console.error(chalk.red('\n✗ Force sync failed:'), error.message);
+    console.log(chalk.gray('\nThis usually means the server is unreachable.\n'));
+    throw error;
+  } finally {
+    syncEngine.stop();
+  }
+}
+
+/**
+ * View sync history
+ */
+async function viewHistory(options = {}) {
+  const syncEngine = new SyncEngine();
+  
+  try {
+    await syncEngine.initialize();
+    
+    const limit = parseInt(options.limit) || 50;
+    const history = syncEngine.getHistory(limit);
+    
+    if (history.length === 0) {
+      console.log(chalk.gray('\nNo sync history available.\n'));
+      return;
+    }
+    
+    console.log(chalk.cyan(`\n📜 Sync History (last ${history.length} events)\n`));
+    
+    // Create table
+    const table = new Table({
+      head: [
+        chalk.cyan('Time'),
+        chalk.cyan('Type'),
+        chalk.cyan('Details')
+      ],
+      colWidths: [20, 20, 50]
+    });
+    
+    // Add rows
+    for (const event of history.reverse()) {
+      const time = new Date(event.timestamp).toLocaleString();
+      const type = event.type;
+      const details = getEventDetails(event);
+      
+      table.push([time, type, details]);
+    }
+    
+    console.log(table.toString() + '\n');
+    
+  } catch (error) {
+    console.error(chalk.red('\n✗ Failed to view history:'), error.message);
+    throw error;
+  } finally {
+    syncEngine.stop();
+  }
+}
+
+// Helper functions
+
+function truncate(str, maxLength) {
+  if (str.length <= maxLength) return str;
+  return str.substring(0, maxLength - 3) + '...';
+}
+
+function getEventDetails(event) {
+  switch (event.type) {
+    case 'conflict-resolution':
+      return `Resolved: ${event.resolution?.requirementId || 'unknown'}`;
+    case 'sync-complete':
+      return `Remote: ${event.remoteChanges || 0}, Local: ${event.localChanges || 0}`;
+    case 'remote-change-applied':
+      return `Applied: ${event.change?.requirementId || 'unknown'}`;
+    case 'local-change-pushed':
+      return `Pushed: ${event.change?.requirementId || 'unknown'}`;
+    default:
+      return JSON.stringify(event).substring(0, 47);
+  }
+}
+
+module.exports = {
+  syncNow,
+  syncStatus,
+  viewQueue,
+  forceSync,
+  viewHistory
+};

package/src/utils/asset-cleanup.js

@@ -0,0 +1,61 @@
+const fs = require('fs-extra');
+const path = require('path');
+const chalk = require('chalk');
+
+/**
+ * Scans the assets directory and removes 0-byte PNG files
+ * This is a mitigation for an issue where empty PNG files are mysteriously created
+ */
+async function cleanupBrokenAssets() {
+    try {
+        // Find assets directory - assuming we are in packages/cli/src/utils
+        // and assets is at root of repo, but simpler to look relative to where VCM runs
+        // effectively, we look for 'assets' in the current working directory as CWD is usually the repo root or where the CLI is run
+        // Better: define path relative to the project root if possible, or CWD if that's how it's structured.
+        // Based on previous search, assets/ is at the root of the repo.
+        // However, when installed, strict paths matters.
+        // But for the user's workspace context, iterating 'assets/' in CWD (repo root) is correct for their dev environment.
+        // Let's try to be smart about finding specific 'assets' dir if we can, but CWD is a safe start for the CLI tool which is often run from root.
+
+        // Actually, looking at previous grep, `assets/` was checked specifically. 
+        // Let's just check `assets` in process.cwd() as a primary target.
+
+        const assetsDir = path.join(process.cwd(), 'assets');
+
+        if (!await fs.pathExists(assetsDir)) {
+            return; // No assets dir, nothing to clean
+        }
+
+        const files = await fs.readdir(assetsDir);
+        let removedCount = 0;
+
+        for (const file of files) {
+            if (file.toLowerCase().endsWith('.png')) {
+                const filePath = path.join(assetsDir, file);
+                try {
+                    const stats = await fs.stat(filePath);
+                    if (stats.size === 0) {
+                        await fs.remove(filePath);
+                        removedCount++;
+                    }
+                } catch (err) {
+                    // Ignore errors accessing specific files
+                }
+            }
+        }
+
+        if (removedCount > 0) {
+            // Only log if we actually did something, to keep noise down
+            // But maybe we want to know? The plan said "verification: verify removed".
+            // We can use a debug flag or just log.
+            // Let's log to console for now as it's a CLI tool.
+            // console.log(chalk.gray(`Cleaned up ${removedCount} broken (0-byte) asset files.`));
+        }
+
+    } catch (error) {
+        // Silently fail to avoid disrupting startup
+        // console.error('Asset cleanup failed:', error);
+    }
+}
+
+module.exports = { cleanupBrokenAssets };