Skip to content

Add some more tests for Anthropic{Beta}ClientExtensions#26

Merged
sd-st merged 2 commits intoanthropics:nextfrom
stephentoub:moretests
Dec 2, 2025
Merged

Add some more tests for Anthropic{Beta}ClientExtensions#26
sd-st merged 2 commits intoanthropics:nextfrom
stephentoub:moretests

Conversation

@stephentoub
Copy link
Contributor

image

cc: @sd-st

Copy link
Collaborator

@sd-st sd-st left a comment

Choose a reason for hiding this comment

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

Thanks for this PR! Looks good but there are some merge conflicts

@stephentoub
Copy link
Contributor Author

Rebased

@sd-st
Copy link
Collaborator

sd-st commented Nov 25, 2025

FYI lint is failing

@stephentoub
Copy link
Contributor Author

Ran format.

@sd-st
Copy link
Collaborator

sd-st commented Nov 25, 2025

Lint is still failing, other than that lgtm

@stephentoub
Copy link
Contributor Author

Lint is still failing, other than that lgtm

dotnet csharpier format . is a nop for me locally. But I've manually changed the two lines reported by CI.

@sd-st
Copy link
Collaborator

sd-st commented Nov 25, 2025

Has a conflict now, sorry 😅

@stephentoub
Copy link
Contributor Author

Rebased to resolve conflicts

@sd-st
Copy link
Collaborator

sd-st commented Dec 1, 2025

@stephentoub Failing lint 😔

@stephentoub
Copy link
Contributor Author

@stephentoub Failing lint 😔

Addressed

@sd-st
Copy link
Collaborator

sd-st commented Dec 2, 2025

@stephentoub Looks like something went weird with your rebase, there are a bunch of extra files being touched now

@stephentoub
Copy link
Contributor Author

@stephentoub Looks like something went weird with your rebase, there are a bunch of extra files being touched now

Ugh, wrong base. Fixed. The fact that main is actually a release branch rather than the main active branch is confusing ;-)

@sd-st sd-st merged commit 5b7052c into anthropics:next Dec 2, 2025
3 checks passed
sd-st pushed a commit that referenced this pull request Dec 10, 2025
* chore(internal): suppress diagnostic for .netstandard2.0

* chore(internal): suppress diagnostic for .netstandard2.0

* fix: remove bad preprocessor directive

* Add some more tests for Anthropic{Beta}ClientExtensions (#26)

* Add some more tests for Anthropic{Beta}ClientExtensions

* Fix csharpier

* chore(internal): update testing setup

* Readd TestBase

* Use single source of truth for test parameter

* Readd baseclass for codegen compatibility

* Fix code style

* chore(internal): equality and more unit tests

* fix(client): handling of null value type
refactor(internal): share get/set logic

* chore(internal): update csproj formatting

* refactor(client): use `System.Net.ServerSentEvents`

* docs: add link to nuget

* fix(client): check response status when `MaxRetries = 0`

* chore(internal): add tests for constants

* feat(internal): add additional object tests

* codegen metadata

* chore(ci): lint and build examples (#43)

* docs: correct reqs

* chore(internal): suppress diagnostic for .netstandard2.0

* feat(client): improve csproj

* chore(internal): set up cron release job

* fix(internal): running net462 tests on ci

* Improve code style on examples
Remove obsolete examples
Include all examples in example sln

* codestyle improvements

* adapt build script to also build examples

---------

Co-authored-by: Tomer Aberbach <tomer@stainless.com>
Co-authored-by: stainless-app[bot] <142633134+stainless-app[bot]@users.noreply.github.com>

* feat(client): improve some names

* docs: add more comments

* fix(client): with expressions for models

* chore(internal): fix release please config

* chore(internal): fix package name (#239)

* Update version for src/Anthropic in manifest

* Add src/Anthropic to release-please config

* feat(client): add x-stainless-retry-count

* fix(ci): fail loudly if version already exists on nuget

* Fix handling of multiple streaming tool calls (#62)

The implementation was missing a clear on the collection of streaming data. As such, it would each tool already yielded every time it yielded another tool.

* Add beta headers automatically for HostedCodeInterpreter/McpServerTool (#63)

* Add beta headers automatically for HostedCodeInterpreter/McpServerTool

* format

* Idiomatic Skills support for MEAI via AsAITool() extension (#58)

* Add WithSkills convenience method to MEAI extensions

* Refactor to use AsAITool approach

* Align with existing AsAITool pattern

* Unify with beta header inclusion from #63

* Fix bug

* Ran /scripts/format

* Update src/Anthropic/Services/Beta/Messages/AnthropicBetaClientExtensions.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>

---------

Co-authored-by: Stephen Toub <stoub@microsoft.com>

* chore(internal): add enum tests

* chore(client): improve union validation

* chore: release main

---------

Co-authored-by: stainless-app[bot] <142633134+stainless-app[bot]@users.noreply.github.com>
Co-authored-by: Tomer Aberbach <tomer@stainless.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: JPVenson <github@jpb.email>
Co-authored-by: meorphis <108296353+meorphis@users.noreply.github.com>
Co-authored-by: Peder Holdgaard Pedersen <127606677+PederHP@users.noreply.github.com>
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