Skip to content

feat(chat): display timestamp on API request messages - #1238

Closed
mazg1987 wants to merge 1 commit into
Zoo-Code-Org:mainfrom
mazg1987:feat/chat-message-timestamp
Closed

feat(chat): display timestamp on API request messages#1238
mazg1987 wants to merge 1 commit into
Zoo-Code-Org:mainfrom
mazg1987:feat/chat-message-timestamp

Conversation

@mazg1987

@mazg1987 mazg1987 commented Aug 13, 2026

Copy link
Copy Markdown

Related GitHub Issue

Closes: #

Description

Test Procedure

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Testing: New and/or updated tests have been added to cover my changes (if applicable).
  • Visual Snapshot (UI changes only): If a user would notice this change at a glance (layout, theme tokens, brand elements, empty/error states), I've added or updated a *.visual.tsx snapshot in webview-ui/. See webview-ui/AGENTS.md → "When a UI change needs a snapshot".
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Visual Snapshots

Videos (interaction / animation only)

Documentation Updates

Additional Notes

Get in Touch

Summary by CodeRabbit

  • New Features

    • Chat messages now display the API request time alongside the request status.
    • Times are shown in a localized format with hours, minutes, and seconds.
  • Tests

    • Added coverage to verify time formatting across supported clock formats.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 32d3f5f3-c327-4070-ae02-d2e27342ba98

📥 Commits

Reviewing files that changed from the base of the PR and between d4023d1 and b36832f.

📒 Files selected for processing (3)
  • webview-ui/src/components/chat/ChatRow.tsx
  • webview-ui/src/utils/__tests__/format.spec.ts
  • webview-ui/src/utils/format.ts

📝 Walkthrough

Walkthrough

The change adds a localized formatTime utility, tests its output, and displays formatted API request timestamps in chat rows.

Changes

Timestamp Display

Layer / File(s) Summary
Add and test timestamp formatting
webview-ui/src/utils/format.ts, webview-ui/src/utils/__tests__/format.spec.ts
Adds the exported formatTime utility and tests localized hour, minute, and second output.
Render request timestamps
webview-ui/src/components/chat/ChatRow.tsx
Displays the formatted API request timestamp beside the request status.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested labels: awaiting-review

Suggested reviewers: hannesrudolph

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

webview-ui/src/components/chat/ChatRow.tsx

ESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox.

webview-ui/src/utils/__tests__/format.spec.ts

ESLint skipped: the ESLint configuration for this file references a package that is not available in the sandbox.

webview-ui/src/utils/format.ts

ESLint skipped: the ESLint configuration for this file references a package that is not available in the sandbox.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
webview-ui/src/utils/format.ts 75.00% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@mazg1987
mazg1987 deleted the feat/chat-message-timestamp branch August 13, 2026 00:17
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.

1 participant