Add a concept of Developer Mode for Luau

Add Developer Mode to LuaManager
Add DeveloperOnly flag support to Luau method tables

Author: Pursche

Source/Scripting/Scripting/LuaManager.cpp

@@ -381,6 +381,9 @@ namespace Scripting
             if (StringUtils::BeginsWith(relativePathAsString, "API/") || StringUtils::BeginsWith(relativePathAsString, "Bootstrap/"))
                 continue;
 
+            if (!_developerMode && StringUtils::BeginsWith(relativePathAsString, "Editor/"))
+                continue;
+
             std::string pathStr = path.string();
             u32 depth = GetPathDepth(scriptBootstrapDirectoryAsString, pathStr);
             paths.push_back({ depth, path });

Source/Scripting/Scripting/LuaManager.h

@@ -30,6 +30,16 @@ namespace Scripting
         void SetDirty() { _isDirty = true; }
         void ClearDirty() { _isDirty = false; }
 
+        // When false, scripts under {scriptDir}/Editor/ are excluded from the load.
+        bool IsDeveloperMode() const { return _developerMode; }
+        void SetDeveloperMode(bool enabled)
+        {
+            if (enabled == _developerMode)
+                return;
+            _developerMode = enabled;
+            SetDirty();
+        }
+
         ZenithStateManager& GetZenithStateManager()
         {
             return _zenithStateManager;
@@ -57,6 +67,7 @@ namespace Scripting
 
     private:
         bool _isDirty = false;
+        bool _developerMode = true;
 
         u64 _currentTick = 0;
         u64 _lastRecompiledTick = 0;

Source/Scripting/Scripting/LuaMethodTable.h

@@ -12,12 +12,33 @@
 
 namespace Scripting
 {
+    // Per-entry filter for LuaMethodTable::Set: entries overlapping excludeFlags are skipped.
+    enum class LuaMethodFlags : u32
+    {
+        None          = 0,
+        DeveloperOnly = 1 << 0,
+    };
+
+    inline constexpr LuaMethodFlags operator|(LuaMethodFlags a, LuaMethodFlags b)
+    {
+        return static_cast<LuaMethodFlags>(static_cast<u32>(a) | static_cast<u32>(b));
+    }
+    inline constexpr LuaMethodFlags operator&(LuaMethodFlags a, LuaMethodFlags b)
+    {
+        return static_cast<LuaMethodFlags>(static_cast<u32>(a) & static_cast<u32>(b));
+    }
+    inline constexpr bool HasAnyFlag(LuaMethodFlags value, LuaMethodFlags mask)
+    {
+        return (value & mask) != LuaMethodFlags::None;
+    }
+
     template <typename UserDataType = void>
     struct LuaRegister
     {
     public:
         const char* name;
         typename std::conditional<std::is_same_v<UserDataType, void>, i32(*)(Zenith*), i32(*)(Zenith*, UserDataType*)>::type func;
+        LuaMethodFlags flags = LuaMethodFlags::None;
     };
 
     template <typename UserDataType>
@@ -112,7 +133,7 @@ namespace Scripting
         }
 
         template <class MethodTableUserDataType, size_t N>
-        static void Set(Zenith* zenith, const LuaRegister<MethodTableUserDataType> (&methodTable)[N], const char* globalName = nullptr)
+        static void Set(Zenith* zenith, const LuaRegister<MethodTableUserDataType> (&methodTable)[N], const char* globalName = nullptr, LuaMethodFlags excludeFlags = LuaMethodFlags::None)
         {
             static_assert(std::is_same_v<UserDataType, void> || std::is_same_v<UserDataType, MethodTableUserDataType> || std::is_base_of_v<MethodTableUserDataType, UserDataType>, "LuaMetaTable::Set - LuaRegister's UserDataType must either match LuaMetaTable's UserDataType, be derived from it or be void");
             constexpr bool isGlobal = std::is_same_v<UserDataType, void>;
@@ -147,6 +168,9 @@ namespace Scripting
             {
                 const auto& method = methodTable[i];
 
+                if (HasAnyFlag(method.flags, excludeFlags))
+                    continue;
+
                 lua_pushstring(zenith->state, method.name);
 
                 lua_pushlightuserdata(zenith->state, (void*)&method);