Fix: Async component loader wrong path for 'async!...' imports (#50)

* fix: wrong build path for 'async!' imports. Fixes #47

* fix: Incorrect async-component-loader refactor

* chore: Configure eslint & fix linting errors

package.json

@@ -10,6 +10,7 @@
     "build": "babel src -d lib -D",
     "prepublish": "npm run -s build",
     "dev": "babel-node src",
+    "lint": "eslint 'src/**/*.js'",
     "test:build": "babel-node src build --cwd examples/root",
     "test:serve": "npm run -s test:build && babel-node src serve --port 3000 --cwd examples/root",
     "test:serve:config": "npm run -s test:build && babel-node src serve --server config --cwd examples/root",
@@ -51,6 +52,8 @@
     "rules": {
       "no-console": 1,
       "no-empty": 0,
+      "semi": 2,
+      "keyword-spacing": 2,
       "react/no-string-refs": 2,
       "react/no-find-dom-node": 2,
       "react/no-is-mounted": 2,

src/commands/create.js

@@ -148,7 +148,7 @@ export default asyncCommand({
 			  \u001b[32mnpm run serve\u001b[39m
 		`.trim().replace(/^\t+/gm, '') + '\n';
 	}
-})
+});
 
 
 const npm = (cwd, args) => spawn('npm', args, { cwd, stdio: 'ignore' });

src/commands/serve.js

@@ -121,7 +121,7 @@ function createHeadersFromPushManifest(pushManifest) {
 						`<${url}>; rel=preload; as=${type}`
 					).join(', ')
 				}]
-			})
+			});
 		}
 	}
 

src/commands/watch.js

@@ -35,4 +35,4 @@ export default asyncCommand({
 		let stats = await runWebpack(true, config, showStats);
 		showStats(stats);
 	}
-})
+});

src/lib/async-command.js

@@ -16,5 +16,5 @@ export default function asyncCommand(options) {
 			let r = options.handler(argv, done);
 			if (r && r.then) r.then(result => done(null, result), done);
 		}
-	}
+	};
 }

src/lib/async-component-loader.js

@@ -6,7 +6,14 @@ module.exports.pitch = function(remainingRequest) {
 	this.cacheable && this.cacheable();
 	var query = loaderUtils.getOptions(this) || {};
 	var routeName = typeof query.name === 'function' ? query.name(this.resourcePath) : null;
-	var name = routeName !== null ? routeName : ('name' in query ? query.name : (query.formatName || String)(this.resourcePath));
+	var name;
+	if (routeName !== null) {
+		name = routeName;
+	} else if ('name' in query) {
+		name = query.name;
+	} else if ('formatName' in query) {
+		name = query.formatName(this.resourcePath);
+	}
 
 	return `
 		import async from ${JSON.stringify(path.resolve(__dirname, '../components/async'))};

src/lib/babel-config.js

@@ -28,4 +28,4 @@ export default (env, options={}) => ({
 			import: 'h'
 		}]
 	]
-})
+});

src/lib/prerender.js

@@ -37,6 +37,7 @@ export default function prerender(config, params) {
 		app = m && m.default || m;
 
 	if (typeof app!=='function') {
+		// eslint-disable-next-line no-console
 		console.warn('Entry does not export a Component function/class, aborting prerendering.');
 		return '';
 	}

src/lib/push-manifest.js

@@ -36,4 +36,4 @@ module.exports = class PushManifestPlugin {
 			callback();
 		});
 	}
-}
+};

src/lib/run-webpack.js

@@ -13,7 +13,7 @@ export default (watch=false, config, onprogress) => new Promise( (resolve, rejec
 			// Timeout for plugins that work on `after-emit` event of webpack
 			setTimeout(()=>{
 				resolve(stats);
-			},20)
+			},20);
 		}
 	};