Skip to content

Conversation

@stainless-app
Copy link
Contributor

@stainless-app stainless-app bot commented Nov 25, 2025

Automated Release PR

1.19.1 (2025-12-06)

Full Changelog: v1.19.0...v1.19.1

Bug Fixes

  • mcp: correct code tool API endpoint (6c8a083)
  • rename param to avoid collision (6d1cf75)

Chores

  • elide duplicate aliases (c8e2ee1)
  • internal: codegen related update (931c976)

This pull request is managed by Stainless's GitHub App.

The semver version number is based on included commit messages. Alternatively, you can manually set the version number in the title of this pull request.

For a better experience, it is recommended to use either rebase-merge or squash-merge when merging this pull request.

🔗 Stainless website
📚 Read the docs
🙋 Reach out for help or questions

copy from next without all message utils

add utils

revert diffs for betamessage

migrate param utils
add more to param converters
clean up test

clean up test
…m-fixes

fix: update accumulators and param converters
Copy link

@packyg packyg left a comment

Choose a reason for hiding this comment

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

A few small things plus:

  • Can we add a conventional commit message so we get a changelog?
  • Could we also add a unit test to explicitly test the issue from #164 ?

p := variant.ToParam()
return ContentBlockParamUnion{OfWebSearchToolResult: &p}
default:
panic(fmt.Sprintf("unexpected anthropic.anyContentBlock: %#v", variant))
Copy link

Choose a reason for hiding this comment

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

Can we not panic and perhaps log something here instead and then return like we do for beta?

Suggested change
panic(fmt.Sprintf("unexpected anthropic.anyContentBlock: %#v", variant))
return ContentBlockParamUnion{}

There's some risk we forget a new content block type here. That or add a unit test to catch some cases.


// Param converters

func (r BetaContentBlockUnion) ToParam() BetaContentBlockParamUnion {
Copy link

Choose a reason for hiding this comment

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

Looks like BetaToolSearchToolResultBlock is missing, could we add that?

// }
func (acc *Message) Accumulate(event MessageStreamEventUnion) error {
if acc == nil {
return fmt.Errorf("accumulate: cannot accumlate into nil Message")
Copy link

Choose a reason for hiding this comment

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

Suggested change
return fmt.Errorf("accumulate: cannot accumlate into nil Message")
return fmt.Errorf("accumulate: cannot accumulate into nil Message")

@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next branch 2 times, most recently from 1456002 to ba13a14 Compare November 27, 2025 05:03
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next branch 7 times, most recently from f35ec3d to db37e73 Compare December 3, 2025 05:03
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next branch 3 times, most recently from 35b8e9e to 9398b8b Compare December 5, 2025 05:03
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next branch from 9398b8b to 0b7cf94 Compare December 5, 2025 20:41
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next branch from 0b7cf94 to f003ce9 Compare December 5, 2025 21:18
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next branch from f003ce9 to e0b17e1 Compare December 5, 2025 23:52
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next branch from e0b17e1 to 3e9a487 Compare December 5, 2025 23:53
@stainless-app stainless-app bot force-pushed the release-please--branches--main--changes--next branch from 3e9a487 to aa8de63 Compare December 6, 2025 05:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants