Fix: Allow internal members in ObjectType and InterfaceType source generators (#8621)

Author: PHILLIPS71

src/HotChocolate/Core/src/Types.Analyzers/Inspectors/InterfaceTypeInfoInspector.cs

@@ -47,7 +47,10 @@ public bool TryHandle(GeneratorSyntaxContext context, [NotNullWhen(true)] out Sy
 
         foreach (var member in members)
         {
-            if (member.DeclaredAccessibility is Accessibility.Public or Accessibility.Internal)
+            if (member.DeclaredAccessibility is
+                Accessibility.Public or
+                Accessibility.Internal or
+                Accessibility.ProtectedAndInternal)
             {
                 if (member is IMethodSymbol { MethodKind: MethodKind.Ordinary } methodSymbol)
                 {

src/HotChocolate/Core/src/Types.Analyzers/Inspectors/ObjectTypeInspector.cs

@@ -58,7 +58,11 @@ public bool TryHandle(GeneratorSyntaxContext context, [NotNullWhen(true)] out Sy
 
         foreach (var member in members)
         {
-            if (member.DeclaredAccessibility is Accessibility.Public && !member.IsIgnored())
+            if (member.DeclaredAccessibility is
+                Accessibility.Public or
+                Accessibility.Internal or
+                Accessibility.ProtectedAndInternal
+                && !member.IsIgnored())
             {
                 if (member is IMethodSymbol { MethodKind: MethodKind.Ordinary } methodSymbol)
                 {

src/HotChocolate/Core/test/Types.Analyzers.Tests/ResolverTests.cs

@@ -241,4 +241,33 @@ internal class Test
             }
             """).MatchMarkdownAsync();
     }
+
+    [Fact]
+    public async Task Internal_NodeResolver_Should_Generate_Source()
+    {
+        await TestHelper.GetGeneratedSourceSnapshot(
+            """
+            using HotChocolate;
+            using HotChocolate.Types;
+            using HotChocolate.Types.Relay;
+            using System.Threading.Tasks;
+
+            namespace TestNamespace;
+
+            [ObjectType<Test>]
+            internal static partial class TestType
+            {
+                [NodeResolver]
+                internal static Task<Test?> GetTestByIdAsync(int id)
+                    => Task.FromResult<Test?>(null);
+            }
+
+            internal class Test
+            {
+                public int Id { get; set; }
+
+                public string Name { get; set; }
+            }
+            """).MatchMarkdownAsync();
+    }
 }

src/HotChocolate/Core/test/Types.Analyzers.Tests/__snapshots__/ResolverTests.Internal_NodeResolver_Should_Generate_Source.md

@@ -0,0 +1,86 @@
+# Internal_NodeResolver_Should_Generate_Source
+
+## HotChocolateTypeModule.735550c.g.cs
+
+```csharp
+// <auto-generated/>
+
+#nullable enable
+#pragma warning disable
+
+using System;
+using System.Runtime.CompilerServices;
+using HotChocolate;
+using HotChocolate.Types;
+using HotChocolate.Execution.Configuration;
+
+namespace Microsoft.Extensions.DependencyInjection
+{
+    public static partial class TestsTypesRequestExecutorBuilderExtensions
+    {
+        public static IRequestExecutorBuilder AddTestsTypes(this IRequestExecutorBuilder builder)
+        {
+            builder.ConfigureDescriptorContext(ctx => ctx.TypeConfiguration.TryAdd<global::TestNamespace.Test>(
+                "Tests::TestNamespace.TestType",
+                () => global::TestNamespace.TestType.Initialize));
+            builder.AddType<ObjectType<global::TestNamespace.Test>>();
+            return builder;
+        }
+    }
+}
+
+```
+
+## TestType.WaAdMHmlGJHjtEI4nqY7WA.hc.g.cs
+
+```csharp
+// <auto-generated/>
+
+#nullable enable
+#pragma warning disable
+
+using System;
+using System.Runtime.CompilerServices;
+using HotChocolate;
+using HotChocolate.Types;
+using HotChocolate.Execution.Configuration;
+using Microsoft.Extensions.DependencyInjection;
+using HotChocolate.Internal;
+
+namespace TestNamespace
+{
+    internal static partial class TestType
+    {
+        internal static void Initialize(global::HotChocolate.Types.IObjectTypeDescriptor<global::TestNamespace.Test> descriptor)
+        {
+            var thisType = typeof(global::TestNamespace.TestType);
+            var bindingResolver = descriptor.Extend().Context.ParameterBindingResolver;
+            var resolvers = new __Resolvers();
+
+            descriptor
+                .ImplementsNode()
+                .ResolveNode(resolvers.GetTestByIdAsync().Resolver!);
+
+            Configure(descriptor);
+        }
+
+        static partial void Configure(global::HotChocolate.Types.IObjectTypeDescriptor<global::TestNamespace.Test> descriptor);
+
+        private sealed class __Resolvers
+        {
+            public HotChocolate.Resolvers.FieldResolverDelegates GetTestByIdAsync()
+                => new global::HotChocolate.Resolvers.FieldResolverDelegates(resolver: GetTestByIdAsync);
+
+            private async global::System.Threading.Tasks.ValueTask<global::System.Object?> GetTestByIdAsync(global::HotChocolate.Resolvers.IResolverContext context)
+            {
+                var args0 = context.GetLocalState<int>(global::HotChocolate.WellKnownContextData.InternalId);
+                var result = await global::TestNamespace.TestType.GetTestByIdAsync(args0);
+                return result;
+            }
+        }
+    }
+}
+
+
+```
+