chore: change to safely access nameof(_serviceScope)

addons/GodotDI/GodotDI.cs

@@ -1,18 +1,17 @@
 #if TOOLS
 using Godot;
-using GodotHostTest.addons.godot_di;
 
 namespace GodotHostTest.GodotDI;
 
 [Tool]
 public partial class GodotDI : EditorPlugin
 {
-    private ScopeInspectorPlugin? _plugin;
+    private ServiceSource.ScopeInspectorPlugin? _plugin;
 
     public override void _EnterTree()
     {
         GD.Print("Initializing Godot DI");
-        _plugin = new ScopeInspectorPlugin();
+        _plugin = new ServiceSource.ScopeInspectorPlugin();
         AddInspectorPlugin(_plugin);
     }
 

addons/GodotDI/ScopeInspectorPlugin.cs

@@ -4,28 +4,33 @@ using GodotHostTest.addons.godot_di;
 
 namespace GodotHostTest.GodotDI;
 
-public partial class ScopeInspectorPlugin : EditorInspectorPlugin
+public partial class ServiceSource
 {
-    public override bool _CanHandle(GodotObject @object)
+    // it's inside ServiceSource so that it can safely access nameof(_serviceScope)
+    public partial class ScopeInspectorPlugin : EditorInspectorPlugin
     {
-        var result = @object.GetScript().AsGodotObject() is CSharpScript script &&
+        public override bool _CanHandle(GodotObject @object)
-                     GodotScriptPathCache.TryGetTypeFromPath(script.ResourcePath, out var type) &&
+        {
-                     type.IsAssignableTo(typeof(ServiceSource));
+            var result = @object.GetScript().AsGodotObject() is CSharpScript script &&
-        return result;
+                         GodotScriptPathCache.TryGetTypeFromPath(script.ResourcePath, out var type) &&
-    }
+                         type.IsAssignableTo(typeof(ServiceSource));
+            return result;
+        }
 
-    public override bool _ParseProperty(GodotObject @object, Variant.Type type, string name, PropertyHint hintType,
+        public override bool _ParseProperty(GodotObject @object, Variant.Type type, string name, PropertyHint hintType,
-        string hintString,
+            string hintString,
-        PropertyUsageFlags usageFlags, bool wide)
+            PropertyUsageFlags usageFlags, bool wide)
-    {
+        {
-        // We handle properties of type integer.
+            // We handle properties of type integer.
-        if (name != "_serviceScope") return false;
+            if (name != nameof(_serviceScope)) return false;
-        // Create an instance of the custom property editor and register
+            // Create an instance of the custom property editor and register
-        // it to a specific property path.
+            // it to a specific property path.
-        AddPropertyEditor(name, new ScopePickerEditor());
+            AddPropertyEditor(name, new ScopePickerEditor());
-        // Inform the editor to remove the default property editor for
+            // Inform the editor to remove the default property editor for
-        // this property type.
+            // this property type.
-        return true;
+            return true;
+        }
     }
 }
+
 #endif