Skip to content

Filter transcript streaming nodes#791

Merged
blove merged 4 commits into
mainfrom
blove/streaming-markdown-e2e-regressions
Jul 8, 2026
Merged

Filter transcript streaming nodes#791
blove merged 4 commits into
mainfrom
blove/streaming-markdown-e2e-regressions

Conversation

@blove

@blove blove commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add transcriptNodeNames to the LangGraph adapter so side-effect LLM nodes do not stream into the main chat transcript.
  • Configure the canonical chat example to stream only the generate node into visible assistant messages.
  • Add aimock-backed streaming markdown regressions for tables, blockquote/table rendering, and closing code fences, plus docs for the new option.

Root cause

The canonical LangGraph graph streams messages-tuple chunks from both the chat generation node and side-effect nodes such as title generation. The bridge treated every metadata-bearing message tuple as transcript content, so side-effect output could temporarily render as an extra table/message while streaming.

Validation

  • npx nx test langgraph --runInBand -- stream-manager.bridge.spec.ts
  • npx nx test chat --runInBand -- streaming-markdown.table-stream.spec.ts
  • npx nx run examples-chat-angular:e2e -- --grep "streaming markdown table|streaming code fence"
  • npx nx lint examples-chat-angular

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview, Comment Jul 8, 2026 9:57pm

Request Review

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@blove blove enabled auto-merge July 8, 2026 21:17
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove blove merged commit 0a7158c into main Jul 8, 2026
61 checks passed
@blove blove deleted the blove/streaming-markdown-e2e-regressions branch July 8, 2026 23:18
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