Skip to content

Conversation

@threepointone
Copy link
Contributor

  • use streamdown for text
  • move the input down
  • scroll to the bottom when messages change
  • fix the bug where it would stop streaming

- use streamdown for text
- move the input down
- scroll to the bottom when messages change
- fix the bug where it would stop streaming
@changeset-bot
Copy link

changeset-bot bot commented Nov 19, 2025

⚠️ No Changeset found

Latest commit: d47dbf4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

}

const App = () => {
const [error, setError] = useState("");
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this wasn't used anywhere

@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 19, 2025

Open in StackBlitz

npm i https://pkg.pr.new/cloudflare/agents@662

commit: d47dbf4

@claude
Copy link

claude bot commented Nov 19, 2025

Claude Code Review

PR adds streamdown for markdown, fixes stream stop bug, and improves UI layout.

Issues:

  1. ai-react.tsx:273 - Code correctly handles empty body and done separately. No issue here on review.

  2. app.tsx:195-199 - Auto-scroll useEffect may fire on message updates without new content. Consider checking scroll position or message length.

  3. app.tsx:195 - Scroll timing: Streamdown may not finish rendering before scroll. Use requestAnimationFrame.

Testing: No tests for stream stop fix or auto-scroll behavior.

Good: Streamdown integration, sticky input positioning.

@threepointone threepointone merged commit ef38e84 into main Nov 19, 2025
7 checks passed
@threepointone threepointone deleted the playground-fixes branch November 19, 2025 00:32
naji247 pushed a commit to MCPCat/agents that referenced this pull request Nov 20, 2025
- use streamdown for text
- move the input down
- scroll to the bottom when messages change
- fix the bug where it would stop streaming
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