amend exports

[sinisterstumble]

export generateGraphQLString and generateContentsFromGraphqlString

[louisscruz]

Just out of curiosity: what was the motivation for exporting these functions?

[sinisterstumble]

@louisscruz for a vite plugin. To avoid the need to fork the repository for this:

export const viteGraphQLPlugin = (
  options: GraphQLLoaderPluginOptions = {}
): VitePlugin => ({
  name: 'graphql',
  transform(graphqlString: string, id: string) {
    if ((options.filterRegex ?? /\.graphql$/).test(id)) {
      return {
        code: generateContentsFromGraphqlString(
          graphqlString,
          options.mapDocumentNode
        ),
        map: null
      }
    }

    return
  }
})

[sinisterstumble]

@louisscruz unplugin, unified plugin system for vite, rollup, and webpack, is another possible option.

[louisscruz]

After a lot of internal deliberation over this one, I've decided to allow for these functions to be exposed. Generally, I am not in favor of exposing more than is needed in order to keep future implementation and API changes as simple as possible. But this seems harmless enough. Hopefully this doesn't become some source of regret in the future 😆 .

Merged here https://github.com/luckycatfactory/esbuild-graphql-loader/pull/25 and made available as v3.7.0.

[sinisterstumble]

@louisscruz thank you!