Fixes #164 - add capabilities response to InitializeRequest.

This commit is in preparation for adding conditional breakpoint and function breakpoint support.  Also cleaned up some doc comments.  The two primary files that were changed functionally are InitializeRequest.cs and DebugAdapaterBase.cs.

Author: rkeithhill

src/PowerShellEditorServices.Protocol/DebugAdapter/InitializeRequest.cs

@@ -10,8 +10,8 @@ namespace Microsoft.PowerShell.EditorServices.Protocol.DebugAdapter
     public class InitializeRequest
     {
         public static readonly
-            RequestType<InitializeRequestArguments, object> Type =
-            RequestType<InitializeRequestArguments, object>.Create("initialize");
+            RequestType<InitializeRequestArguments, InitializeResponseBody> Type =
+            RequestType<InitializeRequestArguments, InitializeResponseBody>.Create("initialize");
     }
 
     public class InitializeRequestArguments
@@ -26,4 +26,36 @@ public class InitializeRequestArguments
 
         public string GeneratedCodeDirectory { get; set; }
     }
+
+    public class InitializeResponseBody
+    {
+        /// <summary>
+        /// Gets or sets a boolean value that determines whether the debug adapter 
+        /// supports the configurationDoneRequest.
+        /// </summary>
+        public bool SupportsConfigurationDoneRequest { get; set; }
+
+        /// <summary>
+        /// Gets or sets a boolean value that determines whether the debug adapter 
+        /// supports functionBreakpoints.
+        /// </summary>
+        public bool SupportsFunctionBreakpoints { get; set; }
+
+        /// <summary>
+        /// Gets or sets a boolean value that determines whether the debug adapter 
+        /// supports conditionalBreakpoints.
+        /// </summary>
+        public bool SupportsConditionalBreakpoints { get; set; }
+
+        /// <summary>
+        /// Gets or sets a boolean value that determines whether the debug adapter 
+        /// supports a (side effect free) evaluate request for data hovers.
+        /// </summary>
+        public bool SupportsEvaluateForHovers { get; set; }
+
+        /// <summary>
+        /// Gets or sets the available filters for the setExceptionBreakpoints request.
+        /// </summary>
+        public object ExceptionBreakpointFilters { get; set; }
+    }
 }

src/PowerShellEditorServices.Protocol/DebugAdapter/LaunchRequest.cs

@@ -3,8 +3,8 @@
 // Licensed under the MIT license. See LICENSE file in the project root for full license information.
 //
 
-using Microsoft.PowerShell.EditorServices.Protocol.MessageProtocol;
 using System.Collections.Generic;
+using Microsoft.PowerShell.EditorServices.Protocol.MessageProtocol;
 
 namespace Microsoft.PowerShell.EditorServices.Protocol.DebugAdapter
 {
@@ -17,25 +17,43 @@ public static readonly
 
     public class LaunchRequestArguments
     {
-    //        /** An absolute path to the program to debug. */
+        /// <summary>
+        /// Gets or sets the absolute path to the program to debug.
+        /// </summary>
         public string Program { get; set; }
 
-    //        /** Automatically stop target after launch. If not specified, target does not stop. */
+        /// <summary>
+        /// Gets or sets a boolean value that determines whether to automatically stop 
+        /// target after launch. If not specified, target does not stop.
+        /// </summary>
         public bool StopOnEntry { get; set; }
 
-    //        /** Optional arguments passed to the debuggee. */
+        /// <summary>
+        /// Gets or sets optional arguments passed to the debuggee.
+        /// </summary>
         public string[] Args { get; set; }
 
-    //        /** Launch the debuggee in this working directory (specified as an absolute path). If omitted the debuggee is lauched in its own directory. */
+        /// <summary>
+        /// Gets or sets the working directory of the launched debuggee (specified as an absolute path).
+        /// If omitted the debuggee is lauched in its own directory.
+        /// </summary>
         public string Cwd { get; set; }
 
-    //        /** Absolute path to the runtime executable to be used. Default is the runtime executable on the PATH. */
+        /// <summary>
+        /// Gets or sets the absolute path to the runtime executable to be used. 
+        /// Default is the runtime executable on the PATH.
+        /// </summary>
         public string RuntimeExecutable { get; set; }
 
-    //        /** Optional arguments passed to the runtime executable. */
+        /// <summary>
+        /// Gets or sets the optional arguments passed to the runtime executable.
+        /// </summary>
         public string[] RuntimeArgs { get; set; }
 
-    //        /** Optional environment variables to pass to the debuggee. The string valued properties of the 'environmentVariables' are used as key/value pairs. */
+        /// <summary>
+        /// Gets or sets optional environment variables to pass to the debuggee. The string valued 
+        /// properties of the 'environmentVariables' are used as key/value pairs.
+        /// </summary>
         public Dictionary<string, string> Env { get; set; }
     }
 }

