pulse-js-framework 1.7.4 → 1.7.6

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,17 @@ 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
+ * @param {boolean} options.fix - Auto-fix issues
+ * @param {boolean} options.dryRun - Show fixes without applying
+ * @param {boolean} options.quiet - Suppress output
+ * @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, dryRun = false, quiet = false } = options;
+  const timer = createTimer();
 
   let totalErrors = 0;
   let totalWarnings = 0;
@@ -725,7 +723,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 +738,117 @@ 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 dryRun = options['dry-run'] || false;
+  const watchMode = options.watch || options.w || false;
+
+  // Dry-run only makes sense with --fix
+  if (dryRun && !fix) {
+    log.warn('Note: --dry-run has no effect without --fix');
+  }
+
+  // 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)...${dryRun ? ' (dry-run)' : ''}\n`);
+  const summary = await lintFiles(files, { fix, dryRun });
+
+  // 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, dryRun }).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})`);
   }
 }

package/cli/logger.js

@@ -1,9 +1,14 @@
 /**
  * Pulse CLI Logger
- * Lightweight logger for CLI tools with support for verbose mode
+ * Adapter for CLI tools using the unified runtime logger
  * @module pulse-cli/logger
  */
 
+import { createLogger, setLogLevel, LogLevel } from '../runtime/logger.js';
+
+// Create CLI-namespaced logger
+const cliLogger = createLogger('CLI');
+
 /** @type {boolean} */
 let verboseMode = false;
 
