@cyprnet/node-red-contrib-uibuilder-formgen 0.5.12 → 0.5.13

package/CHANGELOG.md

@@ -6,6 +6,10 @@ All notable changes to this package will be documented in this file.
 
 - Schema Builder: added a **Convert multiline → arrays** helper in the Lookups editor to migrate older lookup lists (newline strings) into arrays-of-strings.
 
+## 0.5.13
+
+- Schema Builder: Field editor now **stays open on validation errors** so dynamic Lookup/Auto-fill settings (e.g. Node-RED source requiring Lookup ID) don’t appear to “revert” after clicking OK.
+
 ## 0.5.11
 
 - Lookup / Auto-fill: lookup items can now store lists as **arrays** (e.g. <code>["a","b"]</code>) and the portal will format arrays for textarea/keyvalue targets. Builder Lookups editor now includes a **Simple list** mode (one item per line → JSON array).

package/examples/formgen-builder/src/index.js

@@ -942,16 +942,25 @@
           this.$bvModal.show('field-modal');
         },
         
-        saveField() {
+        saveField(bvModalEvt) {
+          const preventClose = () => {
+            if (bvModalEvt && typeof bvModalEvt.preventDefault === 'function') {
+              bvModalEvt.preventDefault();
+            }
+          };
+          const fail = (msg) => {
+            preventClose();
+            this.showAlertMessage(msg, 'warning');
+            return false;
+          };
+
           if (!this.currentField.id || !this.currentField.label) {
-            this.showAlertMessage('Field ID and Label are required', 'warning');
-            return;
+            return fail('Field ID and Label are required');
           }
           
           // Validate field ID
           if (!/^[a-zA-Z0-9._-]+$/.test(this.currentField.id)) {
-            this.showAlertMessage('Field ID must be alphanumeric with underscores or hyphens', 'warning');
-            return;
+            return fail('Field ID must be alphanumeric with underscores or hyphens');
           }
           
           const { sectionIdx, fieldIdx } = this.editingFieldLocation;
@@ -961,8 +970,7 @@
           if (fieldIdx === null) {
             const exists = section.fields.some(f => f.id === this.currentField.id);
             if (exists) {
-              this.showAlertMessage('Field ID already exists in this section', 'warning');
-              return;
+              return fail('Field ID already exists in this section');
             }
           } else {
             // Check if ID changed and conflicts
@@ -970,8 +978,7 @@
             if (oldField.id !== this.currentField.id) {
               const exists = section.fields.some((f, idx) => idx !== fieldIdx && f.id === this.currentField.id);
               if (exists) {
-                this.showAlertMessage('Field ID already exists in this section', 'warning');
-                return;
+                return fail('Field ID already exists in this section');
               }
             }
           }
@@ -1022,8 +1029,7 @@
             delete field.options;
           } else if (this.currentField.type === 'select' || this.currentField.type === 'radio') {
             if (!this.currentField.options || this.currentField.options.length === 0) {
-              this.showAlertMessage(`${this.currentField.type} fields require at least one option`, 'warning');
-              return;
+              return fail(`${this.currentField.type} fields require at least one option`);
             }
             // Filter out empty options and ensure both value and text exist
             const validOptions = this.currentField.options
@@ -1034,8 +1040,7 @@
               }));
             
             if (validOptions.length === 0) {
-              this.showAlertMessage(`${this.currentField.type} fields require at least one valid option`, 'warning');
-              return;
+              return fail(`${this.currentField.type} fields require at least one valid option`);
             }
             
             field.options = validOptions;
