Change TryParseElicitations out parameter to IList, remove unnecessary ToList()

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>

Author: Copilot

src/ModelContextProtocol.Core/UrlElicitationRequiredException.cs

@@ -66,7 +66,7 @@ internal static bool TryCreateFromError(
         return true;
     }
 
-    private static bool TryParseElicitations(JsonElement dataElement, out IReadOnlyList<ElicitRequestParams> elicitations)
+    private static bool TryParseElicitations(JsonElement dataElement, out IList<ElicitRequestParams> elicitations)
     {
         elicitations = [];
 
@@ -89,7 +89,7 @@ private static bool TryParseElicitations(JsonElement dataElement, out IReadOnlyL
             }
         }
 
-        elicitations = elicitationsFromPayload.ToList();
+        elicitations = elicitationsFromPayload;
         return true;
     }