@onexapis/cli 1.1.18 → 1.1.20

package/dist/cli.js

@@ -2587,6 +2587,79 @@ async function validateCommand(options) {
       }
     }
   }
+  if (fs__default.default.existsSync(sectionsDir)) {
+    const sections = fs__default.default.readdirSync(sectionsDir).filter(
+      (name) => fs__default.default.statSync(path8__default.default.join(sectionsDir, name)).isDirectory()
+    );
+    for (const sectionName of sections) {
+      const sectionPath = path8__default.default.join(sectionsDir, sectionName);
+      const tsxFiles = fs__default.default.readdirSync(sectionPath).filter((f) => f.endsWith(".tsx") && !f.endsWith(".schema.ts"));
+      for (const tsxFile of tsxFiles) {
+        const filePath = path8__default.default.join(sectionPath, tsxFile);
+        const content = fs__default.default.readFileSync(filePath, "utf-8");
+        const relPath = `sections/${sectionName}/${tsxFile}`;
+        if (!content.includes('"use client"') && !content.includes("'use client'")) {
+          issues.push({
+            type: "error",
+            file: relPath,
+            message: 'Missing "use client" directive at top of file'
+          });
+        }
+        if (!content.includes("ComponentRenderer") && !content.includes("BlockRenderer")) {
+          issues.push({
+            type: "warning",
+            file: relPath,
+            message: "No ComponentRenderer or BlockRenderer found \u2014 sections should use core renderers for content"
+          });
+        }
+        if (!content.includes("data-section-id")) {
+          issues.push({
+            type: "error",
+            file: relPath,
+            message: "Missing data-section-id attribute \u2014 editor cannot select this section"
+          });
+        }
+        if (/\beval\s*\(/.test(content)) {
+          issues.push({
+            type: "error",
+            file: relPath,
+            message: "eval() detected \u2014 arbitrary code execution risk"
+          });
+        }
+        if (content.includes("document.cookie")) {
+          issues.push({
+            type: "error",
+            file: relPath,
+            message: "document.cookie access \u2014 session hijacking risk"
+          });
+        }
+        if (/\brequire\s*\(/.test(content)) {
+          issues.push({
+            type: "warning",
+            file: relPath,
+            message: "require() detected \u2014 themes should use ES module imports"
+          });
+        }
+      }
+    }
+  }
+  const registryPath = path8__default.default.join(themePath, "sections-registry.ts");
+  const bundleEntryPath = path8__default.default.join(themePath, "bundle-entry.ts");
+  const registryContent = fs__default.default.existsSync(registryPath) ? fs__default.default.readFileSync(registryPath, "utf-8") : fs__default.default.existsSync(bundleEntryPath) ? fs__default.default.readFileSync(bundleEntryPath, "utf-8") : "";
+  if (fs__default.default.existsSync(sectionsDir) && registryContent) {
+    const sections = fs__default.default.readdirSync(sectionsDir).filter(
+      (name) => fs__default.default.statSync(path8__default.default.join(sectionsDir, name)).isDirectory()
+    );
+    for (const sectionName of sections) {
+      if (!registryContent.includes(`sections/${sectionName}`) && !registryContent.includes(`"${sectionName}"`)) {
+        issues.push({
+          type: "warning",
+          file: `sections/${sectionName}/`,
+          message: "Section not found in sections-registry.ts or bundle-entry.ts \u2014 may not be included in build"
+        });
+      }
+    }
+  }
   logger.stopSpinner(true, "Validation complete");
   const errors = issues.filter((i) => i.type === "error");
   const warnings = issues.filter((i) => i.type === "warning");
@@ -2699,7 +2772,7 @@ async function buildCommand(options) {
   logger.stopSpinner(true, "Lint passed");
   const pkgJson = fs__default.default.readJsonSync(packageJsonPath);
   const buildScript = pkgJson.scripts?.build || "";
-  const isRecursive = buildScript.includes("onexthm build") || buildScript.includes("onex-cli build");
+  const isRecursive = buildScript.includes("onexthm build") || buildScript.includes("onex build") || buildScript.includes("onex-cli build");
   logger.startSpinner(
     options.watch ? "Building (watch mode)..." : "Building..."
   );
@@ -2733,18 +2806,36 @@ function runCommand(command, args, cwd) {
   return new Promise((resolve) => {
     const proc = child_process.spawn(command, args, {
       cwd,
-      stdio: "pipe",
+      stdio: ["pipe", "pipe", "pipe"],
       shell: true
     });
-    let hasError = false;
+    let stdout = "";
+    let stderr = "";
+    proc.stdout.on("data", (data) => {
+      stdout += data.toString();
+    });
     proc.stderr.on("data", (data) => {
-      const message = data.toString();
-      if (message.includes("error") || message.includes("Error") || message.includes("ERROR")) {
-        hasError = true;
-      }
+      stderr += data.toString();
     });
     proc.on("close", (code) => {
-      resolve(code === 0 && !hasError);
+      if (code !== 0) {
+        const output = (stderr + stdout).trim();
+        if (output) {
+          logger.newLine();
+          for (const line of output.split("\n")) {
+            const t = line.trim();
+            if (!t) continue;
+            if (t.includes("error") || t.includes("Error") || t.includes("TS")) {
+              logger.log(`  \u274C ${t}`);
+            } else if (t.includes("warning") || t.includes("Warning")) {
+              logger.log(`  \u26A0\uFE0F  ${t}`);
+            } else {
+              logger.log(`     ${t}`);
+            }
+          }
+        }
+      }
+      resolve(code === 0);
     });
     proc.on("error", () => {
       resolve(false);
@@ -4532,18 +4623,17 @@ async function publishCommand(options) {
   }
   logger.startSpinner("Building theme...");
   try {
-    const { execSync: execSync3 } = await import('child_process');
-    execSync3(
-      "npx tsup bundle-entry.ts --format esm --target es2020 --outDir dist",
-      {
-        cwd: themePath,
-        stdio: "ignore"
-      }
-    );
+    const { compileStandaloneTheme: compileStandaloneTheme2 } = await Promise.resolve().then(() => (init_compile_theme(), compile_theme_exports));
+    const buildSuccess = await compileStandaloneTheme2(themePath, themeId);
+    if (!buildSuccess) {
+      logger.stopSpinner(false, "Build failed");
+      logger.error("Run 'onexthm build' to see build errors");
+      process.exit(1);
+    }
     logger.stopSpinner(true, "Theme compiled");
-  } catch {
+  } catch (error) {
     logger.stopSpinner(false, "Build failed");
-    logger.error("Run 'onexthm build' to see build errors");
+    logger.error(error instanceof Error ? error.message : "Build error");
     process.exit(1);
   }
   logger.startSpinner("Getting upload URL...");
@@ -4583,7 +4673,12 @@ async function publishCommand(options) {
       process.exit(1);
     }
     const bundleZipPath = path8__default.default.join(themePath, "dist", "bundle.zip");
-    await createZip(distDir, bundleZipPath, ["bundle.zip"]);
+    await createZip(distDir, bundleZipPath, [
+      "bundle.zip",
+      "source.zip",
+      "preview-runtime.js",
+      "preview-runtime.js.map"
+    ]);
     const bundleBuffer = fs__default.default.readFileSync(bundleZipPath);
     const bundleRes = await fetch(bundleUploadUrl, {
       method: "PUT",
@@ -4679,7 +4774,7 @@ async function createZip(sourceDir, outputPath, exclude) {
     archive.pipe(output);
     archive.glob("**/*", {
       cwd: sourceDir,
-      ignore: exclude.map((e) => `${e}/**`),
+      ignore: exclude.flatMap((e) => [e, `${e}/**`]),
       dot: false
     });
     archive.finalize();