[client] Make node-specific middleware be excluded from bundles (#164)

Author: rexxars

packages/@sanity/client/package.json

@@ -16,6 +16,10 @@
     "prepublish": "in-publish && npm run build || not-in-publish",
     "test": "NODE_ENV=test tape -r babel-register test/*.test.js"
   },
+  "browser": {
+    "./src/http/nodeMiddleware.js": "./src/http/browserMiddleware.js",
+    "./lib/http/nodeMiddleware.js": "./lib/http/browserMiddleware.js"
+  },
   "dependencies": {
     "@sanity/eventsource": "^0.111.5",
     "@sanity/generate-help-url": "^0.111.5",

packages/@sanity/client/src/http/browserMiddleware.js

@@ -0,0 +1 @@
+module.exports = []

packages/@sanity/client/src/http/nodeMiddleware.js

@@ -0,0 +1,10 @@
+const debug = require('get-it/lib/middleware/debug')
+const headers = require('get-it/lib/middleware/headers')
+const pkg = require('../../package.json')
+
+const middleware = [
+  debug({verbose: true, namespace: 'sanity:client'}),
+  headers({'User-Agent': `${pkg.name} ${pkg.version}`})
+]
+
+module.exports = middleware

packages/@sanity/client/src/http/request.js

@@ -20,25 +20,16 @@ const httpError = ({
   }
 })
 
-const middleware = [
+// Environment-specific middleware.
+const envSpecific = require('./nodeMiddleware')
+
+const middleware = envSpecific.concat([
   jsonRequest(),
   jsonResponse(),
   progress(),
   httpError,
   observable({implementation: SanityObservable})
-]
-
-// Node-specifics
-if (process.env.BROWSERIFY_ENV !== 'build') {
-  // Only include debug middleware in browsers
-  const debug = require('get-it/lib/middleware/debug')
-  middleware.unshift(debug({verbose: true, namespace: 'sanity:client'}))
-
-  // Assign user agent in node
-  const headers = require('get-it/lib/middleware/headers')
-  const pkg = require('../../package.json')
-  middleware.unshift(headers({'User-Agent': `${pkg.name} ${pkg.version}`}))
-}
+])
 
 const request = getIt(middleware)