response to code review

Author: petehunt

grunt/config/browserify.js

@@ -5,6 +5,7 @@
 
 var envify = require('envify/custom');
 var grunt = require('grunt');
+var recast = require('recast');
 var UglifyJS = require('uglify-js');
 
 var SIMPLE_TEMPLATE =
@@ -39,7 +40,16 @@ function minify(src) {
 // allow browserified libraries to work with requirejs and
 // other pacakgers that get confused by these calls.
 function muffinize(src) {
-  return src.replace(/require/g, 'muffin');
+  return recast.print(
+    recast.types.traverse(recast.parse(src), muffinizeVisitor)
+  );
+}
+
+function muffinizeVisitor(node, traverse) {
+  if (recast.namedTypes.Identifier.check(node) &&
+      node.name === 'require') {
+    node.name = 'muffin';
+  }
 }
 
 // TODO: move this out to another build step maybe.

npm-react-core/README.md

@@ -1,8 +1,8 @@
 # react-core
 
-An npm package to get you immediate access to `React`, without also requiring
-the JSX transformer. This is especially useful for cases where you want to
-[`browserify`](https://github.com/substack/node-browserify) your module using
+An npm package to get you immediate access to [React](http://facebook.github.io/react/),
+without also requiring the JSX transformer. This is especially useful for cases where you
+want to [`browserify`](https://github.com/substack/node-browserify) your module using
 `React`.
 
 ## Example Usage

vendor/constants.js

@@ -57,7 +57,7 @@ function transform(ast, constants) {
       // There could in principle be a constant called "hasOwnProperty",
       // so be careful always to use Object.prototype.hasOwnProperty.
       if (node.name === '__DEV__') {
-        // replace __DEV__ with process.env.NODE_ENV === 'dev'
+        // replace __DEV__ with process.env.NODE_ENV !== 'production'
         this.replace(DEV_EXPRESSION);
         return false;
       } else if (hasOwn.call(constants, node.name)) {