@taicode/common-base 4.0.0-beta.3 → 4.0.0-beta.5

package/output/disposer/disposer.d.ts

@@ -1,6 +1,24 @@
 type CleanupFunction = () => unknown | Promise<unknown>;
+/**
+ * 清理失败时的日志出口（结构性类型）。
+ *
+ * 既兼容 base 全局 logger 的可变参形式 `(...args: unknown[])`，
+ * 也兼容 pino 的 `log.error(obj, msg)`；调用方传入应用自己的 logger 后，
+ * 失败日志就不会落进 console，而是进应用日志流。
+ */
+export interface DisposerLogSink {
+    error(payload: unknown, message?: string): void;
+}
+export interface DisposerOptions {
+    /** 失败日志出口；不传则回退到 base 全局 logger */
+    log?: DisposerLogSink;
+    /** 日志标签，用于区分是哪个阶段的 disposer（缺省 `disposer`） */
+    label?: string;
+}
 export declare class Disposer {
+    private readonly opts;
     private cleanupFunctions;
+    constructor(opts?: DisposerOptions);
     add(cleanupFn: CleanupFunction): void;
     dispose(): Promise<void>;
 }

package/output/disposer/disposer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"disposer.d.ts","sourceRoot":"","sources":["../../source/disposer/disposer.ts"],"names":[],"mappings":"AAEA,KAAK,eAAe,GAAG,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,CAAA;AAEvD,qBAAa,QAAQ;IACnB,OAAO,CAAC,gBAAgB,CAAwB;IAGhD,GAAG,CAAC,SAAS,EAAE,eAAe,GAAG,IAAI;IAK/B,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;CAa/B"}
+{"version":3,"file":"disposer.d.ts","sourceRoot":"","sources":["../../source/disposer/disposer.ts"],"names":[],"mappings":"AAEA,KAAK,eAAe,GAAG,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,CAAA;AAEvD;;;;;;GAMG;AACH,MAAM,WAAW,eAAe;IAC9B,KAAK,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;CAChD;AAED,MAAM,WAAW,eAAe;IAC9B,mCAAmC;IACnC,GAAG,CAAC,EAAE,eAAe,CAAA;IACrB,8CAA8C;IAC9C,KAAK,CAAC,EAAE,MAAM,CAAA;CACf;AAED,qBAAa,QAAQ;IAGP,OAAO,CAAC,QAAQ,CAAC,IAAI;IAFjC,OAAO,CAAC,gBAAgB,CAAwB;gBAEnB,IAAI,GAAE,eAAoB;IAGvD,GAAG,CAAC,SAAS,EAAE,eAAe,GAAG,IAAI;IAK/B,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;CAgB/B"}

package/output/disposer/disposer.js

@@ -1,6 +1,10 @@
 import { logger } from '../logger/index.js';
 export class Disposer {
+    opts;
     cleanupFunctions = [];
+    constructor(opts = {}) {
+        this.opts = opts;
+    }
     // 添加清理函数
     add(cleanupFn) {
         this.cleanupFunctions.push(cleanupFn);
@@ -15,7 +19,10 @@ export class Disposer {
                 await cleanupFn();
             }
             catch (error) {
-                logger.error('Error during cleanup:', error);
+                // 单个清理失败不阻断后续，也不向调用方抛出
+                const label = this.opts.label ?? 'disposer';
+                const sink = this.opts.log ?? logger;
+                sink.error({ error }, `[${label}] cleanup failed`);
             }
         }
     }

package/output/disposer/disposer.test.js

@@ -98,7 +98,7 @@ describe('Disposer', () => {
             expect(errorFn).toHaveBeenCalled();
             expect(workingFn2).toHaveBeenCalled();
             // 错误应该被记录
-            expect(loggerErrorSpy).toHaveBeenCalledWith('Error during cleanup:', expect.any(Error));
+            expect(loggerErrorSpy).toHaveBeenCalledWith({ error: expect.any(Error) }, '[disposer] cleanup failed');
             loggerErrorSpy.mockRestore();
         });
         it('应该在执行后清空清理函数列表', async () => {
@@ -132,6 +132,54 @@ describe('Disposer', () => {
             expect(cleanupFn).toHaveBeenCalledOnce();
         });
     });
