Merge pull request #42 from Navigraph/build/bundle-axios

Author: SkySails

.changeset/weak-geese-swim.md

@@ -0,0 +1,8 @@
+---
+"@navigraph/app": patch
+"@navigraph/auth": patch
+"@navigraph/charts": patch
+"navigraph": patch
+---
+
+Included dependencies like `axios` and `@navigraph/pkce` in the final bundle. Also bumped the target to ES6, which should fix some warnings when using Rollup related to usage of global `this`.

package.json

@@ -33,9 +33,9 @@
     "jest-environment-jsdom": "^29.2.2",
     "prettier": "^2.7.1",
     "rimraf": "^3.0.2",
-    "ts-jest": "^29.0.3",
+    "ts-jest": "^29.0.5",
     "tsup": "^6.3.0",
-    "turbo": "^1.2.16",
-    "typescript": "^4.7.4"
+    "turbo": "^1.8.5",
+    "typescript": "^5.0.2"
   }
 }

packages/auth/package.json

@@ -39,11 +39,10 @@
     "postpack": "clean-package restore"
   },
   "dependencies": {
-    "@navigraph/pkce": "^1.0.3",
-    "@navigraph/app": "1.3.0",
-    "axios": "^0.24.0"
+    "@navigraph/app": "1.3.0"
   },
   "devDependencies": {
-    "@types/qr-image": "^3.2.5"
+    "@navigraph/pkce": "^1.0.3",
+    "axios": "^0.24.0"
   }
 }

tsup.config.ts

@@ -3,8 +3,7 @@ import { defineConfig } from "tsup";
 export default defineConfig({
   clean: true,
   format: ["cjs", "esm"],
-  target: "es5",
-  outExtension(ctx) {
-    return { js: `.${ctx.format}.js` };
-  },
+  target: "es6",
+  noExternal: ["axios", "@navigraph/pkce"],
+  outExtension: (ctx) => ({ js: `.${ctx.format}.js` }),
 });