From 530d2b6fe949fcbee7e5b1c21d1d4b63b64e4636 Mon Sep 17 00:00:00 2001
From: Tiago Costa <tiagoffcc@hotmail.com>
Date: Tue, 1 Oct 2019 20:38:33 +0100
Subject: [PATCH] chore(NA): fix logic behind cleaning x-pack node modules on
 build

---
 package.json                                         |  2 --
 packages/kbn-babel-code-parser/src/can_require.js    | 12 ++++++------
 packages/kbn-babel-code-parser/src/code_parser.js    |  2 +-
 packages/kbn-babel-code-parser/src/strategies.js     |  8 ++++++--
 .../kbn-babel-code-parser/src/strategies.test.js     |  6 +++---
 x-pack/package.json                                  |  2 ++
 6 files changed, 18 insertions(+), 14 deletions(-)

diff --git a/package.json b/package.json
index 6f54c8683410a..47a74262d0d01 100644
--- a/package.json
+++ b/package.json
@@ -160,7 +160,6 @@
     "expiry-js": "0.1.7",
     "file-loader": "4.2.0",
     "font-awesome": "4.7.0",
-    "fp-ts": "^2.0.5",
     "getos": "^3.1.0",
     "glob": "^7.1.2",
     "glob-all": "^3.1.0",
@@ -177,7 +176,6 @@
     "https-proxy-agent": "^2.2.2",
     "inert": "^5.1.0",
     "inline-style": "^2.0.0",
-    "io-ts": "^2.0.1",
     "joi": "^13.5.2",
     "jquery": "^3.4.1",
     "js-yaml": "3.13.1",
diff --git a/packages/kbn-babel-code-parser/src/can_require.js b/packages/kbn-babel-code-parser/src/can_require.js
index e590c249e9806..4d85910abe6ed 100644
--- a/packages/kbn-babel-code-parser/src/can_require.js
+++ b/packages/kbn-babel-code-parser/src/can_require.js
@@ -17,18 +17,18 @@
  * under the License.
  */
 
-export function canRequire(cwd, entry) {
+export function canRequire(entry, cwd = require.resolve.paths(entry) || []) {
   try {
     // We will try to test if we can resolve
     // this entry through the require.resolve
     // setting as the start looking path the
-    // given cwd. Require.resolve will keep
+    // given cwd. That cwd variable could be
+    // a path or an array of paths
+    // from where Require.resolve will keep
     // looking recursively as normal starting
-    // from that location.
+    // from those locations.
     return require.resolve(entry, {
-      paths: [
-        cwd
-      ]
+      paths: [].concat(cwd)
     });
   } catch (e) {
     return false;
diff --git a/packages/kbn-babel-code-parser/src/code_parser.js b/packages/kbn-babel-code-parser/src/code_parser.js
index 8d76b1032561a..0f53bd249bb5c 100644
--- a/packages/kbn-babel-code-parser/src/code_parser.js
+++ b/packages/kbn-babel-code-parser/src/code_parser.js
@@ -79,7 +79,7 @@ export async function parseEntries(cwd, entries, strategy, results, wasParsed =
   const sanitizedCwd = cwd || process.cwd();
 
   // Test each entry against canRequire function
-  const entriesQueue = entries.map(entry => canRequire(sanitizedCwd, entry));
+  const entriesQueue = entries.map(entry => canRequire(entry));
 
   while(entriesQueue.length) {
     // Get the first element in the queue as
diff --git a/packages/kbn-babel-code-parser/src/strategies.js b/packages/kbn-babel-code-parser/src/strategies.js
index 317ded014210b..89621bc53bd53 100644
--- a/packages/kbn-babel-code-parser/src/strategies.js
+++ b/packages/kbn-babel-code-parser/src/strategies.js
@@ -62,8 +62,12 @@ export async function dependenciesParseStrategy(cwd, parseSingleFile, mainEntry,
   // new dependencies
   return dependencies.reduce((filteredEntries, entry) => {
     const absEntryPath = resolve(cwd, dirname(mainEntry), entry);
-    const requiredPath = canRequire(cwd, absEntryPath);
-    const requiredRelativePath = canRequire(cwd, entry);
+
+    // NOTE: cwd for following canRequires is absEntryPath
+    // because we should start looking from there
+    const requiredPath = canRequire(absEntryPath, absEntryPath);
+    const requiredRelativePath = canRequire(entry, absEntryPath);
+
     const isRelativeFile = !isAbsolute(entry);
     const isNodeModuleDep = isRelativeFile && !requiredPath && requiredRelativePath;
     const isNewEntry = isRelativeFile && requiredPath;
diff --git a/packages/kbn-babel-code-parser/src/strategies.test.js b/packages/kbn-babel-code-parser/src/strategies.test.js
index 5a84edf560af1..d7caa8b95d4a2 100644
--- a/packages/kbn-babel-code-parser/src/strategies.test.js
+++ b/packages/kbn-babel-code-parser/src/strategies.test.js
@@ -59,8 +59,8 @@ describe('Code Parser Strategies', () => {
       cb(null, `require('dep_from_node_modules')`);
     });
 
-    canRequire.mockImplementation((mockCwd, entry) => {
-      if (entry === `${mockCwd}dep1/dep_from_node_modules`) {
+    canRequire.mockImplementation((entry, cwd) => {
+      if (entry === `${cwd}dep1/dep_from_node_modules`) {
         return false;
       }
 
@@ -78,7 +78,7 @@ describe('Code Parser Strategies', () => {
       cb(null, `require('./relative_dep')`);
     });
 
-    canRequire.mockImplementation((mockCwd, entry) => {
+    canRequire.mockImplementation((entry) => {
       if (entry === `${mockCwd}dep1/relative_dep`) {
         return `${entry}/index.js`;
       }
diff --git a/x-pack/package.json b/x-pack/package.json
index d15d66aa18ebe..cb3c961faef71 100644
--- a/x-pack/package.json
+++ b/x-pack/package.json
@@ -241,6 +241,7 @@
     "file-type": "^10.9.0",
     "font-awesome": "4.7.0",
     "formsy-react": "^1.1.5",
+    "fp-ts": "^2.0.5",
     "geojson-rewind": "^0.3.1",
     "get-port": "4.2.0",
     "getos": "^3.1.0",
@@ -262,6 +263,7 @@
     "immer": "^1.5.0",
     "inline-style": "^2.0.0",
     "intl": "^1.2.5",
+    "io-ts": "^2.0.1",
     "isbinaryfile": "4.0.2",
     "isomorphic-git": "0.55.5",
     "joi": "^13.5.2",