fix(web): keep tool timestamps before disclosure chevrons

[Yash-Singh1]

What Changed

Moved tool-row timestamps before trailing disclosure chevrons so revealing a timestamp does not shift the chevron.

Why

The timestamp previously rendered as the rightmost flex child, which could move disclosure controls when it appeared. This keeps the chevron position stable.

UI Changes

The same git diff --check tool row is hovered in both screenshots.

Before

After

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Summary by CodeRabbit

• UI Improvements
  • Adjusted work entry rows so timestamps appear before the disclosure chevron.
  • Timestamp formatting and display behavior remain unchanged.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 775e0de5-15b1-43c0-a161-6d0b2df31d2a

📥 Commits

Reviewing files that changed from the base of the PR and between dda44ed and 561c329.

📒 Files selected for processing (1)

• apps/web/src/components/chat/MessagesTimeline.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

PlainWorkEntryRow now renders TimelineRowTimestamp before the trailing disclosure chevron. The timestamp display behavior remains unchanged. The related documentation now records this placement convention.

Changes

Timeline timestamp placement

Layer / File(s)	Summary
Place timestamp before disclosure control apps/web/src/components/chat/MessagesTimeline.tsx	The documentation now requires timestamps to appear before trailing disclosure controls. PlainWorkEntryRow moves the existing timestamp rendering before the chevron.

Priority: ⬇️ Low

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

Change: Bug fix

Suggested reviewers: t3dotgg

Merge Risk: ⚪ Minimal · up to 561c3

The timestamp placement change has no established functional or availability risk and is ready to merge.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description check	✅ Passed	The description clearly states what changed and why. It includes before-and-after screenshots for the UI change and completes the relevant checklist items. A video is not required because the change d…
Title check	✅ Passed	The title clearly and concisely describes the main change: keeping tool timestamps before disclosure chevrons.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch t3code/move-tool-time-before-chevron

---

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

[macroscopeapp]

Approvability

Verdict: Approved at 561c329

Macroscope's review found this PR approvable — This is a small, self-contained UI bug fix that reorders an existing timestamp so hover/focus disclosure does not move the chevron. It changes no data contracts, defaults, infrastructure, or static-analysis settings.

^{You can add or adjust custom eligibility rules. Learn more.}

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.6 KiB	13.6 KiB	−41 B (−0.3%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.1 KiB	7.1 KiB	−2 B (−0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.5 KiB	−39 B (−0.6%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.1 KiB	57.0 KiB	−44 B (−0.1%)	66.4 KiB	✅
Codex	Live turn messages	10	9	−1 (−10.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.6 KiB	+18 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.1 KiB	7.1 KiB	−3 B (−0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	+21 B (+0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.8 KiB	+44 B (+0.1%)	66.4 KiB	✅
Claude	Live turn messages	8	9	+1 (+12.5%)	21	✅

Baseline: dda44ed · PR result: 561c329 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.9 KiB
• Claude decoded thread snapshot: 114.6 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.