feat: upgrade Bugsnag to v8 (#123)

* feat: tsup and bugsnag upgrade Signed-off-by: Žiga Strgar <ziga.strgar@gmail.com> * docs: Update readme Signed-off-by: Žiga Strgar <ziga.strgar@gmail.com> * fix: dist ignore Signed-off-by: Žiga Strgar <ziga.strgar@gmail.com> (cherry picked from commit fb9775c) --------- Signed-off-by: Žiga Strgar <ziga.strgar@gmail.com>
ZigaStrgar · Sep 24, 2024 · 12baa1d · 12baa1d
1 parent 747067c
commit 12baa1d
Show file tree

Hide file tree

Showing 9 changed files with 43 additions and 61 deletions.
diff --git a/README.md b/README.md
@@ -29,7 +29,7 @@ the request data to the error. That includes `body`, `query` and `params` which
 |         | Version                                                            |
 |---------|--------------------------------------------------------------------|
 | Fastify | `^4.x`                                                             |
-| Bugsnag | `^7.17.0`                                                          |
+| Bugsnag | `^8.0.0`                                                           |
 | Node    | <code>^18.12.1 &#124;&#124; ^20.10.0 &#124;&#124; >= 21.0.0</code> |
 
 ## Installation

diff --git a/fix-package-json.sh b/fix-package-json.sh
diff --git a/package.json b/package.json
@@ -2,24 +2,20 @@
   "name": "fastify-bugsnag",
   "version": "4.0.0",
   "description": "Fastify plugin for Bugsnag",
-  "main": "./dist/cjs/index.js",
-  "module": "./dist/esm/index.js",
-  "types": "./dist/cjs/index.d.ts",
+  "main": "./dist/index.js",
+  "module": "./dist/index.mjs",
+  "types": "./dist/index.d.ts",
   "exports": {
     ".": {
-      "import": "./dist/esm/index.js",
-      "require": "./dist/cjs/index.js"
+      "import": "./dist/index.mjs",
+      "require": "./dist/index.js"
     }
   },
   "scripts": {
-    "prebuild": "rimraf dist",
-    "build": "npm run build:cjs && npm run build:esm",
-    "postbuild": "./fix-package-json.sh",
-    "build:cjs": "tsc -p ./tsconfig.build.cjs.json",
-    "build:esm": "tsc -p ./tsconfig.build.esm.json",
+    "build": "tsup-node",
     "lint": "ts-standard --verbose | snazzy",
     "lint:fix": "ts-standard --fix | snazzy",
-    "prepublish": "npm run build",
+    "prepublishOnly": "npm run build",
     "postversion": "git push --tags && git push origin main && echo \"Successfully published new package version $npm_package_version\"",
     "preversion": "npm run lint && npm run test && npm run test:d && npm run build",
     "test": "nyc --reporter=lcov ts-mocha",
@@ -50,8 +46,8 @@
   },
   "homepage": "https://github.com/ZigaStrgar/fastify-bugsnag#readme",
   "dependencies": {
-    "@bugsnag/js": "^7.20.2",
-    "@bugsnag/node": "^7.17.0",
+    "@bugsnag/js": "^8.0.0",
+    "@bugsnag/node": "^8.0.0",
     "fastify-plugin": "^4.0.0"
   },
   "engines": {
@@ -72,15 +68,17 @@
     "ts-node": "^10.8.0",
     "ts-standard": "^12.0.0",
     "tsd": "^0.30.3",
-    "typescript": "~5.1.6"
+    "tsup": "^8.3.0",
+    "typescript": "~5.6"
   },
   "ts-standard": {
     "ignore": [
       "dist"
     ],
     "files": [
       "src/**/*.ts",
-      "test/**/*.ts"
+      "test/**/*.ts",
+      "tsup.config.ts"
     ]
   },
   "nyc": {
@@ -90,7 +88,9 @@
     ],
     "exclude": [
       "**/*.d.ts",
-      "test/**"
+      "test/**",
+      "tsup.config.ts",
+      "dist"
     ],
     "reporter": [
       "lcov"

diff --git a/src/index.ts b/src/index.ts
@@ -1,8 +1,8 @@
 import Bugsnag from '@bugsnag/js'
-import { FastifyInstance, FastifyReply, FastifyRequest } from 'fastify'
+import type { FastifyInstance, FastifyReply, FastifyRequest } from 'fastify'
 import fp from 'fastify-plugin'
 
-import { PluginOptions, RequestInfoWithMetadata, RequestMetadata } from './types.js'
+import type { PluginOptions, RequestInfoWithMetadata, RequestMetadata } from './types.js'
 
 function extractRequestMetadata (request: FastifyRequest): RequestMetadata {
   return {

diff --git a/src/types.ts b/src/types.ts
@@ -1,6 +1,6 @@
-import { Client, BrowserConfig } from '@bugsnag/js'
-import { NodeConfig } from '@bugsnag/node'
-import { FastifyRequest } from 'fastify'
+import type { Client, BrowserConfig } from '@bugsnag/js'
+import type { NodeConfig } from '@bugsnag/node'
+import type { FastifyRequest } from 'fastify'
 
 export type PluginOptions = NodeConfig | BrowserConfig
 

diff --git a/tsconfig.build.cjs.json b/tsconfig.build.cjs.json
diff --git a/tsconfig.build.esm.json b/tsconfig.build.esm.json
diff --git a/tsconfig.json b/tsconfig.json
@@ -1,18 +1,26 @@
 {
-  "extends": "@tsconfig/node18",
+  "extends": "@tsconfig/node18/tsconfig.json",
   "compilerOptions": {
+    "allowJs": true,
+    "moduleDetection": "force",
+    "isolatedModules": true,
+    "verbatimModuleSyntax": false,
+    "noUncheckedIndexedAccess": true,
+    "noImplicitOverride": true,
+    "module": "NodeNext",
+    "outDir": "dist",
+    "lib": ["es2022"],
     "declaration": true,
     "resolveJsonModule": true,
     "removeComments": true,
     "sourceMap": true,
     "noEmit": true,
-    "types": [
-      "node"
-    ]
+    "esModuleInterop": true
   },
   "include": [
     "src/**/*.ts",
-    "test/**/*.ts"
+    "test/**/*.ts",
+    "tsup.config.ts"
   ],
   "exclude": [
     "node_modules"

diff --git a/tsup.config.ts b/tsup.config.ts
@@ -0,0 +1,8 @@
+import { defineConfig } from 'tsup'
+
+export default defineConfig({
+  entry: ['src/index.ts', 'src/types.ts'],
+  clean: true,
+  dts: true,
+  format: ['cjs', 'esm']
+})