Skip to content

Conversation

@jacoblee93
Copy link
Contributor

@jacoblee93 jacoblee93 commented Jan 14, 2026

Summary

The default stdio stream buffer size is only 64kb, which means you get errors like this when trying to send messages larger than that via ACP:

Screenshot 2026-01-14 at 10 10 18 AM

Filesizes above that are quite common for multimodal data

The new default of 50MB should handle most practical multimodal cases. Definitely open to tweaking this default though if anyone has strong feelings! And it's now configurable if needed

Related issues

Testing

Added tests and ran appropriate make commands

Docs & screenshots

Not sure how to best document, assuming this will automatically propagate to API refs?

Checklist

  • Conventional Commit title (e.g. feat:, fix:).
  • Tests cover the change or are not required (explain above).
  • Docs/examples updated when behaviour is user-facing.
  • Schema regenerations (make gen-all) are called out if applicable.

@PsiACE
Copy link
Member

PsiACE commented Jan 30, 2026

Thanks for your PR. The default 50MB limit doesn't seem reasonable. I think it's sufficient to just allow it to be configured.

@phil65
Copy link

phil65 commented Jan 30, 2026

@PsiACE I would at least increase the default limit to at least 1MB or sth.. its quite easy to reach the 64kb when it comes to sending the models and slash commands (model lists can easily reach 1000 items for some agents)

Copy link
Member

@PsiACE PsiACE left a comment

Choose a reason for hiding this comment

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

thanks @phil65, i will merge it first and consider a small and enough default value. and cc @stdrc for suggestion

@PsiACE PsiACE merged commit 5510db2 into agentclientprotocol:main Jan 31, 2026
8 checks passed
@jacoblee93
Copy link
Contributor Author

Thank you! Yeah 50MB is probably more than needed outside of niche cases. I've seen some screenshots around 5MB though so wouldn't go too small.

@PsiACE
Copy link
Member

PsiACE commented Jan 31, 2026

Thank you! Yeah 50MB is probably more than needed outside of niche cases. I've seen some screenshots around 5MB though so wouldn't go too small.

talked with stdrc, i will keep 50MB as default

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.

3 participants