Merge pull request #32 from getsumer/ih/abstract-observer-refactor

fix: repare rpcMethodArgs

Author: Isaac Herrera

package.json

@@ -1,6 +1,6 @@
 {
   "name": "sumer-sdk",
-  "version": "6.0.0",
+  "version": "6.0.1",
   "description": "SDK that allows observing EVM-based provider requests",
   "author": "Sumer Communities SL (by Wagmi Venture Studio) <[email protected]>",
   "license": "AGPL-3.0-only",

src/core/Observer.ts

@@ -24,8 +24,10 @@ export abstract class Observer {
 
   protected getMethodArgs({ methodArgs }: TargetExecution): string[] | Record<string, string> {
     if (methodArgs && methodArgs.length && methodArgs[0]['params']) {
-      if (methodArgs[0]['params'].length > 1) {
-        return methodArgs[0]['params'].filter((p: unknown) => typeof p === 'string')
+      if (methodArgs[0]['params'].length >= 1) {
+        return methodArgs[0]['params'].map((p: unknown) =>
+          typeof p === 'string' ? p : JSON.stringify(p),
+        )
       }
       return methodArgs[0]['params'][0]
     }

src/observers/TransactionObserver.ts

@@ -16,10 +16,7 @@ export class TransactionObserver extends Observer {
         data: methodArgs['data'] ?? undefined,
         gas: methodArgs['gas'] ?? undefined,
         rpcMethodName: this.getMethodName(execution),
-        ...(Array.isArray(methodArgs) && {
-          rpcMethodArgs: methodArgs,
-        }),
-
+        rpcMethodArgs: Array.isArray(methodArgs) ? methodArgs : [JSON.stringify(methodArgs)],
         error: this.getTransactionError(result),
         metadata: this.getMetadata(execution),
       })