Skip to content

feat(mobile): improve session transcript details - #5020

Merged
iscekic merged 8 commits into
mainfrom
session-transcript-ux-b01c
Aug 5, 2026
Merged

feat(mobile): improve session transcript details#5020
iscekic merged 8 commits into
mainfrom
session-transcript-ux-b01c

Conversation

@iscekic

@iscekic iscekic commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Make transcript tool and reasoning parts fixed single-line rows.
  • Open live page-sheet details for tools and reasoning.
  • Add subtle timestamps to user and assistant messages.

Why

Keep streaming transcripts stable while preserving access to part details and message context.

How

  • Reuse existing tool-card bodies inside a shared detail sheet.
  • Resolve open parts by id from live transcript messages.
  • Keep reasoning preference modes and child-session navigation intact.
  • Add pure, mounted, routing, and regression tests.

Verification

  • apps/mobile: pnpm format, pnpm typecheck, pnpm lint, pnpm check:unused, pnpm test, pnpm format:check, and git diff --check pass.
  • Root repository test was attempted but failed during database setup because the configured PostgreSQL service was unavailable.
  • Bot E2E reverify passed S1, S2, S3, S4, and S5 on iOS.
  • S2 passed with a running remote tool and live output in the open sheet.
  • S3 passed collapsed reasoning and expanded reasoning without a collapse control.
  • S5 passed message details; no task row materialized, and child-navigation seam tests cover that path.
  • The first E2E S2 run exposed a workflow defect. Kilo workflow PR 39 now emits KILO_REMOTE=1 for npm-installed CLIs and classifies read-only exec run sessions as environment failures.

Visual Changes

Before: s2-before.png

After: s2-after.png

@iscekic
iscekic requested a review from jeanduplessis August 5, 2026 05:11
@iscekic iscekic self-assigned this Aug 5, 2026
@iscekic
iscekic requested a review from pandemicsyn August 5, 2026 05:11
@kilo-code-bot

kilo-code-bot Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Reviewed the mobile session-transcript overhaul (fixed part rows, live part-detail sheets, message time labels) across all 45 changed files: the row/body card split preserves the previous display logic verbatim, part-detail hosts resolve parts from the same live messages that render the rows, hooks ordering and accessibility states are correct, and no memory leaks (timers, subscriptions, listeners, reanimated shared values) are introduced — high confidence.

Files Reviewed (45 files)
  • apps/mobile/src/components/agents/message-time-label.ts + test
  • apps/mobile/src/components/agents/message-details-content.ts + message-details-sheet.test.ts
  • apps/mobile/src/components/agents/fixed-part-row.tsx + mounted test
  • apps/mobile/src/components/agents/open-part-detail-context.ts
  • apps/mobile/src/components/agents/tool-card-display.ts + test
  • apps/mobile/src/components/agents/suggest-tool-card.tsx + test
  • apps/mobile/src/components/agents/tool-card-shell.tsx (deleted; no stale references)
  • apps/mobile/src/components/agents/tool-cards/{bash,edit,generic,glob,grep,list,read,task,todo,web-search,write}-tool-card.tsx + index.ts + edit/write tests
  • apps/mobile/src/components/agents/part-detail-model.ts + test
  • apps/mobile/src/components/agents/part-detail-sheet.tsx
  • apps/mobile/src/components/agents/part-detail-sheet-host.tsx + mounted test
  • apps/mobile/src/components/agents/tool-part-detail-body.tsx + test
  • apps/mobile/src/components/agents/child-session-sheet.tsx
  • apps/mobile/src/components/agents/session-detail-content.tsx
  • apps/mobile/src/components/agents/part-renderer.tsx + test
  • apps/mobile/src/components/agents/reasoning-part-renderer.tsx + mounted test
  • apps/mobile/src/components/agents/message-bubble.tsx + test
  • apps/mobile/src/components/agents/child-session-message.test.ts
  • apps/mobile/src/components/agents/tool-part-renderer.test.ts
  • apps/mobile/src/components/security-agent/collapsible-section.tsx (comment-only)

Reviewed by kimi-k3 · Input: 298.4K · Output: 21.6K · Cached: 2.9M

Review guidance: REVIEW.md from base branch main

@iscekic iscekic added the human-ready The PR is ready for human review. label Aug 5, 2026
@iscekic
iscekic merged commit 4f0c2f2 into main Aug 5, 2026
21 checks passed
@iscekic
iscekic deleted the session-transcript-ux-b01c branch August 5, 2026 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

human-ready The PR is ready for human review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants