Records and init only properties (#442)

Made all lsp/dap models records where appropraite.  for lsp registration options and capabilities are not records and do not have init only properties.  These values can be adjusted during the startup lifecycle.
+semver:minor

Author: david-driscoll

sample/SampleServer/TextDocumentHandler.cs

@@ -69,8 +69,8 @@ public override Task<Unit> Handle(DidCloseTextDocumentParams notification, Cance
 
         protected override TextDocumentSyncRegistrationOptions CreateRegistrationOptions(SynchronizationCapability capability, ClientCapabilities clientCapabilities) => new TextDocumentSyncRegistrationOptions() {
             DocumentSelector = _documentSelector,
-            SyncKind = Change,
-            IncludeText = true
+            Change = Change,
+            Save = new SaveOptions() { IncludeText = true }
         };
 
         public override TextDocumentAttributes GetTextDocumentAttributes(DocumentUri uri) => new TextDocumentAttributes(uri, "csharp");

src/Dap.Protocol/Feature/Events/BreakpointFeature.cs

@@ -11,71 +11,70 @@ namespace OmniSharp.Extensions.DebugAdapter.Protocol
 {
     namespace Models
     {
-
         /// <summary>
         /// Information about a Breakpoint created in setBreakpoints or setFunctionBreakpoints.
         /// </summary>
-        public class Breakpoint
+        public record Breakpoint
         {
             /// <summary>
             /// An optional identifier for the breakpoint. It is needed if breakpoint events are used to update or remove breakpoints.
             /// </summary>
             [Optional]
-            public long? Id { get; set; }
+            public long? Id { get; init; }
 
             /// <summary>
             /// If true breakpoint could be set (but not necessarily at the desired location).
             /// </summary>
-            public bool Verified { get; set; }
+            public bool Verified { get; init; }
 
             /// <summary>
             /// An optional message about the state of the breakpoint. This is shown to the user and can be used to explain why a breakpoint could not be verified.
             /// </summary>
             [Optional]
-            public string? Message { get; set; }
+            public string? Message { get; init; }
 
             /// <summary>
             /// The source where the breakpoint is located.
             /// </summary>
             [Optional]
-            public Source? Source { get; set; }
+            public Source? Source { get; init; }
 
             /// <summary>
             /// The start line of the actual range covered by the breakpoint.
             /// </summary>
             [Optional]
-            public int? Line { get; set; }
+            public int? Line { get; init; }
 
             /// <summary>
             /// An optional start column of the actual range covered by the breakpoint.
             /// </summary>
             [Optional]
-            public int? Column { get; set; }
+            public int? Column { get; init; }
 
             /// <summary>
             /// An optional end line of the actual range covered by the breakpoint.
             /// </summary>
             [Optional]
-            public int? EndLine { get; set; }
+            public int? EndLine { get; init; }
 
             /// <summary>
             /// An optional end column of the actual range covered by the breakpoint. If no end line is given, then the end column is assumed to be in the start line.
             /// </summary>
             [Optional]
-            public int? EndColumn { get; set; }
+            public int? EndColumn { get; init; }
 
             /// <summary>
             /// An optional memory reference to where the breakpoint is set.
             /// </summary>
             [Optional]
-            public string? InstructionReference { get; set; }
+            public string? InstructionReference { get; init; }
 
             /// <summary>
             /// An optional offset from the instruction reference.
             /// This can be negative.
             /// </summary>
             [Optional]
-            public int? Offset { get; set; }
+            public int? Offset { get; init; }
         }
     }
     namespace Events
@@ -87,18 +86,18 @@ namespace Events
             GenerateHandlerMethods,
             GenerateRequestMethods
         ]
-        public class BreakpointEvent : IRequest
+        public record BreakpointEvent : IRequest
         {
             /// <summary>
             /// The reason for the event.
             /// Values: 'changed', 'new', 'removed', etc.
             /// </summary>
-            public string Reason { get; set; } = null!;
+            public string Reason { get; init; }
 
             /// <summary>
             /// The 'id' attribute is used to find the target breakpoint and the other attributes are used as the new values.
             /// </summary>
-            public Breakpoint Breakpoint { get; set; } = null!;
+            public Breakpoint Breakpoint { get; init; }
         }
     }
 }

src/Dap.Protocol/Feature/Events/CapabilitiesFeature.cs

@@ -14,7 +14,7 @@ namespace Models
         /// <summary>
         /// Information about the capabilities of a debug adapter.
         /// </summary>
-        public class Capabilities
+        public record Capabilities
         {
             /// <summary>
             /// The debug adapter supports the 'configurationDone' request.
@@ -232,12 +232,12 @@ namespace Events
             GenerateHandlerMethods,
             GenerateRequestMethods
         ]
-        public class CapabilitiesEvent : IRequest
+        public record CapabilitiesEvent : IRequest
         {
             /// <summary>
             /// The set of updated capabilities.
             /// </summary>
-            public Capabilities Capabilities { get; set; } = null!;
+            public Capabilities Capabilities { get; init; }
         }
     }
 }

src/Dap.Protocol/Feature/Events/ContinuedFeature.cs

@@ -17,18 +17,18 @@ namespace Events
             GenerateHandlerMethods,
             GenerateRequestMethods
         ]
-        public class ContinuedEvent : IRequest
+        public record ContinuedEvent : IRequest
         {
             /// <summary>
             /// The thread which was continued.
             /// </summary>
-            public long ThreadId { get; set; }
+            public long ThreadId { get; init; }
 
             /// <summary>
             /// If 'allThreadsContinued' is true, a debug adapter can announce that all threads have continued.
             /// </summary>
             [Optional]
-            public bool AllThreadsContinued { get; set; }
+            public bool AllThreadsContinued { get; init; }
         }
     }
 }

src/Dap.Protocol/Feature/Events/ExitedFeature.cs

@@ -16,12 +16,12 @@ namespace Events
             GenerateHandlerMethods,
             GenerateRequestMethods
         ]
-        public class ExitedEvent : IRequest
+        public record ExitedEvent : IRequest
         {
             /// <summary>
             /// The exit code returned from the debuggee.
             /// </summary>
-            public long ExitCode { get; set; }
+            public long ExitCode { get; init; }
         }
     }
 }

src/Dap.Protocol/Feature/Events/InitializedFeature.cs

@@ -16,7 +16,7 @@ namespace Events
             GenerateHandlerMethods,
             GenerateRequestMethods
         ]
-        public class InitializedEvent : IRequest
+        public record InitializedEvent : IRequest
         {
         }
     }

src/Dap.Protocol/Feature/Events/LoadedSourceFeature.cs

@@ -19,17 +19,17 @@ namespace Events
             GenerateHandlerMethods,
             GenerateRequestMethods
         ]
-        public class LoadedSourceEvent : IRequest
+        public record LoadedSourceEvent : IRequest
         {
             /// <summary>
             /// The reason for the event.
             /// </summary>
-            public LoadedSourceReason Reason { get; set; }
+            public LoadedSourceReason Reason { get; init; }
 
             /// <summary>
             /// The new, changed, or removed source.
             /// </summary>
-            public Source Source { get; set; } = null!;
+            public Source Source { get; init; }
         }
 
         [JsonConverter(typeof(StringEnumConverter))]

src/Dap.Protocol/Feature/Events/ModuleFeature.cs

@@ -19,17 +19,17 @@ namespace Events
             GenerateHandlerMethods,
             GenerateRequestMethods
         ]
-        public class ModuleEvent : IRequest
+        public record ModuleEvent : IRequest
         {
             /// <summary>
             /// The reason for the event.
             /// </summary>
-            public ModuleEventReason Reason { get; set; }
+            public ModuleEventReason Reason { get; init; }
 
             /// <summary>
             /// The new, changed, or removed module. In case of 'removed' only the module id is used.
             /// </summary>
-            public Module Module { get; set; } = null!;
+            public Module Module { get; init; }
         }
 
         [JsonConverter(typeof(StringEnumConverter))]

src/Dap.Protocol/Feature/Events/OutputFeature.cs

@@ -19,49 +19,49 @@ namespace Events
             GenerateHandlerMethods,
             GenerateRequestMethods
         ]
-        public class OutputEvent : IRequest
+        public record OutputEvent : IRequest
         {
             /// <summary>
             /// The output category. If not specified, 'console' is assumed.
             /// Values: 'console', 'stdout', 'stderr', 'telemetry', etc.
             /// </summary>
             [Optional]
-            public string? Category { get; set; }
+            public string? Category { get; init; }
 
             /// <summary>
             /// The output to report.
             /// </summary>
-            public string Output { get; set; } = null!;
+            public string Output { get; init; }
 
             /// <summary>
             /// If an attribute 'variablesReference' exists and its value is > 0, the output contains objects which can be retrieved by passing 'variablesReference' to the 'variables' request.
             /// </summary>
             [Optional]
-            public long? VariablesReference { get; set; }
+            public long? VariablesReference { get; init; }
 
             /// <summary>
             /// An optional source location where the output was produced.
             /// </summary>
             [Optional]
-            public Source? Source { get; set; }
+            public Source? Source { get; init; }
 
             /// <summary>
             /// An optional source location line where the output was produced.
             /// </summary>
             [Optional]
-            public long? Line { get; set; }
+            public long? Line { get; init; }
 
             /// <summary>
             /// An optional source location column where the output was produced.
             /// </summary>
             [Optional]
-            public long? Column { get; set; }
+            public long? Column { get; init; }
 
             /// <summary>
             /// Optional data to report. For the 'telemetry' category the data will be sent to telemetry, for the other categories the data is shown in JSON format.
             /// </summary>
             [Optional]
-            public JToken? Data { get; set; }
+            public JToken? Data { get; init; }
         }
     }
 }

src/Dap.Protocol/Feature/Events/ProcessFeature.cs

@@ -19,24 +19,24 @@ namespace Events
             GenerateHandlerMethods,
             GenerateRequestMethods
         ]
-        public class ProcessEvent : IRequest
+        public record ProcessEvent : IRequest
         {
             /// <summary>
             /// The logical name of the process. This is usually the full path to process's executable file. Example: /home/example/myproj/program.js.
             /// </summary>
-            public string Name { get; set; } = null!;
+            public string Name { get; init; }
 
             /// <summary>
             /// The system process id of the debugged process. This property will be missing for non-system processes.
             /// </summary>
             [Optional]
-            public long? SystemProcessId { get; set; }
+            public long? SystemProcessId { get; init; }
 
             /// <summary>
             /// If true, the process is running on the same computer as the debug adapter.
             /// </summary>
             [Optional]
-            public bool IsLocalProcess { get; set; }
+            public bool IsLocalProcess { get; init; }
 
             /// <summary>
             /// Describes how the debug engine started debugging this process.
@@ -45,13 +45,13 @@ public class ProcessEvent : IRequest
             /// 'attachForSuspendedLaunch': A project launcher component has launched a new process in a suspended state and then asked the debugger to attach.
             /// </summary>
             [Optional]
-            public ProcessEventStartMethod? StartMethod { get; set; }
+            public ProcessEventStartMethod? StartMethod { get; init; }
 
             /// <summary>
             /// The size of a pointer or address for this process, in bits. This value may be used by clients when formatting addresses for display.
             /// </summary>
             [Optional]
-            public long? PointerSize { get; set; }
+            public long? PointerSize { get; init; }
         }
 
         [JsonConverter(typeof(StringEnumConverter))]