Skip to content

fix(web): stop rendering sidechain prompt as user message#340

Merged
tiann merged 1 commit intotiann:mainfrom
hqhq1025:fix/sidechain-prompt-display
Mar 23, 2026
Merged

fix(web): stop rendering sidechain prompt as user message#340
tiann merged 1 commit intotiann:mainfrom
hqhq1025:fix/sidechain-prompt-display

Conversation

@hqhq1025
Copy link
Contributor

Summary

  • Sub-agent (sidechain) prompts were incorrectly converted to user-text blocks in reducerTimeline.ts, causing them to appear as standalone messages in the web chat UI
  • The prompt text is already visible in the parent Task tool call's input, so this was a duplicate/misattributed display
  • Fixed by skipping sidechain content blocks instead of creating user-text blocks

Test plan

  • Open a session that uses sub-agents (Agent tool) in the web UI
  • Verify the sub-agent prompt no longer appears as a separate user message
  • Verify the Task tool call still shows the prompt in its input parameters
  • Verify sub-agent responses still render correctly as children of the Task tool call

The sub-agent (sidechain) prompt was incorrectly converted to a
`user-text` block in reducerTimeline, causing it to appear as a
standalone message in the chat UI. The prompt is already visible
in the parent Task tool call's input, so the sidechain content
block is now skipped to avoid the duplicate/misattributed display.

via [HAPI](https://hapi.run)

Co-Authored-By: HAPI <noreply@hapi.run>
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Findings

  • None.

Summary
Review mode: initial. No issues found in diff. Residual risk: UI behavior change untested; manual verification recommended.

Testing

  • Not run (automation)

@tiann tiann merged commit 4c11fbe into tiann:main Mar 23, 2026
2 checks passed
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.

2 participants