Skip to content

chore: remove SDK struct usage from requests in messages API#218

Draft
pawbana wants to merge 2 commits intomainfrom
pb/messages-remove-paramswrap
Draft

chore: remove SDK struct usage from requests in messages API#218
pawbana wants to merge 2 commits intomainfrom
pb/messages-remove-paramswrap

Conversation

@pawbana
Copy link
Contributor

@pawbana pawbana commented Mar 16, 2026

Replaces MessageNewParamsWrapper that wrapped anthropic.MessageNewParams
with MessagesRequestPayload that operates on raw bytes.

Copy link
Contributor Author

pawbana commented Mar 16, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@pawbana pawbana force-pushed the pb/messages-remove-paramswrap branch 6 times, most recently from 7cc40c0 to 18c3ec7 Compare March 18, 2026 13:09
@pawbana pawbana force-pushed the pb/responses-remove-paramswrap-part2 branch from af9c082 to 746f730 Compare March 18, 2026 13:09
@pawbana pawbana force-pushed the pb/messages-remove-paramswrap branch 2 times, most recently from 3c808e8 to e300abd Compare March 18, 2026 18:24
@pawbana pawbana changed the base branch from pb/responses-remove-paramswrap-part2 to graphite-base/218 March 18, 2026 18:30
Replaces MessageNewParamsWrapper that wrapped anthropic.MessageNewParams
with MessagesRequestPayload that operates on raw bytes.
@pawbana pawbana force-pushed the graphite-base/218 branch from 2e26427 to 95ae940 Compare March 18, 2026 18:32
@pawbana pawbana force-pushed the pb/messages-remove-paramswrap branch from e300abd to 0ea4f18 Compare March 18, 2026 18:32
@graphite-app graphite-app bot changed the base branch from graphite-base/218 to main March 18, 2026 18:33
@pawbana pawbana force-pushed the pb/messages-remove-paramswrap branch from 0ea4f18 to 40237e3 Compare March 18, 2026 18:33
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