Use provider's chainId as default if present

[ramirotw]

Close #3

This improves how the chainId was handled due to the possibility of a mismatch between the chainId used in the constructor to instantiate the sdk and the provider's chainId:

// before
const provider = new ethers.providers.Web3Provider(window.ethereum, "any") // network is Mainnet
const chainId = 4;
const cowSdk = new CowSdk(chainId, { signer: provider.getSigner() })
// this will use Rinkeby as the chainId
// now
const provider = new ethers.providers.Web3Provider(window.ethereum, "any") // network is Mainnet
const chainId = 4;
const cowSdk = new CowSdk(chainId, { signer: provider.getSigner() })
// this will use Mainnet as the chainId

You can still instantiate the sdk without passing any signer just for querying purposes and also update the current chainId

const chainId = 1;
const cowSdk = new CowSdk(chainId)
const quoteResponse = await cowSdk.cowApi.getQuote({...}) // will query Mainnet
cowSdk.updateChainId(4)
const quoteResponse = await cowSdk.cowApi.getQuote({...}) // will query Rinkeby

The chainId getter/setter was moved to the Context and made it async.

[github-actions]

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type	Occurrences	Fixable
Errors	0	0
Warnings	13	0
Ignored	0	N/A

• Result: ✅ success

• Annotations: 13 total

  ---

[failure] @typescript-eslint/no-explicit-any

• src/api/cow/errors/OperatorError.ts Line 9 - Unexpected any. Specify a different type.
• src/api/cow/errors/OperatorError.ts Line 140 - Unexpected any. Specify a different type.
• src/api/cow/errors/QuoteError.ts Line 8 - Unexpected any. Specify a different type.
• src/api/cow/errors/QuoteError.ts Line 64 - Unexpected any. Specify a different type.
• src/api/cow/errors/QuoteError.ts Line 113 - Unexpected any. Specify a different type.
• src/api/cow/index.ts Line 67 - Unexpected any. Specify a different type.
• src/api/cow/index.ts Line 334 - Unexpected any. Specify a different type.
• src/api/cow/index.ts Line 343 - Unexpected any. Specify a different type.
• src/api/cow/index.ts Line 352 - Unexpected any. Specify a different type.
• src/api/cow/index.ts Line 364 - Unexpected any. Specify a different type.
• src/utils/common.ts Line 12 - Unexpected any. Specify a different type.
• src/utils/sign.ts Line 133 - Unexpected any. Specify a different type.

---

[failure] @typescript-eslint/no-non-null-assertion

• src/utils/sign.ts Line 197 - Forbidden non-null assertion.

---

^{Report generated by eslint-plus-action}

[alfetopito]

What's the reason for using TS private (private) for context and JS private (#) for chainId?

[ramirotw]

good catch! when I updated the chainId accessor didn't change the context one. This was to enforce runtime access outside of TS