treeshake and skipNodeModulesBundle (#402)

* treeshake and skipNodeModulesBundle

* add to CHANGELOG

* tsup clean is not clean enough

* notes for future self

* Apply suggestions from code review

Co-authored-by: Nayyir Jutha <nayyir.jutha@gmail.com>

* remove watch because we arent a app

---------

Co-authored-by: mix irving <mix@protozoa.nz>
Co-authored-by: Nayyir Jutha <nayyir.jutha@gmail.com>

CHANGELOG.md

@@ -21,7 +21,7 @@ Version header format: `[version] Name - year-month-day (entropy-core compatibil
 ### Broke
 
 ### Dev
-
+ - no longer building node modules into dist
 ### Meta
 
 

index.cjs

@@ -1 +1,6 @@
+// this is here to make the exports less "messy"
+// and also if you dont know about package.json you can
+// still import or require see test tests/require.test.cjs
+// next time you look at the build system
+// ask your self is node done with require?
 module.exports = require('./dist/index.cjs')

package.json

@@ -53,8 +53,7 @@
     "lint:eslint": "eslint . --ext .ts --fix",
     "peek": "tsc --build --clean",
     "burn": "rm -Rf node_modules && yarn cache clean --all && yarn install",
-    "build": "tsup",
-    "build:watch": "tsup --watch",
+    "build": "rm -rf dist && tsup",
     "build:node": "tsup --platform node ",
     "build:browser": "tsup --platform browser",
     "build:link": "yarn build && yarn unlink && yarn link",

tsup.config.ts

@@ -9,6 +9,8 @@ export default defineConfig((options) => {
       'src/utils/index.ts',
       'src/utils/crypto/index.ts',
     ],
+    treeshake: true,
+    skipNodeModulesBundle: true,
     replaceNodeEnv: true,
     format: ['esm', 'cjs'],
     dts: true,
@@ -17,6 +19,5 @@ export default defineConfig((options) => {
     clean: true,
     target: 'es2022',
     minify: options.minify,
-    watch: options.watch,
   }
 })