Вызов получения переменных модуля в зависимости от версии протокола

Author: EvilBeaver

src/OneScript.DebugProtocol/IDebuggerService.cs

@@ -52,6 +52,15 @@ public interface IDebuggerService
         /// <param name="path"></param>
         /// <returns></returns>
         Variable[] GetVariables(int threadId, int frameIndex, int[] path);
+        
+        /// <summary>
+        /// Получает переменные модуля для указанного фрейма
+        /// </summary>
+        /// <param name="threadId"></param>
+        /// <param name="frameIndex"></param>
+        /// <param name="path"></param>
+        /// <returns></returns>
+        Variable[] GetModuleVariables(int threadId, int frameIndex, int[] path);
 
         /// <summary>
         /// Получает значения переменных вычисленного выражения

src/OneScript.DebugServices/DefaultDebugger.cs

@@ -20,7 +20,7 @@ public class DefaultDebugger : IDebugger
         // NB! должен быть согласован с перечислением TransportProtocols в адаптере
         private const short JSON_FORMAT_MARKER = 2;
         // NB! должен быть согласован с файлом ProtocolVersions в адаптере
-        private const short SUPPORTED_FORMAT_VERSION = 3;
+        private const short SUPPORTED_FORMAT_VERSION = 4;
 
         private readonly IDebugServer _transport;
         private IDebugSession _session;

src/OneScript.DebugServices/Internal/DebuggerServiceImpl.cs

@@ -138,6 +138,21 @@ public Variable[] GetVariables(int threadId, int frameIndex, int[] path)
 
             return GetDebugVariables(locals);
         }
+        
+        public Variable[] GetModuleVariables(int threadId, int frameIndex, int[] path)
+        {
+            var machine = GetMachine(threadId);
+
+            var moduleVars = machine.GetModuleVariables(frameIndex);
+
+            foreach (var step in path)
+            {
+                var variable = moduleVars[step];
+                moduleVars = GetChildVariables(variable);
+            }
+
+            return GetDebugVariables(moduleVars);
+        }
 
         public Variable[] GetEvaluatedVariables(string expression, int threadId, int frameIndex, int[] path)
         {

src/VSCode.DebugAdapter/OneScriptDebuggerClient.cs

@@ -132,6 +132,18 @@ public Variable[] GetVariables(int threadId, int frameIndex, int[] path)
 
             return GetResponse<Variable[]>();
         }
+        
+        public Variable[] GetModuleVariables(int threadId, int frameIndex, int[] path)
+        {
+            WriteCommand(new object[]
+            {
+                threadId,
+                frameIndex,
+                path
+            });
+
+            return GetResponse<Variable[]>();
+        }
 
         public Variable[] GetEvaluatedVariables(string expression, int threadId, int frameIndex, int[] path)
         {

src/VSCode.DebugAdapter/OscriptDebugSession.cs

@@ -379,11 +379,13 @@ public override void Scopes(Response response, dynamic arguments)
             var localHandle = _threadState.RegisterVariablesProvider(localProvider);
             scopes.Add(new Scope("Локальные переменные", localHandle));
 
-            // Scope 2: Переменные модуля (для будущего расширения)
-            // Раскомментировать когда будет готов протокол:
-            // var moduleProvider = new ModuleScopeProvider(frame.ThreadId, frame.Index);
-            // var moduleHandle = _threadState.RegisterVariablesProvider(moduleProvider);
-            // scopes.Add(new Scope("Переменные модуля", moduleHandle));
+            // Scope 2: Переменные модуля (начиная с протокола версии 4)
+            if (_debuggee.ProtocolVersion >= ProtocolVersions.Version4)
+            {
+                var moduleProvider = new ModuleScopeProvider(frame.ThreadId, frame.Index);
+                var moduleHandle = _threadState.RegisterVariablesProvider(moduleProvider);
+                scopes.Add(new Scope("Переменные модуля", moduleHandle));
+            }
 
             SendResponse(response, new ScopesResponseBody(scopes.ToArray()));
         }

src/VSCode.DebugAdapter/Transport/ProtocolVersions.cs

@@ -41,6 +41,11 @@ public static int Adjust(int valueToAdjust)
         /// Выделенный тип для параметра в SetExceptionBreakpoints 
         /// </summary>
         public const int Version3 = 3;
+        
+        /// <summary>
+        /// Переменные модуля 
+        /// </summary>
+        public const int Version4 = 4;
 
         /// <summary>
         /// Значение, безопасное для всех версий движка
@@ -50,6 +55,6 @@ public static int Adjust(int valueToAdjust)
         /// <summary>
         /// Контрольное значение
         /// </summary>
