fix: remove "browser" field in package.json (#424)

* fix: remove "browser" field in package.json

* fix: esm build

Author: antfu

package.json

@@ -13,7 +13,6 @@
   },
   "main": "index.js",
   "module": "dist/vue-composition-api.esm.js",
-  "browser": "dist/vue-composition-api.prod.js",
   "unpkg": "dist/vue-composition-ap.prod.js",
   "jsdelivr": "dist/vue-composition-api.prod.js",
   "typings": "dist/index.d.ts",

rollup.config.js

@@ -69,9 +69,12 @@ function genConfig({ outFile, format, mode }) {
       }),
       resolve(),
       replace({
-        'process.env.NODE_ENV': JSON.stringify(
-          isProd ? 'production' : 'development'
-        ),
+        'process.env.NODE_ENV':
+          format === 'es'
+            ? // preserve to be handled by bundlers
+              'process.env.NODE_ENV'
+            : // hard coded dev/prod builds
+              JSON.stringify(isProd ? 'production' : 'development'),
         __DEV__:
           format === 'es'
             ? // preserve to be handled by bundlers

src/apis/inject.ts

@@ -53,7 +53,7 @@ export function inject(
     if (val !== NOT_FOUND) {
       return val
     } else {
-      if (defaultValue === undefined && process.env.NODE_ENV !== 'production') {
+      if (defaultValue === undefined && __DEV__) {
         warn(`Injection "${String(key)}" not found`, vm)
       }
       return defaultValue