Skip to content

Commit 583fa97

Browse files
committed
fix: resolve TypeScript build errors
- Remove unused redactSensitiveData function - Fix pino logger initialization - Add proper type annotations - Clean up logger configuration
1 parent 22f4f4e commit 583fa97

File tree

9 files changed

+43
-92
lines changed

9 files changed

+43
-92
lines changed

dist/cjs/utils/logger.d.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,16 @@
33
* @description Logger utility for the application
44
*/
55
import pino from 'pino';
6+
import { Logger } from 'pino';
67
/**
7-
* Creates a logger instance with the given name
8+
* Creates a logger instance with the specified name
89
* @param name - Name for the logger instance
910
* @returns Pino logger instance
1011
*/
11-
export declare function createLogger(name: string): pino.Logger<never>;
12+
export declare function createLogger(name: string): Logger;
1213
/**
1314
* Default logger instance for the library
1415
*/
15-
export declare const logger: pino.Logger<never>;
16+
export declare const logger: pino.Logger;
1617
export type { Logger } from 'pino';
1718
//# sourceMappingURL=logger.d.ts.map

dist/cjs/utils/logger.d.ts.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/cjs/utils/logger.js

Lines changed: 12 additions & 28 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/cjs/utils/logger.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/utils/logger.d.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,16 @@
33
* @description Logger utility for the application
44
*/
55
import pino from 'pino';
6+
import { Logger } from 'pino';
67
/**
7-
* Creates a logger instance with the given name
8+
* Creates a logger instance with the specified name
89
* @param name - Name for the logger instance
910
* @returns Pino logger instance
1011
*/
11-
export declare function createLogger(name: string): pino.Logger<never>;
12+
export declare function createLogger(name: string): Logger;
1213
/**
1314
* Default logger instance for the library
1415
*/
15-
export declare const logger: pino.Logger<never>;
16+
export declare const logger: pino.Logger;
1617
export type { Logger } from 'pino';
1718
//# sourceMappingURL=logger.d.ts.map

dist/utils/logger.d.ts.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/utils/logger.js

Lines changed: 12 additions & 28 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/utils/logger.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/utils/logger.ts

Lines changed: 7 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -4,36 +4,15 @@
44
*/
55

66
import pino from 'pino';
7+
import { Logger } from 'pino';
78

89
/**
9-
* Redacts sensitive data from objects before logging
10-
* @param obj - Object to redact
11-
* @param sensitiveKeys - Array of sensitive keys to redact
12-
* @returns Redacted object
13-
*/
14-
function redactSensitiveData(obj: any, sensitiveKeys: string[] = ['privateKey', 'secret', 'password', 'token']): any {
15-
if (typeof obj !== 'object' || obj === null) {
16-
return obj;
17-
}
18-
19-
const redacted = { ...obj };
20-
for (const key in redacted) {
21-
if (sensitiveKeys.includes(key)) {
22-
redacted[key] = '[REDACTED]';
23-
} else if (typeof redacted[key] === 'object') {
24-
redacted[key] = redactSensitiveData(redacted[key], sensitiveKeys);
25-
}
26-
}
27-
return redacted;
28-
}
29-
30-
/**
31-
* Creates a logger instance with the given name
10+
* Creates a logger instance with the specified name
3211
* @param name - Name for the logger instance
3312
* @returns Pino logger instance
3413
*/
35-
export function createLogger(name: string) {
36-
return pino({
14+
export function createLogger(name: string): Logger {
15+
const logger = pino.default({
3716
name,
3817
level: process.env.LOG_LEVEL || 'info',
3918
redact: {
@@ -42,7 +21,7 @@ export function createLogger(name: string) {
4221
},
4322
timestamp: () => `,"time":"${new Date(Date.now()).toISOString()}"`,
4423
formatters: {
45-
level: (label) => {
24+
level: (label: string) => {
4625
return { level: label };
4726
},
4827
},
@@ -53,6 +32,8 @@ export function createLogger(name: string) {
5332
},
5433
},
5534
});
35+
36+
return logger;
5637
}
5738

5839
/**

0 commit comments

Comments
 (0)