-        public const int LatestKnownVersion = Version3;
+        public const int LatestKnownVersion = Version4;
     }
 }

src/VSCode.DebugAdapter/Variables/ChildVariablesProvider.cs

@@ -11,32 +11,38 @@ This Source Code Form is subject to the terms of the
 namespace VSCode.DebugAdapter
 {
     /// <summary>
-    /// Провайдер вложенных переменных объекта
+    /// Универсальный провайдер вложенных переменных
     /// </summary>
     public class ChildVariablesProvider : IVariablesProvider
     {
         private readonly int _threadId;
         private readonly int _frameIndex;
         private readonly int[] _path;
+        private readonly Func<IDebuggerService, int, int, int[], Variable[]> _fetchFunc;
 
-        public ChildVariablesProvider(int threadId, int frameIndex, int[] path)
+        public ChildVariablesProvider(
+            int threadId, 
+            int frameIndex, 
+            int[] path,
+            Func<IDebuggerService, int, int, int[], Variable[]> fetchFunc)
         {
             _threadId = threadId;
             _frameIndex = frameIndex;
             _path = path;
+            _fetchFunc = fetchFunc;
         }
 
         public Variable[] FetchVariables(IDebuggerService service)
         {
-            return service.GetVariables(_threadId, _frameIndex, _path);
+            return _fetchFunc(service, _threadId, _frameIndex, _path);
         }
 
         public IVariablesProvider CreateChildProvider(int variableIndex)
         {
             var newPath = new int[_path.Length + 1];
             Array.Copy(_path, newPath, _path.Length);
             newPath[_path.Length] = variableIndex;
-            return new ChildVariablesProvider(_threadId, _frameIndex, newPath);
+            return new ChildVariablesProvider(_threadId, _frameIndex, newPath, _fetchFunc);
         }
     }
 }

src/VSCode.DebugAdapter/Variables/EvaluatedExpressionProvider.cs

@@ -46,7 +46,11 @@ public IVariablesProvider CreateChildProvider(int variableIndex)
             var newPath = new int[_path.Length + 1];
             Array.Copy(_path, newPath, _path.Length);
             newPath[_path.Length] = variableIndex;
-            return new EvaluatedExpressionProvider(_expression, _threadId, _frameIndex, newPath);
+            return new ChildVariablesProvider(
+                _threadId, 
+                _frameIndex, 
+                newPath,
+                (service, tid, fid, path) => service.GetEvaluatedVariables(_expression, tid, fid, path));
         }
     }
 }

src/VSCode.DebugAdapter/Variables/LocalScopeProvider.cs

@@ -32,7 +32,11 @@ public Variable[] FetchVariables(IDebuggerService service)
 
         public IVariablesProvider CreateChildProvider(int variableIndex)
         {
-            return new ChildVariablesProvider(_threadId, _frameIndex, new[] { variableIndex });
+            return new ChildVariablesProvider(
+                _threadId, 
+                _frameIndex, 
+                new[] { variableIndex },
+                (service, tid, fid, path) => service.GetVariables(tid, fid, path));
         }
     }
 }

src/VSCode.DebugAdapter/Variables/ModuleScopeProvider.cs

@@ -11,14 +11,12 @@ This Source Code Form is subject to the terms of the
 namespace VSCode.DebugAdapter
 {
     /// <summary>
-    /// Провайдер переменных модуля (для будущего расширения)
-    /// Требует расширения протокола отладки для получения переменных модуля
+    /// Провайдер переменных модуля
     /// </summary>
     public class ModuleScopeProvider : IVariablesProvider
     {
         private readonly int _threadId;
         private readonly int _frameIndex;
-        // В будущем может потребоваться moduleId или другой идентификатор
 
         public ModuleScopeProvider(int threadId, int frameIndex)
         {
@@ -28,14 +26,17 @@ public ModuleScopeProvider(int threadId, int frameIndex)
 
         public Variable[] FetchVariables(IDebuggerService service)
         {
-            // TODO: когда будет расширен протокол - заменить на GetModuleVariables или подобное
-            // Пока возвращаем пустой массив
-            throw new NotImplementedException("Module scope requires protocol extension");
+            // path пустой - получаем переменные модуля верхнего уровня фрейма
+            return service.GetModuleVariables(_threadId, _frameIndex, Array.Empty<int>());
         }
 
         public IVariablesProvider CreateChildProvider(int variableIndex)
         {
-            throw new NotImplementedException("Module scope requires protocol extension");
+            return new ChildVariablesProvider(
+                _threadId, 
+                _frameIndex, 
+                new[] { variableIndex },
+                (service, tid, fid, path) => service.GetModuleVariables(tid, fid, path));
         }
     }
 }