Fix TS error caused by importing internal files directly (#3339)

Author: IvanGoncharov

integrationTests/ts/internalImports-test.ts

@@ -0,0 +1,8 @@
+import type { NameNode } from 'graphql/language';
+
+// Parser class is internal API so so any changes to it are never considered breaking changes.
+// We just want to test that we are able to import it.
+import { Parser } from 'graphql/language/parser';
+
+const parser = new Parser('foo');
+const ast: NameNode = parser.parseName();

integrationTests/ts/test.js

@@ -11,7 +11,9 @@ const tsVersions = Object.keys(dependencies)
 
 for (const version of tsVersions) {
   console.log(`Testing on ${version} ...`);
+  childProcess.execSync(tscPath(version), { stdio: 'inherit' });
+}
 
-  const tscPath = path.join(__dirname, 'node_modules', version, 'bin/tsc');
-  childProcess.execSync(tscPath, { stdio: 'inherit' });
+function tscPath(version) {
+  return path.join(__dirname, 'node_modules', version, 'bin/tsc');
 }

package.json

@@ -6,17 +6,10 @@
   "private": true,
   "main": "index",
   "module": "index.mjs",
-  "types": "NotSupportedTSVersion.d.ts",
   "typesVersions": {
     ">=4.1.0": {
       "*": [
-        "index.d.ts"
-      ],
-      "*/*": [
-        "*/index.d.ts"
-      ],
-      "*/*/*": [
-        "*/*"
+        "*"
       ]
     }
   },

resources/build-npm.js

@@ -50,18 +50,23 @@ if (require.main === module) {
     'Fail to generate `*.d.ts` files, please run `npm run check`',
   );
 
-  assert(packageJSON.types, 'Missing "types".');
+  assert(packageJSON.types === undefined, 'Unexpected "types" in package.json');
   const supportedTSVersions = Object.keys(packageJSON.typesVersions);
   assert(
     supportedTSVersions.length === 1,
     'Property "typesVersions" should have exactly one key.',
   );
-  // TODO: revisit once TS implements https://github.com/microsoft/TypeScript/issues/44795
+  // TODO: revisit once TS implements https://github.com/microsoft/TypeScript/issues/32166
+  const notSupportedTSVersionFile = 'NotSupportedTSVersion.d.ts';
   fs.writeFileSync(
-    path.join('./npmDist', packageJSON.types),
+    path.join('./npmDist', notSupportedTSVersionFile),
     // Provoke syntax error to show this message
     `"Package 'graphql' support only TS versions that are ${supportedTSVersions[0]}".`,
   );
+  packageJSON.typesVersions = {
+    ...packageJSON.typesVersions,
+    '*': { '*': [notSupportedTSVersionFile] },
+  };
 
   fs.copyFileSync('./LICENSE', './npmDist/LICENSE');
   fs.copyFileSync('./README.md', './npmDist/README.md');