pulse-js-framework 1.7.3 → 1.7.5

package/cli/format.js

@@ -3,9 +3,11 @@
  * Formats .pulse files consistently
  */
 
-import { readFileSync, writeFileSync } from 'fs';
+import { readFileSync, writeFileSync, watch } from 'fs';
+import { dirname } from 'path';
 import { findPulseFiles, parseArgs, relativePath } from './utils/file-utils.js';
 import { log } from './logger.js';
+import { createTimer, formatDuration } from './utils/cli-ui.js';
 
 /**
  * Default format options
@@ -641,6 +643,7 @@ export async function runFormat(args) {
   const { options, patterns } = parseArgs(args);
   const check = options.check || false;
   const write = !check; // Default to write unless --check is specified
+  const watchMode = options.watch || options.w || false;
 
   // Find files to format
   const files = findPulseFiles(patterns);
@@ -650,7 +653,57 @@ export async function runFormat(args) {
     return;
   }
 
-  log.info(`${check ? 'Checking' : 'Formatting'} ${files.length} file(s)...\n`);
+  // Run initial format
+  const result = await runFormatOnFiles(files, { check, write, options });
+
+  // If watch mode, set up file watchers
+  if (watchMode) {
+    if (check) {
+      log.warn('--watch mode is not available with --check');
+      return;
+    }
+
+    log.info('\nWatching for changes... (Ctrl+C to stop)\n');
+
+    // Get unique directories to watch
+    const watchedDirs = new Set(files.map(f => dirname(f)));
+
+    // Debounce timer
+    let debounceTimer = null;
+    const debounceDelay = 100;
+
+    for (const dir of watchedDirs) {
+      watch(dir, { recursive: false }, (_eventType, filename) => {
+        if (!filename || !filename.endsWith('.pulse')) return;
+
+        // Debounce rapid changes
+        if (debounceTimer) clearTimeout(debounceTimer);
+        debounceTimer = setTimeout(() => {
+          const changedFiles = findPulseFiles(patterns);
+          runFormatOnFiles(changedFiles, { check: false, write: true, options, isRerun: true })
+            .then(() => {
+              log.info('Watching for changes...\n');
+            });
+        }, debounceDelay);
+      });
+    }
+
+    // Keep process alive
+    process.stdin.resume();
+  } else if (check && result.changedCount > 0) {
+    process.exit(1);
+  }
+}
+
+/**
+ * Run format on a list of files
+ */
+async function runFormatOnFiles(files, { check, write, options, isRerun = false }) {
+  const timer = createTimer();
+
+  if (!isRerun) {
+    log.info(`${check ? 'Checking' : 'Formatting'} ${files.length} file(s)...\n`);
+  }
 
   let changedCount = 0;
   let errorCount = 0;
@@ -675,12 +728,14 @@ export async function runFormat(args) {
         log.info(`  ${relPath} - formatted`);
       }
     } else {
-      if (!check) {
+      if (!check && !isRerun) {
         log.info(`  ${relPath} - unchanged`);
       }
     }
   }
 
+  const elapsed = timer.elapsed();
+
   // Summary
   log.info('\n' + '─'.repeat(60));
 
@@ -690,16 +745,17 @@ export async function runFormat(args) {
 
   if (check) {
     if (changedCount > 0) {
-      log.error(`✗ ${changedCount} file(s) need formatting`);
-      process.exit(1);
+      log.error(`✗ ${changedCount} file(s) need formatting (${formatDuration(elapsed)})`);
     } else {
-      log.success(`✓ All ${files.length} file(s) are properly formatted`);
+      log.success(`✓ All ${files.length} file(s) are properly formatted (${formatDuration(elapsed)})`);
     }
   } else {
     if (changedCount > 0) {
-      log.success(`✓ ${changedCount} file(s) formatted`);
+      log.success(`✓ ${changedCount} file(s) formatted (${formatDuration(elapsed)})`);
     } else {
-      log.success(`✓ All ${files.length} file(s) were already formatted`);
+      log.success(`✓ All ${files.length} file(s) were already formatted (${formatDuration(elapsed)})`);
     }
   }
+
+  return { changedCount, errorCount };
 }

package/cli/lint.js

@@ -3,9 +3,11 @@
  * Validates .pulse files for errors and style issues
  */
 
-import { readFileSync, writeFileSync } from 'fs';
+import { readFileSync, writeFileSync, watch } from 'fs';
+import { dirname } from 'path';
 import { findPulseFiles, parseArgs, relativePath } from './utils/file-utils.js';
 import { log } from './logger.js';
+import { createTimer, formatDuration } from './utils/cli-ui.js';
 
 /**
  * Lint rules configuration
@@ -701,21 +703,14 @@ export async function lintFile(filePath, options = {}) {
 }
 
 /**
- * Main lint command handler
+ * Lint files and return summary
+ * @param {string[]} files - Files to lint
+ * @param {Object} options - Lint options
+ * @returns {Object} Summary with totals
  */
