Skip to content

fix(logger): remove timestamp from production log format - #396

Open
sjinks wants to merge 1 commit into
trunkfrom
fix/production-log-format
Open

fix(logger): remove timestamp from production log format#396
sjinks wants to merge 1 commit into
trunkfrom
fix/production-log-format

Conversation

@sjinks

@sjinks sjinks commented Jun 12, 2026

Copy link
Copy Markdown
Member

Description

This pull request updates the logging format in the src/logger module to remove timestamps from production log output. Production log lines will now omit the timestamp, resulting in a simpler and more consistent log structure.

Logging format changes:

  • The production log format in src/logger/index.ts no longer includes the timestamp in the output string; only the app, app_type, and the JSON-encoded log object are included.
  • The corresponding test in __tests__/logger.spec.ts is updated to match the new log output format without the timestamp.

Why

Production logs already have a timestamp. By removing the timestamp from the message itself, we make it easier to filter out uninteresting duplicate log lines.

Steps to Test

Unit tests have been updated.

@sjinks sjinks self-assigned this Jun 12, 2026
Copilot AI review requested due to automatic review settings June 12, 2026 00:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request updates the src/logger module’s production log line formatting to omit the inline timestamp (since production log aggregation already provides timestamps), and adjusts unit tests accordingly.

Changes:

  • Removed the timestamp prefix from production log output in src/logger/index.ts.
  • Updated the production logging test assertion in __tests__/logger.spec.ts to match the new output format.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/logger/index.ts Updates the production printf formatter to no longer prepend timestamp to log lines.
__tests__/logger.spec.ts Updates the production log output regex to match the new timestamp-free format.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants