clients: Support Brave wallet default

[aefhm]

Description

Close #385 maybe.

Problem

Wagmi is concerned with bad implementations of EIP-1193 and wants to check that the removeListener function exists. However, the Brave wallet overrides ethereum properties and does not like the way we currently use Proxy objects, giving a typeerror upon a get call.

Our current implementation is a classic Proxy pattern in order to preserve this.

const proxy = new Proxy(aSecret, {
  get(target, prop, receiver) {
    const value = target[prop];
    if (value instanceof Function) {
      return function (...args) {
        return value.apply(this === receiver ? target : this, args);
      };
    }
    return value;
  },
});

simplified to:

sapphire-paratime/clients/js/src/provider.ts

Line 211 in 42c1a69

return typeof value === 'function' ? value.bind(upstream) : value;

One solution is to not forward context in order to maintain the function signature for Brave browsers as noted in this PR.

[netlify]

✅ Deploy Preview for oasisprotocol-sapphire-paratime canceled.

Name	Link
🔨 Latest commit	60e355a
🔍 Latest deploy log	https://app.netlify.com/sites/oasisprotocol-sapphire-paratime/deploys/66f2477eb705020008cc549d