Skip to content

feat: add ISO timestamps to console logs#226

Closed
liyishuai wants to merge 1 commit intoEvoMap:mainfrom
liyishuai:fix/add-log-timestamps
Closed

feat: add ISO timestamps to console logs#226
liyishuai wants to merge 1 commit intoEvoMap:mainfrom
liyishuai:fix/add-log-timestamps

Conversation

@liyishuai
Copy link

This PR addresses issue #225 by monkey-patching the console methods (log, warn, error) to include standard ISO timestamps when running in --loop mode. This improves observability for long-running evolver instances.

Copilot AI review requested due to automatic review settings March 9, 2026 14:08
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses issue #225 by monkey-patching console.log, console.warn, and console.error in --loop (daemon) mode to prepend ISO timestamps to every log line, improving observability for long-running evolver instances.

Changes:

  • Console methods are overridden inside the if (isLoop) branch with timestamp-prefixing wrappers.
  • A helper function ts() is introduced to generate [ISO-timestamp] strings.

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

@autogame-17
Copy link
Collaborator

Thank you @liyishuai for this contribution! The changes have been reviewed and merged into the upstream codebase with full Co-authored-by attribution.

Your improvement will be included in the next public release. This also resolves issue #225.

If you have an EvoMap account, we would like to award you credits for this contribution. Please reply with your node_id or registered email so we can send the reward.

Closing this PR as the changes have been cherry-picked upstream (the public repo is a one-way publish from the private codebase). Thanks again!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants