aa

graphql · dimaMachina · Aug 23, 2024 · Aug 23, 2024 · Aug 23, 2024 · Aug 23, 2024
commit 9da88f310f33efeb4012ca1f038c7f449a1ccdec
diff --git a/.changeset/silent-ghosts-fix.md b/.changeset/silent-ghosts-fix.md
@@ -0,0 +1,5 @@
+---
+'@graphiql/toolkit': minor
+---
+
+compile with `tsup` instead of `tsc`
diff --git a/packages/graphiql-toolkit/docs/create-fetcher.md b/packages/graphiql-toolkit/docs/create-fetcher.md
@@ -67,10 +67,7 @@ const url = 'https://my-schema.com/graphql';
 
 const subscriptionUrl = 'wss://my-schema.com/graphql';
 
-const fetcher = createGraphiQLFetcher({
-  url,
-  subscriptionUrl,
-});
+const fetcher = createGraphiQLFetcher({ url, subscriptionUrl });
 
 export const App = () => <GraphiQL fetcher={fetcher} />;
 
@@ -209,10 +206,7 @@ import { createGraphiQLFetcher } from '@graphiql/toolkit';
 
 const url = 'https://my-schema.com/graphql';
 
-const fetcher = createGraphiQLFetcher({
-  url,
-  fetch,
-});
+const fetcher = createGraphiQLFetcher({ url, fetch });
 
 export const App = () => <GraphiQL fetcher={fetcher} />;
 

diff --git a/packages/graphiql-toolkit/package.json b/packages/graphiql-toolkit/package.json
@@ -15,10 +15,15 @@
     "url": "https://github.com/graphql/graphiql/issues?q=issue+label:@graphiql/toolkit"
   },
   "license": "MIT",
