@bonsae/nrg 0.5.3 → 0.5.4

package/build/vite/index.js

@@ -1594,7 +1594,9 @@ async function build2(clientBuildOptions, buildContext) {
     throw new BuildError("client", error);
   } finally {
     if (generatedEntry) {
-      fs10.unlinkSync(entryPath);
+      if (fs10.existsSync(entryPath)) {
+        fs10.unlinkSync(entryPath);
+      }
     }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bonsae/nrg",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "description": "NRG framework — build Node-RED nodes with Vue 3, TypeScript, and JSON Schema",
   "author": "Allan Oricil <allanoricil@duck.com>",
   "license": "MIT",
@@ -17,6 +17,10 @@
     "pnpm": ">=10.11.0"
   },
   "scripts": {
+    "test": "vitest run && vitest run --config vitest.e2e.config.ts",
+    "test:unit": "vitest run",
+    "test:unit:watch": "vitest",
+    "test:e2e": "vitest run --config vitest.e2e.config.ts",
     "build": "node build.mjs",
     "typecheck": "tsc -p src/core/server/tsconfig.json --noEmit && tsc -p src/core/client/tsconfig.json --noEmit",
     "lint": "eslint src/",
@@ -103,6 +107,7 @@
     "@types/node": "^22.15.18",
     "@typescript-eslint/eslint-plugin": "^8.32.1",
     "@typescript-eslint/parser": "^8.32.1",
+    "@vitest/coverage-v8": "^4.1.5",
     "eslint": "^9.27.0",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-vue": "^10.1.0",
@@ -113,6 +118,7 @@
     "semantic-release": "^24.2.4",
     "typescript-eslint": "^8.32.1",
     "vite": "^6.3.4",
+    "vitest": "^4.1.5",
     "vue": "^3.5.14"
   }
 }

package/src/core/client/app.vue

@@ -162,7 +162,7 @@ export default defineComponent({
             error.instancePath,
           );
           if (
-            error.parentSchema.format === "password" &&
+            error.parentSchema?.format === "password" &&
             errorValue === "__PWD__"
           ) {
             return acc;

package/src/core/client/components/node-red-json-schema-form.vue

@@ -197,7 +197,8 @@ interface FormField {
     | "select"
     | "typed"
     | "config"
-    | "editor";
+    | "editor"
+    | "array-text";
   required: boolean;
   htmlType?: "text" | "number" | "password";
   options?: Array<{ value: string; label: string }>;

package/src/core/client/components/node-red-typed-input.vue

@@ -136,6 +136,12 @@ export default defineComponent({
       this.onChange();
     });
   },
+  beforeUnmount() {
+    if (this._observer) {
+      this._observer.disconnect();
+      this._observer = null;
+    }
+  },
   methods: {
     onChange() {
       const newValue = this.$input.typedInput("value");

package/src/core/client/index.ts

@@ -203,7 +203,7 @@ function getNodeState(node: Node): NodeState {
   const state: NodeState = {
     credentials: {},
   };
-  Object.keys(node._def.defaults).forEach((prop) => {
+  Object.keys(node._def.defaults ?? {}).forEach((prop) => {
     state[prop] = node[prop];
   });
   if (node._def.credentials) {
@@ -300,8 +300,6 @@ function defineNode<T extends NodeDefinition>(options: T): T {
 async function registerType(definition: NodeDefinition): Promise<void> {
   const { type } = definition;
   try {
-    console.log(`Registering node type: ${type}`);
-
     const nodeDefinition = {
       ...(_schemas[type] ?? {}),
       ...definition,
@@ -311,9 +309,6 @@ async function registerType(definition: NodeDefinition): Promise<void> {
     const defaults = nodeDefinition.defaults ?? undefined;
     const credentials = nodeDefinition.credentials ?? undefined;
 
-    console.log("defaults", defaults);
-    console.log("credentials", credentials);
-
     const appContainerId = `nrg-app-${type}`;
 
     $("<script>", {
@@ -323,21 +318,20 @@ async function registerType(definition: NodeDefinition): Promise<void> {
     }).appendTo("body");
 
     function oneditprepare(this: Node) {
-      console.log("oneditprepare");
-      console.log(this);
-
-      const validationSchema = nodeDefinition.credentialsSchema?.properties
-        ? {
-            ...nodeDefinition.configSchema,
-            properties: {
-              ...nodeDefinition.configSchema.properties,
-              credentials: {
-                type: "object",
-                properties: nodeDefinition.credentialsSchema.properties,
+      const validationSchema =
+        nodeDefinition.configSchema &&
+        nodeDefinition.credentialsSchema?.properties
+          ? {
+              ...nodeDefinition.configSchema,
+              properties: {
+                ...nodeDefinition.configSchema.properties,
+                credentials: {
+                  type: "object",
+                  properties: nodeDefinition.credentialsSchema.properties,
+                },
               },
-            },
-          }
-        : nodeDefinition.configSchema;
+            }
+          : nodeDefinition.configSchema;
 
       const form =
         definition.form ??
@@ -360,7 +354,7 @@ async function registerType(definition: NodeDefinition): Promise<void> {
       const changed = !!Object.keys(changes)?.length;
       if (!changed) return false;
 
-      Object.keys(node._def.defaults).forEach((prop) => {
+      Object.keys(node._def.defaults ?? {}).forEach((prop) => {
         if (!node._def.defaults?.[prop]?.type) return;
         const oldConfigNodeId: string = node[prop] as string;
         const newConfigNodeId: string = node._newState![prop] as string;
@@ -376,7 +370,7 @@ async function registerType(definition: NodeDefinition): Promise<void> {
         }
       });
 
-      Object.keys(node._def.defaults).forEach((prop) => {
+      Object.keys(node._def.defaults ?? {}).forEach((prop) => {
         if (!node._def.defaults?.[prop]?.type) return;
         const newConfigNodeId: string = node._newState![prop] as string;
         if (!newConfigNodeId) return;
@@ -400,7 +394,7 @@ async function registerType(definition: NodeDefinition): Promise<void> {
       // overwriting the correctly-typed values already set by merge() above.
       const isConfigNode = definition.category === "config";
       if (isConfigNode) {
-        Object.keys(node._def.defaults).forEach((prop) => {
+        Object.keys(node._def.defaults ?? {}).forEach((prop) => {
           if (node._def.defaults[prop].type) return; // config-node refs handled separately
           const inputId = `node-config-input-${prop}`;
           let input = $(`#${inputId}`);
@@ -484,9 +478,7 @@ async function registerType(definition: NodeDefinition): Promise<void> {
  */
 async function registerTypes(nodes: NodeDefinition[]): Promise<void> {
   try {
-    console.log("Registering node types in parallel");
     await Promise.all(nodes.map((definition) => registerType(definition)));
-    console.log("All node types registered in parallel");
   } catch (error) {
     console.error("Error registering node types:", error);
     throw error;

package/src/vite/client/build.ts

@@ -217,7 +217,9 @@ async function build(
     throw new BuildError("client", error as Error);
   } finally {
     if (generatedEntry) {
-      fs.unlinkSync(entryPath);
+      if (fs.existsSync(entryPath)) {
+        fs.unlinkSync(entryPath);
+      }
     }
   }
 }