[pull] main from fern-api:main

generators/csharp/sdk/changes/2.63.1/fix-comma-delimited-query-params-wiremock.yml

@@ -0,0 +1,6 @@
+- summary: |
+    Fix WireMock test generation to use percent-encoded query parameter keys,
+    matching what spec-compliant HTTP clients send. Also split comma-delimited
+    query parameter examples into multiple values in a single `WithParam()` call,
+    since WireMock.Net parses comma-separated values into arrays.
+  type: fix

generators/csharp/sdk/changes/2.63.2/fix-multivalue-query-param-list-example-snippets.yml

@@ -0,0 +1,6 @@
+- summary: |
+    Fix multi-value query parameter snippets when the example value is already
+    a list, which previously produced uncompilable list-of-list initializers
+    like `[new List<T>() { value }]` in mock server tests, reference docs, and
+    snippets.
+  type: fix

generators/csharp/sdk/src/test-generation/mock-server/MockEndpointGenerator.ts

@@ -96,7 +96,13 @@ export class MockEndpointGenerator extends WithGeneration {
                 for (const parameter of example.queryParameters) {
                     const maybeParameterValue = this.exampleToQueryOrHeaderValue(parameter);
                     if (maybeParameterValue != null) {
-                        writer.write(`.WithParam("${getWireValue(parameter.name)}", "${maybeParameterValue}")`);
+                        const encodedKey = percentEncodeQueryKey(getWireValue(parameter.name));
+                        // WireMock.Net splits comma-delimited query values into separate array
+                        // entries, so pass all values in a single WithParam call.
+                        const paramValues = maybeParameterValue
+                            .split(",")
+                            .map((v) => `"${v.replace(/\\/g, "\\\\").replace(/"/g, '\\"')}"`);
+                        writer.write(`.WithParam("${encodedKey}", ${paramValues.join(", ")})`);
                     }
                 }
                 for (const header of [...example.serviceHeaders, ...example.endpointHeaders]) {
@@ -973,3 +979,27 @@ export class MockEndpointGenerator extends WithGeneration {
         return pairs.join(", ");
     }
 }
+
+// Characters the C# SDK's QueryStringBuilder treats as safe for query keys.
+// Mirrors: unreserved + (sub-delims \ {& = +}) + : @ / ?
+// See QueryStringBuilder.Template.cs SafeQueryKeyChars.
+const SAFE_QUERY_KEY_CHARS = new Set("ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789-_.~!$'()*,;:@/?");
+
+/**
+ * Percent-encodes a query parameter key to match the C# SDK's QueryStringBuilder.
+ * Characters not in SafeQueryKeyChars are percent-encoded with uppercase hex digits.
+ */
+function percentEncodeQueryKey(key: string): string {
+    const encoder = new TextEncoder();
+    let encoded = "";
+    for (const char of key) {
+        if (SAFE_QUERY_KEY_CHARS.has(char)) {
+            encoded += char;
+        } else {
+            for (const byte of encoder.encode(char)) {
+                encoded += `%${byte.toString(16).toUpperCase().padStart(2, "0")}`;
+            }
+        }
+    }
+    return encoded;
+}

generators/csharp/sdk/src/wrapped-request/WrappedRequestGenerator.ts

@@ -331,19 +331,22 @@ export class WrappedRequestGenerator extends FileGenerator<CSharpFile, SdkGenera
         for (const exampleQueryParameter of example.queryParameters) {
             const isSingleQueryParameter =
                 exampleQueryParameter.shape == null || exampleQueryParameter.shape.type === "single";
+            const exampleShape = exampleQueryParameter.value.shape;
+            const isExampleAlreadyList = exampleShape.type === "container" && exampleShape.container.type === "list";
             const singleValueSnippet = this.exampleGenerator.getSnippetForTypeReference({
                 exampleTypeReference: exampleQueryParameter.value,
                 parseDatetimes
             });
-            const value = isSingleQueryParameter
-                ? singleValueSnippet
-                : this.csharp.codeblock((writer: Writer) =>
-                      writer.writeNode(
-                          this.csharp.list({
-                              entries: [singleValueSnippet]
-                          })
-                      )
-                  );
+            const value =
+                isSingleQueryParameter || isExampleAlreadyList
+                    ? singleValueSnippet
+                    : this.csharp.codeblock((writer: Writer) =>
+                          writer.writeNode(
+                              this.csharp.list({
+                                  entries: [singleValueSnippet]
+                              })
+                          )
+                      );
             orderedFields.push({
                 name: exampleQueryParameter.name,
                 value

generators/csharp/sdk/versions.yml

@@ -1,4 +1,24 @@
 # yaml-language-server: $schema=../../../fern-versions-yml.schema.json
+- version: 2.63.2
+  changelogEntry:
+    - summary: |
+        Fix multi-value query parameter snippets when the example value is already
+        a list, which previously produced uncompilable list-of-list initializers
+        like `[new List<T>() { value }]` in mock server tests, reference docs, and
+        snippets.
+      type: fix
+  createdAt: "2026-04-29"
+  irVersion: 66
+- version: 2.63.1
+  changelogEntry:
+    - summary: |
+        Fix WireMock test generation to use percent-encoded query parameter keys,
+        matching what spec-compliant HTTP clients send. Also split comma-delimited
+        query parameter examples into multiple values in a single `WithParam()` call,
+        since WireMock.Net parses comma-separated values into arrays.
+      type: fix
+  createdAt: "2026-04-29"
+  irVersion: 66
 - version: 2.63.0
   changelogEntry:
     - summary: |

generators/go-v2/ast/src/custom-config/BaseGoCustomConfigSchema.ts

@@ -29,7 +29,8 @@ export const baseGoCustomConfigSchema = z.strictObject({
     customPagerName: z.string().optional(),
     offsetSemantics: z.enum(["item-index", "page-index"]).optional(),
     omitFernHeaders: z.boolean().optional(),
-    maxRetries: z.number().int().min(0).optional()
+    maxRetries: z.number().int().min(0).optional(),
+    retryStatusCodes: z.optional(z.enum(["legacy", "recommended"]))
 });
 
 export type BaseGoCustomConfigSchema = z.infer<typeof baseGoCustomConfigSchema>;

generators/go-v2/base/src/asIs/internal/retrier.go_

@@ -138,9 +138,7 @@ func (r *Retrier) run(
 // shouldRetry returns true if the request should be retried based on the given
 // response status code.
 func (r *Retrier) shouldRetry(response *http.Response) bool {
-	return response.StatusCode == http.StatusTooManyRequests ||
-		response.StatusCode == http.StatusRequestTimeout ||
-		response.StatusCode >= http.StatusInternalServerError
+	return {{RETRY_STATUS_CHECK}}
 }
 
 // retryDelay calculates the delay time based on response headers,

generators/go-v2/base/src/project/GoProject.ts

@@ -205,6 +205,12 @@ export class GoProject extends AbstractProject<AbstractGoGeneratorContext<BaseGo
             contents = contents.replace(doubleRegex, value);
         }
 
+        const retryStatusCheck =
+            this.context.customConfig.retryStatusCodes === "recommended"
+                ? "response.StatusCode == http.StatusTooManyRequests ||\n\t\tresponse.StatusCode == http.StatusRequestTimeout ||\n\t\tresponse.StatusCode == http.StatusBadGateway ||\n\t\tresponse.StatusCode == http.StatusServiceUnavailable ||\n\t\tresponse.StatusCode == http.StatusGatewayTimeout"
+                : "response.StatusCode == http.StatusTooManyRequests ||\n\t\tresponse.StatusCode == http.StatusRequestTimeout ||\n\t\tresponse.StatusCode >= http.StatusInternalServerError";
+        contents = contents.replace(/\{\{RETRY_STATUS_CHECK\}\}/g, retryStatusCheck);
+
         return new File(filename.replace(".go_", ".go"), RelativeFilePath.of(""), contents);
     }
 

generators/go-v2/sdk/features.yml

@@ -61,11 +61,19 @@ features:
       as the request is deemed retryable and the number of retry attempts has not grown larger than the configured
       retry limit (default: 2).
 
-      A request is deemed retryable when any of the following HTTP status codes is returned:
+      Which status codes are retried depends on the `retryStatusCodes` generator configuration:
 
+      **`legacy`** (current default): retries on
       - [408](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/408) (Timeout)
       - [429](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/429) (Too Many Requests)
-      - [5XX](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/500) (Internal Server Errors)
+      - [5XX](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status#server_error_responses) (All server errors, including 500)
+
+      **`recommended`**: retries on
+      - [408](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/408) (Timeout)
+      - [429](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/429) (Too Many Requests)
+      - [502](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/502) (Bad Gateway)
+      - [503](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/503) (Service Unavailable)
+      - [504](https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/504) (Gateway Timeout)
 
       If the `Retry-After` header is present in the response, the SDK will prioritize respecting its value exactly
       over the default exponential backoff.

generators/go/internal/testdata/sdk/auth/fixtures/internal/retrier.go

@@ -130,7 +130,7 @@ func (r *Retrier) run(
 func (r *Retrier) shouldRetry(response *http.Response) bool {
 	return response.StatusCode == http.StatusTooManyRequests ||
 		response.StatusCode == http.StatusRequestTimeout ||
-		response.StatusCode >= http.StatusInternalServerError
+		(response.StatusCode > http.StatusInternalServerError && response.StatusCode < 600)
 }
 
 // retryDelay calculates the delay time in milliseconds based on the retry attempt.

generators/go/internal/testdata/sdk/auth/fixtures/internal/retrier_test.go

@@ -84,9 +84,24 @@ func TestRetrier(t *testing.T) {
 			giveStatusCodes: []int{http.StatusRequestTimeout, http.StatusOK},
 		},
 		{
-			description:     "retries occur on status code 500",
+			description:     "retries occur on status code 502",
 			giveAttempts:    2,
-			giveStatusCodes: []int{http.StatusInternalServerError, http.StatusOK},
+			giveStatusCodes: []int{http.StatusBadGateway, http.StatusOK},
+		},
+		{
+			description:     "no retries on status code 500",
+			giveAttempts:    1,
+			giveStatusCodes: []int{http.StatusInternalServerError},
+		},
+		{
+			description:     "retries occur on status code 501",
+			giveAttempts:    2,
+			giveStatusCodes: []int{501, http.StatusOK},
+		},
+		{
+			description:     "retries occur on status code 599",
+			giveAttempts:    2,
+			giveStatusCodes: []int{599, http.StatusOK},
 		},
 	}
 

generators/go/internal/testdata/sdk/basic/fixtures/internal/retrier.go

@@ -130,7 +130,7 @@ func (r *Retrier) run(
 func (r *Retrier) shouldRetry(response *http.Response) bool {
 	return response.StatusCode == http.StatusTooManyRequests ||
 		response.StatusCode == http.StatusRequestTimeout ||
-		response.StatusCode >= http.StatusInternalServerError
+		(response.StatusCode > http.StatusInternalServerError && response.StatusCode < 600)
 }
 
 // retryDelay calculates the delay time in milliseconds based on the retry attempt.

generators/go/internal/testdata/sdk/basic/fixtures/internal/retrier_test.go

@@ -84,9 +84,24 @@ func TestRetrier(t *testing.T) {
 			giveStatusCodes: []int{http.StatusRequestTimeout, http.StatusOK},
 		},
 		{
-			description:     "retries occur on status code 500",
+			description:     "retries occur on status code 502",
 			giveAttempts:    2,
-			giveStatusCodes: []int{http.StatusInternalServerError, http.StatusOK},
+			giveStatusCodes: []int{http.StatusBadGateway, http.StatusOK},
+		},
+		{
+			description:     "no retries on status code 500",
+			giveAttempts:    1,
+			giveStatusCodes: []int{http.StatusInternalServerError},
+		},
+		{
+			description:     "retries occur on status code 501",
+			giveAttempts:    2,
+			giveStatusCodes: []int{501, http.StatusOK},
+		},
+		{
+			description:     "retries occur on status code 599",
+			giveAttempts:    2,
+			giveStatusCodes: []int{599, http.StatusOK},
 		},
 	}
 

generators/go/internal/testdata/sdk/bearer-token-name/fixtures/internal/retrier.go

@@ -130,7 +130,7 @@ func (r *Retrier) run(
 func (r *Retrier) shouldRetry(response *http.Response) bool {
 	return response.StatusCode == http.StatusTooManyRequests ||
 		response.StatusCode == http.StatusRequestTimeout ||
-		response.StatusCode >= http.StatusInternalServerError
+		(response.StatusCode > http.StatusInternalServerError && response.StatusCode < 600)
 }
 
 // retryDelay calculates the delay time in milliseconds based on the retry attempt.

generators/go/internal/testdata/sdk/bearer-token-name/fixtures/internal/retrier_test.go

@@ -84,9 +84,24 @@ func TestRetrier(t *testing.T) {
 			giveStatusCodes: []int{http.StatusRequestTimeout, http.StatusOK},
 		},
 		{
-			description:     "retries occur on status code 500",
+			description:     "retries occur on status code 502",
 			giveAttempts:    2,
-			giveStatusCodes: []int{http.StatusInternalServerError, http.StatusOK},
+			giveStatusCodes: []int{http.StatusBadGateway, http.StatusOK},
+		},
+		{
+			description:     "no retries on status code 500",
+			giveAttempts:    1,
+			giveStatusCodes: []int{http.StatusInternalServerError},
+		},
+		{
+			description:     "retries occur on status code 501",
+			giveAttempts:    2,
+			giveStatusCodes: []int{501, http.StatusOK},
+		},
+		{
+			description:     "retries occur on status code 599",
+			giveAttempts:    2,
+			giveStatusCodes: []int{599, http.StatusOK},
 		},
 	}
 

generators/go/internal/testdata/sdk/bearer/fixtures/internal/retrier.go

@@ -130,7 +130,7 @@ func (r *Retrier) run(
 func (r *Retrier) shouldRetry(response *http.Response) bool {
 	return response.StatusCode == http.StatusTooManyRequests ||
 		response.StatusCode == http.StatusRequestTimeout ||
-		response.StatusCode >= http.StatusInternalServerError
+		(response.StatusCode > http.StatusInternalServerError && response.StatusCode < 600)
 }
 
 // retryDelay calculates the delay time in milliseconds based on the retry attempt.

generators/go/internal/testdata/sdk/bearer/fixtures/internal/retrier_test.go

@@ -84,9 +84,24 @@ func TestRetrier(t *testing.T) {
 			giveStatusCodes: []int{http.StatusRequestTimeout, http.StatusOK},
 		},
 		{
-			description:     "retries occur on status code 500",
+			description:     "retries occur on status code 502",
 			giveAttempts:    2,
-			giveStatusCodes: []int{http.StatusInternalServerError, http.StatusOK},
+			giveStatusCodes: []int{http.StatusBadGateway, http.StatusOK},
+		},
+		{
+			description:     "no retries on status code 500",
+			giveAttempts:    1,
+			giveStatusCodes: []int{http.StatusInternalServerError},
+		},
+		{
+			description:     "retries occur on status code 501",
+			giveAttempts:    2,
+			giveStatusCodes: []int{501, http.StatusOK},
+		},
+		{
+			description:     "retries occur on status code 599",
+			giveAttempts:    2,
+			giveStatusCodes: []int{599, http.StatusOK},
 		},
 	}
 

generators/go/internal/testdata/sdk/bytes/fixtures/internal/retrier.go

@@ -130,7 +130,7 @@ func (r *Retrier) run(
 func (r *Retrier) shouldRetry(response *http.Response) bool {
 	return response.StatusCode == http.StatusTooManyRequests ||
 		response.StatusCode == http.StatusRequestTimeout ||
-		response.StatusCode >= http.StatusInternalServerError
+		(response.StatusCode > http.StatusInternalServerError && response.StatusCode < 600)
 }
 
 // retryDelay calculates the delay time in milliseconds based on the retry attempt.

generators/go/internal/testdata/sdk/bytes/fixtures/internal/retrier_test.go

@@ -84,9 +84,24 @@ func TestRetrier(t *testing.T) {
 			giveStatusCodes: []int{http.StatusRequestTimeout, http.StatusOK},
 		},
 		{
-			description:     "retries occur on status code 500",
+			description:     "retries occur on status code 502",
 			giveAttempts:    2,
-			giveStatusCodes: []int{http.StatusInternalServerError, http.StatusOK},
+			giveStatusCodes: []int{http.StatusBadGateway, http.StatusOK},
+		},
+		{
+			description:     "no retries on status code 500",
+			giveAttempts:    1,
+			giveStatusCodes: []int{http.StatusInternalServerError},
+		},
+		{
+			description:     "retries occur on status code 501",
+			giveAttempts:    2,
+			giveStatusCodes: []int{501, http.StatusOK},
+		},
+		{
+			description:     "retries occur on status code 599",
+			giveAttempts:    2,
+			giveStatusCodes: []int{599, http.StatusOK},
 		},
 	}
 

generators/go/internal/testdata/sdk/client-options-core/fixtures/internal/retrier.go

@@ -130,7 +130,7 @@ func (r *Retrier) run(
 func (r *Retrier) shouldRetry(response *http.Response) bool {
 	return response.StatusCode == http.StatusTooManyRequests ||
 		response.StatusCode == http.StatusRequestTimeout ||
-		response.StatusCode >= http.StatusInternalServerError
+		(response.StatusCode > http.StatusInternalServerError && response.StatusCode < 600)
 }
 
 // retryDelay calculates the delay time in milliseconds based on the retry attempt.

generators/go/internal/testdata/sdk/client-options-core/fixtures/internal/retrier_test.go

@@ -84,9 +84,24 @@ func TestRetrier(t *testing.T) {
 			giveStatusCodes: []int{http.StatusRequestTimeout, http.StatusOK},
 		},
 		{
-			description:     "retries occur on status code 500",
+			description:     "retries occur on status code 502",
 			giveAttempts:    2,
-			giveStatusCodes: []int{http.StatusInternalServerError, http.StatusOK},
+			giveStatusCodes: []int{http.StatusBadGateway, http.StatusOK},
+		},
+		{
+			description:     "no retries on status code 500",
+			giveAttempts:    1,
+			giveStatusCodes: []int{http.StatusInternalServerError},
+		},
+		{
+			description:     "retries occur on status code 501",
+			giveAttempts:    2,
+			giveStatusCodes: []int{501, http.StatusOK},
+		},
+		{
+			description:     "retries occur on status code 599",
+			giveAttempts:    2,
+			giveStatusCodes: []int{599, http.StatusOK},
 		},
 	}
 

generators/go/internal/testdata/sdk/client-options-filename/fixtures/internal/retrier.go

@@ -130,7 +130,7 @@ func (r *Retrier) run(
 func (r *Retrier) shouldRetry(response *http.Response) bool {
 	return response.StatusCode == http.StatusTooManyRequests ||
 		response.StatusCode == http.StatusRequestTimeout ||
-		response.StatusCode >= http.StatusInternalServerError
+		(response.StatusCode > http.StatusInternalServerError && response.StatusCode < 600)
 }
 
 // retryDelay calculates the delay time in milliseconds based on the retry attempt.