Add linting to babel plugin (#542)

Author: doug-wade

packages/babel-plugin-react-server/package.json

@@ -18,7 +18,7 @@
   "scripts": {
     "clean": "rm -rf lib npm-debug.log*",
     "build": "babel src -d lib",
-    "test": "mocha --compilers js:babel-register",
+    "test": "mocha --compilers js:babel-register && eslint src/",
     "test:watch": "npm run test -- --watch",
     "prepublish": "npm run clean && npm run build"
   },

packages/babel-plugin-react-server/src/index.js

@@ -1,34 +1,35 @@
 import loggerSpec from 'react-server-module-tagger';
 import path from 'path';
 
-module.exports = function({types: t }) {
-  return {
-    visitor: {
-      Identifier(p, state) {
-        const {node} = p;
-        const {name, type} = node;
+module.exports = function() {
+	return {
+		visitor: {
+			Identifier(p, state) {
+				const {node} = p;
+				const {name} = node;
 
-        const config = { trim: state.opts.trim };
-        const parent = path.resolve(path.join(process.cwd(), '..')) + path.sep;
-        const fp = this.file.opts.filename.replace(parent, '');
-        const file =  { path: fp };
-        //TODO: Support labels
-        const moduleTag = loggerSpec.bind({ file, config })(fp);
+				const config = { trim: state.opts.trim };
+				const parent = path.resolve(path.join(process.cwd(), '..')) + path.sep;
+				const fp = this.file.opts.filename.replace(parent, '');
+				const file =  { path: fp };
 
-       let tokens;
-       if (state.opts.tokens) {
-         tokens = new Set(state.opts.tokens);
-       } else {
-         tokens = new Set(["__LOGGER__", "__CHANNEL__", "__CACHE__"]);
-       }
+				//TODO: Support labels
+				const moduleTag = loggerSpec.bind({ file, config })(fp);
 
-        if (tokens.has(name)) {
-          // this strikes me as a dirty, nasty hack.  I think it would be better
-          // to parse the object as json and coerce it to an array of
-          // ObjectProperties to construct an ObjectExpression
-          p.node.name = moduleTag;
-        }
-      }
-    }
-  };
+				let tokens;
+				if (state.opts.tokens) {
+					tokens = new Set(state.opts.tokens);
+				} else {
+					tokens = new Set(["__LOGGER__", "__CHANNEL__", "__CACHE__"]);
+				}
+
+				if (tokens.has(name)) {
+					// this strikes me as a dirty, nasty hack.  I think it would be better
+					// to parse the object as json and coerce it to an array of
+					// ObjectProperties to construct an ObjectExpression
+					p.node.name = moduleTag;
+				}
+			},
+		},
+	};
 }