Fix(lint): Add import eslint and fix import errors

.eslintrc

@@ -13,8 +13,19 @@
   },
   "parser": "babel-eslint",
   "plugins": [
-    "react"
+    "react",
+    "import"
   ],
+  "settings": {
+    "import/ignore": [
+      "node_modules",
+      "\\.json$"
+    ],
+    "import/extensions": [
+      ".js",
+      ".jsx"
+    ]
+  },
   "globals": {
     "Promise": true,
     "window": true,
@@ -223,9 +234,9 @@
     "no-bitwise": 2,
     "no-plusplus": 0,
 
+    "jsx-quotes": [2, "prefer-single"],
     "react/display-name": 2,
     "react/jsx-boolean-value": [2, "always"],
-    "jsx-quotes": [2, "prefer-single"],
     "react/jsx-no-undef": 2,
     "react/jsx-sort-props": [2, { "ignoreCase": true }],
     "react/jsx-uses-react": 2,
@@ -237,6 +248,15 @@
     "react/react-in-jsx-scope": 2,
     "react/self-closing-comp": 2,
     "react/wrap-multilines": 2,
-    "react/jsx-closing-bracket-location": [ 2, { "selfClosing": "line-aligned", "nonEmpty": "props-aligned" } ]
+    "react/jsx-closing-bracket-location": [ 2, { "selfClosing": "line-aligned", "nonEmpty": "props-aligned" } ],
+
+    "import/no-unresolved": 2,
+    "import/named": 2,
+    "import/namespace": 2,
+    "import/default": 2,
+    "import/export": 2,
+    "import/imports-first": 2,
+    "import/no-duplicates": 2,
+    "import/newline-after-import": 2
   }
 }

client/rechallenge/transformers.js

@@ -1,5 +1,7 @@
 import { Observable } from 'rx';
+/* eslint-disable import/no-unresolved */
 import loopProtect from 'loop-protect';
+/* eslint-enable import/no-unresolved */
 
 import { updateContents } from '../../common/utils/polyvinyl';
 

client/sagas/frame-saga.js

@@ -1,5 +1,7 @@
 import Rx, { Observable, Subject } from 'rx';
+/* eslint-disable import/no-unresolved */
 import loopProtect from 'loop-protect';
