Skip to content

feat(cli): add optional message timestamps between exchanges - #2258

Closed
dieutx wants to merge 1 commit into
NousResearch:mainfrom
dieutx:feat/cli-message-timestamps
Closed

feat(cli): add optional message timestamps between exchanges#2258
dieutx wants to merge 1 commit into
NousResearch:mainfrom
dieutx:feat/cli-message-timestamps

Conversation

@dieutx

@dieutx dieutx commented Mar 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add display.show_timestamps config option (default: false)
  • When enabled, prepends HH:MM:SS to the separator line between user/assistant exchanges
  • Useful for tracking how long tasks take during long CLI sessions

Enable in ~/.hermes/config.yaml:

display:
  show_timestamps: true

Example output:

14:32:05 ────────────────────────────────────

Closes #1569

Test plan

  • Timestamps shown when show_timestamps: true in config
  • No change when config option is absent or false
  • Uses datetime.now().strftime("%H:%M:%S") for consistent formatting
  • Run pytest tests/ to confirm no regressions

Add a display.show_timestamps config option that prepends HH:MM:SS to
the separator line between user/assistant exchanges. Useful for tracking
how long tasks take during long sessions.

Enable in ~/.hermes/config.yaml:
  display:
    show_timestamps: true

Closes NousResearch#1569
@dieutx
dieutx force-pushed the feat/cli-message-timestamps branch from 4794f19 to 4aeffe7 Compare March 21, 2026 02:44
@teknium1

Copy link
Copy Markdown
Contributor

Closing — we've reviewed the batch of 30 PRs submitted in ~36 hours and found multiple duplicates of other contributors' work, at least one no-op (identical bytes to main), and no tests or filled checklists. We appreciate the intent, but please focus on quality over quantity: check for existing PRs before submitting, verify your diff is real, add tests, and fill out the PR template. If you have a specific fix you've tested and verified, we're happy to review it.

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.

[Feature]: timestamp every message sent to cli

2 participants