Named exports no longer trigger stdlib imports

- Fixes wcjohnson/lightscript#56
wcjohnson · Oct 21, 2017 · 9f7065c · 9f7065c
1 parent 1328737
commit 9f7065c
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 0 deletions.
diff --git a/src/helpers/variables.lsc b/src/helpers/variables.lsc
@@ -83,5 +83,12 @@ export checkFalsePositiveReference(refPath) ->
     !parentNode.computed
   elif parentNode.type == "FunctionTypeParam":
     true
+  elif parentNode.type == "ExportSpecifier":
+    // In `export { x } from 'y'`, x is not actually referenced.
+    parent2 = refPath.parentPath.parentPath.node
+    if parent2.type == "ExportNamedDeclaration" and parent2.source:
+      true
+    else:
+      false
   else:
     false
diff --git a/test/fixtures/stdlib/export-from/actual.js b/test/fixtures/stdlib/export-from/actual.js
@@ -0,0 +1,2 @@
+export { map }
+export { toPlainObject } from 'otherLib'
diff --git a/test/fixtures/stdlib/export-from/expected.js b/test/fixtures/stdlib/export-from/expected.js
@@ -0,0 +1,2 @@
+import map from 'lodash/map';export { map };
+export { toPlainObject } from 'otherLib';