fix(sdk): accept null for optional fields in swapApprovalResponseSchema

.changeset/tolerate-null-swap-approval-fields.md

@@ -0,0 +1,5 @@
+---
+"@across-protocol/app-sdk": patch
+---
+
+Accept `null` (in addition to `undefined`) for optional fields in `swapApprovalResponseSchema`: `approvalTxns`, `steps.originSwap`, `steps.destinationSwap`, and `swapTx`. The `/swap/approval` endpoint now serializes these as explicit `null` for cases where they don't apply (e.g. `bridgeableToBridgeable` routes), which previously caused `getSwapQuote` to throw `Invalid swap approval response`. Consumers already coerce to truthy, so behavior is unchanged.

packages/sdk/src/api/swap-approval.ts

@@ -201,11 +201,11 @@ export const swapApprovalResponseSchema = z.object({
         data: z.string(),
       }),
     )
-    .optional(),
+    .nullish(),
   steps: z.object({
-    originSwap: swapStepSchema.optional(),
+    originSwap: swapStepSchema.nullish(),
     bridge: bridgeStepSchema,
-    destinationSwap: swapStepSchema.optional(),
+    destinationSwap: swapStepSchema.nullish(),
   }),
   inputToken: z.object({
     address: anyChainAddress,
@@ -280,7 +280,7 @@ export const swapApprovalResponseSchema = z.object({
   expectedOutputAmount: bigNumberString,
   minOutputAmount: bigNumberString,
   expectedFillTime: positiveInteger,
-  swapTx: z.union([swapTxSchema, permitSwapTxSchema]).optional(),
+  swapTx: z.union([swapTxSchema, permitSwapTxSchema]).nullish(),
   id: z.string().optional(),
 });
 

packages/sdk/test/common/sdk.ts

@@ -7,7 +7,6 @@ import {
   polygon,
   linea,
   lisk,
-  scroll,
   zora,
   sepolia,
 } from "viem/chains";
@@ -22,7 +21,6 @@ export const MAINNET_SUPPORTED_CHAINS = [
   polygon,
   linea,
   lisk,
-  scroll,
   zora,
 ] as const;