@onexapis/cli 1.1.18 → 1.1.19

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);