src/PowerShellEditorServices.Protocol/DebugAdapter/SetExceptionBreakpointsRequest.cs

@@ -7,24 +7,25 @@
 
 namespace Microsoft.PowerShell.EditorServices.Protocol.DebugAdapter
 {
-    //    /** SetExceptionBreakpoints request; value of command field is "setExceptionBreakpoints".
-    //        Enable that the debuggee stops on exceptions with a StoppedEvent (event type 'exception').
-    //    */
+    /// <summary>
+    /// SetExceptionBreakpoints request; value of command field is "setExceptionBreakpoints".
+    /// Enable that the debuggee stops on exceptions with a StoppedEvent (event type 'exception').
+    /// </summary>
     public class SetExceptionBreakpointsRequest
     {
         public static readonly
             RequestType<SetExceptionBreakpointsRequestArguments, object> Type =
             RequestType<SetExceptionBreakpointsRequestArguments, object>.Create("setExceptionBreakpoints");
     }
 
+    /// <summary>
+    /// Arguments for "setExceptionBreakpoints" request.
+    /// </summary>
     public class SetExceptionBreakpointsRequestArguments
     {
-    //    /** Arguments for "setExceptionBreakpoints" request. */
-    //    export interface SetExceptionBreakpointsArguments {
-    //        /** Names of enabled exception breakpoints. */
-    //        filters: string[];
-    //    }
+        /// <summary>
+        /// Gets or sets the names of enabled exception breakpoints.
+        /// </summary>
         public string[] Filters { get; set; }
     }
 }
-

src/PowerShellEditorServices.Protocol/DebugAdapter/SourceRequest.cs

@@ -16,7 +16,9 @@ public static readonly
 
     public class SourceRequestArguments
     {
-    //        /** The reference to the source. This is the value received in Source.reference. */
+        /// <summary>
+        /// Gets or sets the reference to the source. This is the value received in Source.reference.
+        /// </summary>
         public int SourceReference { get; set; }
     }
 
@@ -25,4 +27,3 @@ public class SourceResponseBody
         public string Content { get; set; }
     }
 }
-

src/PowerShellEditorServices.Protocol/DebugAdapter/StackTraceRequest.cs

@@ -20,7 +20,9 @@ public class StackTraceRequestArguments
     {
         public int ThreadId { get; private set; }
 
-    //        /** The maximum number of frames to return. If levels is not specified or 0, all frames are returned. */
+        /// <summary>
+        /// Gets the maximum number of frames to return. If levels is not specified or 0, all frames are returned.
+        /// </summary>
         public int Levels { get; private set; }
     }
 

src/PowerShellEditorServices.Protocol/Server/DebugAdapterBase.cs

@@ -52,15 +52,15 @@ protected override Task OnStop()
 
         private async Task HandleInitializeRequest(
             object shutdownParams,
-            RequestContext<object> requestContext)
+            RequestContext<InitializeResponseBody> requestContext)
         {
             // Send the Initialized event first so that we get breakpoints
             await requestContext.SendEvent(
                 InitializedEvent.Type,
                 null);
 
             // Now send the Initialize response to continue setup
-            await requestContext.SendResult(new object());
+            await requestContext.SendResult(new InitializeResponseBody());
         }
     }
 }