Merge pull request #10 from argentlabs/fix/errors

fix: hybrid session error

README.md

@@ -58,6 +58,7 @@ The following snippet show how to create and use an hybrid session account
 
 ```typescript
 import {
+  SignSessionError,
   SessionParams,
   createSessionAccount
 } from "@argent/x-sessions"
@@ -100,15 +101,20 @@ const sessionAccount = await createSessionAccount({
 })
 
 // this transaction should get executed without the user having to approve again
-const tx = sessionAccount.execute({
-  // lets assume this is a erc20 contract
-  contractAddress: "0x...",
-  selector: "transfer",
-  calldata: [
-    "0x..."
-    // ...
-  ]
-})
+
+try {
+  const tx = sessionAccount.execute({
+    // lets assume this is a erc20 contract
+    contractAddress: "0x...",
+    selector: "transfer",
+    calldata: [
+      "0x..."
+      // ...
+    ]
+  })
+} catch (e) {
+  console.error((e as SignSessionError).cause, e.message)
+}
 ```
 
 Use this account to send transactions.

src/errors.ts

@@ -0,0 +1,10 @@
+export class SignSessionError extends Error {
+  constructor(
+    message: string,
+    public cause: any,
+  ) {
+    super(message)
+
+    Error.captureStackTrace(this, this.constructor)
+  }
+}

src/hybridSessionBackendService.ts

@@ -18,6 +18,7 @@ import {
   BackendSignSessionBody,
   BackendSignatureResponse,
 } from "./hybridSessionTypes"
+import { SignSessionError } from "./errors"
 
 export class ArgentBackendService {
   constructor(
@@ -129,8 +130,13 @@ export class ArgentBackendService {
       },
       body: JSON.stringify(body),
     })
-    const json = await response.json()
 
+    if (!response.ok) {
+      const error: { status: string } = await response.json()
+      throw new SignSessionError("Sign session error", error.status)
+    }
+
+    const json = await response.json()
     return json.signature
   }
 

src/main.ts

@@ -9,3 +9,5 @@ export type {
   CreateSessionParams,
   SessionParams,
 } from "./hybridSessionTypes"
+
+export { type SignSessionError } from "./errors"