Fix field initialization and editor creation

- Add Set() method to RmlUiField (alias for SetValue) for Chili API compatibility
- Fix GroupField to auto-generate name like Chili does (_groupField1, etc.)
- Register GroupField child fields in Editor:AddField() for RmlUi mode
- Initialize ALL editors upfront in View:InitializeAllEditors()
  * Called during View:InitializeRmlUi() instead of lazy creation
  * Catches field errors early instead of when users click buttons
  * Generates RML for all editors during init via Finalize()
- Update OpenEditor() to use pre-initialized editors

This fixes:
- "Attempted to add field without name" errors from GroupField children
- "attempt to call method 'Set' (a nil value)" errors
- Late error detection when clicking UI buttons

Author: claude

scen_edit/view/editor.lua

@@ -362,6 +362,16 @@ function Editor:AddField(field)
         field.ctrl = self:_AddControl(field.name, field.components)
     end
     self:_AddField(field)
+
+    -- In RmlUi mode, register child fields of GroupField
+    if SB.view and SB.view.useRmlUi and field.fields then
+        for _, childField in ipairs(field.fields) do
+            if childField.name then
+                self:_AddField(childField)
+            end
+        end
+    end
+
     -- Only call Added() if the method exists (Chili fields)
     if field.Added then
         field:Added()

scen_edit/view/rmlui_field_compat.lua

@@ -23,7 +23,18 @@ ObjectField = RmlUiObjectField
 ObjectTypeField = RmlUiObjectTypeField
 TeamField = RmlUiTeamField
 ArrayField = RmlUiArrayField
-GroupField = RmlUiGroupField
+
+-- GroupField wrapper that matches Chili's constructor pattern
+-- Auto-generates name like Chili does
+local _GROUP_INDEX = 0
+function GroupField(fields)
+    _GROUP_INDEX = _GROUP_INDEX + 1
+    local name = "_groupField" .. tostring(_GROUP_INDEX)
+    return RmlUiGroupField({
+        name = name,
+        fields = fields
+    })
+end
 
 -- Picker windows
 AssetPickerWindow = RmlUiAssetPickerWindow

scen_edit/view/rmlui_fields.lua

@@ -40,6 +40,11 @@ function RmlUiField:SetValue(value)
     end
 end
 
+-- Alias for compatibility with Chili field API
+function RmlUiField:Set(value)
+    self:SetValue(value)
+end
+
 function RmlUiField:UpdateDisplay()
     -- Override in subclass
 end

scen_edit/view/view.lua

@@ -83,6 +83,10 @@ function View:InitializeRmlUi()
         return
     end
 
+    -- Initialize ALL editors upfront from editorRegistry
+    -- This generates RML during init so we catch errors early
+    self:InitializeAllEditors()
+
     -- Initialize tab system
     self.currentTab = "Objects"
     self:BindTabEvents()
@@ -159,6 +163,29 @@ function View:InitializeRmlUi()
     Log.Notice("RmlUi UI initialized successfully with all dialogs, editors, and floating windows")
 end
 
+function View:InitializeAllEditors()
+    -- Initialize SB.editors table if it doesn't exist
+    if not SB.editors then
+        SB.editors = {}
+    end
+
+    -- Instantiate all editors from editorRegistry
+    -- This calls their init() which calls Finalize() which generates RML
+    -- Doing this upfront catches field errors early instead of when users click buttons
+    Log.Notice("Initializing all editors from editorRegistry...")
+    local count = 0
+    for name, editorCfg in pairs(SB.editorRegistry) do
+        if editorCfg.editor then
+            Log.Notice("  Creating editor: " .. name)
+            SB.editors[name] = editorCfg.editor()
+            count = count + 1
+        else
+            Log.Warning("  Editor " .. name .. " has no constructor function")
+        end
+    end
+    Log.Notice("Initialized " .. count .. " editors")
+end
+
 function View:SetupRmlUiEvents()
     if not self.mainDocument then
         return
@@ -422,22 +449,13 @@ end
 function View:OpenEditor(editorName)
     Log.Notice("Opening editor: " .. editorName)
 
-    -- Create editor instance if not exists
-    if not SB.editors then
-        SB.editors = {}
-    end
-
-    if not SB.editors[editorName] then
-        local editorCfg = SB.editorRegistry[editorName]
-        if editorCfg and editorCfg.editor then
-            SB.editors[editorName] = editorCfg.editor()
-        else
-            Log.Error("Editor not found: " .. editorName)
-            return
-        end
+    -- Editor should already be initialized during View:InitializeAllEditors()
+    local editor = SB.editors and SB.editors[editorName]
+    if not editor then
+        Log.Error("Editor not initialized: " .. editorName)
+        return
     end
 
-    local editor = SB.editors[editorName]
     local editorCfg = SB.editorRegistry[editorName]
     local mainContent = self.mainDocument:GetElementById("main-content")