Remove TS types from exports when they come from node_modules

CHANGELOG.md

@@ -1,6 +1,8 @@
 ## [`master`](https://github.com/elastic/eui/tree/master)
 
-No public interface changes since `13.1.1`.
+**Bug fixes**
+
+- Remove exported TypeScript type and interface exports from built artifacts when they originate from `node_modules` ([#2191](https://github.com/elastic/eui/pull/2191))
 
 ## [`13.1.1`](https://github.com/elastic/eui/tree/v13.1.1)
 

scripts/babel/proptypes-from-ts-props/index.js

@@ -1100,15 +1100,33 @@ module.exports = function propTypesFromTypeScript({ types }) {
           // this prevents TS-only identifiers from leaking into ES code
           programPath.traverse({
             ExportNamedDeclaration: (path) => {
+              debugger;
               const specifiers = path.get('specifiers');
+              const source = path.get('source');
               specifiers.forEach(specifierPath => {
                 if (types.isExportSpecifier(specifierPath)) {
                   const { node: { local } } = specifierPath;
                   if (types.isIdentifier(local)) {
                     const { name } = local;
-                    const def = typeDefinitions[name];
-                    if (isTSType(def)) {
-                      specifierPath.remove();
+                    if (typeDefinitions.hasOwnProperty(name)) {
+                      // this is a locally-known value
+                      const def = typeDefinitions[name];
+                      if (isTSType(def)) {
+                        specifierPath.remove();
+                      }
+                    } else if (types.isStringLiteral(source)) {
+                      const libraryName = source.get('value').node;
+                      const isRelativeSource = libraryName.startsWith('.');
+                      if (isRelativeSource === false) {
+                        // comes from a 3rd-party library
+                        // best way to reliably check if this is
+                        // a type or value is to require the
+                        // library and check it's exports
+                        const library = require(libraryName);
+                        if (library.hasOwnProperty(name) === false) {
+                          specifierPath.remove();
+                        }
+                      }
                     }
                   }
                 }

scripts/babel/proptypes-from-ts-props/index.test.js

@@ -2485,6 +2485,19 @@ export type Foo = string;
 
       expect(result.code).toBe('');
     });
+
+    it('removes 3rd-party type exports', () => {
+      const result = transform(
+        `
+export { DraggableLocation, Draggable, DragDropContextProps } from 'react-beautiful-dnd';
+`,
+        babelOptions
+      );
+
+      expect(result.code).toBe(
+        `export { Draggable } from 'react-beautiful-dnd';`
+      );
+    });
   });
 
 });