fix!: Switch to a network and client param solution

[janniks]

This PR was published to npm with the version 6.14.1-pr.78+8cf2e30c
e.g. npm install @stacks/common@6.14.1-pr.78+8cf2e30c --save-exact

• another pivot on network params

---

The previous solution had some confusing aspects. E.g when using devnet (which is only a URL, and otherwise not different from testnet)

Now, all functions that need fetch, can take a network param to specify which URL+fetchFn to use. All of these function ALSO take the new client param. But it's okay to supply network, client, both or neither. Stacks.js will default to mainnet or infer which network to use. This way we can easily override URL for APIs, but also create manageable network objects that can be passed around everywhere (what people do currently in Stacks.js code).

Want to create your custom network?

const network = {
  ...STACKS_TESTNET,
  chainId: 666,
  client: { baseUrl: "node6.org" }
}

makeSTXTokenTransfer({ network, // ...

Override things if you want for client stuff.

makeSTXTokenTransfer({
  network,
  client: { baseUrl: "node7.com" } // client overrides client in network
  // ...

This also makes it easier to combine network and API/RPC clients.

const client = createClient();

makeSTXTokenTransfer({
  client, // objects are mostly compatible

This way we always know if a function does something with it's fetching/client -- e.g.

selectStxDerivation({
  network,
  // doesn't take `client` param, we know it doesn't do networking

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
stacksjs-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 25, 2024 7:00pm

[codecov]

Codecov Report

Attention: Patch coverage is 84.55285% with 19 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
packages/cli/src/cli.ts	54.54%	5 Missing ⚠️
packages/transactions/src/builders.ts	75.00%	5 Missing ⚠️
packages/profile/src/profile.ts	33.33%	3 Missing and 1 partial ⚠️
packages/network/src/network.ts	60.00%	1 Missing and 1 partial ⚠️
packages/api/src/api.ts	50.00%	1 Missing ⚠️
packages/transactions/src/fetch.ts	97.91%	1 Missing ⚠️
packages/wallet-sdk/src/models/wallet.ts	80.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[zone117x]

Solid solution 👍