Skip to content

refactor(studio): retire the old Observe panels - #755

Merged
LinuxDevil merged 4 commits into
mainfrom
revamp/f18-retire-observe
Oct 28, 2026
Merged

LinuxDevil merged 4 commits into
mainfrom
revamp/f18-retire-observe

Conversation

@LinuxDevil

Copy link
Copy Markdown
Owner

The last old Observe pieces go: the Agents drawer's Trace tab now uses the shared trace viewer (SpanEvent adapter, no Playground links there), and dead status/trace CSS is removed.

Ticket: F18 (Forge Observe vs LangSmith evaluation follow-up)

Changes

  • fix(studio): no Playground links in the drawer trace tab
  • refactor(studio): retire the old Observe panels
  • refactor(studio): drawer trace tab uses the shared trace viewer

Verification: focused vitest, tsc --noEmit, eslint on changed files (details in the review package); full suite and fallow run in CI.

🤖 Generated with Claude Code

@LinuxDevil LinuxDevil changed the title refactor(studio): drawer trace tab uses the shared trace viewer refactor(studio): retire the old Observe panels Oct 28, 2026
LinuxDevil and others added 4 commits October 29, 2026 00:32
The Agents drawer's Trace tab renders ui/run/TraceViewer (tree or
waterfall beside the inspector) instead of its own span waterfall. A pure
spansToObservations adapter maps runtime SpanEvents to observation rows
(kind from gen_ai.operation.name, error message to attrs.error, model,
usage, cost, input/output). Trace history, node highlighting, Logs jump
and ticking running steps are kept. useNow moves to ui/run for both the
drawer and #/traces/:id; orderSpansForWaterfall and the old .trace-row
CSS are gone. New e2e covers the tab.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Drop the pre-StatusPill .status-* rules from ui/base.css and the unused
.traces-detail rule, and point the docs at the Traces, Threads,
Monitoring and Scores screens instead of the removed Observe tabs.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
TraceViewer, RunTree and RunInspector take `playgroundLinks` (default
true); the drawer's TracePanel passes false, since Studio runs reach the
observe daemon only when LOUSHO_OBSERVE is set. Also drops the unused
.chip-list rule, fixes the wireTypes comment and rewraps the touched
doc paragraphs.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@LinuxDevil
LinuxDevil force-pushed the revamp/f18-retire-observe branch from da99a4c to ccdd81d Compare October 28, 2026 21:32
@LinuxDevil
LinuxDevil merged commit 4717928 into main Oct 28, 2026
2 checks passed
@LinuxDevil
LinuxDevil deleted the revamp/f18-retire-observe branch October 28, 2026 21:43
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