Add response types and improve test suite

.github/dependabot.yml

@@ -1,32 +1,46 @@
 version: 2
 updates:
-  - package-ecosystem: "github-actions"
+  - package-ecosystem: 'github-actions'
     directory: /
     schedule:
       interval: monthly
     labels:
-      - "no-changelog"
+      - 'no-changelog'
 
-  - package-ecosystem: "pip"
+  - package-ecosystem: 'github-actions'
+    directory: .github/actions/lint
+    schedule:
+      interval: monthly
+    labels:
+      - 'no-changelog'
+
+  - package-ecosystem: 'github-actions'
+    directory: .github/actions/build-package
+    schedule:
+      interval: monthly
+    labels:
+      - 'no-changelog'
+
+  - package-ecosystem: 'pip'
     directory: /generator
     schedule:
       interval: daily
     labels:
-      - "no-changelog"
+      - 'no-changelog'
 
-  - package-ecosystem: "pip"
+  - package-ecosystem: 'pip'
     directory: /tests
     schedule:
       interval: daily
     labels:
-      - "no-changelog"
+      - 'no-changelog'
 
-  - package-ecosystem: "pip"
+  - package-ecosystem: 'pip'
     directory: /
     schedule:
       interval: daily
     labels:
-      - "debt"
+      - 'debt'
     commit-message:
-      include: "scope"
-      prefix: "pip"
+      include: 'scope'
+      prefix: 'pip'

.github/workflows/pr-check.yml

@@ -80,6 +80,10 @@ jobs:
       - name: Generate C# Code
         run: python -m generator --plugin dotnet
 
+      - name: Dotnet Build Tests
+        run: dotnet build tests/dotnet/lsprotocol_tests/lsprotocol_tests.csproj
+        shell: bash
+
       - name: Dotnet Run Tests
         run: dotnet test tests/dotnet/lsprotocol_tests/lsprotocol_tests.csproj
         shell: bash

generator/plugins/dotnet/custom/IResponse.cs

@@ -1,9 +1,9 @@
 public interface IResponse<TResponse> : IMessage
 {
 
-    OrType<int, string> Id { get; }
+    OrType<string, int> Id { get; }
 
-    TResponse Result { get; }
+    TResponse? Result { get; }
 
-    IResponseError? Error { get; }
+    ResponseError? Error { get; }
 }

generator/plugins/dotnet/custom/LSPRequest.cs

@@ -0,0 +1,22 @@
+using System;
+
+[AttributeUsage(AttributeTargets.Class)]
+public class LSPRequestAttribute : Attribute
+{
+    public LSPRequestAttribute(string method, Type response)
+    {
+        Method = method;
+        Response = response;
+    }
+
+    public LSPRequestAttribute(string method, Type response, Type partialResponse)
+    {
+        Method = method;
+        Response = response;
+        PartialResponse = partialResponse;
+    }
+
+    public string Method { get; }
+    public Type Response { get; }
+    public Type? PartialResponse { get; }
+}

generator/plugins/dotnet/custom/LSPResponse.cs

@@ -0,0 +1,13 @@
+using System;
+
+[AttributeUsage(AttributeTargets.Class)]
+public class LSPResponseAttribute : Attribute
+{
+    public LSPResponseAttribute(Type request)
+    {
+        Request = request;
+    }
+
+
+    public Type Request { get; }
+}

generator/plugins/dotnet/custom/OrTypeArrayConverter.cs

@@ -3,34 +3,34 @@
 using System;
 
 
