@siliconst/metaui 1.4.34 → 1.4.37

package/src/nz/co/siliconst/ui5/metaui/controls/DynamicHost.ts

@@ -5,6 +5,11 @@ import { Logger } from "../utils/Logger";
 import { ODataDelegate } from "./host/delegates/ODataDelegate";
 import ODataV4Context from "sap/ui/model/odata/v4/Context";
 import { SchemaBuilderRegistry } from "../core/SchemaBuilderRegistry";
+import BusyIndicator from "sap/ui/core/BusyIndicator";
+import MessageToast from "sap/m/MessageToast";
+import MessageBox from "sap/m/MessageBox";
+import { SchemaProvider } from "../services/SchemaProvider";
+import { AIConfig } from "../ai/AIConfig";
 
 /**
  * A transparent wrapper control (Facade) that automatically chooses between Explicit Schema Mode 
@@ -111,7 +116,7 @@ export default class DynamicHost extends Control {
             }
 
             // Guard: If the user explicitly bound 'data' to a specific property path, do not steal the context.
-            const dataBinding = this.getBindingInfo("data") as any;
+            const dataBinding = this.getBindingInfo("data") as Record<string, unknown> | undefined;
             if (dataBinding && dataBinding.path && dataBinding.path !== "") {
                 return;
             }
@@ -139,8 +144,8 @@ export default class DynamicHost extends Control {
      * @param {string} [sModelName] Optional model name
      * @returns {this} The control instance for chaining
      */
