Skip to content

fix(studio): run inspector shows the error pill for a failed open step - #745

Merged
LinuxDevil merged 2 commits into
mainfrom
revamp/f15-trace-screen
Oct 24, 2026
Merged

LinuxDevil merged 2 commits into
mainfrom
revamp/f15-trace-screen

Conversation

@LinuxDevil

Copy link
Copy Markdown
Owner

A trace gets its own deep-linkable screen (#/traces/:id?obs=) with a header, the unified viewer and human feedback; the observe proxy gains a one-entry write allowlist (POST /api/v1/scores).

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

Changes

  • feat(studio): trace screen with header, deep links and feedback
  • fix(studio): run inspector shows the error pill for a failed open step

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 and others added 2 commits October 25, 2026 01:45
An observation with end === null and status error showed 'running'; a stored
error now wins in displayStatus (inspector and tree).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
#/traces/:traceId now renders TraceScreen (new 'trace' ScreenId): a header
(root name, status, agent link, model, start, live duration, tokens, cost,
session link, copy link) over the unified TraceViewer. ?obs= selects a step on
load and is written with replace on selection. The inspector's Feedback
section takes thumbs up/down + comment, posted via observeClient.postScore to
POST /api/v1/scores; the studio /observe proxy gains a write allowlist for it.
A missing trace shows an EmptyState with Back to traces; a dead daemon shows
the shared down state, keeps the URL and recovers in place.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@LinuxDevil
LinuxDevil merged commit b07773c into main Oct 24, 2026
2 checks passed
@LinuxDevil
LinuxDevil deleted the revamp/f15-trace-screen branch October 24, 2026 22:55
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