-public class OrTypeArrayConverter<T, U, V, W> : JsonConverter<OrType<T, U, V, W>[]>
+public class OrTypeArrayConverter<T, U> : JsonConverter<OrType<T, U>[]>
 {
-    private OrTypeConverter<T, U, V, W> _converter;
+    private OrTypeConverter<T, U> _converter;
 
     public OrTypeArrayConverter()
     {
-        _converter = new OrTypeConverter<T, U, V, W>();
+        _converter = new OrTypeConverter<T, U>();
     }
 
-    public override OrType<T, U, V, W>[] ReadJson(JsonReader reader, Type objectType, OrType<T, U, V, W>[]? existingValue, bool hasExistingValue, JsonSerializer serializer)
+    public override OrType<T, U>[] ReadJson(JsonReader reader, Type objectType, OrType<T, U>[]? existingValue, bool hasExistingValue, JsonSerializer serializer)
     {
         if (reader.TokenType == JsonToken.Null)
         {
             return null;
         }
 
         JArray array = JArray.Load(reader);
-        var result = new OrType<T, U, V, W>[array.Count];
+        var result = new OrType<T, U>[array.Count];
 
         for (int i = 0; i < array.Count; i++)
         {
-            result[i] = (OrType<T, U, V, W>)_converter.ReadJson(array[i].CreateReader(), typeof(OrType<T, U, V, W>), null, serializer);
+            result[i] = (OrType<T, U>)_converter.ReadJson(array[i].CreateReader(), typeof(OrType<T, U>), null, serializer);
         }
 
         return result;
     }
 
-    public override void WriteJson(JsonWriter writer, OrType<T, U, V, W>[] value, JsonSerializer serializer)
+    public override void WriteJson(JsonWriter writer, OrType<T, U>[]? value, JsonSerializer serializer)
     {
         if (value is null)
         {
@@ -49,34 +49,34 @@ public override void WriteJson(JsonWriter writer, OrType<T, U, V, W>[] value, Js
         }
     }
 }
-public class OrTypeArrayConverter<T, U> : JsonConverter<OrType<T, U>[]>
+public class OrTypeArrayConverter<T, U, V> : JsonConverter<OrType<T, U, V>[]>
 {
-    private OrTypeConverter<T, U> _converter;
+    private OrTypeConverter<T, U, V> _converter;
 
     public OrTypeArrayConverter()
     {
-        _converter = new OrTypeConverter<T, U>();
+        _converter = new OrTypeConverter<T, U, V>();
     }
 
-    public override OrType<T, U>[] ReadJson(JsonReader reader, Type objectType, OrType<T, U>[]? existingValue, bool hasExistingValue, JsonSerializer serializer)
+    public override OrType<T, U, V>[] ReadJson(JsonReader reader, Type objectType, OrType<T, U, V>[]? existingValue, bool hasExistingValue, JsonSerializer serializer)
     {
         if (reader.TokenType == JsonToken.Null)
         {
             return null;
         }
 
         JArray array = JArray.Load(reader);
-        var result = new OrType<T, U>[array.Count];
+        var result = new OrType<T, U, V>[array.Count];
 
         for (int i = 0; i < array.Count; i++)
         {
-            result[i] = (OrType<T, U>)_converter.ReadJson(array[i].CreateReader(), typeof(OrType<T, U>), null, serializer);
+            result[i] = (OrType<T, U, V>)_converter.ReadJson(array[i].CreateReader(), typeof(OrType<T, U, V>), null, serializer);
         }
 
         return result;
     }
 
-    public override void WriteJson(JsonWriter writer, OrType<T, U>[] value, JsonSerializer serializer)
+    public override void WriteJson(JsonWriter writer, OrType<T, U, V>[]? value, JsonSerializer serializer)
     {
         if (value is null)
         {
@@ -95,34 +95,36 @@ public override void WriteJson(JsonWriter writer, OrType<T, U>[] value, JsonSeri
         }
     }
 }
-public class OrTypeArrayConverter<T, U, V> : JsonConverter<OrType<T, U, V>[]>
+
+
+public class OrTypeArrayConverter<T, U, V, W> : JsonConverter<OrType<T, U, V, W>[]>
 {
-    private OrTypeConverter<T, U, V> _converter;
+    private OrTypeConverter<T, U, V, W> _converter;
 
     public OrTypeArrayConverter()
     {
-        _converter = new OrTypeConverter<T, U, V>();
+        _converter = new OrTypeConverter<T, U, V, W>();
     }
 
-    public override OrType<T, U, V>[] ReadJson(JsonReader reader, Type objectType, OrType<T, U, V>[]? existingValue, bool hasExistingValue, JsonSerializer serializer)
+    public override OrType<T, U, V, W>[] ReadJson(JsonReader reader, Type objectType, OrType<T, U, V, W>[]? existingValue, bool hasExistingValue, JsonSerializer serializer)
     {
         if (reader.TokenType == JsonToken.Null)
         {
             return null;
         }
 
         JArray array = JArray.Load(reader);
-        var result = new OrType<T, U, V>[array.Count];
+        var result = new OrType<T, U, V, W>[array.Count];
 
         for (int i = 0; i < array.Count; i++)
         {
-            result[i] = (OrType<T, U, V>)_converter.ReadJson(array[i].CreateReader(), typeof(OrType<T, U, V>), null, serializer);
+            result[i] = (OrType<T, U, V, W>)_converter.ReadJson(array[i].CreateReader(), typeof(OrType<T, U, V, W>), null, serializer);
         }
 
         return result;
     }
 
-    public override void WriteJson(JsonWriter writer, OrType<T, U, V>[] value, JsonSerializer serializer)
+    public override void WriteJson(JsonWriter writer, OrType<T, U, V, W>[]? value, JsonSerializer serializer)
     {
         if (value is null)
         {