fix: Fix propTypes and lint warnings (#31)

package.json

@@ -7,15 +7,15 @@
     "playroom": "bin/cli.js"
   },
   "scripts": {
-    "test": "npm run cypress",
+    "test": "npm run lint && npm run cypress",
     "cypress": "start-server-and-test cypress:prepare http://localhost:9000 cypress:run",
     "cypress:dev": "start-server-and-test cypress:prepare http://localhost:9000 cypress:open",
     "cypress:prepare": "./bin/cli.js start --config cypress/projects/basic/playroom.config.js",
     "cypress:open": "cypress open",
     "cypress:run": "cypress run",
     "commit": "git-cz",
-    "lint": "eslint .",
-    "format": "prettier --config ./.prettierrc --write '**/*.{js,md,less}'",
+    "lint": "eslint . && prettier --list-different '**/*.{js,md,less}'",
+    "format": "prettier --write '**/*.{js,md,less}'",
     "semantic-release": "semantic-release"
   },
   "husky": {

src/Playroom/Playroom.js

@@ -1,5 +1,4 @@
 import React, { Component } from 'react';
-import PropTypes from 'prop-types';
 import parsePropTypes from 'parse-prop-types';
 import flatMap from 'lodash/flatMap';
 import debounce from 'lodash/debounce';
@@ -36,13 +35,13 @@ const resizableConfig = {
 
 const completeAfter = (cm, predicate) => {
   const CodeMirror = cm.constructor;
-  const cur = cm.getCursor();
-  if (!predicate || predicate())
+  if (!predicate || predicate()) {
     setTimeout(() => {
       if (!cm.state.completionActive) {
         cm.showHint({ completeSingle: false });
       }
     }, 100);
+  }
 
   return CodeMirror.Pass;
 };
@@ -52,6 +51,7 @@ const completeIfAfterLt = cm => {
 
   return completeAfter(cm, () => {
     const cur = cm.getCursor();
+    // eslint-disable-next-line new-cap
     return cm.getRange(CodeMirror.Pos(cur.line, cur.ch - 1), cur) === '<';
   });
 };
@@ -62,9 +62,9 @@ const completeIfInTag = cm => {
   return completeAfter(cm, () => {
     const tok = cm.getTokenAt(cm.getCursor());
     if (
-      tok.type == 'string' &&
+      tok.type === 'string' &&
       (!/['"]/.test(tok.string.charAt(tok.string.length - 1)) ||
-        tok.string.length == 1)
+        tok.string.length === 1)
     ) {
       return false;
     }
@@ -189,7 +189,6 @@ export default class Playroom extends Component {
     const tags = Object.assign(
       {},
       ...componentNames.map(componentName => {
-        const { propTypes = {} } = components[componentName];
         const parsedPropTypes = parsePropTypes(components[componentName]);
         const filteredPropTypes = omit(
           parsedPropTypes,

src/Playroom/Preview/Preview.js

@@ -10,15 +10,15 @@ import styles from './Preview.less';
 export default class Preview extends Component {
   static propTypes = {
     code: PropTypes.string.isRequired,
-    components: PropTypes.node.isRequired,
+    components: PropTypes.object.isRequired,
     themes: PropTypes.object,
     frames: PropTypes.arrayOf(
       PropTypes.shape({
         theme: PropTypes.string.isRequired,
-        width: PropTypes.string.isRequired
+        width: PropTypes.number.isRequired
       })
     ),
-    frameComponent: PropTypes.node.isRequired
+    frameComponent: PropTypes.func.isRequired
   };
 
   static defaultProps = { themes: [], frames: [] };