-  "main": "dist/index.js",
-  "module": "esm/index.js",
-  "typings": "dist/index.d.ts",
-  "scripts": {},
+  "main": "dist/cjs/index.js",
+  "module": "dist/esm/index.mjs",
+  "typings": "dist/esm/index.d.mts",
+  "scripts": {
+    "build": "tsup",
+    "dev": "tsup --watch",
+    "prebuild": "yarn types:check",
+    "types:check": "tsc --noEmit"
+  },
   "dependencies": {
     "@n1ru4l/push-pull-async-iterable-iterator": "^3.1.0",
     "meros": "^1.1.4"
@@ -27,7 +32,8 @@
     "graphql": "^17.0.0-alpha.7",
     "graphql-ws": "^5.5.5",
     "isomorphic-fetch": "^3.0.0",
-    "subscriptions-transport-ws": "0.11.0"
+    "subscriptions-transport-ws": "0.11.0",
+    "tsup": "^8.2.4"
   },
   "peerDependencies": {
     "graphql": "^15.5.0 || ^16.0.0 || ^17.0.0-alpha.2",

diff --git a/packages/graphiql-toolkit/src/create-fetcher/__tests__/lib.spec.ts b/packages/graphiql-toolkit/src/create-fetcher/__tests__/lib.spec.ts
@@ -68,30 +68,21 @@ describe('getWsFetcher', () => {
   });
   it('provides an observable wsClient when custom wsClient option is provided', () => {
     createClient.mockReturnValue(true);
-    getWsFetcher({
-      url: '',
-      wsClient: true,
-    });
+    getWsFetcher({ url: '', wsClient: true });
     // @ts-ignore
     expect(createClient.mock.calls).toHaveLength(0);
   });
   it('creates a subscriptions-transports-ws observable when custom legacyClient option is provided', () => {
     createClient.mockReturnValue(true);
-    getWsFetcher({
-      url: '',
-      legacyClient: true,
-    });
+    getWsFetcher({ url: '', legacyClient: true });
     // @ts-ignore
     expect(createClient.mock.calls).toHaveLength(0);
     expect(SubscriptionClient.mock.calls).toHaveLength(0);
   });
 
   it('if subscriptionsUrl is provided, create a client on the fly', () => {
     createClient.mockReturnValue(true);
-    getWsFetcher({
-      url: '',
-      subscriptionUrl: 'wss://example',
-    });
+    getWsFetcher({ url: '', subscriptionUrl: 'wss://example' });
     expect(createClient.mock.calls[0]).toEqual([
       { connectionParams: {}, url: 'wss://example' },
     ]);

diff --git a/packages/graphiql-toolkit/src/create-fetcher/createFetcher.ts b/packages/graphiql-toolkit/src/create-fetcher/createFetcher.ts
@@ -11,27 +11,15 @@ import {
  * build a GraphiQL fetcher that is:
  * - backwards compatible
  * - optionally supports graphql-ws or `
- *
- * @param options {CreateFetcherOptions}
- * @returns {Fetcher}
  */
 export function createGraphiQLFetcher(options: CreateFetcherOptions): Fetcher {
-  let httpFetch;
-  if (typeof window !== 'undefined' && window.fetch) {
-    httpFetch = window.fetch;
-  }
-  if (
-    options?.enableIncrementalDelivery === null ||
-    options.enableIncrementalDelivery !== false
-  ) {
-    options.enableIncrementalDelivery = true;
-  }
-  if (options.fetch) {
-    httpFetch = options.fetch;
-  }
+  const httpFetch =
+    options.fetch || (typeof window !== 'undefined' && window.fetch);
   if (!httpFetch) {
     throw new Error('No valid fetcher implementation available');
   }
+  options.enableIncrementalDelivery =
+    options.enableIncrementalDelivery !== false;
   // simpler fetcher for schema requests
   const simpleFetcher = createSimpleFetcher(options, httpFetch);
 

diff --git a/packages/graphiql-toolkit/src/create-fetcher/lib.ts b/packages/graphiql-toolkit/src/create-fetcher/lib.ts
@@ -32,7 +32,7 @@ const errorHasCode = (err: unknown): err is { code: string } => {
  */
 export const isSubscriptionWithName = (
   document: DocumentNode,
-  name: string | undefined,
+  name?: string,
 ): boolean => {
   let isSubscription = false;
   visit(document, {
@@ -97,12 +97,10 @@ export const createWebsocketsFetcherFromUrl = (
 
 /**
  * Create ws/s fetcher using provided wsClient implementation
- *
- * @param wsClient {Client}
- * @returns {Fetcher}
  */
 export const createWebsocketsFetcherFromClient =
-  (wsClient: Client) => (graphQLParams: FetcherParams) =>
+  (wsClient: Client): Fetcher =>
+  (graphQLParams: FetcherParams) =>
     makeAsyncIterableIteratorFromSink<ExecutionResult>(sink =>
       wsClient.subscribe(graphQLParams, {
         ...sink,
@@ -125,12 +123,9 @@ export const createWebsocketsFetcherFromClient =
 /**
  * Allow legacy websockets protocol client, but no definitions for it,
  * as the library is deprecated and has security issues
- *
- * @param legacyWsClient
- * @returns
  */
 export const createLegacyWebsocketsFetcher =
-  (legacyWsClient: { request: (params: FetcherParams) => unknown }) =>
+  (legacyWsClient: { request: (params: FetcherParams) => unknown }): Fetcher =>
   (graphQLParams: FetcherParams) => {
     const observable = legacyWsClient.request(graphQLParams);
     return makeAsyncIterableIteratorFromSink<ExecutionResult>(
@@ -139,11 +134,8 @@ export const createLegacyWebsocketsFetcher =
     );
   };
 /**
- * create a fetcher with the `IncrementalDelivery` HTTP/S spec for
+ * Create a fetcher with the `IncrementalDelivery` HTTP/S spec for
  * `@stream` and `@defer` support using `fetch-multipart-graphql`
- *
- * @param options {CreateFetcherOptions}
- * @returns {Fetcher}
  */
 export const createMultipartFetcher = (
   options: CreateFetcherOptions,
@@ -187,13 +179,11 @@ export const createMultipartFetcher = (
 
 /**
  * If `wsClient` or `legacyClient` are provided, then `subscriptionUrl` is overridden.
- * @param options {CreateFetcherOptions}
- * @returns
  */
 export const getWsFetcher = (
   options: CreateFetcherOptions,
   fetcherOpts: FetcherOpts | undefined,
-) => {
+): Fetcher | void => {
   if (options.wsClient) {
     return createWebsocketsFetcherFromClient(options.wsClient);
   }

diff --git a/packages/graphiql-toolkit/tsconfig.esm.json b/packages/graphiql-toolkit/tsconfig.esm.json
diff --git a/packages/graphiql-toolkit/tsconfig.json b/packages/graphiql-toolkit/tsconfig.json
@@ -1,21 +1,16 @@
 {
-  "extends": "../../resources/tsconfig.base.cjs.json",
   "compilerOptions": {
-    "rootDir": "./src",
-    "outDir": "./dist",
-    "composite": true,
-    "jsx": "react",
-    "target": "es5",
-    "baseUrl": ".",
-    "strictPropertyInitialization": false
+    "target": "es2016",
+    "module": "ESNext",
+    "declaration": true,
+    "noEmit": true,
+    "esModuleInterop": true,
+    "strict": true,
+    "skipLibCheck": true,
+    "allowJs": true,
+    "lib": ["es2022", "dom"],
+    "moduleResolution": "node"
   },
-  "include": ["src"],
-  "exclude": [
-    "**/__tests__/**",
-    "**/dist/**.*",
-    "**/*.spec.ts",
-    "**/*.spec.js",
-    "**/*-test.ts",
-    "**/*-test.js"
-  ]
+  "include": ["src", "tsup.config.ts"],
+  "exclude": ["**/*.spec.ts"]
 }
diff --git a/packages/graphiql-toolkit/tsup.config.ts b/packages/graphiql-toolkit/tsup.config.ts
@@ -0,0 +1,21 @@
+import { defineConfig, Options } from 'tsup';
+
+const opts: Options = {
+  entry: ['src/**/*.ts', '!**/__tests__'],
+  bundle: false,
+  clean: true,
+  dts: true,
+};
+
+export default defineConfig([
+  {
+    ...opts,
+    format: 'esm',
+    outDir: 'dist/esm',
+  },
+  {
+    ...opts,
+    format: 'cjs',
+    outDir: 'dist/cjs',
+  },
+]);
diff --git a/packages/graphiql/tsconfig.esm.json b/packages/graphiql/tsconfig.esm.json
@@ -21,9 +21,6 @@
     {
       "path": "../codemirror-graphql"
     },
-    {
-      "path": "../graphiql-toolkit"
-    },
     {
       "path": "../graphql-language-service"
     }

diff --git a/packages/graphiql/tsconfig.json b/packages/graphiql/tsconfig.json
@@ -11,9 +11,6 @@
   "include": ["src"],
   "exclude": ["**/__tests__/**", "**/dist/**.*", "cypress/**"],
   "references": [
-    {
-      "path": "../graphiql-toolkit"
-    },
     {
       "path": "../codemirror-graphql"
     },

diff --git a/resources/tsconfig.build.cjs.json b/resources/tsconfig.build.cjs.json
@@ -5,9 +5,6 @@
   "files": [],
   "include": [],
   "references": [
-    {
-      "path": "../packages/graphiql-toolkit"
-    },
     {
       "path": "../packages/monaco-graphql"
     },

diff --git a/resources/tsconfig.build.esm.json b/resources/tsconfig.build.esm.json
@@ -5,9 +5,6 @@
   "files": [],
   "include": [],
   "references": [
-    {
-      "path": "../packages/graphiql-toolkit/tsconfig.esm.json"
-    },
     {
       "path": "../packages/cm6-graphql/tsconfig.esm.json"
     },