Connector Implementation & Inheritance: Anthropic Claude

[qoweh]

Purpose

• close Connector Implementation & Inheritance: Anthropic Claude #261

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] New feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

README updated?

The top-level readme for this repo contains a link to each sample in the repo. If you're adding a new sample did you update the readme?

[x] Yes
[ ] No
[ ] N/A

How to Test

• Get the code

git clone https://github.com/qoweh/open-chat-playground.git
cd open-chat-playground
git checkout feat/261-ConnectorImplementation-Inheritance-AnthropicClaude

• Test the code

dotnet test test/OpenChat.PlaygroundApp.Tests/OpenChat.PlaygroundApp.Tests.csproj --filter FullyQualifiedName~AnthropicConnectorTests

What to Check

• Default model and token configuration applied correctly
• Exception handling for invalid settings
• Inheritance and type casting behavior validation
• All related unit tests pass successfully
• Docs updated and consistent with Anthropic connector section

Other Information about AnthropicConnectorTests.cs

생성/타입 관련

• Given_Null_Settings_When_Instantiated_Then_It_Should_Throw()
• Given_Settings_When_Instantiated_Then_It_Should_Return()
• Given_BaseType_Then_It_Should_Be_AssignableFrom_DerivedType()

EnsureLanguageModelSettingsValid()

• Given_Settings_Is_Null_When_EnsureLanguageModelSettingsValid_Invoked_Then_It_Should_Throw()
• Given_Invalid_ApiKey_When_EnsureLanguageModelSettingsValid_Invoked_Then_It_Should_Throw()
• Given_Invalid_Model_When_EnsureLanguageModelSettingsValid_Invoked_Then_It_Should_Throw()
• Given_Valid_Settings_When_EnsureLanguageModelSettingsValid_Invoked_Then_It_Should_Return_True()

GetChatClientAsync()

• Given_Valid_Settings_When_GetChatClientAsync_Invoked_Then_It_Should_Return_ChatClient()
• Given_Settings_Is_Null_When_GetChatClientAsync_Invoked_Then_It_Should_Throw()
• Given_Missing_ApiKey_When_GetChatClientAsync_Invoked_Then_It_Should_Throw()

CreateChatClientAsync()

• Given_Valid_Settings_When_CreateChatClientAsync_Invoked_Then_It_Should_Return_ChatClient()
• Given_Invalid_Settings_When_CreateChatClientAsync_Invoked_Then_It_Should_Throw()

[justinyoo]

@qoweh #494 PR이 머지가 됐으므로 테스트 코드는 이를 참고해서 작성해 주세욥! 촘촘하게 작성하셔야 합니다.

[qoweh]

@sikutisa 죄송합니다.. 각 이슈별로 처리하다가 현재는 3까지 진행한 상태입니다!

1. appsettings.json Modelling: Anthropic Claude #342 -> 해당 commit
2. Model Class Implementation: Anthropic Claude #258 -> 해당 commit
3. Command-Line Argument Parsing: Anthropic Claude #259 -> 해당 commit
4. Command-Line Argument Options Modelling: Anthropic Claude #341
5. Environment Variables Testing: Anthropic Claude #260

그리고 현재 브랜치 병합하면서 문제생긴 부분이 있는데 해결하고 댓글 남겨놓겠습니다!
-> ArgumentOptionConstants.cs에 추가 안 한 부분있어서 추가했습니다!

[sikutisa]

일단은 코드부터 보겠습니다. Connector는 작업이 다 안 된 것 같은데 맞나요?

[sikutisa]

수고하셨습니다!

[qoweh]

수고하셨습니다!

바쁘실텐데 죄송합니다 고생하셨습니다!!