Skip to content

Draw calls to MCP servers, and what each Evidence position carries - #48

Merged
wu-sheng merged 1 commit into
mainfrom
renderer-horizon-c903e77
Sep 28, 2026
Merged

wu-sheng merged 1 commit into
mainfrom
renderer-horizon-c903e77

Conversation

@wu-sheng

Copy link
Copy Markdown
Member

The page's renderer is Horizon's, and two Horizon changes since the pin (88e0110) draw what asz.view already carries. This moves the embedded renderer to them.

What the page shows now

  • A call to an MCP server sits on its own MCP lane, named by its server and tool. Its card shows how it ended and the time the plugin measured around it, and the inspector gains an Execution tab: the server and where its configuration came from, the outcome, the measured time, and the size and digest of the arguments and the answer.
  • The Evidence tab lists the step's own positions, then each of its change and execution records. The picked position shows what the document carries for it: the step's text, the call's result on a result position, or the record itself. It used to show the step's text under every position, so a call's request and result showed the same thing.

Checked

  • make conversation-view-check passes: the embedded copy is what c903e77 builds.
  • make check passes.
  • I built asz from this branch and served a root of real Claude Code sessions that called an MCP server. The call sits on the MCP lane as web · echo, the Execution tab is there, and Evidence lists request, result and the execution record; the result chip shows the call's result, echo: {"text": "over http"}.

The page's renderer is Horizon's, and two Horizon changes since the pin
draw what asz.view already carries. apache/skywalking-horizon-ui#175 puts
a call to an MCP server on its own lane, with how it ended, the measured
time and an Execution tab. #176 makes the Evidence tab show what the
document carries for the picked position: the step's text, a call's
result, or a change or execution record. It showed the step's text under
every position.

internal/view/conversation-view is rebuilt with
tools/conversation-view.sh update c903e77876c99358873b89f4803becdc5470ca30.
@wu-sheng wu-sheng added this to the 0.6.0 milestone Sep 28, 2026
@wu-sheng
wu-sheng merged commit aeff5f7 into main Sep 28, 2026
24 checks passed
@wu-sheng
wu-sheng deleted the renderer-horizon-c903e77 branch September 28, 2026 03:33
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