Skip to content

Commit 82effcb

Browse files
alban bertoliniclaude
andcommitted
refactor(workflow-executor): remove redundant level field from ConsoleLogger
console.log (info) vs console.error (error) already distinguishes the level via stdout/stderr. The JSON level field is redundant — users who need it can implement their own Logger. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 31cdb47 commit 82effcb

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

packages/workflow-executor/src/adapters/console-logger.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,10 @@ import type { Logger } from '../ports/logger-port';
33
export default class ConsoleLogger implements Logger {
44
info(message: string, context: Record<string, unknown>): void {
55
// eslint-disable-next-line no-console
6-
console.log(
7-
JSON.stringify({ level: 'info', message, timestamp: new Date().toISOString(), ...context }),
8-
);
6+
console.log(JSON.stringify({ message, timestamp: new Date().toISOString(), ...context }));
97
}
108

119
error(message: string, context: Record<string, unknown>): void {
12-
console.error(
13-
JSON.stringify({ level: 'error', message, timestamp: new Date().toISOString(), ...context }),
14-
);
10+
console.error(JSON.stringify({ message, timestamp: new Date().toISOString(), ...context }));
1511
}
1612
}

packages/workflow-executor/test/adapters/console-logger.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ describe('ConsoleLogger', () => {
1414

1515
expect(spy).toHaveBeenCalledTimes(1);
1616
const output = JSON.parse(spy.mock.calls[0][0]);
17-
expect(output).toMatchObject({ level: 'info', message: 'test message', key: 'value' });
17+
expect(output).toMatchObject({ message: 'test message', key: 'value' });
1818
expect(output.timestamp).toBeDefined();
1919

2020
spy.mockRestore();

0 commit comments

Comments
 (0)