pulse-js-framework 1.11.3 → 1.11.4

package/cli/analyze.js

@@ -3,7 +3,7 @@
  * Analyzes bundle size and dependencies
  */
 
-import { readFileSync, statSync, existsSync } from 'fs';
+import { readFileSync, statSync, readdirSync } from 'fs';
 import { join, dirname, basename, relative } from 'path';
 import { findPulseFiles, parseArgs, formatBytes, relativePath, resolveImportPath } from './utils/file-utils.js';
 import { log } from './logger.js';
@@ -82,12 +82,20 @@ async function analyzeFiles(files, parse) {
 
   for (const file of files) {
     try {
-      const stats = statSync(file);
-      const source = readFileSync(file, 'utf-8');
+      let source;
+      let fileSize;
+      try {
+        const buffer = readFileSync(file);
+        source = buffer.toString('utf-8');
+        fileSize = buffer.length;
+      } catch (e) {
+        if (e.code !== 'ENOENT') throw e;
+        continue;
+      }
 
       const info = {
         path: relativePath(file),
-        size: stats.size,
+        size: fileSize,
         sizeFormatted: formatBytes(stats.size),
         lines: source.split('\n').length
       };
@@ -548,10 +556,15 @@ export async function runAnalyze(args) {
   const root = patterns[0] || '.';
 
   // Check if src directory exists
-  if (!existsSync(join(root, 'src'))) {
-    log.error('Error: No src/ directory found.');
-    log.info('Run this command from your Pulse project root.');
-    process.exit(1);
+  try {
+    readdirSync(join(root, 'src'));
+  } catch (err) {
+    if (err.code === 'ENOENT' || err.code === 'ENOTDIR') {
+      log.error('Error: No src/ directory found.');
+      log.info('Run this command from your Pulse project root.');
+      process.exit(1);
+    }
+    throw err;
   }
 
   const timer = createTimer();

package/cli/build.js

@@ -4,7 +4,7 @@
  * Builds Pulse projects for production
  */
 
-import { readFileSync, writeFileSync, existsSync, mkdirSync, readdirSync, statSync, copyFileSync } from 'fs';
+import { readFileSync, writeFileSync, mkdirSync, readdirSync, copyFileSync } from 'fs';
 import { join, extname, relative, dirname } from 'path';
 import { compile } from '../compiler/index.js';
 import { preprocessStylesSync, isSassAvailable, getSassVersion } from '../compiler/preprocessor.js';
@@ -39,13 +39,12 @@ export async function buildProject(args) {
   // Check if vite is available
   try {
     const viteConfig = join(root, 'vite.config.js');
-    if (existsSync(viteConfig)) {
-      const spinner = createSpinner('Building with Vite...');
-      const { build } = await import('vite');
-      await build({ root });
-      spinner.success(`Built with Vite in ${timer.format()}`);
-      return;
-    }
+    readFileSync(viteConfig, 'utf-8'); // Throws ENOENT if missing
+    const spinner = createSpinner('Building with Vite...');
+    const { build } = await import('vite');
+    await build({ root });
+    spinner.success(`Built with Vite in ${timer.format()}`);
+    return;
   } catch (e) {
     // Vite not available, use built-in build
   }
@@ -60,22 +59,29 @@ export async function buildProject(args) {
   }
 
   // Create output directory
-  if (!existsSync(outDir)) {
-    mkdirSync(outDir, { recursive: true });
-  }
+  mkdirSync(outDir, { recursive: true });
 
   // Copy public files
   const publicDir = join(root, 'public');
-  if (existsSync(publicDir)) {
+  try {
+    readdirSync(publicDir);
     copyDir(publicDir, outDir);
+  } catch (e) {
+    if (e.code !== 'ENOENT') throw e;
+    // No public directory, skip
   }
 
   // Count files for progress bar
   const srcDir = join(root, 'src');
-  const fileCount = existsSync(srcDir) ? countFiles(srcDir) : 0;
+  let fileCount = 0;
+  try {
+    fileCount = countFiles(srcDir);
+  } catch (e) {
+    if (e.code !== 'ENOENT') throw e;
+  }
 
   // Process source files with progress bar
-  if (existsSync(srcDir) && fileCount > 0) {
+  if (fileCount > 0) {
     const progress = createProgressBar({
       total: fileCount,
       label: 'Compiling',
@@ -87,7 +93,7 @@ export async function buildProject(args) {
 
   // Copy and process index.html
   const indexHtml = join(root, 'index.html');
-  if (existsSync(indexHtml)) {
+  try {
     let html = readFileSync(indexHtml, 'utf-8');
 
     // Update script paths for production
@@ -100,6 +106,9 @@ export async function buildProject(args) {
     html = html.replace(/\.pulse"/g, '.js"');
 
     writeFileSync(join(outDir, 'index.html'), html);
+  } catch (e) {
+    if (e.code !== 'ENOENT') throw e;
+    // No index.html, skip
   }
 
   // Bundle runtime
@@ -146,13 +155,11 @@ To preview:
  */
 function countFiles(dir) {
   let count = 0;
-  const files = readdirSync(dir);
+  const entries = readdirSync(dir, { withFileTypes: true });
 
-  for (const file of files) {
-    const fullPath = join(dir, file);
-    const stat = statSync(fullPath);
-    if (stat.isDirectory()) {
-      count += countFiles(fullPath);
+  for (const entry of entries) {
+    if (entry.isDirectory()) {
+      count += countFiles(join(dir, entry.name));
     } else {
       count++;
     }
@@ -168,21 +175,25 @@ function countFiles(dir) {
  * @param {Object} [progress] - Progress bar instance
  */
 function processDirectory(srcDir, outDir, progress = null) {
-  if (!existsSync(outDir)) {
-    mkdirSync(outDir, { recursive: true });
-  }
+  mkdirSync(outDir, { recursive: true });
 
-  const files = readdirSync(srcDir);
+  const entries = readdirSync(srcDir, { withFileTypes: true });
 
-  for (const file of files) {
+  for (const entry of entries) {
+    const file = entry.name;
     const srcPath = join(srcDir, file);
-    const stat = statSync(srcPath);
 
-    if (stat.isDirectory()) {
+    if (entry.isDirectory()) {
       processDirectory(srcPath, join(outDir, file), progress);
     } else if (file.endsWith('.pulse')) {
       // Compile .pulse files
-      const source = readFileSync(srcPath, 'utf-8');
+      let source;
+      try {
+        source = readFileSync(srcPath, 'utf-8');
+      } catch (e) {
+        if (e.code !== 'ENOENT') throw e;
+        continue;
+      }
       const result = compile(source, {
         runtime: './runtime.js',
         minify: true
@@ -212,7 +223,13 @@ function processDirectory(srcDir, outDir, progress = null) {
         }
 
         const outPath = join(outDir, file.replace('.pulse', '.js'));
-        writeFileSync(outPath, code);
+        try {
+          writeFileSync(outPath, code);
+        } catch (writeErr) {
+          log.error(`  Failed to write ${outPath}: ${writeErr.message}`);
+          if (progress) progress.tick();
+          continue;
+        }
       } else {
         log.error(`  Error compiling ${file}:`);
         for (const error of result.errors) {
@@ -222,7 +239,13 @@ function processDirectory(srcDir, outDir, progress = null) {
       if (progress) progress.tick();
     } else if (file.endsWith('.js') || file.endsWith('.mjs')) {
       // Process JS files - rewrite imports
-      let content = readFileSync(srcPath, 'utf-8');
+      let content;
+      try {
+        content = readFileSync(srcPath, 'utf-8');
+      } catch (e) {
+        if (e.code !== 'ENOENT') throw e;
+        continue;
+      }
 
       // Rewrite .pulse imports to .js
       content = content.replace(/from\s+['"]([^'"]+)\.pulse['"]/g, "from '$1.js'");
@@ -283,8 +306,10 @@ function readRuntimeFile(filename) {
   ];
 
   for (const path of paths) {
-    if (existsSync(path)) {
+    try {
       return readFileSync(path, 'utf-8');
+    } catch (e) {
+      if (e.code !== 'ENOENT') throw e;
     }
   }
 
@@ -421,18 +446,15 @@ export function minifyJS(code) {
  * Copy a directory recursively
  */
 function copyDir(src, dest) {
-  if (!existsSync(dest)) {
-    mkdirSync(dest, { recursive: true });
-  }
+  mkdirSync(dest, { recursive: true });
 
-  const files = readdirSync(src);
+  const entries = readdirSync(src, { withFileTypes: true });
 
-  for (const file of files) {
-    const srcPath = join(src, file);
-    const destPath = join(dest, file);
-    const stat = statSync(srcPath);
+  for (const entry of entries) {
+    const srcPath = join(src, entry.name);
+    const destPath = join(dest, entry.name);
 
-    if (stat.isDirectory()) {
+    if (entry.isDirectory()) {
       copyDir(srcPath, destPath);
     } else {
       copyFileSync(srcPath, destPath);
@@ -448,24 +470,28 @@ export async function previewBuild(args) {
   const root = process.cwd();
   const distDir = join(root, 'dist');
 
-  if (!existsSync(distDir)) {
-    log.error('No dist folder found. Run "pulse build" first.');
-    process.exit(1);
+  try {
+    readdirSync(distDir);
+  } catch (e) {
+    if (e.code === 'ENOENT' || e.code === 'ENOTDIR') {
+      log.error('No dist folder found. Run "pulse build" first.');
+      process.exit(1);
+    }
+    throw e;
   }
 
   // Check if vite is available for preview
   try {
     const viteConfig = join(root, 'vite.config.js');
-    if (existsSync(viteConfig)) {
-      log.info('Using Vite preview...');
-      const { preview } = await import('vite');
-      const server = await preview({
-        root,
-        preview: { port }
-      });
-      server.printUrls();
-      return;
-    }
+    readFileSync(viteConfig, 'utf-8'); // Throws ENOENT if missing
+    log.info('Using Vite preview...');
+    const { preview } = await import('vite');
+    const server = await preview({
+      root,
+      preview: { port }
+    });
+    server.printUrls();
+    return;
   } catch (e) {
     // Vite not available, use built-in server
   }
@@ -497,7 +523,7 @@ export async function previewBuild(args) {
 
     const filePath = join(distDir, pathname);
 
-    if (existsSync(filePath) && statSync(filePath).isFile()) {
+    try {
       const ext = filePath.substring(filePath.lastIndexOf('.'));
       const mimeType = MIME_TYPES[ext] || 'application/octet-stream';
 
@@ -506,7 +532,8 @@ export async function previewBuild(args) {
         'Cache-Control': 'public, max-age=31536000'
       });
       res.end(readFileSync(filePath));
-    } else {
+    } catch (e) {
+      if (e.code !== 'ENOENT') throw e;
       res.writeHead(404, { 'Content-Type': 'text/plain' });
       res.end('Not Found');
     }

package/cli/dev.js

@@ -5,7 +5,7 @@
  */
 
 import { createServer } from 'http';
-import { readFileSync, existsSync, statSync, watch } from 'fs';
+import { readFileSync, watch } from 'fs';
 import { join, extname, resolve, dirname } from 'path';
 import { compile } from '../compiler/index.js';
 import { preprocessStylesSync, isSassAvailable, getSassVersion } from '../compiler/preprocessor.js';
@@ -76,17 +76,16 @@ export async function startDevServer(args) {
   // Check if vite is available, use it if so
   try {
     const viteConfig = join(root, 'vite.config.js');
-    if (existsSync(viteConfig)) {
-      log.info('Vite config detected, using Vite...');
-      const { createServer: createViteServer } = await import('vite');
-      const server = await createViteServer({
-        root,
-        server: { port }
-      });
-      await server.listen();
-      server.printUrls();
-      return;
-    }
+    readFileSync(viteConfig, 'utf-8'); // Throws ENOENT if missing
+    log.info('Vite config detected, using Vite...');
+    const { createServer: createViteServer } = await import('vite');
+    const server = await createViteServer({
+      root,
+      server: { port }
+    });
+    await server.listen();
+    server.printUrls();
+    return;
   } catch (e) {
     // Vite not available, use built-in server
   }
@@ -128,60 +127,61 @@ export async function startDevServer(args) {
 
     // Check for .pulse files and compile them
     if (pathname.endsWith('.pulse')) {
-      if (existsSync(filePath)) {
-        try {
-          const source = readFileSync(filePath, 'utf-8');
-          const result = compile(source, {
-            runtime: '/runtime/index.js',
-            sourceMap: true,
-            inlineSourceMap: true,
-            sourceFileName: pathname
-          });
+      try {
+        const source = readFileSync(filePath, 'utf-8');
+        const result = compile(source, {
+          runtime: '/runtime/index.js',
+          sourceMap: true,
+          inlineSourceMap: true,
+          sourceFileName: pathname
+        });
 
-          if (result.success) {
-            let code = result.code;
-
-            // Preprocess SASS/SCSS if available
-            if (sassAvailable) {
-              const stylesMatch = code.match(/const styles = `([\s\S]*?)`;/);
-              if (stylesMatch) {
-                try {
-                  const preprocessed = preprocessStylesSync(stylesMatch[1], {
-                    filename: filePath,
-                    loadPaths: [dirname(filePath)]
-                  });
-                  if (preprocessed.wasSass) {
-                    code = code.replace(stylesMatch[0], `const styles = \`${preprocessed.css}\`;`);
-                  }
-                } catch (sassError) {
-                  console.warn(`[Pulse] SASS warning: ${sassError.message}`);
+        if (result.success) {
+          let code = result.code;
+
+          // Preprocess SASS/SCSS if available
+          if (sassAvailable) {
+            const stylesMatch = code.match(/const styles = `([\s\S]*?)`;/);
+            if (stylesMatch) {
+              try {
+                const preprocessed = preprocessStylesSync(stylesMatch[1], {
+                  filename: filePath,
+                  loadPaths: [dirname(filePath)]
+                });
+                if (preprocessed.wasSass) {
+                  code = code.replace(stylesMatch[0], `const styles = \`${preprocessed.css}\`;`);
                 }
+              } catch (sassError) {
+                console.warn(`[Pulse] SASS warning: ${sassError.message}`);
               }
             }
-
-            res.writeHead(200, {
-              'Content-Type': 'application/javascript',
-              'Cache-Control': 'no-cache, no-store, must-revalidate'
-            });
-            res.end(code);
-          } else {
-            const errorDetails = result.errors.map(e => `${e.message} at line ${e.line || '?'}:${e.column || '?'}`).join('\n');
-            console.error(`[Pulse] Compilation error in ${filePath}:`, result.errors);
-            res.writeHead(500, { 'Content-Type': 'text/plain' });
-            res.end(`Compilation error: ${errorDetails}\nFile: ${filePath}`);
           }
-        } catch (error) {
+
+          res.writeHead(200, {
+            'Content-Type': 'application/javascript',
+            'Cache-Control': 'no-cache, no-store, must-revalidate'
+          });
+          res.end(code);
+        } else {
+          const errorDetails = result.errors.map(e => `${e.message} at line ${e.line || '?'}:${e.column || '?'}`).join('\n');
+          console.error(`[Pulse] Compilation error in ${filePath}:`, result.errors);
+          res.writeHead(500, { 'Content-Type': 'text/plain' });
+          res.end(`Compilation error: ${errorDetails}\nFile: ${filePath}`);
+        }
+        return;
+      } catch (error) {
+        if (error.code !== 'ENOENT') {
           console.error(`[Pulse] Error compiling ${filePath}:`, error);
           res.writeHead(500, { 'Content-Type': 'text/plain' });
           res.end(`Error: ${error.message}\nFile: ${filePath}`);
+          return;
         }
-        return;
       }
     }
 
     // Serve JS files
     if (pathname.endsWith('.js') || pathname.endsWith('.mjs')) {
-      if (existsSync(filePath)) {
+      try {
         const content = readFileSync(filePath, 'utf-8');
         res.writeHead(200, {
           'Content-Type': 'application/javascript',
@@ -189,62 +189,65 @@ export async function startDevServer(args) {
         });
         res.end(content);
         return;
+      } catch (e) {
+        if (e.code !== 'ENOENT') throw e;
       }
 
       // Check if there's a .pulse file that should be compiled to .js
       const pulseFilePath = filePath.replace(/\.(js|mjs)$/, '.pulse');
-      if (existsSync(pulseFilePath)) {
-        try {
-          const source = readFileSync(pulseFilePath, 'utf-8');
-          const result = compile(source, {
-            runtime: '/runtime/index.js',
-            sourceMap: true,
-            inlineSourceMap: true,
-            sourceFileName: pulseFilePath.replace(root, '')
-          });
+      try {
+        const source = readFileSync(pulseFilePath, 'utf-8');
+        const result = compile(source, {
+          runtime: '/runtime/index.js',
+          sourceMap: true,
+          inlineSourceMap: true,
+          sourceFileName: pulseFilePath.replace(root, '')
+        });
 
-          if (result.success) {
-            let code = result.code;
-
-            // Preprocess SASS/SCSS if available
-            if (sassAvailable) {
-              const stylesMatch = code.match(/const styles = `([\s\S]*?)`;/);
-              if (stylesMatch) {
-                try {
-                  const preprocessed = preprocessStylesSync(stylesMatch[1], {
-                    filename: pulseFilePath,
-                    loadPaths: [dirname(pulseFilePath)]
-                  });
-                  if (preprocessed.wasSass) {
-                    code = code.replace(stylesMatch[0], `const styles = \`${preprocessed.css}\`;`);
-                  }
-                } catch (sassError) {
-                  console.warn(`[Pulse] SASS warning: ${sassError.message}`);
+        if (result.success) {
+          let code = result.code;
+
+          // Preprocess SASS/SCSS if available
+          if (sassAvailable) {
+            const stylesMatch = code.match(/const styles = `([\s\S]*?)`;/);
+            if (stylesMatch) {
+              try {
+                const preprocessed = preprocessStylesSync(stylesMatch[1], {
+                  filename: pulseFilePath,
+                  loadPaths: [dirname(pulseFilePath)]
+                });
+                if (preprocessed.wasSass) {
+                  code = code.replace(stylesMatch[0], `const styles = \`${preprocessed.css}\`;`);
                 }
+              } catch (sassError) {
+                console.warn(`[Pulse] SASS warning: ${sassError.message}`);
               }
             }
-
-            res.writeHead(200, {
-              'Content-Type': 'application/javascript',
-              'Cache-Control': 'no-cache, no-store, must-revalidate'
-            });
-            res.end(code);
-          } else {
-            res.writeHead(500, { 'Content-Type': 'text/plain' });
-            res.end(`Compilation error: ${result.errors.map(e => e.message).join('\n')}`);
           }
-        } catch (error) {
+
+          res.writeHead(200, {
+            'Content-Type': 'application/javascript',
+            'Cache-Control': 'no-cache, no-store, must-revalidate'
+          });
+          res.end(code);
+        } else {
           res.writeHead(500, { 'Content-Type': 'text/plain' });
-          res.end(`Error: ${error.message}`);
+          res.end(`Compilation error: ${result.errors.map(e => e.message).join('\n')}`);
         }
         return;
+      } catch (error) {
+        if (error.code !== 'ENOENT') {
+          res.writeHead(500, { 'Content-Type': 'text/plain' });
+          res.end(`Error: ${error.message}`);
+          return;
+        }
       }
     }
 
     // Handle node_modules
     if (pathname.startsWith('/node_modules/pulse-js-framework/')) {
       const modulePath = join(root, '..', 'pulse', pathname.replace('/node_modules/pulse-js-framework/', ''));
-      if (existsSync(modulePath)) {
+      try {
         const content = readFileSync(modulePath, 'utf-8');
         res.writeHead(200, {
           'Content-Type': 'application/javascript',
@@ -252,6 +255,8 @@ export async function startDevServer(args) {
         });
         res.end(content);
         return;
+      } catch (e) {
+        if (e.code !== 'ENOENT') throw e;
       }
     }
 
@@ -267,7 +272,7 @@ export async function startDevServer(args) {
       ];
 
       for (const runtimePath of possiblePaths) {
-        if (existsSync(runtimePath)) {
+        try {
           const content = readFileSync(runtimePath, 'utf-8');
           res.writeHead(200, {
             'Content-Type': 'application/javascript',
@@ -275,12 +280,14 @@ export async function startDevServer(args) {
           });
           res.end(content);
           return;
+        } catch (e) {
+          if (e.code !== 'ENOENT') throw e;
         }
       }
     }
 
     // Serve static files
-    if (existsSync(filePath) && statSync(filePath).isFile()) {
+    try {
       const ext = extname(filePath);
       const mimeType = MIME_TYPES[ext] || 'application/octet-stream';
 
@@ -294,6 +301,8 @@ export async function startDevServer(args) {
       res.writeHead(200, { 'Content-Type': mimeType });
       res.end(content);
       return;
+    } catch (e) {
+      if (e.code !== 'ENOENT') throw e;
     }
 
     // SPA fallback: serve index.html for routes without file extensions
@@ -301,12 +310,14 @@ export async function startDevServer(args) {
     const ext = extname(pathname);
     if (!ext || ext === '') {
       const indexPath = join(root, 'index.html');
-      if (existsSync(indexPath)) {
+      try {
         let content = readFileSync(indexPath, 'utf-8');
         content = content.replace('</body>', LIVERELOAD_SCRIPT);
         res.writeHead(200, { 'Content-Type': 'text/html' });
         res.end(content);
         return;
+      } catch (e) {
+        if (e.code !== 'ENOENT') throw e;
       }
     }
 
@@ -336,7 +347,7 @@ function watchFiles(root) {
   const srcDir = join(root, 'src');
   let debounceTimer = null;
 
-  if (existsSync(srcDir)) {
+  try {
     watch(srcDir, { recursive: true }, (eventType, filename) => {
       if (filename && (filename.endsWith('.pulse') || filename.endsWith('.js') || filename.endsWith('.css'))) {
         // Debounce to avoid multiple reloads
@@ -348,6 +359,9 @@ function watchFiles(root) {
       }
     });
     log.debug('Watching src/ for changes...');
+  } catch (e) {
+    if (e.code !== 'ENOENT') throw e;
+    // No src directory, skip watching
   }
 }