-export async function runLint(args) {
-  const { options, patterns } = parseArgs(args);
-  const fix = options.fix || false;
-
-  // Find files to lint
-  const files = findPulseFiles(patterns);
-
-  if (files.length === 0) {
-    log.info('No .pulse files found to lint.');
-    return;
-  }
-
-  log.info(`Linting ${files.length} file(s)...\n`);
+async function lintFiles(files, options = {}) {
+  const { fix = false, quiet = false } = options;
+  const timer = createTimer();
 
   let totalErrors = 0;
   let totalWarnings = 0;
@@ -725,7 +720,7 @@ export async function runLint(args) {
     const result = await lintFile(file, { fix });
     const relPath = relativePath(file);
 
-    if (result.diagnostics.length > 0) {
+    if (result.diagnostics.length > 0 && !quiet) {
       log.info(`\n${relPath}`);
 
       for (const diag of result.diagnostics) {
@@ -740,21 +735,111 @@ export async function runLint(args) {
     }
   }
 
-  // Summary
+  return {
+    errors: totalErrors,
+    warnings: totalWarnings,
+    info: totalInfo,
+    elapsed: timer.elapsed()
+  };
+}
+
+/**
+ * Main lint command handler
+ */
+export async function runLint(args) {
+  const { options, patterns } = parseArgs(args);
+  const fix = options.fix || false;
+  const watchMode = options.watch || options.w || false;
+
+  // Find files to lint
+  const files = findPulseFiles(patterns);
+
+  if (files.length === 0) {
+    log.info('No .pulse files found to lint.');
+    return;
+  }
+
+  // Initial lint run
+  log.info(`Linting ${files.length} file(s)...\n`);
+  const summary = await lintFiles(files, { fix });
+
+  // Print summary
+  printLintSummary(summary, files.length);
+
+  // Watch mode
+  if (watchMode) {
+    log.info('\nWatching for changes... (Ctrl+C to stop)\n');
+
+    const watchedDirs = new Set();
+    const debounceTimers = new Map();
+
+    // Collect directories to watch
+    for (const file of files) {
+      watchedDirs.add(dirname(file));
+    }
+
+    // Watch each directory
+    for (const dir of watchedDirs) {
+      watch(dir, { recursive: false }, (_eventType, filename) => {
+        if (!filename || !filename.endsWith('.pulse')) return;
+
+        const filePath = files.find(f => f.endsWith(filename));
+        if (!filePath) return;
+
+        // Debounce rapid changes
+        if (debounceTimers.has(filePath)) {
+          clearTimeout(debounceTimers.get(filePath));
+        }
+
+        debounceTimers.set(filePath, setTimeout(() => {
+          debounceTimers.delete(filePath);
+
+          log.info(`\n[${new Date().toLocaleTimeString()}] File changed: ${relativePath(filePath)}`);
+          lintFiles([filePath], { fix }).then(result => {
+            printLintSummary(result, 1, true);
+          });
+        }, 100));
+      });
+    }
+
+    // Keep process running
+    return new Promise(() => {});
+  } else {
+    // Exit with error code if errors found
+    if (summary.errors > 0) {
+      process.exit(1);
+    }
+  }
+}
+
+/**
+ * Print lint summary
+ */
+function printLintSummary(summary, fileCount, compact = false) {
+  const { errors, warnings, info, elapsed } = summary;
+  const timeStr = formatDuration(elapsed);
+
+  if (compact) {
+    const parts = [];
+    if (errors > 0) parts.push(`${errors} error(s)`);
+    if (warnings > 0) parts.push(`${warnings} warning(s)`);
+    if (parts.length === 0) {
+      log.success(`✓ Passed (${timeStr})`);
+    } else {
+      log.error(`✗ ${parts.join(', ')} (${timeStr})`);
+    }
+    return;
+  }
+
   log.info('\n' + '─'.repeat(60));
   const parts = [];
-  if (totalErrors > 0) parts.push(`${totalErrors} error(s)`);
-  if (totalWarnings > 0) parts.push(`${totalWarnings} warning(s)`);
-  if (totalInfo > 0) parts.push(`${totalInfo} info`);
+  if (errors > 0) parts.push(`${errors} error(s)`);
+  if (warnings > 0) parts.push(`${warnings} warning(s)`);
+  if (info > 0) parts.push(`${info} info`);
 
   if (parts.length === 0) {
-    log.success(`✓ ${files.length} file(s) passed`);
+    log.success(`✓ ${fileCount} file(s) passed (${timeStr})`);
   } else {
-    log.error(`✗ ${parts.join(', ')} in ${files.length} file(s)`);
-  }
-
-  // Exit with error code if errors found
-  if (totalErrors > 0) {
-    process.exit(1);
+    log.error(`✗ ${parts.join(', ')} in ${fileCount} file(s) (${timeStr})`);
   }
 }