Strip propTypes in production

Author: mjackson

.babelrc

@@ -3,5 +3,8 @@
     "es2015-loose",
     "stage-1",
     "react"
+  ],
+  "plugins": [
+    "babel-plugin-dev-expression"
   ]
 }

.eslintrc

@@ -16,5 +16,8 @@
   "rules": {
     "prefer-arrow-callback": 2,
     "semi": [ 2, "never" ]
+  },
+  "globals": {
+    "__DEV__": true
   }
 }

modules/BrowserRouter.js

@@ -26,15 +26,17 @@ const BrowserRouter = ({ basename, forceRefresh, getUserConfirmation, keyLength,
   </BrowserHistory>
 )
 
-BrowserRouter.propTypes = {
-  basename: PropTypes.string,
-  forceRefresh: PropTypes.bool,
-  getUserConfirmation: PropTypes.func,
-  keyLength: PropTypes.number,
-  children: PropTypes.oneOfType([
-    PropTypes.func,
-    PropTypes.node
-  ])
+if (__DEV__) {
+  BrowserRouter.propTypes = {
+    basename: PropTypes.string,
+    forceRefresh: PropTypes.bool,
+    getUserConfirmation: PropTypes.func,
+    keyLength: PropTypes.number,
+    children: PropTypes.oneOfType([
+      PropTypes.func,
+      PropTypes.node
+    ])
+  }
 }
 
 export default BrowserRouter

modules/HashRouter.js

@@ -25,14 +25,16 @@ const HashRouter = ({ basename, getUserConfirmation, hashType, ...props }) => (
   </HashHistory>
 )
 
-HashRouter.propTypes = {
-  basename: PropTypes.string,
-  getUserConfirmation: PropTypes.func,
-  hashType: PropTypes.string,
-  children: PropTypes.oneOfType([
-    PropTypes.func,
-    PropTypes.node
-  ])
+if (__DEV__) {
+  HashRouter.propTypes = {
+    basename: PropTypes.string,
+    getUserConfirmation: PropTypes.func,
+    hashType: PropTypes.string,
+    children: PropTypes.oneOfType([
+      PropTypes.func,
+      PropTypes.node
+    ])
+  }
 }
 
 export default HashRouter

modules/Link.js

@@ -5,23 +5,6 @@ import {
 } from './PropTypes'
 
 class Link extends React.Component {
-  static propTypes = {
-    to: PropTypes.oneOfType([ PropTypes.string, PropTypes.object ]).isRequired,
-    activeStyle: PropTypes.object,
-    activeClassName: PropTypes.string,
-    location: PropTypes.object,
-    activeOnlyWhenExact: PropTypes.bool,
-    isActive: PropTypes.func,
-    children: PropTypes.oneOfType([PropTypes.node, PropTypes.func]),
-
-    // props we have to deal with but aren't necessarily
-    // part of the Link API
-    style: PropTypes.object,
-    className: PropTypes.string,
-    target: PropTypes.string,
-    onClick: PropTypes.func
-  }
-
   static defaultProps = {
     activeOnlyWhenExact: false,
     className: '',
@@ -121,6 +104,25 @@ class Link extends React.Component {
   }
 }
 
+if (__DEV__) {
+  Link.propTypes = {
+    to: PropTypes.oneOfType([ PropTypes.string, PropTypes.object ]).isRequired,
+    activeStyle: PropTypes.object,
+    activeClassName: PropTypes.string,
+    location: PropTypes.object,
+    activeOnlyWhenExact: PropTypes.bool,
+    isActive: PropTypes.func,
+    children: PropTypes.oneOfType([PropTypes.node, PropTypes.func]),
+
+    // props we have to deal with but aren't necessarily
+    // part of the Link API
+    style: PropTypes.object,
+    className: PropTypes.string,
+    target: PropTypes.string,
+    onClick: PropTypes.func
+  }
+}
+
 // we should probably use LocationUtils.createLocationDescriptor
 const createLocationDescriptor = (to) =>
   typeof to === 'object' ? to : { pathname: to }

modules/Match.js

@@ -4,11 +4,6 @@ import matchPattern from './matchPattern'
 import { LocationSubscriber } from './locationEmission'
 
 class RegisterMatch extends React.Component {
-  static propTypes = {
-    children: PropTypes.node.isRequired,
-    match: PropTypes.any
-  }
-
   static contextTypes = {
     match: PropTypes.object,
     serverRouter: PropTypes.object
@@ -58,17 +53,14 @@ class RegisterMatch extends React.Component {
   }
 }
 
-class Match extends React.Component {
-  static propTypes = {
-    pattern: PropTypes.string,
-    exactly: PropTypes.bool,
-    location: PropTypes.object,
-
-    children: PropTypes.func,
-    render: PropTypes.func,
-    component: PropTypes.func
+if (__DEV__) {
+  RegisterMatch.propTypes = {
+    children: PropTypes.node.isRequired,
+    match: PropTypes.any
   }
+}
 
+class Match extends React.Component {
   static defaultProps = {
     exactly: false
   }
@@ -110,4 +102,16 @@ class Match extends React.Component {
   }
 }
 
+if (__DEV__) {
+  Match.propTypes = {
+    pattern: PropTypes.string,
+    exactly: PropTypes.bool,
+    location: PropTypes.object,
+
+    children: PropTypes.func,
+    render: PropTypes.func,
+    component: PropTypes.func
+  }
+}
+
 export default Match

modules/MatchProvider.js

@@ -4,11 +4,6 @@ import {
 } from './PropTypes'
 
 class MatchProvider extends React.Component {
-  static propTypes = {
-    match: PropTypes.any,
-    children: PropTypes.node
-  }
-
   static childContextTypes = {
     match: matchContextType.isRequired
   }
@@ -88,4 +83,11 @@ class MatchProvider extends React.Component {
   }
 }
 
+if (__DEV__) {
+  MatchProvider.propTypes = {
+    match: PropTypes.any,
+    children: PropTypes.node
+  }
+}
+
 export default MatchProvider

modules/MemoryRouter.js

@@ -25,15 +25,17 @@ const MemoryRouter = ({ getUserConfirmation, initialEntries, initialIndex, keyLe
   </MemoryHistory>
 )
 
-MemoryRouter.propTypes = {
-  getUserConfirmation: PropTypes.func,
-  initialEntries: PropTypes.array,
-  initialIndex: PropTypes.number,
-  keyLength: PropTypes.number,
-  children: PropTypes.oneOfType([
-    PropTypes.func,
-    PropTypes.node
-  ])
+if (__DEV__) {
+  MemoryRouter.propTypes = {
+    getUserConfirmation: PropTypes.func,
+    initialEntries: PropTypes.array,
+    initialIndex: PropTypes.number,
+    keyLength: PropTypes.number,
+    children: PropTypes.oneOfType([
+      PropTypes.func,
+      PropTypes.node
+    ])
+  }
 }
 
 export default MemoryRouter

modules/Miss.js

@@ -2,20 +2,12 @@ import React, { PropTypes } from 'react'
 import { location as locationType } from './PropTypes'
 
 class Miss extends React.Component {
-  static propTypes = {
-    children: PropTypes.node,
-    location: locationType,
-    render: PropTypes.func,
-    component: PropTypes.func
-  }
-
   static contextTypes = {
     match: PropTypes.object,
     location: PropTypes.object,
     serverRouter: PropTypes.object
   }
 
-
   constructor(props, context) {
     super(props, context)
 
@@ -67,4 +59,13 @@ class Miss extends React.Component {
   }
 }
 
+if (__DEV__) {
+  Miss.propTypes = {
+    children: PropTypes.node,
+    location: locationType,
+    render: PropTypes.func,
+    component: PropTypes.func
+  }
+}
+
 export default Miss

modules/Redirect.js

@@ -4,13 +4,6 @@ import {
 } from './PropTypes'
 
 class Redirect extends React.Component {
-  static propTypes = {
-    to: PropTypes.oneOfType([
-      PropTypes.string,
-      PropTypes.object
-    ]).isRequired
-  }
-
   static contextTypes = {
     router: routerContextType,
     serverRouter: PropTypes.object
@@ -37,4 +30,13 @@ class Redirect extends React.Component {
   }
 }
 
+if (__DEV__) {
+  Redirect.propTypes = {
+    to: PropTypes.oneOfType([
+      PropTypes.string,
+      PropTypes.object
+    ]).isRequired
+  }
+}
+
 export default Redirect

modules/ServerRouter.js

@@ -2,16 +2,6 @@ import React, { PropTypes } from 'react'
 import StaticRouter from './StaticRouter'
 
 class ServerRouter extends React.Component {
-
-  static propTypes = {
-    context: PropTypes.object.isRequired,
-    location: PropTypes.string.isRequired,
-    children: PropTypes.oneOfType([
-      PropTypes.func,
-      PropTypes.node
-    ])
-  }
-
   static childContextTypes = {
     serverRouter: PropTypes.object.isRequired
   }
@@ -39,4 +29,15 @@ class ServerRouter extends React.Component {
   }
 }
 
+if (__DEV__) {
+  ServerRouter.propTypes = {
+    context: PropTypes.object.isRequired,
+    location: PropTypes.string.isRequired,
+    children: PropTypes.oneOfType([
+      PropTypes.func,
+      PropTypes.node
+    ])
+  }
+}
+
 export default ServerRouter

modules/StaticRouter.js

@@ -17,23 +17,6 @@ const stringifyQuery = (query) => (
 )
 
 class StaticRouter extends React.Component {
-  static propTypes = {
-    children: PropTypes.oneOfType([ PropTypes.node, PropTypes.func ]),
-
-    action: actionType.isRequired,
-    location: PropTypes.oneOfType([ PropTypes.object, PropTypes.string ]).isRequired,
-
-    onPush: PropTypes.func.isRequired,
-    onReplace: PropTypes.func.isRequired,
-    blockTransitions: PropTypes.func,
-
-    stringifyQuery: PropTypes.func.isRequired,
-    parseQueryString: PropTypes.func.isRequired,
-    createHref: PropTypes.func.isRequired, // TODO: Clarify why this is useful
-
-    basename: PropTypes.string // TODO: Feels like we should be able to remove this
-  }
-
   static defaultProps = {
     stringifyQuery,
     parseQueryString,
@@ -120,4 +103,23 @@ class StaticRouter extends React.Component {
   }
 }
 
+if (__DEV__) {
+  StaticRouter.propTypes = {
+    children: PropTypes.oneOfType([ PropTypes.node, PropTypes.func ]),
+
+    action: actionType.isRequired,
+    location: PropTypes.oneOfType([ PropTypes.object, PropTypes.string ]).isRequired,
+
+    onPush: PropTypes.func.isRequired,
+    onReplace: PropTypes.func.isRequired,
+    blockTransitions: PropTypes.func,
+
+    stringifyQuery: PropTypes.func.isRequired,
+    parseQueryString: PropTypes.func.isRequired,
+    createHref: PropTypes.func.isRequired, // TODO: Clarify why this is useful
+
+    basename: PropTypes.string // TODO: Feels like we should be able to remove this
+  }
+}
+
 export default StaticRouter

package.json

@@ -39,6 +39,7 @@
     "babel-core": "^6.9.1",
     "babel-eslint": "^6.0.4",
     "babel-loader": "^6.2.4",
+    "babel-plugin-dev-expression": "^0.2.1",
     "babel-preset-es2015": "^6.14.0",
     "babel-preset-es2015-loose": "^8.0.0",
     "babel-preset-react": "^6.5.0",