collect errors in build

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "behavior3editor",
-  "version": "1.8.4",
+  "version": "1.8.5",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "behavior3editor",
-      "version": "1.8.4",
+      "version": "1.8.5",
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "behavior3editor",
-  "version": "1.8.4",
+  "version": "1.8.5",
   "main": "dist-electron/main/index.js",
   "description": "行为树编辑器",
   "author": "zhandouxiaojiji & codetypes",

sample/scripts/build.js

@@ -44,12 +44,12 @@ export const onSetup = (env) => {
   console.log(env.fs.readFileSync("sample/scripts/build.js", "utf8"));
 };
 
-export const onProcessTree = (tree) => {
+export const onProcessTree = (tree, errors) => {
   console.log(`processTree ${tree.name}`);
   return tree;
 };
 
-export const onProcessNode = (node, tree) => {
+export const onProcessNode = (node, tree, errors) => {
   console.log(`processNode ${tree.name} ${node.name}#${node.id}`);
   return node;
 };

src/contexts/workspace-context.ts

@@ -426,7 +426,12 @@ export const useWorkspace = create<WorkspaceStore>((set, get) => ({
         });
         workspace.allFiles.forEach((file) => {
           let tree: TreeData | null = readTree(file.path);
-          tree = b3util.processBatch(tree, file.path, batch);
+          const errors: string[] = [];
+          tree = b3util.processBatch(tree, file.path, batch, errors);
+          if (errors.length) {
+            errors.forEach((error) => message.error(error));
+            hasError = true;
+          }
           if (tree) {
             batch.onWriteFile?.(file.path, tree);
             writeTree(file.path, tree);

src/misc/b3type.ts

@@ -1,6 +1,6 @@
 import { NodeDef } from "../behavior3/src/behavior3";
 
-export const VERSION = "1.8.4";
+export const VERSION = "1.8.5";
 
 export const keyWords = ["true", "false", "null", "undefined", "NaN", "Infinity"];
 

src/misc/b3util.ts

@@ -37,8 +37,8 @@ type Env = {
 
 export interface BatchScript {
   onSetup?(env: Env): void;
-  onProcessTree?(tree: TreeData, path: string): TreeData | null;
+  onProcessTree?(tree: TreeData, path: string, errors: string[]): TreeData | null;
-  onProcessNode?(node: NodeData): NodeData | null;
+  onProcessNode?(node: NodeData, errors: string[]): NodeData | null;
   onWriteFile?(path: string, tree: TreeData): void;
   onComplete?(status: "success" | "failure"): void;
 }
@@ -767,12 +767,17 @@ export const createBuildData = (path: string) => {
   return null;
 };
 
-export const processBatch = (tree: TreeData | null, path: string, batch: BatchScript) => {
+export const processBatch = (
+  tree: TreeData | null,
+  path: string,
+  batch: BatchScript,
+  errors: string[]
+) => {
   if (!tree) {
     return null;
   }
   if (batch.onProcessTree) {
-    tree = batch.onProcessTree(tree, path);
+    tree = batch.onProcessTree(tree, path, errors);
   }
   if (!tree) {
     return null;
@@ -789,7 +794,7 @@ export const processBatch = (tree: TreeData | null, path: string, batch: BatchSc
         });
         node.children = children;
       }
-      return batch.onProcessNode?.(node);
+      return batch.onProcessNode?.(node, errors);
     };
     tree.root = processNode(tree.root) ?? ({} as NodeData);
   }
@@ -823,10 +828,10 @@ export const buildProject = async (project: string, buildDir: string) => {
   for (const path of Path.ls(Path.dirname(project), true)) {
     if (path.endsWith(".json")) {
       const buildpath = buildDir + "/" + path.substring(workdir.length + 1);
-      console.log("build:", buildpath);
       let tree = createBuildData(path);
+      const errors: string[] = [];
       if (buildScript) {
-        tree = processBatch(tree, path, buildScript);
+        tree = processBatch(tree, path, buildScript, errors);
       }
       if (!tree) {
         continue;
@@ -835,6 +840,11 @@ export const buildProject = async (project: string, buildDir: string) => {
         console.log("skip:", buildpath);
         continue;
       }
+      console.log("build:", buildpath);
+      if (errors.length) {
+        errors.forEach((msg) => console.error(msg));
+        hasError = true;
+      }
       const declare: FileVarDecl = {
         import: tree.import.map((v) => ({ path: v, vars: [], depends: [] })),
         vars: tree.vars.map((v) => ({ name: v.name, desc: v.desc })),