-    public setBindingContext(oContext: any, sModelName?: string): this {
-        super.setBindingContext(oContext, sModelName);
+    public setBindingContext(oContext: unknown, sModelName?: string): this {
+        super.setBindingContext(oContext as sap.ui.model.Context, sModelName);
         this.initODataDelegate();
         return this;
     }
@@ -154,8 +159,8 @@ export default class DynamicHost extends Control {
      * @param {object} [mParameters] Optional mapping parameters
      * @returns {this} Reference to this instance for chaining
      */
-    public bindElement(vPath: any, mParameters?: object): this {
-        super.bindElement(vPath, mParameters);
+    public bindElement(vPath: unknown, mParameters?: object): this {
+        super.bindElement(vPath as string | { path: string }, mParameters);
         const sModelName = typeof vPath === "object" ? vPath.model : undefined;
         const oBinding = this.getElementBinding(sModelName);
         if (oBinding) {
@@ -180,41 +185,33 @@ export default class DynamicHost extends Control {
         if (!this._initializedInner) {
             let schemaRaw = this.getProperty("schemaDefinition");
 
-            // 1. URL Detection & Fetching
             if (typeof schemaRaw === "string" && (schemaRaw.startsWith("http://") || schemaRaw.startsWith("https://"))) {
                 const self = this as unknown as { _fetchingSchema?: boolean };
                 if (self._fetchingSchema) return; // Prevent re-entry
                 self._fetchingSchema = true;
                 
-                sap.ui.require(["sap/ui/core/BusyIndicator", "sap/m/MessageToast"], (BusyIndicator: { show: (n: number) => void, hide: () => void }, MessageToast: { show: (s: string) => void }) => {
-                    BusyIndicator.show(0);
-                    fetch(schemaRaw as string)
-                        .then(res => {
-                            if (!res.ok) throw new Error("HTTP " + res.status);
-                            return res.json();
-                        })
-                        .then(json => {
-                            const plugin = SchemaBuilderRegistry.getBuilderFor(json);
-                            const finalSchema = plugin ? plugin.build(json, this.getProperty("schemaTarget") as string | undefined) : json;
-                            super.setProperty("schemaDefinition", finalSchema, true);
-                            if (finalSchema.definitions) {
-                                super.setProperty("schemaDefinitions", finalSchema.definitions, true);
-                            }
-                            const self = this as unknown as { _fetchingSchema?: boolean };
-                            self._fetchingSchema = false;
-                            BusyIndicator.hide();
-                            this.invalidate(); // Force re-render with resolved schema
-                        })
-                        .catch(err => {
-                            const self = this as unknown as { _fetchingSchema?: boolean };
-                            Logger.warn("[MetaUI DynamicHost]", "Failed to fetch remote schema: " + err, "DynamicHost");
-                            if (MessageToast) MessageToast.show("Failed to fetch remote schema");
-                            super.setProperty("schemaDefinition", null, true); // Fallback to full inference
-                            self._fetchingSchema = false;
-                            BusyIndicator.hide();
-                            this.invalidate();
-                        });
-                });
+                BusyIndicator.show(0);
+                SchemaProvider.resolve(schemaRaw)
+                    .then(json => {
+                        const plugin = SchemaBuilderRegistry.getBuilderFor(json);
+                        const finalSchema = plugin ? plugin.build(json, this.getProperty("schemaTarget") as string | undefined) : json;
+                        super.setProperty("schemaDefinition", finalSchema, true);
+                        if (finalSchema.definitions) {
+                            super.setProperty("schemaDefinitions", finalSchema.definitions, true);
+                        }
+                        const self = this as unknown as { _fetchingSchema?: boolean };
+                        self._fetchingSchema = false;
+                        BusyIndicator.hide();
+                        this.invalidate(); // Force re-render with resolved schema
+                    })
+                    .catch(err => {
+                        const self = this as unknown as { _fetchingSchema?: boolean };
+                        MessageToast.show("Failed to fetch remote schema");
+                        super.setProperty("schemaDefinition", null, true); // Fallback to full inference
+                        self._fetchingSchema = false;
+                        BusyIndicator.hide();
+                        this.invalidate();
+                    });
                 return; // Abort this render cycle until fetch completes
             }
 
@@ -234,8 +231,8 @@ export default class DynamicHost extends Control {
                     schema = plugin.build(schema, this.getProperty("schemaTarget") as string | undefined);
                     // Silently update the wrapper property so the inner host gets the final parsed MetaUI format
                     super.setProperty("schemaDefinition", schema, true);
-                    if ((schema as any).definitions) {
-                        super.setProperty("schemaDefinitions", (schema as any).definitions, true);
+                    if ((schema as Record<string, unknown>).definitions) {
+                        super.setProperty("schemaDefinitions", (schema as Record<string, unknown>).definitions, true);
                     }
                 }
             }
@@ -244,7 +241,7 @@ export default class DynamicHost extends Control {
             const data = this.getProperty("data");
 
             let hasSchema = !!schema;
-            if (hasSchema && typeof schema === "object" && Object.keys(schema as any).length === 0) {
+            if (hasSchema && typeof schema === "object" && Object.keys(schema as Record<string, unknown>).length === 0) {
                 hasSchema = false; // Treat empty object as no schema
             }
 
@@ -258,36 +255,34 @@ export default class DynamicHost extends Control {
 
                 let rawDataObj = data;
                 if (!rawDataObj && dataJson) {
-                    try { rawDataObj = JSON.parse(dataJson as string); } catch(e: any) { Logger.debug("[DynamicHost] Ignored invalid dataJson parse: " + e.message); }
+                    try { rawDataObj = JSON.parse(dataJson as string); } catch(e: unknown) { 
+                        const msg = "Ignored invalid dataJson parse: " + (e as Error).message;
+                        Logger.error("[DynamicHost]", msg, "DynamicHost"); 
+                        MessageBox.error(msg, { title: "Invalid JSON Data" });
+                    }
                 }
 
                 if (rawDataObj) {
-                    sap.ui.require([
-                        "sap/ui/core/BusyIndicator", 
-                        "sap/m/MessageToast",
-                        "nz/co/siliconst/ui5/metaui/ai/AIConfig"
-                    ], (BusyIndicator: { show: (n: number) => void, hide: () => void }, MessageToast: { show: (s: string) => void }, AIConfig: { getProxy: () => { generateSchema: (d: unknown, s?: unknown) => Promise<unknown> } }) => {
-                        BusyIndicator.show(0);
-                        const proxy = AIConfig.getProxy();
-                        
-                        proxy.generateSchema(rawDataObj, schema || undefined)
-                            .then((generatedSchema: unknown) => {
-                                const self = this as unknown as { _fetchingSchema?: boolean };
-                                super.setProperty("schemaDefinition", generatedSchema, true);
-                                self._fetchingSchema = false;
-                                BusyIndicator.hide();
-                                this.invalidate(); // Force re-render with the new schema
-                            })
-                            .catch((err: Error) => {
-                                const self = this as unknown as { _fetchingSchema?: boolean };
-                                Logger.warn("[MetaUI DynamicHost]", "AI inference failed, falling back to RuleBased. Error: " + err.message, "DynamicHost");
-                                if (MessageToast) MessageToast.show("AI Inference failed, using fallback.");
-                                super.setProperty("inferenceStrategy", "RuleBased", true); // Fallback to normal inference
-                                self._fetchingSchema = false;
-                                BusyIndicator.hide();
-                                this.invalidate(); 
-                            });
-                    });
+                    BusyIndicator.show(0);
+                    const proxy = AIConfig.getProxy();
+                    
+                    proxy.generateSchema(rawDataObj, schema || undefined)
+                        .then((generatedSchema: unknown) => {
+                            const self = this as unknown as { _fetchingSchema?: boolean };
+                            super.setProperty("schemaDefinition", generatedSchema, true);
+                            self._fetchingSchema = false;
+                            BusyIndicator.hide();
+                            this.invalidate(); // Force re-render with the new schema
+                        })
+                        .catch((err: Error) => {
+                            const self = this as unknown as { _fetchingSchema?: boolean };
+                            Logger.warn("[MetaUI DynamicHost]", "AI inference failed, falling back to RuleBased. Error: " + err.message, "DynamicHost");
+                            MessageToast.show("AI Inference failed, using fallback.");
+                            super.setProperty("inferenceStrategy", "RuleBased", true); // Fallback to normal inference
+                            self._fetchingSchema = false;
+                            BusyIndicator.hide();
+                            this.invalidate(); 
+                        });
                     return; // Abort render cycle until AI is done
                 }
             }
@@ -328,7 +323,7 @@ export default class DynamicHost extends Control {
                 // and perfectly receive the inner host's submission event.
                 const events = this.getMetadata().getEvents();
                 for (const eventName in events) {
-                    this._innerHost.attachEvent(eventName, (oEvent: any) => {
+                    this._innerHost.attachEvent(eventName, (oEvent: sap.ui.base.Event) => {
                         // Natively push updated payload out to any bound Fiori Element properties.
                         // Using super.setProperty prevents an infinite loop back down to _innerHost.
                         if (eventName === "submit" || eventName === "fieldChange") {

package/src/nz/co/siliconst/ui5/metaui/controls/host/GeneratorHost-dbg.js

@@ -1,4 +1,4 @@
-sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNormalizer", "../../core/SchemaValidator", "../../core/Engine", "../../core/StateManager", "./delegates/DataSyncDelegate", "./delegates/ValidationDelegate", "./delegates/DialogDelegate", "sap/m/MessageBox", "sap/m/MessageStrip", "sap/m/VBox", "../../core/PluginRegistry", "../../core/LayoutScorer", "../../core/LayoutMutator", "../../core/DefaultLayoutGenerator"], function (Control, ____utils_Logger, ____core_SchemaNormalizer, ____core_SchemaValidator, ____core_Engine, ____core_StateManager, ___delegates_DataSyncDelegate, ___delegates_ValidationDelegate, ___delegates_DialogDelegate, MessageBox, MessageStrip, VBox, ____core_PluginRegistry, ____core_LayoutScorer, ____core_LayoutMutator, ____core_DefaultLayoutGenerator) {
+sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNormalizer", "../../core/SchemaValidator", "../../core/Engine", "../../core/StateManager", "../../core/PolicyEngine", "./delegates/DataSyncDelegate", "./delegates/ValidationDelegate", "./delegates/DialogDelegate", "sap/m/MessageBox", "sap/m/MessageStrip", "sap/m/VBox", "../../core/PluginRegistry", "../../core/LayoutScorer", "../../core/LayoutMutator", "../../core/DefaultLayoutGenerator"], function (Control, ____utils_Logger, ____core_SchemaNormalizer, ____core_SchemaValidator, ____core_Engine, ____core_StateManager, ____core_PolicyEngine, ___delegates_DataSyncDelegate, ___delegates_ValidationDelegate, ___delegates_DialogDelegate, MessageBox, MessageStrip, VBox, ____core_PluginRegistry, ____core_LayoutScorer, ____core_LayoutMutator, ____core_DefaultLayoutGenerator) {
   "use strict";
 
   const Logger = ____utils_Logger["Logger"];
@@ -6,6 +6,7 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
   const SchemaValidator = ____core_SchemaValidator["SchemaValidator"];
   const Engine = ____core_Engine["Engine"];
   const StateManager = ____core_StateManager["StateManager"];
+  const PolicyEngine = ____core_PolicyEngine["PolicyEngine"];
   const DataSyncDelegate = ___delegates_DataSyncDelegate["DataSyncDelegate"];
   const ValidationDelegate = ___delegates_ValidationDelegate["ValidationDelegate"];
   const DialogDelegate = ___delegates_DialogDelegate["DialogDelegate"];
@@ -193,7 +194,14 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
       Control.prototype.constructor.call(this, idOrSettings, settings);
       this.stateManager = null;
       this.generatedContent = null;
+      /** The embedded engine responsible for rendering the UI controls */
       this.engine = null;
+      /** 
+       * The Policy-Based Rules Engine.
+       * Continuously evaluates declarative cross-field rules and pushes visual/structural mutations down into the plugins.
+       */
+      this.policyEngine = null;
+      this.policyDebounceTimer = null;
       this.activeModelName = "";
       /** Cached schema for intelligent diffing during inference mode hot-swaps */
       this.parsedSchema = null;
@@ -286,6 +294,53 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
         payload: payload,
         isValid: isValid
       });
+
+      // Trigger PBRE Live Evaluation
+      if (this.policyEngine && fieldKey) {
+        if (this.policyDebounceTimer) clearTimeout(this.policyDebounceTimer);
+        this.policyDebounceTimer = setTimeout(() => {
+          this.evaluatePolicies();
+        }, 200);
+      }
+    },
+    /**
+     * Executes the PBRE policy evaluations and orchestrates the delta updates to the UI plugins.
+     * Acts as the bridge between the headless PolicyEngine calculations and the physical UI5 plugins.
+     * 
+     * @param changedPath The specific JSON path that triggered the evaluation (used for incremental sweeps).
+     *                    If undefined, a full engine sweep is performed (e.g. during initial render or manual submit).
+     */
+    evaluatePolicies: function _evaluatePolicies() {
+      if (!this.policyEngine || !this.stateManager || !this.engine) return;
+      const payload = this.stateManager.extractPayload();
+      const deltas = this.policyEngine.evaluate(payload);
+      for (const delta of deltas) {
+        const plugin = this.engine.getPluginByPath(delta.target);
+        if (plugin) {
+          // Systemic Failsafe: State Pruning on Hide/Disable
+          if ((delta.property === "visibility" || delta.property === "editable") && delta.value === false) {
+            this.stateManager.setProperty(delta.target, undefined);
+          }
+          switch (delta.property) {
+            case "visibility":
+              if (plugin.applyVisibility) plugin.applyVisibility(delta.value);
+              break;
+            case "validity":
+              if (delta.value) {
+                this.engine.reportPolicyError(delta.target, null);
+              } else {
+                this.engine.reportPolicyError(delta.target, delta.errorMessage || "Validation failed by policy");
+              }
+              break;
+            case "required":
+              if (plugin.applyRequired) plugin.applyRequired(delta.value);
+              break;
+            case "editable":
+              if (plugin.applyEditable) plugin.applyEditable(delta.value);
+              break;
+          }
+        }
+      }
     },
     /**
      * Standard UI5 Exit Hook. Ensures complete teardown of the engine and state manager.
@@ -343,14 +398,22 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
         let isPrevented = false;
         this.validationDelegate.removeAllMessages();
 
-        // Only apply local fallback visual states if the global MessageManager is explicitly disabled
-        const applyVisualFallback = this.getProperty("useMessageManager") !== true;
-        const errors = this.engine.validateAll(applyVisualFallback);
+        // Run PBRE Full Sweep before validation to ensure visibility states are perfect
+        if (this.policyEngine) {
+          this.evaluatePolicies();
+        }
+
+        // PBRE Architecture Rule 12: Controls are deliberately isolated from UI5's native 
+        // MessageManager auto-sync. We must explicitly instruct plugins to paint their 
+        // visual red borders (ValueState) manually during a full form submission sweep.
+        const errors = this.engine.validateAll(true);
         if (errors.length > 0) {
-          this.validationDelegate.pushMessage("", "One or more fields failed schema validation. Please review the highlighted fields.");
+          if (this.getProperty("useMessageManager") === true) {
+            this.validationDelegate.pushMessage("", "One or more fields failed schema validation. Please review the highlighted fields.");
+          }
           for (const err of errors) {
             if (err.fieldKey && err.errorMessage) {
-              this.validationDelegate.pushMessage(err.fieldKey, err.errorMessage);
+              this.validationDelegate.pushMessage(err.fieldKey, err.errorMessage, err.fieldLabel);
             }
           }
           return false;
@@ -364,8 +427,10 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
           },
           addError: (propertyPath, errorMessage) => {
             isPrevented = true;
-            this.validationDelegate.pushMessage(propertyPath, errorMessage);
-            if (applyVisualFallback && this.engine) {
+            if (this.getProperty("useMessageManager") === true) {
+              this.validationDelegate.pushMessage(propertyPath, errorMessage);
+            }
+            if (this.engine) {
               const plugin = this.engine.getPluginByPath(propertyPath);
               if (plugin && typeof plugin.setVisualValidationState === "function") {
                 plugin.setVisualValidationState(false, errorMessage);
@@ -489,38 +554,42 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
           this.stateManager.getModel().destroy();
           this.stateManager = null;
         }
-        const rawSchema = this.getProperty("schemaDefinition");
+        let rawSchema = this.getProperty("schemaDefinition");
+        if (rawSchema && typeof rawSchema === "object") {
+          rawSchema = JSON.parse(JSON.stringify(rawSchema));
+        }
         const isSchemaEmpty = !rawSchema || typeof rawSchema === "object" && Object.keys(rawSchema).length === 0;
         let inputData = this.getProperty("data");
         const inputDataJson = this.getProperty("dataJson");
+        let dataParseError = "";
         if (inputDataJson) {
           try {
             inputData = JSON.parse(inputDataJson);
           } catch (e) {
-            const msg = "Failed to parse inputDataJson string: " + e.message;
-            Logger.error("[MetaUI]", msg, "GeneratorHost");
+            dataParseError = "Failed to parse inputDataJson string: " + e.message;
+            Logger.error("[MetaUI]", dataParseError, "GeneratorHost");
             this.fireEvent("error", {
-              message: msg,
+              message: dataParseError,
               exception: e
             });
-            throw new Error(msg); // Critical error instead of silent fallback
+            inputData = {}; // Fallback to empty object to allow schema to render
           }
         } else if (typeof inputData === "string") {
           try {
             inputData = JSON.parse(inputData);
           } catch (e) {
-            const msg = "Failed to parse inputData string: " + e.message;
-            Logger.error("[MetaUI]", msg, "GeneratorHost");
+            dataParseError = "Failed to parse inputData string: " + e.message;
+            Logger.error("[MetaUI]", dataParseError, "GeneratorHost");
             this.fireEvent("error", {
-              message: msg,
+              message: dataParseError,
               exception: e
             });
-            throw new Error(msg); // Critical error instead of silent fallback
+            inputData = {}; // Fallback to empty object to allow schema to render
           }
         }
 
-        // The internal payload represents the user's latest un-extracted edits. It must take priority.
-        const finalData = internalPayload || inputData;
+        // The internal payload represents the user's latest un-extracted edits. It must take priority, unless an external data-swap failed.
+        const finalData = dataParseError ? inputData : internalPayload || inputData;
         if (!finalData || Object.keys(finalData).length === 0) {
           if (isSchemaEmpty) {
             const msg = "No payload provided and no explicit schema defined. Cannot infer UI.";
@@ -537,6 +606,11 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
         // Cache the parsed/inferred schema so DataSyncDelegate can diff against it later
         this.parsedSchema = normalizedSchema;
         if (normalizedSchema) {
+          if (normalizedSchema.uiPolicies && normalizedSchema.uiPolicies.length > 0) {
+            this.policyEngine = new PolicyEngine(normalizedSchema.uiPolicies);
+          } else {
+            this.policyEngine = null;
+          }
           const schemaErrors = SchemaValidator.validateSchemaStructure(normalizedSchema);
           if (schemaErrors.length > 0) {
             const errorMsg = "Schema Structural Errors Found:\n- " + schemaErrors.join("\n- ");
@@ -610,6 +684,11 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
           this.engine.host = this;
         }
         this.generatedContent = this.engine.build(normalizedSchema, this.stateManager.getModel(), this.activeModelName, this.triggerSubmit.bind(this), this.getId(), this.onInternalFieldChange.bind(this));
+
+        // Defect #15: PBRE Initialization Blackout - Force headless evaluation of initial visibility state
+        if (this.policyEngine) {
+          this.evaluatePolicies();
+        }
         let contentContainer = this.generatedContent;
         const hasProperties = normalizedSchema && Object.keys(normalizedSchema).length > 0;
         if (!hasProperties) {
@@ -617,12 +696,29 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
             text: "Warning: The generated UI is blank. Reason: The schema (or inferred schema) contains no valid properties to render.",
             type: "Warning",
             showIcon: true,
-            showCloseButton: false
+            showCloseButton: true
           });
-          strip.addStyleClass("sapUiSmallMarginBottom");
+          strip.addStyleClass("sapUiSmallMargin");
           contentContainer = new VBox({
-            items: [strip, this.generatedContent]
+            items: [strip]
+          });
+        } else if (dataParseError) {
+          const strip = new MessageStrip({
+            text: "Data Error: " + dataParseError + ". The form has been rendered with a blank dataset.",
+            type: "Error",
+            showIcon: true,
+            showCloseButton: true
           });
+          strip.addStyleClass("sapUiSmallMargin");
+          if (typeof contentContainer.insertItem === "function") {
+            contentContainer.insertItem(strip, 0);
+          } else if (typeof contentContainer.insertContent === "function") {
+            contentContainer.insertContent(strip, 0);
+          } else {
+            contentContainer = new VBox({
+              items: [strip, this.generatedContent]
+            });
+          }
         } else if (this.engine.hasPartialRenderingErrors) {
           const strip = new MessageStrip({
             text: "Layout partially rendered. Some fields failed to generate due to configuration errors.",
@@ -641,8 +737,7 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
         this.dataSyncDelegate.pushToBindings(payload);
 
         // Retroactively expand the dialog if the heuristic determines it needs more space
-        const optimalWidth = this.calculateOptimalDialogWidth(this.parsedSchema);
-        this.dialogDelegate.updateDialogWidthDynamically(optimalWidth);
+        this.dialogDelegate.calculateAndApplyOptimalWidth(this.parsedSchema);
       } catch (error) {
         this.setBusy(false);
         const msg = "Fatal crash during layout generation: " + error.message;
@@ -655,50 +750,12 @@ sap.ui.define(["sap/ui/core/Control", "../../utils/Logger", "../../core/SchemaNo
         throw error; // Completely eliminate silent error swallowing
       }
     },
-    /**
-     * Helper to heuristically calculate optimal dialog width based on schema content.
-     */
-    calculateOptimalDialogWidth: function _calculateOptimalDialogWidth(schema) {
-      if (!schema) return "auto";
-      let requiresWideDialog = false;
-      const scanProperties = props => {
-        if (requiresWideDialog) return;
-        for (const key in props) {
-          const prop = props[key];
-          if (prop.type === "array") {
-            requiresWideDialog = true;
-            return;
-          }
-          if (prop.ui?.fullWidth === true) {
-            requiresWideDialog = true;
-            return;
-          }
-          if (prop.ui?.widget === "codeEditor" || prop.ui?.widget === "textArea" || prop.ui?.widget === "richText") {
-            requiresWideDialog = true;
-            return;
-          }
-          if (prop.properties) scanProperties(prop.properties);
-          if (prop.items && prop.items.properties) scanProperties(prop.items.properties);
-        }
-      };
-      if (schema.type === "array") return "80vw";
-      if (schema.properties) {
-        scanProperties(schema.properties);
-      }
-      return requiresWideDialog ? "80vw" : "auto";
-    },
     /**
      * Programmatic API. Re-parents the generated content into a dialog and opens it.
      */
     openInDialog: function _openInDialog(title = "Form", submitButtonText = "OK", cancelButtonText = "Cancel", dialogWidth = "auto", parentView) {
       const isGenerated = !!this.generatedContent;
-
-      // If we already have the schema parsed, we can calculate it instantly to prevent flicker
-      let finalWidth = dialogWidth;
-      if (finalWidth === "auto" && this.parsedSchema) {
-        finalWidth = this.calculateOptimalDialogWidth(this.parsedSchema);
-      }
-      this.dialogDelegate.openInDialog(title, submitButtonText, isGenerated, cancelButtonText, finalWidth, parentView);
+      this.dialogDelegate.openInDialog(title, submitButtonText, isGenerated, cancelButtonText, dialogWidth, parentView);
     }
   });
   return GeneratorHost;