@khanacademy/graphql-flow 3.0.0 → 3.0.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @khanacademy/graphql-flow
 
+## 3.0.1
+
+### Patch Changes
+
+-   c6be76f: Search through untracked files for graphql-tag. Don't error out on files that can't be found.
+
 ## 3.0.0
 
 ### Major Changes

package/dist/cli/run.js

@@ -42,7 +42,10 @@ function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj &&
 
 /** Step (1) */
 const findGraphqlTagReferences = root => {
-  const response = (0, _child_process.execSync)("git grep -I --word-regexp --name-only --fixed-strings 'graphql-tag' -- '*.js' '*.jsx' '*.ts' '*.tsx'", {
+  // NOTE(john): We want to include untracked files here so that we can
+  // generate types for them. This is useful for when we have a new file
+  // that we want to generate types for, but we haven't committed it yet.
+  const response = (0, _child_process.execSync)("git grep -I --word-regexp --name-only --fixed-strings --untracked 'graphql-tag' -- '*.js' '*.jsx' '*.ts' '*.tsx'", {
     encoding: 'utf8',
     cwd: root
   });

package/dist/parser/parse.js

@@ -35,7 +35,10 @@ const listExternalReferences = file => {
     if (v.type === 'import') {
       if (followImports) {
         const absPath = (0, _utils.getPathWithExtension)(v.path);
-        paths[absPath] = true;
+
+        if (absPath) {
+          paths[absPath] = true;
+        }
       }
     } else {
       v.source.expressions.forEach(expr => add(expr, true));

package/dist/parser/utils.js

@@ -28,9 +28,13 @@ const getPathWithExtension = pathWithoutExtension => {
 
   if (_fs.default.existsSync(pathWithoutExtension + '.ts')) {
     return pathWithoutExtension + '.ts';
-  }
+  } // NOTE(john): This is a bit of a hack, but it's necessary for when we
+  // have a file that doesn't exist. This will happen when we delete all of
+  // the type files before re-running graphql-flow again. We want to ensure
+  // that we don't error out in this case.
+
 
-  throw new Error("Can't find file at " + pathWithoutExtension);
+  return "";
 };
 
 exports.getPathWithExtension = getPathWithExtension;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@khanacademy/graphql-flow",
-    "version": "3.0.0",
+    "version": "3.0.1",
     "bin": {
         "graphql-flow": "./dist/cli/run.js"
     },

package/src/cli/run.ts

@@ -32,8 +32,11 @@ import {dirname} from 'path';
 /** Step (1) */
 
 const findGraphqlTagReferences = (root: string): Array<string> => {
+    // NOTE(john): We want to include untracked files here so that we can
+    // generate types for them. This is useful for when we have a new file
+    // that we want to generate types for, but we haven't committed it yet.
     const response = execSync(
-        "git grep -I --word-regexp --name-only --fixed-strings 'graphql-tag' -- '*.js' '*.jsx' '*.ts' '*.tsx'",
+        "git grep -I --word-regexp --name-only --fixed-strings --untracked 'graphql-tag' -- '*.js' '*.jsx' '*.ts' '*.tsx'",
         {
             encoding: 'utf8',
             cwd: root,

package/src/parser/parse.ts

@@ -105,7 +105,9 @@ const listExternalReferences = (file: FileResult): Array<string> => {
         if (v.type === 'import') {
             if (followImports) {
                 const absPath = getPathWithExtension(v.path);
-                paths[absPath] = true;
+                if (absPath) {
+                    paths[absPath] = true;
+                }
             }
         } else {
             v.source.expressions.forEach((expr) => add(expr, true));

package/src/parser/utils.ts

@@ -20,5 +20,9 @@ export const getPathWithExtension = (pathWithoutExtension: string): string => {
     if (fs.existsSync(pathWithoutExtension + '.ts')) {
         return pathWithoutExtension + '.ts';
     }
-    throw new Error("Can't find file at " + pathWithoutExtension);
+    // NOTE(john): This is a bit of a hack, but it's necessary for when we
+    // have a file that doesn't exist. This will happen when we delete all of
+    // the type files before re-running graphql-flow again. We want to ensure
+    // that we don't error out in this case.
+    return "";
 };