use dev/prod conditions to set instanceOf check

Author: yaacovCR

resources/build-npm.ts

@@ -19,11 +19,14 @@ await buildPackage('./npmDist');
 showDirStats('./npmDist');
 
 async function buildPackage(outDir: string): Promise<void> {
+  const devDir = path.join(outDir, '__dev__');
+
   fs.rmSync(outDir, { recursive: true, force: true });
   fs.mkdirSync(outDir);
+  fs.mkdirSync(devDir);
 
-  fs.copyFileSync('./LICENSE', `./${outDir}/LICENSE`);
-  fs.copyFileSync('./README.md', `./${outDir}/README.md`);
+  fs.copyFileSync('./LICENSE', `${outDir}/LICENSE`);
+  fs.copyFileSync('./README.md', `${outDir}/README.md`);
 
   const packageJSON = readPackageJSON();
 
@@ -85,17 +88,55 @@ async function buildPackage(outDir: string): Promise<void> {
     extension: '.js',
   });
 
-  for (const filepath of emittedTSFiles) {
-    if (path.basename(filepath) === 'index.js') {
-      const relativePath =
-        './' + crossPlatformRelativePath('./npmDist', filepath);
-      packageJSON.exports[path.dirname(relativePath)] = relativePath;
+  const devHelper = path.join(outDir, 'development.js');
+
+  for (const prodFile of emittedTSFiles) {
+    const { dir, base } = path.parse(prodFile);
+
+    const match = base.match(/^([^.]*)\.?(.*)$/);
+    assert(match);
+    const [, name, ext] = match;
+
+    if (prodFile === devHelper || ext === 'js.map') {
+      continue;
+    }
+
+    const relativePathToProd = crossPlatformRelativePath(prodFile, outDir);
+    const relativePathAndName = crossPlatformRelativePath(
+      outDir,
+      `${dir}/${name}`,
+    );
+
+    const lines =
+      ext === 'd.ts' ? [] : [`import '${relativePathToProd}/development.js';`];
+    lines.push(
+      `export * from '${relativePathToProd}/${relativePathAndName}.js';`,
+    );
+    const body = lines.join('\n');
+
+    writeGeneratedFile(
+      path.join(devDir, path.relative(outDir, prodFile)),
+      body,
+    );
+
+    if (base === 'index.js') {
+      const dirname = path.dirname(relativePathAndName);
+      packageJSON.exports[dirname === '.' ? dirname : `./${dirname}`] = {
+        development: `./dev/${relativePathAndName}.js`,
+        default: `./${relativePathAndName}.js`,
+      };
     }
   }
 
   // Temporary workaround to allow "internal" imports, no grantees provided
-  packageJSON.exports['./*.js'] = './*.js';
-  packageJSON.exports['./*'] = './*.js';
+  packageJSON.exports['./*.js'] = {
+    development: './dev/*.js',
+    default: './*.js',
+  };
+  packageJSON.exports['./*'] = {
+    development: './dev/*.js',
+    default: './*.js',
+  };
 
   const packageJsonPath = `./${outDir}/package.json`;
   const prettified = await prettify(
@@ -127,7 +168,11 @@ function emitTSFiles(options: {
   const tsHost = ts.createCompilerHost(tsOptions);
   tsHost.writeFile = (filepath, body) => writeGeneratedFile(filepath, body);
 
-  const tsProgram = ts.createProgram(['src/index.ts'], tsOptions, tsHost);
+  const tsProgram = ts.createProgram(
+    ['src/index.ts', 'src/development.ts'],
+    tsOptions,
+    tsHost,
+  );
   const tsResult = tsProgram.emit(undefined, undefined, undefined, undefined, {
     after: [changeExtensionInImportPaths({ extension }), inlineInvariant],
   });

resources/utils.ts

@@ -234,7 +234,12 @@ interface PackageJSON {
   repository?: { url?: string };
   scripts?: { [name: string]: string };
   type?: string;
-  exports: { [path: string]: string };
+  exports: {
+    [path: string]: {
+      development: string;
+      default: string;
+    };
+  };
   types?: string;
   typesVersions: { [ranges: string]: { [path: string]: Array<string> } };
   devDependencies?: { [name: string]: string };

src/development.ts

@@ -0,0 +1,42 @@
+import { inspect } from './jsutils/inspect.js';
+import type { Constructor } from './jsutils/instanceOf.js';
+import { symbolForGraphQLInstanceOfCheck } from './jsutils/instanceOf.js';
+
+/**
+ * A replacement for instanceof which includes an error warning when multi-realm
+ * constructors are detected.
+ */
+export function developmentInstanceOfCheck(
+  value: unknown,
+  constructor: Constructor,
+): void {
+  if (typeof value === 'object' && value !== null) {
+    // Prefer Symbol.toStringTag since it is immune to minification.
+    const className = constructor.prototype[Symbol.toStringTag];
+    const valueClassName =
+      // We still need to support constructor's name to detect conflicts with older versions of this library.
+      Symbol.toStringTag in value
+        ? value[Symbol.toStringTag]
+        : value.constructor?.name;
+    if (className === valueClassName) {
+      const stringifiedValue = inspect(value);
+      throw new Error(
+        `Cannot use ${className} "${stringifiedValue}" 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.`,
+      );
+    }
+  }
+}
+
+(globalThis as any)[symbolForGraphQLInstanceOfCheck] =
+  developmentInstanceOfCheck;

src/jsutils/__tests__/instanceOf-test.ts

@@ -1,7 +1,14 @@
 import { expect } from 'chai';
 import { describe, it } from 'mocha';
 
-import { instanceOf } from '../instanceOf.js';
+import { developmentInstanceOfCheck } from '../../development.js';
+
+import { symbolForGraphQLInstanceOfCheck } from '../instanceOf.js';
+
+(globalThis as any)[symbolForGraphQLInstanceOfCheck] =
+  developmentInstanceOfCheck;
+
+const { instanceOf } = await import(`../instanceOf.js?ts${Date.now()}`);
 
 describe('instanceOf', () => {
   it('do not throw on values without prototype', () => {

src/jsutils/instanceOf.ts

@@ -1,58 +1,22 @@
-import { inspect } from './inspect.js';
+export const symbolForGraphQLInstanceOfCheck = Symbol.for(
+  'graphql.instanceOfCheck',
+);
 
-/* c8 ignore next 3 */
-const isProduction =
-  globalThis.process != null &&
-  // eslint-disable-next-line no-undef
-  process.env.NODE_ENV === 'production';
+const check: (_value: unknown, _constructor: Constructor) => void =
+  (globalThis as any)[symbolForGraphQLInstanceOfCheck] ??
+  ((_value: unknown, _constructor: Constructor) => {
+    /* no-op */
+  });
 
-/**
- * A replacement for instanceof which includes an error warning when multi-realm
- * constructors are detected.
- * See: https://expressjs.com/en/advanced/best-practice-performance.html#set-node_env-to-production
- * See: https://webpack.js.org/guides/production/
- */
-export const instanceOf: (value: unknown, constructor: Constructor) => boolean =
-  /* c8 ignore next 6 */
-  // FIXME: https://github.com/graphql/graphql-js/issues/2317
-  isProduction
-    ? function instanceOf(value: unknown, constructor: Constructor): boolean {
-        return value instanceof constructor;
-      }
-    : function instanceOf(value: unknown, constructor: Constructor): boolean {
-        if (value instanceof constructor) {
-          return true;
-        }
-        if (typeof value === 'object' && value !== null) {
-          // Prefer Symbol.toStringTag since it is immune to minification.
-          const className = constructor.prototype[Symbol.toStringTag];
-          const valueClassName =
-            // We still need to support constructor's name to detect conflicts with older versions of this library.
-            Symbol.toStringTag in value
-              ? value[Symbol.toStringTag]
-              : value.constructor?.name;
-          if (className === valueClassName) {
-            const stringifiedValue = inspect(value);
-            throw new Error(
-              `Cannot use ${className} "${stringifiedValue}" 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.`,
-            );
-          }
-        }
-        return false;
-      };
+export function instanceOf(value: unknown, constructor: Constructor): boolean {
+  if (value instanceof constructor) {
+    return true;
+  }
+  check(value, constructor);
+  return false;
+}
 
-interface Constructor {
+export interface Constructor {
   prototype: {
     [Symbol.toStringTag]: string;
   };