@wordpress-flow/cli 1.2.0 → 1.2.2

package/dist/build/block-build-worker.js

@@ -1837,7 +1837,7 @@ import * as fs from "fs";
 import * as esbuild from "esbuild";
 import { execSync } from "child_process";
 async function buildBlock() {
-  const { block, outputDir, webpackConfigPath, scriptsPath, tempDir } = workerData;
+  const { block, outputDir, scriptsOutputDir, webpackConfigPath, scriptsPath, tempDir } = workerData;
   try {
     fs.mkdirSync(tempDir, { recursive: true });
     const entryPoint = await generateEntryPoint(block, tempDir);
@@ -1845,7 +1845,7 @@ async function buildBlock() {
     fs.mkdirSync(blockOutputDir, { recursive: true });
     let scriptPaths = [];
     if (block.scripts && block.scripts.length > 0 && scriptsPath) {
-      scriptPaths = await buildBlockScripts(block.scripts, scriptsPath, outputDir);
+      scriptPaths = await buildBlockScripts(block.scripts, scriptsPath, scriptsOutputDir);
     }
     await generateBlockJson(block, blockOutputDir, tempDir);
     await generateSSRVersion(block, blockOutputDir);
@@ -1859,10 +1859,22 @@ async function buildBlock() {
     };
   } catch (error) {
     cleanupTempFiles(tempDir);
+    let errorMessage = "Unknown error";
+    if (error instanceof Error) {
+      errorMessage = error.message;
+      if (error.stack) {
+        errorMessage += `
+${error.stack}`;
+      }
+    } else if (typeof error === "string") {
+      errorMessage = error;
+    } else if (error) {
+      errorMessage = JSON.stringify(error);
+    }
     return {
       success: false,
       blockName: block.name,
-      error: error.message || String(error)
+      error: errorMessage
     };
   }
 }
@@ -1889,29 +1901,33 @@ if (document.readyState === 'loading') {
   fs.writeFileSync(entryPath, entryContent, "utf8");
   return entryPath;
 }
-async function buildBlockScripts(scripts, scriptsPath, outputDir) {
-  const scriptsOutputDir = path.join(outputDir, "scripts");
+async function buildBlockScripts(scripts, scriptsPath, scriptsOutputDir) {
   fs.mkdirSync(scriptsOutputDir, { recursive: true });
   const outputPaths = [];
   for (const script of scripts) {
     const scriptPath = path.resolve(scriptsPath, script);
     if (!fs.existsSync(scriptPath)) {
-      continue;
+      throw new Error(`Script file not found: ${scriptPath}`);
     }
     const scriptName = path.basename(script);
     const outputPath = path.join(scriptsOutputDir, scriptName.replace(/\.ts$/, ".js"));
-    await esbuild.build({
-      entryPoints: [scriptPath],
-      outfile: outputPath,
-      bundle: true,
-      minify: true,
-      platform: "browser",
-      target: "es2015",
-      format: "iife",
-      external: ["react", "react-dom", "@wordpress/*", "jquery"],
-      define: { "process.env.NODE_ENV": '"production"' },
-      logLevel: "warning"
-    });
+    try {
+      await esbuild.build({
+        entryPoints: [scriptPath],
+        outfile: outputPath,
+        bundle: true,
+        minify: true,
+        platform: "browser",
+        target: "es2015",
+        format: "iife",
+        external: ["react", "react-dom", "@wordpress/*", "jquery"],
+        define: { "process.env.NODE_ENV": '"production"' },
+        logLevel: "warning",
+        allowOverwrite: true
+      });
+    } catch (err) {
+      throw new Error(`Failed to build script ${script}: ${err.message}`);
+    }
     outputPaths.push(outputPath);
   }
   return outputPaths;

package/dist/index.js

@@ -81015,13 +81015,16 @@ class ConfigManager {
     const theme = raw.paths?.theme || DEFAULTS.paths.theme;
     const content2 = raw.paths?.content || DEFAULTS.paths.content;
     const plugins = raw.paths?.plugins || DEFAULTS.paths.plugins;
+    const dist = `${theme}/dist`;
     return {
       paths: {
         theme,
         content: content2,
         plugins,
         blocks: `${theme}/blocks`,
-        dist: `${theme}/dist`,
+        dist,
+        blocksDist: `${dist}/blocks`,
+        scriptsDist: `${dist}/scripts`,
         scripts: `${theme}/scripts`,
         templates: `${content2}/templates`,
         templateParts: `${content2}/parts`,
@@ -81167,7 +81170,7 @@ class ConfigManager {
       },
       build: {
         blocksDir: config.paths.blocks,
-        outputDir: config.paths.dist,
+        outputDir: config.paths.blocksDist,
         scriptsPath: config.paths.scripts
       },
       templates: {
@@ -113058,7 +113061,7 @@ class PushCommand {
       this.blockRegistry = blockRegistry;
     } else {
       const config = this.configManager.getConfig();
-      const buildOutputDir = this.configManager.resolvePath(config.paths.dist);
+      const buildOutputDir = this.configManager.resolvePath(config.paths.blocksDist);
       this.blockRegistry = new BlockRegistry(buildOutputDir);
     }
   }
@@ -114740,6 +114743,7 @@ import * as os from "os";
 class AbortableWorkerPool {
   concurrency;
   outputDir;
+  scriptsOutputDir;
   webpackConfigPath;
   scriptsPath;
   workerPath;
@@ -114747,6 +114751,7 @@ class AbortableWorkerPool {
   constructor(options) {
     this.concurrency = options.concurrency ?? Math.max(1, os.cpus().length - 1);
     this.outputDir = options.outputDir;
+    this.scriptsOutputDir = options.scriptsOutputDir;
     this.webpackConfigPath = options.webpackConfigPath;
     this.scriptsPath = options.scriptsPath;
     this.workerPath = path9.join(import.meta.dirname, "build", "block-build-worker.js");
@@ -114808,22 +114813,23 @@ class AbortableWorkerPool {
             results.push(result2);
             completed++;
             activeCount--;
-            onProgress?.(completed, total, result2.blockName, result2.success, result2.aborted || false);
+            onProgress?.(completed, total, result2.blockName, result2.success, result2.error);
             if (queue.length > 0) {
               processNext();
             } else if (activeCount === 0) {
               resolve4(results);
             }
           }).catch((error) => {
+            const errorMsg = error.message || String(error);
             const result2 = {
               success: false,
               blockName: block.name,
-              error: error.message || String(error)
+              error: errorMsg
             };
             results.push(result2);
             completed++;
             activeCount--;
-            onProgress?.(completed, total, block.name, false, false);
+            onProgress?.(completed, total, block.name, false, errorMsg);
             if (queue.length > 0) {
               processNext();
             } else if (activeCount === 0) {
@@ -114845,6 +114851,7 @@ class AbortableWorkerPool {
           scripts: block.scripts
         },
         outputDir: this.outputDir,
+        scriptsOutputDir: this.scriptsOutputDir,
         webpackConfigPath: this.webpackConfigPath,
         scriptsPath: this.scriptsPath,
         tempDir
@@ -116261,7 +116268,7 @@ class DockerEnvManager {
 // package.json
 var package_default = {
   name: "@wordpress-flow/cli",
-  version: "1.2.0",
+  version: "1.2.2",
   type: "module",
   description: "TypeScript-based WordPress block creation system",
   main: "dist/index.js",
@@ -116338,6 +116345,7 @@ class DevModeOrchestrator {
   isRunning = false;
   blocksDir;
   outputDir;
+  scriptsOutputDir;
   templatesDir;
   templatesOutputDir;
   templatePartsDir;
@@ -116458,7 +116466,8 @@ class DevModeOrchestrator {
   async initializeConfig() {
     const config = this.configManager.getConfig();
     this.blocksDir = this.configManager.resolvePath(config.paths.blocks);
-    this.outputDir = this.configManager.resolvePath(config.paths.dist);
+    this.outputDir = this.configManager.resolvePath(config.paths.blocksDist);
+    this.scriptsOutputDir = this.configManager.resolvePath(config.paths.scriptsDist);
     this.webpackConfig = path17.join(__dirname2, "..", "webpack.config.cjs");
     this.scriptsPath = this.configManager.resolvePath(config.paths.scripts);
     this.contentDir = this.configManager.resolvePath(config.paths.content);
@@ -116469,11 +116478,15 @@ class DevModeOrchestrator {
     if (!fs15.existsSync(this.outputDir)) {
       fs15.mkdirSync(this.outputDir, { recursive: true });
     }
+    if (!fs15.existsSync(this.scriptsOutputDir)) {
+      fs15.mkdirSync(this.scriptsOutputDir, { recursive: true });
+    }
   }
   initializeManagers() {
     this.workerPool = new AbortableWorkerPool({
       concurrency: this.options.concurrency,
       outputDir: this.outputDir,
+      scriptsOutputDir: this.scriptsOutputDir,
       webpackConfigPath: this.webpackConfig,
       scriptsPath: this.scriptsPath
     });
@@ -116552,12 +116565,12 @@ class DevModeOrchestrator {
       let successCount = 0;
       let failCount = 0;
       const blockScripts = new Map;
-      const results = await this.workerPool.buildAll(blocksToRebuild, (completed, total, blockName, success) => {
+      const results = await this.workerPool.buildAll(blocksToRebuild, (completed, total, blockName, success, error) => {
         if (success) {
           console.log(`   ✅ ${blockName} [${completed}/${total}]`);
           successCount++;
         } else {
-          console.log(`   ❌ ${blockName} failed [${completed}/${total}]`);
+          console.log(`   ❌ ${blockName} failed [${completed}/${total}]: ${error || "unknown error"}`);
           failCount++;
         }
       });
@@ -116575,7 +116588,8 @@ class DevModeOrchestrator {
         }
       }
       if (blockScripts.size > 0) {
-        const phpPath = path17.join(path17.dirname(this.outputDir), "functions-blocks-scripts.php");
+        const distDir = path17.dirname(this.outputDir);
+        const phpPath = path17.join(distDir, "functions-blocks-scripts.php");
         this.phpGenerator.generateScriptRegistrationPHP(blockScripts, phpPath);
       }
       await this.generateTypeDefinitions();
@@ -117610,12 +117624,14 @@ import * as os2 from "os";
 class WorkerPool {
   concurrency;
   outputDir;
+  scriptsOutputDir;
   webpackConfigPath;
   scriptsPath;
   workerPath;
   constructor(options) {
     this.concurrency = options.concurrency ?? Math.max(1, os2.cpus().length - 1);
     this.outputDir = options.outputDir;
+    this.scriptsOutputDir = options.scriptsOutputDir;
     this.webpackConfigPath = options.webpackConfigPath;
     this.scriptsPath = options.scriptsPath;
     this.workerPath = path21.join(import.meta.dirname, "build", "block-build-worker.js");
@@ -117636,7 +117652,7 @@ class WorkerPool {
             results.push(result2);
             completed++;
             activeWorkers--;
-            onProgress?.(completed, total, result2.blockName, result2.success);
+            onProgress?.(completed, total, result2.blockName, result2.success, result2.error);
             if (queue.length > 0) {
               processNext();
             } else if (activeWorkers === 0) {
@@ -117651,7 +117667,7 @@ class WorkerPool {
             results.push(result2);
             completed++;
             activeWorkers--;
-            onProgress?.(completed, total, block.name, false);
+            onProgress?.(completed, total, block.name, false, result2.error);
             if (queue.length > 0) {
               processNext();
             } else if (activeWorkers === 0) {
@@ -117673,6 +117689,7 @@ class WorkerPool {
           scripts: block.scripts
         },
         outputDir: this.outputDir,
+        scriptsOutputDir: this.scriptsOutputDir,
         webpackConfigPath: this.webpackConfigPath,
         scriptsPath: this.scriptsPath,
         tempDir
@@ -117726,7 +117743,7 @@ class BuildCommand {
     try {
       const config = this.configManager.getConfig();
       const blocksDir = this.configManager.resolvePath(options.blocksDir || config.paths.blocks);
-      const outputDir = this.configManager.resolvePath(options.outputDir || config.paths.dist);
+      const outputDir = this.configManager.resolvePath(options.outputDir || config.paths.blocksDist);
       let webpackConfig;
       if (options.webpackConfig) {
         webpackConfig = this.configManager.resolvePath(options.webpackConfig);
@@ -117763,17 +117780,19 @@ class BuildCommand {
       } else {
         logger.info(`Found ${blocks.length} block(s): ${blocks.map((b) => b.name).join(", ")}`);
       }
+      const scriptsOutputDir = this.configManager.resolvePath(config.paths.scriptsDist);
       const workerPool = new WorkerPool({
         concurrency: options.concurrency,
         outputDir,
+        scriptsOutputDir,
         webpackConfigPath: webpackConfig,
         scriptsPath
       });
-      const results = await workerPool.buildAll(blocks, (completed, total, blockName, success) => {
+      const results = await workerPool.buildAll(blocks, (completed, total, blockName, success, error) => {
         if (success) {
           logger.success(`✅ Built: ${blockName} [${completed}/${total}]`);
         } else {
-          logger.error(`❌ Failed: ${blockName} [${completed}/${total}]`);
+          logger.error(`❌ Failed: ${blockName} [${completed}/${total}]: ${error || "unknown error"}`);
         }
       });
       for (const block of blocks) {
@@ -117794,7 +117813,8 @@ Failed to build ${failures.length} block(s):`);
         }
       }
       if (this.blockScripts.size > 0) {
-        const phpPath = path22.join(path22.dirname(outputDir), "functions-blocks-scripts.php");
+        const distDir = path22.dirname(outputDir);
+        const phpPath = path22.join(distDir, "functions-blocks-scripts.php");
         this.phpGenerator.generateScriptRegistrationPHP(this.blockScripts, phpPath);
       }
       await this.generateTypeDefinitions(outputDir);
@@ -117849,7 +117869,7 @@ class BuildTemplatesCommand {
       return;
     }
     fs19.mkdirSync(outputDir, { recursive: true });
-    const blocksOutputDir = this.configManager.resolvePath(config.paths.dist);
+    const blocksOutputDir = this.configManager.resolvePath(config.paths.blocksDist);
     const blockRegistry = new BlockRegistry(blocksOutputDir);
     await blockRegistry.loadBuiltBlocks();
     logger.info(`Loaded ${blockRegistry.getAllComponentMappings().length} built blocks`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wordpress-flow/cli",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "type": "module",
   "description": "TypeScript-based WordPress block creation system",
   "main": "dist/index.js",