fixup visitor

jorgerangel-msft · jorgerangel-msft · commit aeb8b1b4aff5 · 2025-10-22T10:40:49.000-05:00
diff --git a/codegen/generator/src/Visitors/MetadataQueryParamVisitor.cs b/codegen/generator/src/Visitors/MetadataQueryParamVisitor.cs
@@ -92,7 +92,7 @@ dictionaryExpression.Original is VariableExpression variable &&
                     if (statement is ExpressionStatement expressionStatement3 &&
                         expressionStatement3.Expression is InvokeMethodExpression invokeMethodExpression &&
                         invokeMethodExpression.MethodName == "AppendQueryDelimited" &&
-                        invokeMethodExpression.Arguments.Count == 5 &&
+                        invokeMethodExpression.Arguments.Count == 4 &&
                         invokeMethodExpression.Arguments[0].ToDisplayString() == "\"metadata\"")
                     {
                         return new SingleLineCommentStatement("Plugin customization: remove unnecessary AppendQueryDelimited for metadata");
diff --git a/src/Generated/ChatClient.RestClient.cs b/src/Generated/ChatClient.RestClient.cs
@@ -40,7 +40,7 @@ internal virtual PipelineMessage CreateGetChatCompletionsRequest(string after, i
                     // Plugin customization: Properly handle metadata query parameters
                     uri.AppendQuery($"metadata[{@param.Key}]", @param.Value, true);
                 }
-                uri.AppendQueryDelimited("metadata", list, ",", escape: true);
+                // Plugin customization: remove unnecessary AppendQueryDelimited for metadata
             }
             if (model != null)
             {

Original file line number	Diff line number	Diff line change
`@@ -92,7 +92,7 @@ dictionaryExpression.Original is VariableExpression variable &&`
`92`	`92`	`if (statement is ExpressionStatement expressionStatement3 &&`
`93`	`93`	`expressionStatement3.Expression is InvokeMethodExpression invokeMethodExpression &&`
`94`	`94`	`invokeMethodExpression.MethodName == "AppendQueryDelimited" &&`
`95`		`- invokeMethodExpression.Arguments.Count == 5 &&`
	`95`	`+ invokeMethodExpression.Arguments.Count == 4 &&`
`96`	`96`	`invokeMethodExpression.Arguments[0].ToDisplayString() == "\"metadata\"")`
`97`	`97`	`{`
`98`	`98`	`return new SingleLineCommentStatement("Plugin customization: remove unnecessary AppendQueryDelimited for metadata");`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ internal virtual PipelineMessage CreateGetChatCompletionsRequest(string after, i`
`40`	`40`	`// Plugin customization: Properly handle metadata query parameters`
`41`	`41`	`uri.AppendQuery($"metadata[{@param.Key}]", @param.Value, true);`
`42`	`42`	`}`
`43`		`- uri.AppendQueryDelimited("metadata", list, ",", escape: true);`
	`43`	`+ // Plugin customization: remove unnecessary AppendQueryDelimited for metadata`
`44`	`44`	`}`
`45`	`45`	`if (model != null)`
`46`	`46`	`{`