Address feedback

Author: eiriktsarpalis

src/ModelContextProtocol.Core/Server/AIFunctionMcpServerResource.cs

@@ -359,7 +359,7 @@ private AIFunctionMcpServerResource(AIFunction function, ResourceTemplate resour
                 return null;
             }
         }
-        else if (new Uri(request.Params.Uri) != new Uri(ProtocolResource!.Uri))
+        else if (UriTemplate.UriTemplateComparer.Instance.Equals(request.Params.Uri, ProtocolResource!.Uri))
         {
             return null;
         }

src/ModelContextProtocol.Core/Server/McpServer.cs

@@ -228,7 +228,7 @@ private void ConfigureResources(McpServerOptions options)
         var unsubscribeHandler = resourcesCapability.UnsubscribeFromResourcesHandler ?? (static async (_, __) => new EmptyResult());
         var resources = resourcesCapability.ResourceCollection;
         var listChanged = resourcesCapability.ListChanged;
-        var subcribe = resourcesCapability.Subscribe;
+        var subscribe = resourcesCapability.Subscribe;
 
         // Handle resources provided via DI.
         if (resources is { IsEmpty: false })
@@ -309,7 +309,7 @@ await originalListResourceTemplatesHandler(request, cancellationToken).Configure
             listChanged = true;
 
             // TODO: Implement subscribe/unsubscribe logic for resource and resource template collections.
-            // subcribe = true;
+            // subscribe = true;
         }
 
         ServerCapabilities.Resources.ListResourcesHandler = listResourcesHandler;
@@ -319,7 +319,7 @@ await originalListResourceTemplatesHandler(request, cancellationToken).Configure
         ServerCapabilities.Resources.SubscribeToResourcesHandler = subscribeHandler;
         ServerCapabilities.Resources.UnsubscribeFromResourcesHandler = unsubscribeHandler;
         ServerCapabilities.Resources.ListChanged = listChanged;
-        ServerCapabilities.Resources.Subscribe = subcribe;
+        ServerCapabilities.Resources.Subscribe = subscribe;
 
         SetHandler(
             RequestMethods.ResourcesList,

src/ModelContextProtocol.Core/UriTemplate.cs

@@ -461,8 +461,9 @@ static void AppendHex(ref DefaultInterpolatedStringHandler builder, char c)
     /// 2. Templated Uris use regular string equality.
     /// 
     /// We do this because non-templated resources are looked up directly from the resource dictionary
-    /// and we need to make sure equality is implemented correctly. Templated Uris are resolved using
-    /// linear traversal so there's no need for equality comparison to be fully accurate.
+    /// and we need to make sure equality is implemented correctly. Templated Uris are resolved in a
+    /// fallback step using linear traversal of the resource dictionary, so their equality is only
+    /// there to distinguish between different templates.
     /// </summary>
     public sealed class UriTemplateComparer : IEqualityComparer<string>
     {
@@ -499,7 +500,7 @@ private static bool TryParseAsNonTemplatedUri(string? uriTemplate, [NotNullWhen(
                 return false;
             }
 
-            return Uri.TryCreate(uriTemplate, UriKind.RelativeOrAbsolute, out uri);
+            return Uri.TryCreate(uriTemplate, UriKind.Absolute, out uri);
         }
     }
 }