Merge branch 'main' into gai/fix-analyzer-errors

Author: glen-84

src/HotChocolate/Fusion-vnext/src/Fusion.Composition/Definitions/CacheControlMutableDirectiveDefinition.cs

@@ -3,7 +3,7 @@
 
 namespace HotChocolate.Fusion.Definitions;
 
-public sealed class CacheControlMutableDirectiveDefinition : MutableDirectiveDefinition
+internal sealed class CacheControlMutableDirectiveDefinition : MutableDirectiveDefinition
 {
     public CacheControlMutableDirectiveDefinition(
         MutableScalarTypeDefinition intType,

src/HotChocolate/Fusion-vnext/src/Fusion.Composition/Definitions/CacheControlScopeMutableEnumTypeDefinition.cs

@@ -2,7 +2,7 @@
 
 namespace HotChocolate.Fusion.Definitions;
 
-public sealed class CacheControlScopeMutableEnumTypeDefinition : MutableEnumTypeDefinition
+internal sealed class CacheControlScopeMutableEnumTypeDefinition : MutableEnumTypeDefinition
 {
     public CacheControlScopeMutableEnumTypeDefinition()
         : base(WellKnownTypeNames.CacheControlScope)

src/HotChocolate/Fusion-vnext/src/Fusion.Composition/Definitions/McpToolAnnotationsMutableDirectiveDefinition.cs

@@ -0,0 +1,28 @@
+using HotChocolate.Types;
+using HotChocolate.Types.Mutable;
+using static HotChocolate.Fusion.WellKnownArgumentNames;
+using static HotChocolate.Fusion.WellKnownDirectiveNames;
+
+namespace HotChocolate.Fusion.Definitions;
+
+internal sealed class McpToolAnnotationsMutableDirectiveDefinition : MutableDirectiveDefinition
+{
+    public McpToolAnnotationsMutableDirectiveDefinition(MutableScalarTypeDefinition booleanType)
+        : base(McpToolAnnotations)
+    {
+        Arguments.Add(new MutableInputFieldDefinition(DestructiveHint, booleanType));
+        Arguments.Add(new MutableInputFieldDefinition(IdempotentHint, booleanType));
+        Arguments.Add(new MutableInputFieldDefinition(OpenWorldHint, booleanType));
+        Locations = DirectiveLocation.FieldDefinition;
+    }
+
+    public static McpToolAnnotationsMutableDirectiveDefinition Create(ISchemaDefinition schema)
+    {
+        if (!schema.Types.TryGetType<MutableScalarTypeDefinition>(SpecScalarNames.Boolean.Name, out var booleanType))
+        {
+            booleanType = BuiltIns.Boolean.Create();
+        }
+
+        return new McpToolAnnotationsMutableDirectiveDefinition(booleanType);
+    }
+}

src/HotChocolate/Fusion-vnext/src/Fusion.Composition/Definitions/ScalarSerializationTypeMutableEnumTypeDefinition.cs

@@ -2,7 +2,7 @@
 
 namespace HotChocolate.Fusion.Definitions;
 
-public sealed class ScalarSerializationTypeMutableEnumTypeDefinition : MutableEnumTypeDefinition
+internal sealed class ScalarSerializationTypeMutableEnumTypeDefinition : MutableEnumTypeDefinition
 {
     public ScalarSerializationTypeMutableEnumTypeDefinition()
         : base(WellKnownTypeNames.ScalarSerializationType)

src/HotChocolate/Fusion-vnext/src/Fusion.Composition/Definitions/SerializeAsMutableDirectiveDefinition.cs

@@ -3,7 +3,7 @@
 
 namespace HotChocolate.Fusion.Definitions;
 
-public sealed class SerializeAsMutableDirectiveDefinition : MutableDirectiveDefinition
+internal sealed class SerializeAsMutableDirectiveDefinition : MutableDirectiveDefinition
 {
     public SerializeAsMutableDirectiveDefinition(
         ScalarSerializationTypeMutableEnumTypeDefinition scalarSerializationTypeType,

src/HotChocolate/Fusion-vnext/src/Fusion.Composition/Definitions/TagMutableDirectiveDefinition.cs

@@ -3,7 +3,7 @@
 
 namespace HotChocolate.Fusion.Definitions;
 
-public sealed class TagMutableDirectiveDefinition : MutableDirectiveDefinition
+internal sealed class TagMutableDirectiveDefinition : MutableDirectiveDefinition
 {
     public TagMutableDirectiveDefinition(MutableScalarTypeDefinition stringType)
         : base(DirectiveNames.Tag.Name)

src/HotChocolate/Fusion-vnext/src/Fusion.Composition/DirectiveMergers/McpToolAnnotationsDirectiveMerger.cs

@@ -0,0 +1,94 @@
+using System.Collections.Immutable;
+using HotChocolate.Fusion.Definitions;
+using HotChocolate.Fusion.Directives;
+using HotChocolate.Fusion.Extensions;
+using HotChocolate.Fusion.Options;
+using HotChocolate.Types;
+using HotChocolate.Types.Mutable;
+using ArgumentNames = HotChocolate.Fusion.WellKnownArgumentNames;
+using DirectiveNames = HotChocolate.Fusion.WellKnownDirectiveNames;
+
+namespace HotChocolate.Fusion.DirectiveMergers;
+
+internal class McpToolAnnotationsDirectiveMerger(DirectiveMergeBehavior mergeBehavior)
+    : DirectiveMergerBase(mergeBehavior)
+{
+    public override string DirectiveName => DirectiveNames.McpToolAnnotations;
+
+    public override MutableDirectiveDefinition GetCanonicalDirectiveDefinition(ISchemaDefinition schema)
+    {
+        return McpToolAnnotationsMutableDirectiveDefinition.Create(schema);
+    }
+
+    public override void MergeDirectives(
+        IDirectivesProvider mergedMember,
+        ImmutableArray<IDirectivesProvider> memberDefinitions,
+        MutableSchemaDefinition mergedSchema)
+    {
+        if (MergeBehavior is DirectiveMergeBehavior.Ignore)
+        {
+            return;
+        }
+
+        if (!mergedSchema.DirectiveDefinitions.TryGetDirective(DirectiveName, out var directiveDefinition))
+        {
+            // Merged definition not found.
+            return;
+        }
+
+        var mcpToolAnnotationsDirectives =
+            memberDefinitions
+                .SelectMany(d => d.Directives.Where(dir => dir.Name == DirectiveNames.McpToolAnnotations))
+                .Select(McpToolAnnotationsDirective.From)
+                .ToArray();
+
+        if (mcpToolAnnotationsDirectives.Length == 0)
+        {
+            return;
+        }
+
+        var isMutationField = ((IFieldDefinition)memberDefinitions[0]).DeclaringMember is IObjectTypeDefinition
+        {
+            Name: WellKnownTypeNames.Mutation
+        };
+
+        var argumentAssignments = new List<ArgumentAssignment>();
+
+        if (!mcpToolAnnotationsDirectives.All(d => d.DestructiveHint is null))
+        {
+            var destructiveHint =
+                mcpToolAnnotationsDirectives.Any(
+                    d => isMutationField
+                        ? d.DestructiveHint is null or true
+                        : d.DestructiveHint is true);
+
+            argumentAssignments.Add(
+                new ArgumentAssignment(ArgumentNames.DestructiveHint, destructiveHint));
+        }
+
+        if (!mcpToolAnnotationsDirectives.All(d => d.IdempotentHint is null))
+        {
+            var idempotentHint =
+                mcpToolAnnotationsDirectives.All(
+                    d => isMutationField
+                        ? d.IdempotentHint is true
+                        : d.IdempotentHint is null or true);
+
+            argumentAssignments.Add(
+                new ArgumentAssignment(ArgumentNames.IdempotentHint, idempotentHint));
+        }
+
+        if (!mcpToolAnnotationsDirectives.All(d => d.OpenWorldHint is null))
+        {
+            var openWorldHint =
+                mcpToolAnnotationsDirectives.Any(d => d.OpenWorldHint is null or true);
+
+            argumentAssignments.Add(
+                new ArgumentAssignment(ArgumentNames.OpenWorldHint, openWorldHint));
+        }
+
+        var mcpToolAnnotationsDirective = new Directive(directiveDefinition, argumentAssignments);
+
+        mergedMember.AddDirective(mcpToolAnnotationsDirective);
+    }
+}

src/HotChocolate/Fusion-vnext/src/Fusion.Composition/Directives/CacheControlDirective.cs

@@ -4,7 +4,7 @@
 
 namespace HotChocolate.Fusion.Directives;
 
-public sealed class CacheControlDirective(
+internal sealed class CacheControlDirective(
     int? maxAge = null,
     CacheControlScope? scope = null,
     bool? inheritMaxAge = null,

src/HotChocolate/Fusion-vnext/src/Fusion.Composition/Directives/McpToolAnnotationsDirective.cs

@@ -0,0 +1,27 @@
+using HotChocolate.Types;
+
+namespace HotChocolate.Fusion.Directives;
+
+internal sealed class McpToolAnnotationsDirective(
+    bool? destructiveHint = null,
+    bool? idempotentHint = null,
+    bool? openWorldHint = null)
+{
+    public bool? DestructiveHint { get; set; } = destructiveHint;
+
+    public bool? IdempotentHint { get; set; } = idempotentHint;
+
+    public bool? OpenWorldHint { get; set; } = openWorldHint;
+
+    public static McpToolAnnotationsDirective From(IDirective directive)
+    {
+        var destructiveHint =
+            (bool?)directive.Arguments.GetValueOrDefault(WellKnownArgumentNames.DestructiveHint)?.Value;
+        var idempotentHint =
+            (bool?)directive.Arguments.GetValueOrDefault(WellKnownArgumentNames.IdempotentHint)?.Value;
+        var openWorldHint =
+            (bool?)directive.Arguments.GetValueOrDefault(WellKnownArgumentNames.OpenWorldHint)?.Value;
+
+        return new McpToolAnnotationsDirective(destructiveHint, idempotentHint, openWorldHint);
+    }
+}

src/HotChocolate/Fusion-vnext/src/Fusion.Composition/Directives/SerializeAsDirective.cs

@@ -4,7 +4,7 @@
 
 namespace HotChocolate.Fusion.Directives;
 
-public sealed class SerializeAsDirective(ScalarSerializationType type, string? pattern)
+internal sealed class SerializeAsDirective(ScalarSerializationType type, string? pattern)
 {
     public ScalarSerializationType Type { get; } = type;