@@ -1053,8 +1058,7 @@
             
             if (field.keyvalueMode === 'pairs') {
               if (!this.currentField.pairs || this.currentField.pairs.length === 0) {
-                this.showAlertMessage('Key-Value fields require at least one pair', 'warning');
-                return;
+                return fail('Key-Value fields require at least one pair');
               }
               // Filter out empty pairs and ensure both key and value exist
               const validPairs = this.currentField.pairs
@@ -1065,16 +1069,14 @@
                 }));
               
               if (validPairs.length === 0) {
-                this.showAlertMessage('Key-Value fields require at least one valid pair with a key', 'warning');
-                return;
+                return fail('Key-Value fields require at least one valid pair with a key');
               }
               
               field.pairs = validPairs;
             } else {
               // Delimiter mode - no pairs needed, but validate delimiter
               if (!field.keyvalueDelimiter || field.keyvalueDelimiter.trim() === '') {
-                this.showAlertMessage('Delimiter is required for delimiter mode', 'warning');
-                return;
+                return fail('Delimiter is required for delimiter mode');
               }
             }
             
@@ -1120,31 +1122,26 @@
               .filter(m => m.targetField && m.valueKey);
 
             if (!fromField) {
-              this.showAlertMessage('Lookup/Auto-fill: "From field" is required when enabled', 'warning');
-              return;
+              return fail('Lookup/Auto-fill: "From field" is required when enabled');
             }
             if (!matchKey) {
-              this.showAlertMessage('Lookup/Auto-fill: "Match key" is required when enabled', 'warning');
-              return;
+              return fail('Lookup/Auto-fill: "Match key" is required when enabled');
             }
             if (cleanedMappings.length === 0) {
-              this.showAlertMessage('Lookup/Auto-fill: add at least one mapping (target field + value key)', 'warning');
-              return;
+              return fail('Lookup/Auto-fill: add at least one mapping (target field + value key)');
             }
 
             let source = null;
             if (sourceType === 'uibuilder') {
               const lookupId = String(af.lookupId || '').trim();
               if (!lookupId) {
-                this.showAlertMessage('Lookup/Auto-fill: "Lookup ID" is required for Node-RED (dynamic) source', 'warning');
-                return;
+                return fail('Lookup/Auto-fill: "Lookup ID" is required for Node-RED (dynamic) source');
               }
               source = { type: 'uibuilder', lookupId };
             } else {
               const listName = String(af.schemaListName || '').trim();
               if (!listName) {
-                this.showAlertMessage('Lookup/Auto-fill: "List name" is required for Schema (static) source', 'warning');
-                return;
+                return fail('Lookup/Auto-fill: "List name" is required for Schema (static) source');
               }
               source = { type: 'schema', path: `lookups.${listName}` };
 
@@ -1156,8 +1153,7 @@
                   if (!this.schema.lookups) this.$set(this.schema, 'lookups', {});
                   this.$set(this.schema.lookups, listName, parsed);
                 } catch (e) {
-                  this.showAlertMessage('Lookup/Auto-fill: schema list JSON is invalid', 'warning');
-                  return;
+                  return fail('Lookup/Auto-fill: schema list JSON is invalid');
                 }
               }
             }

package/nodes/uibuilder-formgen-v3.js

@@ -455,7 +455,7 @@ module.exports = function(RED) {
         await fs.writeJson(targetSchema, schema, { spaces: 2 });
 
         const runtimeData = {
-          generatorVersion: "0.5.12",
+          generatorVersion: "0.5.13",
           generatorNode: "uibuilder-formgen-v3",
           timestamp: new Date().toISOString(),
           instanceName: instanceName,

package/nodes/uibuilder-formgen.js

@@ -484,7 +484,7 @@ module.exports = function(RED) {
 
         // Write runtime metadata
         const runtimeData = {
-          generatorVersion: "0.5.12",
+          generatorVersion: "0.5.13",
           timestamp: new Date().toISOString(),
           instanceName: instanceName,
           storageMode: storageMode,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cyprnet/node-red-contrib-uibuilder-formgen",
-  "version": "0.5.12",
+  "version": "0.5.13",
   "description": "PortalSmith: Generate schema-driven uibuilder form portals from JSON",
   "keywords": [
     "node-red",