Finch v2

[jcblw]

Description

This PR outlines Finch v2, which revamps Finch to allow for things to be split up into smaller modules. This will allow us to have more focused modules, and remove extra dependencies not needed in different areas. For instance, on a website using Finch, you will not need to have the graphql-middleware modules, and if your extension that does not use react you should not need to install it to use Finch.

This is going to change the API of Finch a little, but enough to the point where I think it is useful to make a clean break for v2.

The modules now will all be under the @finch-graphql scope and the new packages are:

• @finch-graphql/api
• @finch-graphql/client
• @finch-graphql/react
• @finch-graphql/types ( shared types )
• @finch-graphql/browser-polyfills
• @finch-graphql/typescript-codgen

This means savings across the board in terms of package size, and flexibility for new integrations.

Testing

• Includes test.
• Manually tested.

Additional notes

No additional information is given

[jcblw]

This is starting to look really good, got a few things I would like to get done before merging and distributing the update.

• Update all the docs, including README, to reflect new changes.
• Add page to docs outlining migration from v1 -> v2.
• Create a v1 branch to allow for updates to still happen if the upgrade path is difficult.
• Publish a new version of v1 with deprecation notice.

[jcblw]

Playing around with this in the Toucan application. Got this error.

Uncaught Error: Cannot use GraphQLObjectType "Query" from another module or realm.

Ensure that there is only one instance of "graphql" in the node_modules
directory. If different versions of "graphql" are the dependencies of other
relied on modules, use "resolutions" to ensure only one version is installed.

https://yarnpkg.com/en/docs/selective-version-resolutions

Duplicate "graphql" modules cannot be used at the same time since different
versions may have different capabilities and behavior. The data from one
version used in the function from another could produce confusing and
spurious results.
    at instanceOf (background.js:146189)
    at isObjectType (background.js:156749)
    at mapArguments (background.js:138014)
    at mapDefaultValues (background.js:137895)
    at Module.mapSchema (background.js:137811)
    at createNewSchemaWithResolvers (background.js:129364)
    at addResolversToSchema (background.js:129257)
    at addMiddlewareToSchema (background.js:127913)
    at normalisedMiddlewares.reduceRight.schema.schema (background.js:127942)
    at Array.reduceRight (<anonymous>)

[jcblw]

https://yarnpkg.com/en/docs/selective-version-resolutions

Fixed, seems like we had a couple of versions of graphql in the modules. These are all in sync now.