@jiangdyjp2/common 1.0.5 → 1.0.9

package/build/index.d.ts

@@ -1,4 +1,3 @@
-import './types/express.js';
 export * from './errors/bad-request-error.js';
 export * from './errors/custom-error.js';
 export * from './errors/database-connection-error.js';

package/build/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,oBAAoB,CAAC;AAE5B,cAAc,+BAA+B,CAAC;AAC9C,cAAc,0BAA0B,CAAA;AACxC,cAAc,uCAAuC,CAAC;AACtD,cAAc,kCAAkC,CAAC;AACjD,cAAc,6BAA6B,CAAC;AAC5C,cAAc,sCAAsC,CAAC;AAErD,cAAc,+BAA+B,CAAC;AAC9C,cAAc,gCAAgC,CAAC;AAC/C,cAAc,+BAA+B,CAAC;AAC9C,cAAc,mCAAmC,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAEA,cAAc,+BAA+B,CAAC;AAC9C,cAAc,0BAA0B,CAAA;AACxC,cAAc,uCAAuC,CAAC;AACtD,cAAc,kCAAkC,CAAC;AACjD,cAAc,6BAA6B,CAAC;AAC5C,cAAc,sCAAsC,CAAC;AAErD,cAAc,+BAA+B,CAAC;AAC9C,cAAc,gCAAgC,CAAC;AAC/C,cAAc,+BAA+B,CAAC;AAC9C,cAAc,mCAAmC,CAAC"}

package/build/index.js

@@ -1,4 +1,4 @@
-import './types/express.js';
+// import './types/express.js';
 export * from './errors/bad-request-error.js';
 export * from './errors/custom-error.js';
 export * from './errors/database-connection-error.js';

package/build/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,oBAAoB,CAAC;AAE5B,cAAc,+BAA+B,CAAC;AAC9C,cAAc,0BAA0B,CAAA;AACxC,cAAc,uCAAuC,CAAC;AACtD,cAAc,kCAAkC,CAAC;AACjD,cAAc,6BAA6B,CAAC;AAC5C,cAAc,sCAAsC,CAAC;AAErD,cAAc,+BAA+B,CAAC;AAC9C,cAAc,gCAAgC,CAAC;AAC/C,cAAc,+BAA+B,CAAC;AAC9C,cAAc,mCAAmC,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,+BAA+B;AAE/B,cAAc,+BAA+B,CAAC;AAC9C,cAAc,0BAA0B,CAAA;AACxC,cAAc,uCAAuC,CAAC;AACtD,cAAc,kCAAkC,CAAC;AACjD,cAAc,6BAA6B,CAAC;AAC5C,cAAc,sCAAsC,CAAC;AAErD,cAAc,+BAA+B,CAAC;AAC9C,cAAc,gCAAgC,CAAC;AAC/C,cAAc,+BAA+B,CAAC;AAC9C,cAAc,mCAAmC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jiangdyjp2/common",
-  "version": "1.0.5",
+  "version": "1.0.9",
   "description": "",
   "main": "./build/index.js",
   "types": "./build/index.d.ts",
@@ -8,26 +8,28 @@
     "build/**/*"
   ],
   "scripts": {
-    "clean": "del ./build/*",
-    "build": "npm run clean && tsc",
-    "pub": "git add . && git commit -m \"Updates\" && npm version patch && npm run build && npm publish"
+    "clean": "rimraf build",
+    "build": "tsc",
+    "prepublishOnly": "npm run clean && npm run build",
+    "pub": "git add . && git commit -m \"release\" && npm version patch && npm publish"
   },
   "keywords": [],
   "author": "",
   "license": "ISC",
   "type": "module",
   "devDependencies": {
+    "@types/jsonwebtoken": "^9.0.10",
     "del-cli": "^7.0.0",
+    "rimraf": "^6.1.2",
     "typescript": "^5.9.3"
   },
   "peerDependencies": {
-    "express": "^5.2.0",
-    "@types/express": "^5.0.0",
+    "@types/cookie-session": "^2.0.0",
+    "@types/express": "^4.17.21 || ^5.0.0",
     "cookie-session": "^2.1.0",
-    "@types/cookie-session": "^2.0.0"
+    "express": "^4.19.2 || ^5.2.0"
   },
   "dependencies": {
-    "@types/jsonwebtoken": "^9.0.10",
     "express-validator": "^7.3.1",
     "jsonwebtoken": "^9.0.3"
   }

package/readme.txt

@@ -56,3 +56,59 @@
 Frameworks like express, react, vue, etc. should almost always be peerDependencies, not normal dependencies, in shared libraries.
 This rule applies whether the package is yours or a 3rd-party package.
 
+
+(10) use Vitest instead of using jest:
+
+If you use Vitest, you do NOT need to compile @jiangdyjp2/common to CommonJS.
+You can keep everything as ESM.
+Why this works (key point)
+
+Your current failure with Jest happens because:
+@jiangdyjp2/common is ESM
+Jest runs tests in CommonJS by default
+Jest tries to require() this file:
+
+import './types/express.js';
+^^^^^^
+→ 💥 boom
+
+Why Vitest fixes this automatically
+Vitest is built on Vite + native ESM:
+Runs Node in ESM mode
+Understands import in node_modules
+No transpiling hacks
+No transformIgnorePatterns
+No ts-jest
+No babel-jest
+
+So this line is perfectly fine under Vitest:
+
+import { currentUser } from '@jiangdyjp2/common';
+
+(11) replace jest with vitest:
+
+--Remove Jest-related packages
+npm uninstall jest ts-jest @types/jest
+
+--Install Vitest
+npm install -D vitest
+
+--Update package.json
+
+
+(11) .d.ts files are types-only and are NEVER imported at runtime.
+
+(12) rm -rf node_modules package-lock.json 
+    npm install
+
+(13) 
+✅ Final checklist for @jiangdyjp2/common
+
+✔ No @types/* in dependencies
+✔ Express + cookie-session in peerDependencies
+✔ Only runtime JS libs in dependencies
+✔ Express augmentation in .d.ts
+✔ No runtime import of type files
+
+(14) Libraries ( say @jiangdyjp2/common ) should never ship @types/* in dependencies
+