Skip utility windows during editor initialization

1. Skip editors with no_serialize=true in View:InitializeAllEditors()
   - These are utility windows (objectPropertyWindow, collisionView, etc.)
   - Should be created on-demand, not during initialization
   - Prevents initialization errors when SB.view not yet set

2. Add nil check in ObjectPropertyWindow:init()
   - Guards SB.view.selectionManager access
   - Needed in case editor is created before SB.view is assigned
   - This is defensive - shouldn't happen now with no_serialize skip

Fixes: "attempt to index field 'view' (a nil value)" in object_property_window.lua:366

Author: claude

scen_edit/view/object/object_property_window.lua

@@ -363,8 +363,13 @@ function ObjectPropertyWindow:init()
     )
 
     self:Finalize(children)
-    SB.view.selectionManager:addListener(self)
-    self:OnSelectionChanged()
+
+    -- Defer listener registration if SB.view not set yet (during init)
+    -- This can happen when editors are pre-initialized in RmlUi mode
+    if SB.view then
+        SB.view.selectionManager:addListener(self)
+        self:OnSelectionChanged()
+    end
     SB.commandManager:addListener(self)
 end
 

scen_edit/view/view.lua

@@ -172,13 +172,16 @@ function View:InitializeAllEditors()
     -- 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
+    -- Skip editors with no_serialize=true (utility windows created on-demand)
     Log.Notice("Initializing all editors from editorRegistry...")
     local count = 0
     for name, editorCfg in pairs(SB.editorRegistry) do
-        if editorCfg.editor then
+        if editorCfg.editor and not editorCfg.no_serialize then
             Log.Notice("  Creating editor: " .. name)
             SB.editors[name] = editorCfg.editor()
             count = count + 1
+        elseif editorCfg.no_serialize then
+            Log.Notice("  Skipping utility window: " .. name .. " (no_serialize)")
         else
             Log.Warning("  Editor " .. name .. " has no constructor function")
         end