@@ -17,6 +22,12 @@ let verboseMode = false;
  */
 export function setVerbose(enabled) {
   verboseMode = enabled;
+  // Update global log level to include debug when verbose
+  if (enabled) {
+    setLogLevel(LogLevel.DEBUG);
+  } else {
+    setLogLevel(LogLevel.INFO);
+  }
 }
 
 /**
@@ -33,6 +44,7 @@ export function isVerbose() {
 
 /**
  * CLI Logger object with console-like API
+ * Uses the runtime logger under the hood for consistency
  * @namespace log
  */
 export const log = {
@@ -44,6 +56,7 @@ export const log = {
    * log.info('Starting server on port', 3000);
    */
   info(...args) {
+    // Use console directly for CLI output (no namespace prefix for info)
     console.log(...args);
   },
 
@@ -66,7 +79,7 @@ export const log = {
    * log.warn('Deprecated feature used');
    */
   warn(...args) {
-    console.warn(...args);
+    cliLogger.warn(...args);
   },
 
   /**
@@ -77,7 +90,7 @@ export const log = {
    * log.error('Failed to compile:', error.message);
    */
   error(...args) {
-    console.error(...args);
+    cliLogger.error(...args);
   },
 
   /**
@@ -89,7 +102,7 @@ export const log = {
    */
   debug(...args) {
     if (verboseMode) {
-      console.log('[debug]', ...args);
+      cliLogger.debug(...args);
     }
   },
 
@@ -116,7 +129,22 @@ export const log = {
    */
   newline() {
     console.log();
+  },
+
+  /**
+   * Create a child logger with a sub-namespace
+   * @param {string} namespace - Child namespace
+   * @returns {import('../runtime/logger.js').Logger} Child logger instance
+   * @example
+   * const buildLog = log.child('Build');
+   * buildLog.info('Starting...'); // [CLI:Build] Starting...
+   */
+  child(namespace) {
+    return cliLogger.child(namespace);
   }
 };
 
+// Re-export LogLevel for convenience
+export { LogLevel };
+
 export default log;

package/cli/release.js

@@ -10,13 +10,15 @@
  * - Push to remote
  */
 
-import { readFileSync, writeFileSync, existsSync } from 'fs';
+import { readFileSync, writeFileSync, existsSync, unlinkSync } from 'fs';
 import { join, dirname } from 'path';
 import { fileURLToPath } from 'url';
 import { execSync } from 'child_process';
+import { tmpdir } from 'os';
 import { createInterface } from 'readline';
 import https from 'https';
 import { log } from './logger.js';
+import { runDocsTest } from './docs-test.js';
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const root = join(__dirname, '..');
@@ -525,14 +527,15 @@ function gitCommitTagPush(newVersion, title, changes, dryRun = false) {
   log.info('  Running: git add -A...');
   execSync('git add -A', { cwd: root, stdio: 'inherit' });
 
-  // git commit with heredoc for proper message formatting
+  // git commit using temp file for cross-platform compatibility
   log.info('  Running: git commit...');
-  const escapedMessage = commitMessage.replace(/'/g, "'\\''");
-  execSync(`git commit -m $'${escapedMessage.replace(/\n/g, '\\n')}'`, {
-    cwd: root,
-    stdio: 'inherit',
-    shell: '/bin/bash'
-  });
+  const tempFile = join(tmpdir(), `pulse-release-${Date.now()}.txt`);
+  writeFileSync(tempFile, commitMessage, 'utf-8');
+  try {
+    execSync(`git commit -F "${tempFile}"`, { cwd: root, stdio: 'inherit' });
+  } finally {
+    unlinkSync(tempFile);
+  }
 
   // git tag
   log.info(`  Running: git tag v${newVersion}...`);
@@ -559,12 +562,13 @@ Types:
   major     Bump major version (1.0.0 -> 2.0.0)
 
 Options:
-  --dry-run        Show what would be done without making changes
-  --no-push        Create commit and tag but don't push
-  --title <text>   Release title (e.g., "Performance Improvements")
-  --skip-prompt    Use empty changelog (for automated releases)
-  --from-commits   Auto-extract changelog from git commits since last tag
-  --yes, -y        Auto-confirm all prompts
+  --dry-run         Show what would be done without making changes
+  --no-push         Create commit and tag but don't push
+  --title <text>    Release title (e.g., "Performance Improvements")
+  --skip-prompt     Use empty changelog (for automated releases)
+  --skip-docs-test  Skip documentation validation before release
+  --from-commits    Auto-extract changelog from git commits since last tag
+  --yes, -y         Auto-confirm all prompts
   --changes <json> Pass changelog as JSON (e.g., '{"added":["Feature 1"],"fixed":["Bug 1"]}')
   --added <items>  Comma-separated list of added features
   --changed <items> Comma-separated list of changes
@@ -599,6 +603,7 @@ export async function runRelease(args) {
   const skipPrompt = args.includes('--skip-prompt');
   const fromCommits = args.includes('--from-commits');
   const autoConfirm = args.includes('--yes') || args.includes('-y');
+  const skipDocsTest = args.includes('--skip-docs-test');
 
   let title = '';
   const titleIndex = args.indexOf('--title');
@@ -675,6 +680,30 @@ export async function runRelease(args) {
     process.exit(1);
   }
 
+  // Run documentation tests
+  if (!skipDocsTest) {
+    log.info('');
+    log.info('Running documentation tests...');
+
+    const docsTestResult = await runDocsTest({ verbose: false, httpTest: true });
+
+    if (!docsTestResult.success) {
+      log.error('Documentation tests failed. Fix errors before releasing.');
+      if (!autoConfirm) {
+        const proceed = await prompt('Continue anyway? (y/N) ');
+        if (proceed.toLowerCase() !== 'y') {
+          log.info('Aborted.');
+          process.exit(1);
+        }
+      } else {
+        log.error('Aborting release due to documentation errors.');
+        process.exit(1);
+      }
+    }
+  } else {
+    log.warn('Skipping documentation tests (--skip-docs-test)');
+  }
+
   // Collect changelog entries
   let changes = { added: [], changed: [], fixed: [], removed: [] };
 
@@ -786,12 +815,13 @@ export async function runRelease(args) {
       // Only commit and tag, no push
       const commitMessage = buildCommitMessage(newVersion, title, changes);
       execSync('git add -A', { cwd: root, stdio: 'inherit' });
-      const escapedMessage = commitMessage.replace(/'/g, "'\\''");
-      execSync(`git commit -m $'${escapedMessage.replace(/\n/g, '\\n')}'`, {
-        cwd: root,
-        stdio: 'inherit',
-        shell: '/bin/bash'
-      });
+      const tempFile = join(tmpdir(), `pulse-release-${Date.now()}.txt`);
+      writeFileSync(tempFile, commitMessage, 'utf-8');
+      try {
+        execSync(`git commit -F "${tempFile}"`, { cwd: root, stdio: 'inherit' });
+      } finally {
+        unlinkSync(tempFile);
+      }
       execSync(`git tag -a v${newVersion} -m "Release v${newVersion}"`, { cwd: root, stdio: 'inherit' });
       log.info('  Created commit and tag (--no-push specified)');
     } else {