Skip to content

Move the conversation renderer to Horizon 88e0110 - #42

Merged
wu-sheng merged 1 commit into
mainfrom
horizon-no-previous-request
Sep 25, 2026
Merged

wu-sheng merged 1 commit into
mainfrom
horizon-no-previous-request

Conversation

@wu-sheng

Copy link
Copy Markdown
Member

This moves the embedded conversation renderer to Horizon 88e0110, the newest commit on Horizon's main. It picks up apache/skywalking-horizon-ui#173.

The Prompt tab's What it added mode said "The request before this one is not among the loaded bodies" whenever it had nothing to compare with. For every LangChain call, and for the first call of any chain, the request names no request before it, so nothing was missing. It now says "This request names no request before it, so there is nothing to compare with." A request that names one the tab can't find keeps the old message.

  • internal/view/conversation-view is rebuilt with tools/conversation-view.sh update 88e0110add7c9d592e1dcf251a4eb6efb8c699a9. #173 is the only Horizon commit since the old pin (297f020) that touches the renderer or the design tokens.
  • The changelog entry for the LangChain Prompt tab names the new pin and the corrected message.

Checked

  • make conversation-view-check and make check pass.
  • I served a real LangGraph conversation with this build and opened "What it added" on a call. It shows the new message.

The Prompt tab's What it added mode said the request before was not among
the loaded bodies whenever it had nothing to compare with. For every
LangChain call, and for the first call of any chain, the request names no
request before it, so nothing was missing. Horizon's renderer now says the
request names none before it (apache/skywalking-horizon-ui#173), and the
embedded copy moves to that commit.
@wu-sheng wu-sheng added this to the 0.6.0 milestone Sep 25, 2026
@wu-sheng wu-sheng added the UI Conversation Local View(UI) label Sep 25, 2026
@wu-sheng
wu-sheng merged commit e88c421 into main Sep 25, 2026
24 checks passed
@wu-sheng
wu-sheng deleted the horizon-no-previous-request branch September 25, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI Conversation Local View(UI)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant