@jiangdyjp2/common 1.0.4 → 1.0.6

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.4",
+  "version": "1.0.6",
   "description": "",
   "main": "./build/index.js",
   "types": "./build/index.d.ts",

package/readme.txt

@@ -50,3 +50,55 @@
     update src/index.ts:
       import './types/express.js';
 
+
+(9) The core principle (very important)
+
+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
+
+