(graphcache) - Change Store constructor to accept options object

[kitten]

This changes cacheExchange to just pass on its options and Store to accept an options object as well to make it easier to use directly.

// before:
new Store(
  undefined, // schema
  { /* resolvers */ },
  { /* updates */ },
  { /* optimistic */ },
  { /* keys */ },
);

// after:
new Store({
  schema: undefined,
  resolvers: {},
  updates: {},
  optimistic: {},
  keys: {},
});

storage is exempted since the rehydration logic is coupled to the cacheExchange (for now). Bundle size remains unchanged.

[changeset-bot]

🦋 Changeset is good to go

Latest commit: 17d88df

We got this.

This PR includes changesets to release 1 package

Name	Type
@urql/exchange-graphcache	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[JoviDeCroock]

That's a good shout 👍

[andyrichardson]

Good shout on removing this!

Default params also save us some conditional logic:

opts?: CacheExchangeOpts = {}

[kitten]

To be fair, I've just moved it 😆 but typically transpilers will jump through more hoops in the generated code with default params rather than reassigning the argument. It's a minor details though, since we do get the type safety from TypeScript in either cases 👍

[JoviDeCroock]

default params are very expensive size wise, see this REPL