While an agent works, its thinking was buried in a one-line preview under a live row that kept changing. Tool calls were stacked in layers that were hard to read, and the fold was confusing. This PR makes the work log simple, close to how Codex shows it.
Read labels show the file name only. The full path shows when you open the row.
| Before | After | |
|---|---|---|
| While working | before, while working | after, while working |
| Finished, fold open | before, finished | after, finished |
streaming after a steering restart, or a local /feedback row sent while a thought streams, can hide or add one "Thinking" row. Both are rare, and nothing wrong is shown.Proof: vp run check:changed passes. Focused tests cover each grouping and "Thinking" rule, and each fix test fails without its fix. I recorded the free Replay (dev) showcase on web, on main and on this branch.
Rounds: 15
| Delegate | R1 | R2 | R3 | R4 | R5 | R6 | R7 | R8 | R9 | R10 | R11 | R12 | R13 | R14 | R15 |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Claude Opus 5.5 | 7/10 | 8/10 | 8/10 | 9/10 | 9/10 | 9/10 | 8/10 | 9/10 | 9/10 | 8/10 | 9/10 | 9/10 | 9/10 | 9/10 | 9/10 |
| Codex GPT-6.1 Sol | 7/10 | 8/10 | 9/10 | 9/10 | 9/10 | 9/10 | 9/10 | 10/10 | 9/10 | 10/10 | 9/10 | 8/10 | 10/10 | 10/10 | 10/10 |
Created with Claude Opus 5.5 in Claude Code.
🤖 Generated with Claude Code
Verdict: Needs Human Approval
Macroscope's review found this PR requires human approval — Reasoning now appears inline by default, with consecutive tool calls grouped into expandable rows. A human must confirm this change to the default activity-log experience for existing web and mobile users.
No code changes detected at 2cddb9c. Prior analysis still applies.
You can add or adjust custom eligibility rules. Learn more.
✅ Thread transfer remains within every enforced ceiling.
| Provider | Metric | Main baseline | This PR | Impact | PR ceiling | |
|---|---|---|---|---|---|---|
| Codex | Total thread wire | 5.0 KiB | 5.0 KiB | −6 B (−0.1%) | 6.8 KiB | ✅ |
| Codex | Thread snapshot wire | 3.8 KiB | 3.8 KiB | 0 B (0.0%) | 4.9 KiB | ✅ |
| Codex | Live turn WebSocket wire | 1.3 KiB | 1.2 KiB | −6 B (−0.5%) | 2.0 KiB | ✅ |
| Codex | Live turn WebSocket decoded | 21.6 KiB | 21.6 KiB | 0 B (0.0%) | 24.4 KiB | ✅ |
| Codex | Live turn messages | 2 | 2 | 0 (0.0%) | 8 | ✅ |
| Claude | Total thread wire | 5.0 KiB | 5.0 KiB | 0 B (0.0%) | 6.8 KiB | ✅ |
| Claude | Thread snapshot wire | 3.8 KiB | 3.8 KiB | 0 B (0.0%) | 4.9 KiB | ✅ |
| Claude | Live turn WebSocket wire | 1.3 KiB | 1.3 KiB | 0 B (0.0%) | 2.0 KiB | ✅ |
| Claude | Live turn WebSocket decoded | 21.9 KiB | 21.9 KiB | 0 B (0.0%) | 24.4 KiB | ✅ |
| Claude | Live turn messages | 2 | 2 | 0 (0.0%) | 8 | ✅ |
Baseline: e925dd6 · PR result: 2cddb9c · Source CI: success
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.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.
No actionable comments were generated in the recent review.
741878c4-da14-48a5-8a8d-05228875d4a8Reviewing files that changed from the base of the PR and between 44acda0091d26a5f0c2370641dd09f9104d8e99b and 2cddb9c43889319864d198ec18ef098869e2e194.
apps/web/src/components/ChatMarkdown.tsxapps/web/src/components/chat/MessagesTimeline.logic.test.tsapps/web/src/components/chat/MessagesTimeline.logic.tspackages/shared/src/String.tsIncluded review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 4 remain after this review.
The change revises mobile and web activity timelines to present reasoning, active work, and tool groups with updated visibility and expansion rules. It also updates streaming text detection, work-log presentation, and Markdown math behavior.
Changes: Activity Timeline
| Layer / File(s) | Summary |
|---|---|
Shared visibility and tool labels packages/shared/src/String.ts, packages/client-runtime/src/state/threadDetail.ts, packages/client-runtime/src/work-log/presentation.ts, related tests | A shared helper detects non-whitespace text. Thread-detail streaming comparisons track whether assistant text is visible. Work-log presentation adds active tool labels for supported tool types. |
Feed rows and active-work state apps/mobile/src/lib/threadActivity.ts, apps/web/src/components/chat/MessagesTimeline.logic.ts, related tests | Mobile and web feed derivation present reasoning separately and group tool work. Thinking rows are suppressed when a qualifying streaming reply or active activity is present. Tests cover feed visibility, grouping, and lifecycle cases. |
Work-group rendering and interactions apps/mobile/src/features/threads/*, apps/web/src/components/chat/*, apps/web/src/index.css, docs/user/activity-log.md | Work-log components render grouped tool entries, reasoning, and Thinking rows without the former live-work row and shimmer presentation. Group expansion, row sizing, Markdown math, and activity-log instructions are updated. |
Estimated code review effort: 4 (Complex) | ~60 minutes
Change: Feature
| 1 | sequenceDiagram |
| 2 | participant ActivityFeed |
| 3 | participant MessagesTimelineLogic |
| 4 | participant MessagesTimeline |
| 5 | participant WorkGroupToggleTimelineRow |
| 6 | ActivityFeed->>MessagesTimelineLogic: activity entries and streaming state |
| 7 | MessagesTimelineLogic->>MessagesTimeline: derived work, work-toggle, and thinking rows |
| 8 | MessagesTimeline->>WorkGroupToggleTimelineRow: grouped entries and active-entry state |
Possibly related issues
Suggested reviewers: juliusmarminge, yash-singh1, maria-rcks
2cddbNo identified issue needs to be resolved before merging.
Failed Failed checks: 1 warning
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Description check | Warning | The description explains the problem, intended behavior, UI changes, known limits, screenshots, recordings, and verification command. However, it does not follow the required section structure and doe… | Add explicit Problem, Change, Scope and approval, and Verification sections. Provide the linked triaged issue or maintainer approval with its approval comment, or explain why this focused fix qualifies for an exemption. List the focused tes… |
| Check name | Status | Explanation |
|---|---|---|
| Title check | Passed | The title clearly and concisely identifies the main changes: inline thinking and folding completed work under “Worked for.” |
| 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. |
Explanation
The description explains the problem, intended behavior, UI changes, known limits, screenshots, recordings, and verification command. However, it does not follow the required section structure and does not provide the required scope and approval information.
Resolution
Add explicit Problem, Change, Scope and approval, and Verification sections. Provide the linked triaged issue or maintainer approval with its approval comment, or explain why this focused fix qualifies for an exemption. List the focused test names or commands and their observed results, including anything not checked. Keep the before/after UI evidence in the Verification section and include the required agent model and harness statement if applicable.
Added 1 commit