review feedback

elastic · Nov 19, 2018 · 422a651 · 422a651
1 parent 9baf6f9
commit 422a651
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 55 deletions.
diff --git a/packages/kbn-interpreter/server/get_plugin_paths.js b/packages/kbn-interpreter/server/get_plugin_paths.js
@@ -23,22 +23,33 @@ import { promisify } from 'util';
 import { flatten } from 'lodash';
 import { pathsRegistry } from '../common/lib/paths_registry';
 
+const lstat = promisify(fs.lstat);
 const readdir = promisify(fs.readdir);
 
+const isDirectory = path =>
+  lstat(path)
+    .then(stat => stat.isDirectory())
+    .catch(() => false);
+
 export const getPluginPaths = type => {
   const typePaths = pathsRegistry.get(type);
   if (!typePaths) {
     throw new Error(`Unknown type: ${type}`);
   }
 
-  return Promise.all(typePaths.map(path => {
-
-    // Get the full path of all files in the directory
-    return readdir(path).then(files => files.map(file => {
-      if (!file.endsWith('.js')) {
-        return;
-      }
-      return resolve(path, file);
-    }).filter(path => path)).catch();
+  return Promise.all(typePaths.map(async path => {
+    const isDir = await isDirectory(path);
+    if (!isDir) {
+      return;
+    }
+    // Get the full path of all js files in the directory
+    return readdir(path).then(files => {
+      return files.reduce((acc, file) => {
+        if (file.endsWith('.js')) {
+          acc.push(resolve(path, file));
+        }
+        return acc;
+      }, []);
+    }).catch();
   })).then(flatten);
 };
diff --git a/packages/kbn-interpreter/server/index.js b/packages/kbn-interpreter/server/index.js
diff --git a/packages/kbn-interpreter/tasks/webpack.plugins.js b/packages/kbn-interpreter/tasks/webpack.plugins.js
@@ -37,6 +37,7 @@ module.exports = {
 
   resolve: {
     extensions: ['.js', '.json'],
+    mainFields: ['browser', 'main'],
   },
 
   plugins: [
@@ -51,12 +52,15 @@ module.exports = {
       });
 
       this.plugin('done', function (stats) {
-        if (stats.compilation.errors && stats.compilation.errors.length) {
-          if (isWatch) {
-            console.error(stats.compilation.errors[0]);
-          } else {
-            throw stats.compilation.errors[0];
-          }
+        if (!stats.hasErrors()) {
+          return;
+        }
+        const errorMessage = stats.toString('errors-only');
+        if (isWatch) {
+          console.error(errorMessage);
+        }
+        else {
+          throw new Error(errorMessage);
         }
       });
     },
@@ -66,28 +70,12 @@ module.exports = {
     rules: [
       {
         test: /\.js$/,
+        exclude: [/node_modules/],
         loaders: 'babel-loader',
         options: {
           babelrc: false,
-          plugins: [
-            'transform-object-rest-spread',
-            'transform-async-to-generator',
-            'transform-class-properties',
-          ],
-          presets: [
-            'es2015',
-            'react',
-            [
-              'env',
-              {
-                targets: {
-                  node: 'current',
-                },
-              },
-            ],
-          ],
+          presets: [require.resolve('@kbn/babel-preset/webpack_preset')],
         },
-        exclude: [/node_modules/],
       },
       {
         test: /\.(png|jpg|gif|jpeg|svg)$/,