params is optional

Author: SamWilsn

client/src/glue.ts

@@ -409,7 +409,7 @@ export class ManualGlue extends Glue {
             // wallet_addEthereumChain isn't exactly the safest endpoint, so we
             // don't expect wallets to implement it. We try optimistically but
             // fall back to human instructions if necessary.
-            console.debug("`wallet_addEthereumChain` failed, going manual");
+            console.debug("`wallet_addEthereumChain` failed, going manual", e);
         }
 
         await this.instruct(

client/src/index.ts

@@ -123,12 +123,17 @@ function main() {
                         throw new TypeError("'method' in body not a string");
                     }
 
-                    if (!("params" in msg.body)) {
-                        throw new TypeError("'params' not in message body");
-                    }
-
-                    if (!(msg.body.params instanceof Array)) {
-                        throw new TypeError("'params' in body not an array");
+                    let params: unknown[];
+                    if ("params" in msg.body) {
+                        if (!(msg.body.params instanceof Array)) {
+                            throw new TypeError(
+                                "'params' in body not an array"
+                            );
+                        }
+
+                        params = msg.body.params;
+                    } else {
+                        params = [];
                     }
 
                     if (!("number" in msg)) {
@@ -137,7 +142,7 @@ function main() {
 
                     result.result = await blockchain.send(
                         msg.body.method,
-                        msg.body.params
+                        params
                     );
                     webSocket?.send(
                         JSON.stringify({