+/* eslint-enable import/no-unresolved */
 import types from '../../common/app/routes/challenges/redux/types';
 import {
   updateOutput,

client/sagas/hard-go-to-saga.js

@@ -1,5 +1,6 @@
-import { hardGoTo } from '../../common/app/redux/types';
+import types from '../../common/app/redux/types';
 
+const { hardGoTo } = types;
 export default function hardGoToSaga(action$, getState, { history }) {
   return action$
     .filter(({ type }) => type === hardGoTo)

client/sagas/window-saga.js

@@ -1,7 +1,8 @@
 import { Observable } from 'rx';
-import { initWindowHeight } from '../../common/app/redux/types';
+import types from '../../common/app/redux/types';
 import { updateWindowHeight } from '../../common/app/redux/actions';
 
+const { initWindowHeight } = types;
 function getWindowSize(document, window) {
   const body = document.getElementsByTagName('body')[0];
   return window.innerHeight ||

common/app/redux/actions.js

@@ -32,6 +32,7 @@ export const addUser = createAction(
   entities => ({ entities })
 );
 export const updateThisUser = createAction(types.updateThisUser);
+export const showSignIn = createAction(types.showSignIn);
 
 // updateUserPoints(username: String, points: Number) => Action
 export const updateUserPoints = createAction(

common/app/redux/entities-reducer.js

@@ -1,8 +1,6 @@
-import {
+import types from './types';
-  updateUserPoints,
-  updateCompletedChallenges
-} from './types';
 
+const { updateUserPoints, updateCompletedChallenges } = types;
 const initialState = {
   superBlock: {},
   block: {},

common/app/redux/fetch-user-saga.js

@@ -1,5 +1,5 @@
 import { Observable } from 'rx';
-import { fetchUser } from './types';
+import types from './types';
 import {
   addUser,
   updateThisUser,
@@ -8,6 +8,8 @@ import {
   showSignIn
 } from './actions';
 
+const { fetchUser } = types;
+
 export default function getUserSaga(action$, getState, { services }) {
   return action$
     .filter(action => action.type === fetchUser)

common/app/redux/index.js

@@ -1,6 +1,6 @@
-export { default as reducer } from './reducer';
-export { default as actions } from './actions';
-export { default as types } from './types';
-
 import fetchUserSaga from './fetch-user-saga';
+
+export { default as reducer } from './reducer';
+export * as actions from './actions';
+export { default as types } from './types';
 export const sagas = [ fetchUserSaga ];

common/app/routes/challenges/components/project/Forms.jsx

@@ -1,5 +1,5 @@
 import React, { PropTypes } from 'react';
-import { resetForm, reduxForm } from 'redux-form';
+import { reduxForm } from 'redux-form';
 import {
   Button,
   FormGroup,
@@ -25,7 +25,6 @@ const propTypes = {
 };
 
 const bindableActions = {
-  resetForm,
   submitChallenge,
   showProjectSubmit
 };

common/app/routes/challenges/redux/fetch-challenges-saga.js

@@ -1,10 +1,6 @@
 import { Observable } from 'rx';
 import { challengeSelector } from './selectors';
-import {
+import types from './types';
-  fetchChallenge,
-  fetchChallenges,
-  replaceChallenge
-} from './types';
 import {
   fetchChallengeCompleted,
   fetchChallengesCompleted,
@@ -13,9 +9,11 @@ import {
 } from './actions';
 import {
   delayedRedirect,
-  createErrorObserable
+  createErrorObservable
 } from '../../../redux/actions';
 
+const { fetchChallenge, fetchChallenges, replaceChallenge } = types;
+
 function createNameIdMap(entities) {
   const { challenge } = entities;
   return {
@@ -77,6 +75,6 @@ export default function fetchChallengesSaga(action$, getState, { services }) {
             initMap(entities, result),
           );
         })
-        .catch(createErrorObserable);
+        .catch(createErrorObservable);
     });
 }

common/app/routes/challenges/redux/index.js

@@ -1,12 +1,12 @@
-export actions from './actions';
-export reducer from './reducer';
-export types from './types';
-
 import fetchChallengesSaga from './fetch-challenges-saga';
 import completionSaga from './completion-saga';
 import nextChallengeSaga from './next-challenge-saga';
 import answerSaga from './answer-saga';
 
+export * as actions from './actions';
+export reducer from './reducer';
+export types from './types';
+
 export projectNormalizer from './project-normalizer';
 
 export const sagas = [

common/app/routes/challenges/redux/next-challenge-saga.js

@@ -1,6 +1,6 @@
 import { Observable } from 'rx';
 import { push } from 'react-router-redux';
-import { moveToNextChallenge } from './types';
+import types from './types';
 import { resetUi, updateCurrentChallenge } from './actions';
 import { createErrorObservable, makeToast } from '../../../redux/actions';
 import {
@@ -10,6 +10,8 @@ import {
 } from '../utils';
 import { randomVerb } from '../../../utils/get-words';
 
+const { moveToNextChallenge } = types;
+
 export default function nextChallengeSaga(actions$, getState) {
   return actions$
     .filter(({ type }) => type === moveToNextChallenge)

common/models/User-Identity.js

@@ -7,8 +7,8 @@ import {
   getUsernameFromProvider,
   getSocialProvider
 } from '../../server/utils/auth';
+import { defaultProfileImage } from '../utils/constantStrings.json';
 
-const { defaultProfileImage } = require('../utils/constantStrings.json');
 const githubRegex = (/github/i);
 const debug = debugFactory('fcc:models:userIdent');
 

package.json

@@ -127,6 +127,7 @@
     "chunk-manifest-webpack-plugin": "0.1.0",
     "del": "^2.2.0",
     "eslint": "^3.1.0",
+    "eslint-plugin-import": "^1.9.2",
     "eslint-plugin-react": "^5.1.1",
     "gulp": "^3.9.0",
     "gulp-babel": "^6.1.1",

server/boot/randomAPIs.js

@@ -1,7 +1,9 @@
 import request from 'request';
 import constantStrings from '../utils/constantStrings.json';
 import testimonials from '../resources/testimonials.json';
-import secrets from '../../config/secrets';
+
+const githubClient = process.env.GITHUB_ID;
+const githubSecret = process.env.GITHUB_SECRET;
 
 module.exports = function(app) {
   const router = app.loopback.Router();
@@ -258,9 +260,9 @@ module.exports = function(app) {
       [
         'https://api.github.com/repos/freecodecamp/',
         'freecodecamp/pulls?client_id=',
-        secrets.github.clientID,
+        githubClient,
         '&client_secret=',
-        secrets.github.clientSecret
+        githubSecret
       ].join(''),
       githubHeaders,
       function(err, status1, pulls) {
@@ -273,9 +275,9 @@ module.exports = function(app) {
           [
             'https://api.github.com/repos/freecodecamp/',
             'freecodecamp/issues?client_id=',
-            secrets.github.clientID,
+            githubClient,
             '&client_secret=',
-            secrets.github.clientSecret
+            githubSecret
           ].join(''),
           githubHeaders,
           function(err, status2, issues) {

server/boot/user.js

@@ -12,7 +12,11 @@ import {
 
 import certTypes from '../utils/certTypes.json';
 
-import { ifNoUser401, ifNoUserRedirectTo } from '../utils/middleware';
+import {
+  ifNoUser401,
+  ifNoUserRedirectTo,
+  flashIfNotVerified
+} from '../utils/middleware';
 import { observeQuery } from '../utils/rx';
 import {
   prepUniqueDays,
@@ -20,8 +24,6 @@ import {
   calcLongestStreak
 } from '../utils/user-stats';
 
-import { flashIfNotVerified } from '../utils/middleware';
-
 const debug = debugFactory('fcc:boot:user');
 const sendNonUserToMap = ifNoUserRedirectTo('/map');
 const certIds = {

server/debug-entry.js

@@ -2,4 +2,5 @@
 // or mocha.
 require('babel-register');
 var app = require('./server');
+
 app.start();

server/middlewares/cookie-parser.js

@@ -1,4 +1,4 @@
 import cookieParser from 'cookie-parser';
-import secrets from '../../config/secrets';
 
-export default cookieParser.bind(cookieParser, secrets.cookieSecret);
+const cookieSecret = process.env.COOKIE_SECRET;
+export default cookieParser.bind(cookieParser, cookieSecret);

server/middlewares/jade-helpers.js

@@ -1,5 +1,7 @@
 import manifest from '../rev-manifest';
+/* eslint-disable import/default */
 import config from '../../webpack.config';
+/* eslint-enable import/default */
 
 let chunkManifest;
 try {

server/middlewares/sessions.js

@@ -1,8 +1,9 @@
 import session from 'express-session';
 import MongoStoreFactory from 'connect-mongo';
-import secrets from '../../config/secrets';
 
 const MongoStore = MongoStoreFactory(session);
+const sessionSecret = process.env.SESSION_SECRET;
+const url = process.env.MONGODB || process.env.MONGOHQ_URL;
 
 export default function sessionsMiddleware() {
   return session({
@@ -10,7 +11,7 @@ export default function sessionsMiddleware() {
     cookie: { maxAge: 900 * 24 * 60 * 60 * 1000 },
     resave: true,
     saveUninitialized: true,
-    secret: secrets.sessionSecret,
+    secret: sessionSecret,
-    store: new MongoStore({ url: secrets.db })
+    store: new MongoStore({ url })
   });
 }

server/passport-providers.js

@@ -1,7 +1,8 @@
 var successRedirect = '/';
 var failureRedirect = '/signin';
 var linkFailureRedirect = '/account';
-module.exports = {
+
+export default {
   local: {
     provider: 'local',
     module: 'passport-local',

server/production-start.js

@@ -5,6 +5,7 @@ var startTime = Date.now();
 var timeoutHandler;
 // this is where server starts booting up
 var app = require('./server');
+
 console.log('waiting for db to connect');
 
 

server/server.js

@@ -1,5 +1,6 @@
 require('dotenv').load();
 var pmx = require('pmx');
+
 pmx.init();
 
 var _ = require('lodash'),

server/utils/commit.js

@@ -1,8 +1,8 @@
 import dedent from 'dedent';
 import debugFactory from 'debug';
 import { Observable } from 'rx';
-
 import commitGoals from './commit-goals.json';
+
 const debug = debugFactory('fcc:utils/commit');
 
 export { commitGoals };

server/utils/constants.js

@@ -1,4 +1,4 @@
-exports.blacklistedUsernames = [
+export const blacklistedUsernames = [
   'bonfire',
   'account',
   'user',

server/utils/index.js

@@ -1,101 +1,22 @@
-var cheerio = require('cheerio'),
+export function dasherize(name) {
-    request = require('request'),
+  return ('' + name)
-    MDNlinks = require('../../seed/bonfireMDNlinks'),
+    .toLowerCase()
-    resources = require('./resources.json');
+    .replace(/\s/g, '-')
+    .replace(/[^a-z0-9\-\.]/gi, '')
+    .replace(/\:/g, '');
+}
 
-/**
+export function nameify(str) {
- * Cached values
+  return ('' + str)
- */
+    .replace(/[^a-zA-Z0-9\s]/g, '')
+    .replace(/\:/g, '');
+}
 
-module.exports = {
+export function unDasherize(name) {
-  dasherize: function dasherize(name) {
+  return ('' + name)
-    return ('' + name)
+    // replace dash with space
-      .toLowerCase()
+    .replace(/\-/g, ' ')
-      .replace(/\s/g, '-')
+    // strip nonalphanumarics chars except whitespace
-      .replace(/[^a-z0-9\-\.]/gi, '')
+    .replace(/[^a-zA-Z\d\s]/g, '')
-      .replace(/\:/g, '');
+    .trim();
-  },
+}
-
-  nameify: function nameify(str) {
-    return ('' + str)
-      .replace(/[^a-zA-Z0-9\s]/g, '')
-      .replace(/\:/g, '');
-  },
-
-  unDasherize: function unDasherize(name) {
-    return ('' + name)
-      // replace dash with space
-      .replace(/\-/g, ' ')
-      // strip nonalphanumarics chars except whitespace
-      .replace(/[^a-zA-Z\d\s]/g, '')
-      .trim();
-  },
-
-  randomPhrase: function() {
-    return resources.phrases[
-      Math.floor(Math.random() * resources.phrases.length)
-      ];
-  },
-
-  randomVerb: function() {
-    return resources.verbs[
-      Math.floor(Math.random() * resources.verbs.length)
-      ];
-  },
-
-  randomCompliment: function() {
-    return resources.compliments[
-      Math.floor(Math.random() * resources.compliments.length)
-      ];
-  },
-
-  whichEnvironment: function() {
-    return process.env.NODE_ENV;
-  },
-
-  getURLTitle: function(url, callback) {
-    var result = {
-      title: '',
-      image: '',
-      url: '',
-      description: ''
-    };
-    request(url, function(err, response, body) {
-      if (err || response.statusCode !== 200) {
-        return callback(new Error('failed'));
-      }
-      var $ = cheerio.load(body);
-      var metaDescription = $("meta[name='description']");
-      var metaImage = $("meta[property='og:image']");
-      var urlImage = metaImage.attr('content') ?
-        metaImage.attr('content') :
-        '';
-
-      var metaTitle = $('title');
-      var description = metaDescription.attr('content') ?
-        metaDescription.attr('content') :
-        '';
-
-      result.title = metaTitle.text().length < 90 ?
-        metaTitle.text() :
-      metaTitle.text().slice(0, 87) + '...';
-
-      result.image = urlImage;
-      result.description = description;
-      return callback(null, result);
-    });
-  },
-
-  getMDNLinks: function(links) {
-    if (!links) {
-      return [];
-    }
-    // takes in an array of links, which are strings
-
-    // for each key value, push the corresponding link
-    // from the MDNlinks object into a new array
-    return links.map(function(value) {
-      return MDNlinks[value];
-    });
-  }
-};