docs: update TypeScript streaming events for sdk-typescript#544#578
docs: update TypeScript streaming events for sdk-typescript#544#578dbschmigelski merged 8 commits intostrands-agents:mainfrom
Conversation
Update docs to reflect breaking changes from sdk-typescript PR strands-agents#544 which wraps all raw data objects in event wrappers for agent.stream(). Raw event types like modelContentBlockDeltaEvent and toolStreamEvent are now accessed through wrapper events (ModelStreamUpdateEvent, ToolStreamUpdateEvent, etc.). Add CLAUDE.md to .gitignore.
|
Assessment: Comment Review SummaryThis PR correctly updates the TypeScript documentation to reflect breaking changes from sdk-typescript#544. The changes are well-structured:
Minor IssueThe Overall, the documentation accurately reflects the SDK changes and the code examples are correct. Nice work keeping the docs in sync with the breaking changes! 👍 |
Documentation Deployment CompleteYour documentation preview has been successfully deployed! Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-578/ |
…script#544 Add new hookable data events (ModelStreamUpdateEvent, ContentBlockEvent, ModelMessageEvent, ToolStreamUpdateEvent, ToolResultEvent, AgentResultEvent) to the Available Events table and mermaid lifecycle diagram. Replace incorrect ts_not_supported_code macros with actual TypeScript snippets for composability and fixed tool arguments sections.
|
Updated Review - Including I've also reviewed the ✅ Lifecycle Diagram Updated: The mermaid diagram now correctly shows the data events flow:
✅ Data Events Documentation: The new "Data Events (TypeScript only)" section properly documents:
✅ TypeScript Examples: The note My previous review assessment remains: Comment (approve with minor suggestion to remove unrelated |
Documentation Deployment CompleteYour documentation preview has been successfully deployed! Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-578/ |
Documentation Deployment CompleteYour documentation preview has been successfully deployed! Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-578/ |
Documentation Deployment CompleteYour documentation preview has been successfully deployed! Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-578/ |
|
Updated Review - The ✅ Separate Lifecycle Diagrams: The mermaid diagrams are now properly split into Python/TypeScript tabs:
✅ Separate Event Tables: Each SDK now has its own "Available Events" table:
✅ Proper Macros: Uses This is a cleaner approach than the previous single diagram with footnotes. TypeScript compilation continues to pass. Assessment remains: Comment (approve with minor suggestion to address unrelated |
Documentation Deployment CompleteYour documentation preview has been successfully deployed! Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-578/ |
Documentation Deployment CompleteYour documentation preview has been successfully deployed! Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-578/ |
Description
wrappers for agent.stream()
'modelContentBlockDeltaEvent')
Event mapping applied
Files changed
Related Issues
Type of Change
Checklist
mkdocs serveBy submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.