Skip to content

refactor(dashboard): default expanded LLM call to raw JSON, show thinking text inline#1017

Merged
zbigniewsobiecki merged 1 commit intodevfrom
refactor/llm-call-detail-default-raw
Mar 23, 2026
Merged

refactor(dashboard): default expanded LLM call to raw JSON, show thinking text inline#1017
zbigniewsobiecki merged 1 commit intodevfrom
refactor/llm-call-detail-default-raw

Conversation

@zbigniewsobiecki
Copy link
Copy Markdown
Member

Summary

  • Expanded row defaults to raw JSONLlmCallDetail now opens in raw view (showRaw=true). The structured parsed view is still accessible via the toggle, but the raw JSON is what's worth expanding to see (tool inputs untruncated, full response structure), whereas the structured summary is already visible inline in the list row.
  • Remove redundant metadata bar from the expanded panel — model, tokens, cost, and timestamp are all already present in the list columns; showing them again in the expanded panel was pure duplication.
  • Thinking text shown inlinelistLlmCalls API now returns thinkingPreview (first 200 chars of thinking text). The Activity column now shows the actual thinking content truncated, with (N chars) suffix when the full text exceeds 200 chars, instead of the opaque "thinking (N chars)" placeholder.

Test plan

  • npm run lint — clean
  • npm run typecheck — clean
  • npm test — all pass
  • Expand any Claude Code row → raw JSON shown immediately, no extra click needed
  • "Structured" button toggles to parsed blocks view
  • Thinking turns in the list show the first ~200 chars of reasoning text

🤖 Generated with Claude Code

… thinking inline

- LlmCallDetail now defaults to raw JSON view (showRaw=true); structured view
  remains accessible via the toggle. Raw JSON is the unique content that isn't
  already visible in the list row, so it should be the default.
- Remove redundant metadata bar from detail panel (model/tokens/cost/time are
  already shown in the list columns).
- listLlmCalls API now also returns thinkingPreview (first 200 chars of thinking
  text, concatenated across blocks) alongside thinkingChars.
- List row Activity column shows the actual thinking text truncated, with char
  count suffix when the full text exceeds 200 chars, instead of just
  "thinking (N chars)".

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 23, 2026

Codecov Report

❌ Patch coverage is 58.33333% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/api/routers/runs.ts 58.33% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

@zbigniewsobiecki zbigniewsobiecki merged commit 156c7f0 into dev Mar 23, 2026
8 of 9 checks passed
@zbigniewsobiecki zbigniewsobiecki deleted the refactor/llm-call-detail-default-raw branch March 23, 2026 21:10
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