Builds, tests & co

fix: ESM compatibility - avoid direct Response object mutation #2774

Triggered via pull request October 24, 2025 19:47

omarmciver

synchronize #1430

omarmciver:fix/esm-response-mutation

Status Failure

Total duration 1m 16s

Artifacts –

main.yml

on: pull_request

Matrix: build-and-test

Annotations

30 errors

tests/extended.test.ts > extended > 'full-swagger-scheme': tests/extended.test.ts#L43

Error: Snapshot `extended > 'full-swagger-scheme' 1` mismatched - Expected + Received @@ -38382,11 +38382,13 @@ target._error = value; return true; } // Prevent mutation of Response properties (ESM safety) - return false; + throw new Error( + `Cannot set property '${String(prop)}' on HttpResponse: only 'data' and 'error' can be set.`, + ); }, }) as HttpResponse<T, E>; const responseToParse = responseFormat ? response.clone() : response; const data = !responseFormat ❯ tests/extended.test.ts:43:21

tests/extended.test.ts > extended > 'explode-param-3': tests/extended.test.ts#L43

Error: Snapshot `extended > 'explode-param-3' 1` mismatched - Expected + Received @@ -341,11 +341,13 @@ target._error = value; return true; } // Prevent mutation of Response properties (ESM safety) - return false; + throw new Error( + `Cannot set property '${String(prop)}' on HttpResponse: only 'data' and 'error' can be set.`, + ); }, }) as HttpResponse<T, E>; const responseToParse = responseFormat ? response.clone() : response; const data = !responseFormat ❯ tests/extended.test.ts:43:21

tests/extended.test.ts > extended > 'enum-type-mismatch': tests/extended.test.ts#L43

Error: Snapshot `extended > 'enum-type-mismatch' 1` mismatched - Expected + Received @@ -294,11 +294,13 @@ target._error = value; return true; } // Prevent mutation of Response properties (ESM safety) - return false; + throw new Error( + `Cannot set property '${String(prop)}' on HttpResponse: only 'data' and 'error' can be set.`, + ); }, }) as HttpResponse<T, E>; const responseToParse = responseFormat ? response.clone() : response; const data = !responseFormat ❯ tests/extended.test.ts:43:21

tests/extended.test.ts > extended > 'components-responses': tests/extended.test.ts#L43

Error: Snapshot `extended > 'components-responses' 1` mismatched - Expected + Received @@ -293,11 +293,13 @@ target._error = value; return true; } // Prevent mutation of Response properties (ESM safety) - return false; + throw new Error( + `Cannot set property '${String(prop)}' on HttpResponse: only 'data' and 'error' can be set.`, + ); }, }) as HttpResponse<T, E>; const responseToParse = responseFormat ? response.clone() : response; const data = !responseFormat ❯ tests/extended.test.ts:43:21

tests/extended.test.ts > extended > 'callback-example': tests/extended.test.ts#L43

Error: Snapshot `extended > 'callback-example' 1` mismatched - Expected + Received @@ -313,11 +313,13 @@ target._error = value; return true; } // Prevent mutation of Response properties (ESM safety) - return false; + throw new Error( + `Cannot set property '${String(prop)}' on HttpResponse: only 'data' and 'error' can be set.`, + ); }, }) as HttpResponse<T, E>; const responseToParse = responseFormat ? response.clone() : response; const data = !responseFormat ❯ tests/extended.test.ts:43:21

tests/extended.test.ts > extended > 'api-with-examples': tests/extended.test.ts#L43

Error: Snapshot `extended > 'api-with-examples' 1` mismatched - Expected + Received @@ -291,11 +291,13 @@ target._error = value; return true; } // Prevent mutation of Response properties (ESM safety) - return false; + throw new Error( + `Cannot set property '${String(prop)}' on HttpResponse: only 'data' and 'error' can be set.`, + ); }, }) as HttpResponse<T, E>; const responseToParse = responseFormat ? response.clone() : response; const data = !responseFormat ❯ tests/extended.test.ts:43:21

tests/extended.test.ts > extended > 'anyof-example': tests/extended.test.ts#L43

Error: Snapshot `extended > 'anyof-example' 1` mismatched - Expected + Received @@ -305,11 +305,13 @@ target._error = value; return true; } // Prevent mutation of Response properties (ESM safety) - return false; + throw new Error( + `Cannot set property '${String(prop)}' on HttpResponse: only 'data' and 'error' can be set.`, + ); }, }) as HttpResponse<T, E>; const responseToParse = responseFormat ? response.clone() : response; const data = !responseFormat ❯ tests/extended.test.ts:43:21

tests/extended.test.ts > extended > 'allof-example': tests/extended.test.ts#L43

Error: Snapshot `extended > 'allof-example' 1` mismatched - Expected + Received @@ -311,11 +311,13 @@ target._error = value; return true; } // Prevent mutation of Response properties (ESM safety) - return false; + throw new Error( + `Cannot set property '${String(prop)}' on HttpResponse: only 'data' and 'error' can be set.`, + ); }, }) as HttpResponse<T, E>; const responseToParse = responseFormat ? response.clone() : response; const data = !responseFormat ❯ tests/extended.test.ts:43:21

tests/extended.test.ts > extended > 'additional-properties2': tests/extended.test.ts#L43

Error: Snapshot `extended > 'additional-properties2' 1` mismatched - Expected + Received @@ -275,11 +275,13 @@ target._error = value; return true; } // Prevent mutation of Response properties (ESM safety) - return false; + throw new Error( + `Cannot set property '${String(prop)}' on HttpResponse: only 'data' and 'error' can be set.`, + ); }, }) as HttpResponse<T, E>; const responseToParse = responseFormat ? response.clone() : response; const data = !responseFormat ❯ tests/extended.test.ts:43:21

tests/extended.test.ts > extended > 'additional-properties': tests/extended.test.ts#L43

Error: Snapshot `extended > 'additional-properties' 1` mismatched - Expected + Received @@ -278,11 +278,13 @@ target._error = value; return true; } // Prevent mutation of Response properties (ESM safety) - return false; + throw new Error( + `Cannot set property '${String(prop)}' on HttpResponse: only 'data' and 'error' can be set.`, + ); }, }) as HttpResponse<T, E>; const responseToParse = responseFormat ? response.clone() : response; const data = !responseFormat ❯ tests/extended.test.ts:43:21

tests/extended.test.ts > extended > 'full-swagger-scheme': tests/extended.test.ts#L43