fix: error interactions regression

failed interactions were never resolved in connect button

examples/simple-dapp/src/main.ts

@@ -217,8 +217,8 @@ resetButton.onclick = () => {
   window.location.replace(window.location.origin)
 }
 
-sendTxButton.onclick = () => {
-  dAppToolkit.walletApi.sendTransaction({
+sendTxButton.onclick = async () => {
+  const res = await dAppToolkit.walletApi.sendTransaction({
     transactionManifest: `
     CALL_METHOD
       Address("component_tdx_2_1cptxxxxxxxxxfaucetxxxxxxxxx000527798379xxxxxxxxxyulkzl")
@@ -232,6 +232,7 @@ sendTxButton.onclick = () => {
       Enum<0u8>()
     ;`,
   })
+  console.log('send tx result', res)
 }
 
 oneTimeRequest.onclick = () => {

packages/dapp-toolkit/src/_types.ts

@@ -7,6 +7,7 @@ import {
   Persona,
   PersonaDataName,
   WalletInteraction,
+  WalletInteractionResponse,
 } from './schemas'
 import type { Logger } from './helpers'
 import type { SdkError } from './error'
@@ -175,7 +176,7 @@ export type TransportProvider = {
   send: (
     walletInteraction: WalletInteraction,
     callbackFns: Partial<CallbackFns>,
-  ) => ResultAsync<unknown, SdkError>
+  ) => ResultAsync<WalletInteractionResponse, SdkError>
   disconnect: () => void
   destroy: () => void
 }

packages/dapp-toolkit/src/helpers/validate-wallet-response.ts

@@ -1,30 +1,22 @@
-import { Result, ResultAsync, errAsync, okAsync } from 'neverthrow'
-import {
-  WalletInteractionResponse,
-  WalletInteractionSuccessResponse,
-} from '../schemas'
+import { Result } from 'neverthrow'
+import { WalletInteractionResponse } from '../schemas'
 import { SdkError } from '../error'
 import { parse } from 'valibot'
 
 export const validateWalletResponse = (
   walletResponse: unknown,
-): ResultAsync<
-  WalletInteractionSuccessResponse,
-  SdkError | { discriminator: 'failure'; interactionId: string; error: string }
-> => {
+): Result<WalletInteractionResponse, SdkError> => {
   const fn = Result.fromThrowable(
     (_) => parse(WalletInteractionResponse, _),
     (error) => error,
   )
 
-  const result = fn(walletResponse)
-  if (result.isErr()) {
-    return errAsync(SdkError('walletResponseValidation', '', 'Invalid input'))
-  } else if (result.isOk()) {
-    return result.value.discriminator === 'success'
-      ? okAsync(result.value)
-      : errAsync(result.value)
-  }
-
-  return errAsync(SdkError('walletResponseValidation', ''))
+  return fn(walletResponse).mapErr((response) =>
+    SdkError(
+      'walletResponseValidation',
+      (walletResponse as any)?.interactionId,
+      'Invalid input',
+      response,
+    ),
+  )
 }

packages/dapp-toolkit/src/modules/wallet-request/request-resolver/request-resolver.module.ts

@@ -1,4 +1,4 @@
-import { err, ok, okAsync, ResultAsync } from 'neverthrow'
+import { err, ok, okAsync, Result, ResultAsync } from 'neverthrow'
 import { validateWalletResponse, type Logger } from '../../../helpers'
 import type { WalletInteractionResponse } from '../../../schemas'
 import type { StorageModule } from '../../storage'
@@ -62,17 +62,19 @@ export const RequestResolverModule = (input: {
     requestItemModule.patch(interactionId, { sentToWallet: true })
 
   const addWalletResponses = (responses: WalletInteractionResponse[]) =>
-    ResultAsync.combine(responses.map(validateWalletResponse)).andThen(() =>
-      walletResponses.setItems(
-        responses.reduce<Record<string, WalletInteractionResponse>>(
-          (acc, response) => {
-            acc[response.interactionId] = response
-            return acc
-          },
-          {},
+    Result.combine(responses.map(validateWalletResponse))
+      .asyncAndThen(() =>
+        walletResponses.setItems(
+          responses.reduce<Record<string, WalletInteractionResponse>>(
+            (acc, response) => {
+              acc[response.interactionId] = response
+              return acc
+            },
+            {},
+          ),
         ),
-      ),
-    )
+      )
+      .mapErr((error) => logger?.error({ method: 'addWalletResponses', error }))
 
   const toRequestItemMap = (items: RequestItem[]) =>
     items.reduce<Record<string, RequestItem>>(

packages/dapp-toolkit/src/modules/wallet-request/transport/connector-extension/connector-extension.module.ts

@@ -27,6 +27,7 @@ import {
   MessageLifeCycleExtensionStatusEvent,
   WalletInteraction,
   WalletInteractionExtensionInteraction,
+  WalletInteractionResponse,
   eventType,
 } from '../../../../schemas'
 import { StorageModule } from '../../../storage'
@@ -143,7 +144,7 @@ export const ConnectorExtensionModule = (input: {
   const sendWalletInteraction = (
     walletInteraction: WalletInteraction,
     callbackFns: Partial<CallbackFns>,
-  ): ResultAsync<unknown, SdkError> => {
+  ): ResultAsync<WalletInteractionResponse, SdkError> => {
     const cancelRequestSubject = new Subject<Err<never, SdkError>>()
 
     const maybeResolved$ = from(

packages/dapp-toolkit/src/modules/wallet-request/transport/radix-connect-relay/radix-connect-relay.module.ts

@@ -156,7 +156,7 @@ export const RadixConnectRelayModule = (input: {
   const sendToWallet = (
     walletInteraction: WalletInteraction,
     callbackFns: Partial<CallbackFns>,
-  ): ResultAsync<unknown, SdkError> =>
+  ): ResultAsync<WalletInteractionResponse, SdkError> =>
     ResultAsync.combine([
       sessionModule
         .getCurrentSession()
@@ -194,6 +194,7 @@ export const RadixConnectRelayModule = (input: {
             walletInteraction.interactionId,
           ),
         )
+        .map((requestItem) => requestItem.walletResponse),
     )
 
   const decryptWalletResponseData = (

packages/dapp-toolkit/src/modules/wallet-request/wallet-request-sdk.ts

@@ -1,12 +1,14 @@
-import { Result, ResultAsync, err, ok } from 'neverthrow'
+import { Result, ResultAsync, err, errAsync, ok, okAsync } from 'neverthrow'
 import { TransportProvider } from '../../_types'
-import { Logger, validateWalletResponse } from '../../helpers'
+import { Logger } from '../../helpers'
 import {
   Metadata,
   CallbackFns,
   WalletInteractionItems,
   WalletInteraction,
+  WalletInteractionFailureResponse,
   WalletInteractionResponse,
+  WalletInteractionSuccessResponse,
 } from '../../schemas'
 import { parse } from 'valibot'
 import { SdkError } from '../../error'
@@ -86,15 +88,24 @@ export const WalletRequestSdk = (input: WalletRequestSdkInput) => {
       items,
     }: { interactionId?: string; items: WalletInteraction['items'] },
     callbackFns: Partial<CallbackFns> = {},
-  ): ResultAsync<unknown, SdkError> =>
+  ): ResultAsync<
+    WalletInteractionSuccessResponse,
+    SdkError | WalletInteractionFailureResponse
+  > =>
     withInterceptor({
       items,
       interactionId,
       metadata,
     }).andThen((walletInteraction) =>
-      getTransport(walletInteraction.interactionId).asyncAndThen((transport) =>
-        transport.send(walletInteraction, callbackFns),
-      ),
+      getTransport(walletInteraction.interactionId)
+        .asyncAndThen((transport) =>
+          transport.send(walletInteraction, callbackFns),
+        )
+        .andThen((response) =>
+          response.discriminator === 'failure'
+            ? errAsync(response)
+            : okAsync(response),
+        ),
     )
 
   return {