+    describe('日志注入', () => {
+        it('应该在传入 log 时使用注入的日志出口，不再走 base logger', async () => {
+            const loggerErrorSpy = vi.spyOn(logger, 'error').mockImplementation(() => { });
+            const logError = vi.fn();
+            const error = new Error('Injected log error');
+            const injected = new Disposer({ log: { error: logError }, label: 'resources' });
+            injected.add(() => {
+                throw error;
+            });
+            await expect(injected.dispose()).resolves.toBeUndefined();
+            expect(logError).toHaveBeenCalledOnce();
+            expect(logError).toHaveBeenCalledWith({ error }, '[resources] cleanup failed');
+            expect(loggerErrorSpy).not.toHaveBeenCalled();
+            loggerErrorSpy.mockRestore();
+        });
+        it('标签应该出现在消息里（注入 log 且自定义 label）', async () => {
+            const logError = vi.fn();
+            const injected = new Disposer({ log: { error: logError }, label: 'middlewares' });
+            injected.add(() => {
+                throw new Error('boom');
+            });
+            await injected.dispose();
+            expect(logError).toHaveBeenCalledWith({ error: expect.any(Error) }, '[middlewares] cleanup failed');
+        });
+        it('未传 log 时应该回退到 base logger', async () => {
+            const loggerErrorSpy = vi.spyOn(logger, 'error').mockImplementation(() => { });
+            const error = new Error('Fallback error');
+            const noLog = new Disposer({ label: 'routers' });
+            noLog.add(() => {
+                throw error;
+            });
+            await noLog.dispose();
+            expect(loggerErrorSpy).toHaveBeenCalledWith({ error }, '[routers] cleanup failed');
+            loggerErrorSpy.mockRestore();
+        });
+        it('传入 log 时单个失败仍不阻断后续，且全部走注入出口', async () => {
+            const logError = vi.fn();
+            const first = vi.fn();
+            const injected = new Disposer({ log: { error: logError }, label: 'services' });
+            injected.add(first);
+            injected.add(() => {
+                throw new Error('boom');
+            });
+            await injected.dispose();
+            expect(first).toHaveBeenCalledOnce();
+            expect(logError).toHaveBeenCalledOnce();
+        });
+    });
     describe('实际使用场景', () => {
         it('应该能够清理事件监听器', async () => {
             const removeEventListener = vi.fn();
@@ -185,7 +233,7 @@ describe('Disposer', () => {
                 throw error;
             });
             await disposer.dispose();
-            expect(loggerErrorSpy).toHaveBeenCalledWith('Error during cleanup:', error);
+            expect(loggerErrorSpy).toHaveBeenCalledWith({ error }, '[disposer] cleanup failed');
             loggerErrorSpy.mockRestore();
         });
         it('应该处理不同类型的错误', async () => {
@@ -210,7 +258,7 @@ describe('Disposer', () => {
                 throw asyncError;
             });
             await disposer.dispose();
-            expect(loggerErrorSpy).toHaveBeenCalledWith('Error during cleanup:', asyncError);
+            expect(loggerErrorSpy).toHaveBeenCalledWith({ error: asyncError }, '[disposer] cleanup failed');
             loggerErrorSpy.mockRestore();
         });
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@taicode/common-base",
-  "version": "4.0.0-beta.3",
+  "version": "4.0.0-beta.5",
   "author": "Alain",
   "license": "ISC",
   "description": "",
@@ -39,7 +39,7 @@
     "test:ci": "vitest run",
     "lint": "eslint && tsc --noEmit",
     "typecheck": "tsc --noEmit",
-    "build": "tsc -p tsconfig.json",
+    "build": "rm -rf output && tsc -p tsconfig.json",
     "dev": "tsc -p tsconfig.json --watch"
   }
 }