Skip to content

fix(grpc-sdk): make chat sendMessage files optional at SDK boundary#1548

Merged
kkopanidis merged 1 commit into
v0.16.xfrom
fix/v0.16-chat-sendmessage-optional-files
Jul 9, 2026
Merged

fix(grpc-sdk): make chat sendMessage files optional at SDK boundary#1548
kkopanidis merged 1 commit into
v0.16.xfrom
fix/v0.16-chat-sendmessage-optional-files

Conversation

@ChrisPdgn

@ChrisPdgn ChrisPdgn commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Expose SendMessageInput with optional files and normalize via SendMessageRequest.fromPartial so existing callers are not broken by ts-proto's required repeated-field typing.

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Build-related changes
  • Other (please describe)

Does this PR introduce a breaking change?

  • Yes
  • No

The PR fulfills these requirements:

  • It's submitted to the main branch
  • When resolving a specific issue, it's referenced in the PR's description (e.g. fix #xxx, where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature

Other information:

@ChrisPdgn
ChrisPdgn marked this pull request as draft July 9, 2026 10:39
@ChrisPdgn
ChrisPdgn marked this pull request as ready for review July 9, 2026 10:43
Expose SendMessageInput with optional files and normalize via
SendMessageRequest.fromPartial so existing callers are not broken by
ts-proto's required repeated-field typing.

Co-authored-by: Cursor <cursoragent@cursor.com>
@ChrisPdgn
ChrisPdgn force-pushed the fix/v0.16-chat-sendmessage-optional-files branch from 334319b to 207d780 Compare July 9, 2026 11:37
@kkopanidis
kkopanidis merged commit 7316b85 into v0.16.x Jul 9, 2026
2 checks passed
@kkopanidis
kkopanidis deleted the fix/v0.16-chat-sendmessage-optional-files branch July 9, 2026 11:40
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