@W-20508471: Implement PWA Adyen integration for Express (#3640)

Support processing adyen payments and handle 409 errors for failure use-cases

Author: amittapalli

packages/commerce-sdk-react/package.json

@@ -40,7 +40,7 @@
     "version": "node ./scripts/version.js"
   },
   "dependencies": {
-    "commerce-sdk-isomorphic": "^4.0.0",
+    "commerce-sdk-isomorphic": "5.0.0-unstable-20260124080731", 
     "js-cookie": "^3.0.1",
     "jwt-decode": "^4.0.0"
   },

packages/commerce-sdk-react/src/auth/index.ts

@@ -174,6 +174,14 @@ const DATA_MAP: AuthDataMap = {
         storageType: 'local',
         key: 'idp_access_token'
     },
+    idp_refresh_token: {
+        storageType: 'local',
+        key: 'idp_refresh_token'
+    },
+    dnt: {
+        storageType: 'local',
+        key: 'dnt'
+    },
     token_type: {
         storageType: 'local',
         key: 'token_type'

packages/commerce-sdk-react/src/hooks/ShopperCustomers/index.test.ts

@@ -18,7 +18,9 @@ describe('Shopper Customers hooks', () => {
         // or add it to the `expected` array with a comment explaining "TODO" or "never" (and why).
         expect(unimplemented).toEqual([
             'getExternalProfile', // TODO: Implement when the endpoint exits closed beta
-            'registerExternalProfile' // TODO: Implement when the endpoint exits closed beta
+            'getPublicProductListItems', // TODO: Implement when the endpoint exits closed beta
+            'registerExternalProfile', // TODO: Implement when the endpoint exits closed beta
+            'updateCustomerPaymentInstrument', // TODO: Implement when needed
         ])
     })
     test('all mutations have cache update logic', () => {

packages/commerce-sdk-react/src/hooks/ShopperExperience/index.test.ts

@@ -11,6 +11,13 @@ import * as queries from './query'
 describe('Shopper Experience hooks', () => {
     test('all endpoints have hooks', () => {
         const unimplemtented = getUnimplementedEndpoints(ShopperExperience, queries)
-        expect(unimplemtented).toEqual([])
+        expect(unimplemtented).toEqual([
+            // TODO: Implement content hooks when needed
+            'getContent',
+            'getContentFolder',
+            'getContentFolders',
+            'getMultipleContent',
+            'searchContent',
+        ])
     })
 })

packages/commerce-sdk-react/src/hooks/ShopperLogin/index.test.ts

@@ -20,7 +20,14 @@ describe('Shopper Login hooks', () => {
             // These endpoints all return data in the response headers, rather than body, so they
             // don't work well with the current implementation of mutation hooks.
             'authenticateCustomer',
-            'getTrustedAgentAuthorizationToken'
+            // TODO: Implement WebAuthn hooks when needed (new endpoints in 5.0.0)
+            'authorizeWebauthnRegistration',
+             // TODO: Implement WebAuthn hooks when needed (new endpoints in 5.0.0)
+            'finishWebauthnAuthentication',
+            'finishWebauthnUserRegistration',
+            'getTrustedAgentAuthorizationToken',
+            'startWebauthnAuthentication',
+            'startWebauthnUserRegistration',
         ])
     })
     test('all mutations have cache update logic', () => {

packages/commerce-sdk-react/src/hooks/ShopperPayments/queryKeyHelpers.ts

@@ -6,7 +6,7 @@
  */
 import {ShopperPayments} from 'commerce-sdk-isomorphic'
 import {Argument, ExcludeTail} from '../types'
-import {pickValidParams} from '../utils'
+import {pickValidParams, omitNullable} from '../utils'
 
 // We must use a client with no parameters in order to have required/optional match the API spec
 type Client = ShopperPayments<{shortCode: string}>
@@ -41,7 +41,10 @@ export const getPaymentConfiguration: QueryKeyHelper<'getPaymentConfiguration'>
     queryKey: (params: Params<'getPaymentConfiguration'>) => {
         return [
             ...getPaymentConfiguration.path(params),
-            pickValidParams(params || {}, ShopperPayments.paramKeys.getPaymentConfiguration)
+            // pickValidParams returns the filtered parameters, TypeScript sees that zoneId 
+            // could be string | null, and complains because null isn't allowed in the query key.
+            // omitNullable removes null values from the parameters (zoneId is optional but NOT nullable)
+            omitNullable(pickValidParams(params || {}, ShopperPayments.paramKeys.getPaymentConfiguration))
         ]
     }
 }