fix: Use elif instead of if for Anthropic provider check

claude[bot] · abrookins · claude[bot] · commit 3df5d15a9c45 · 2025-08-01T21:30:52.000Z
This prevents both OpenAI and Anthropic client wrappers from being
created when the model is OpenAI, addressing the review comment.

Co-authored-by: Andrew Brookins &lt;abrookins@users.noreply.github.com&gt;
diff --git a/agent_memory_server/llms.py b/agent_memory_server/llms.py
@@ -410,7 +410,7 @@ async def get_model_client(
                 api_key=settings.openai_api_key,
                 base_url=settings.openai_api_base,
             )
-        if model_config.provider == ModelProvider.ANTHROPIC:
+        elif model_config.provider == ModelProvider.ANTHROPIC:
             model = AnthropicClientWrapper(
                 api_key=settings.anthropic_api_key,
                 base_url=settings.anthropic_api_base,

Original file line number	Diff line number	Diff line change
`@@ -410,7 +410,7 @@ async def get_model_client(`
`410`	`410`	`api_key=settings.openai_api_key,`
`411`	`411`	`base_url=settings.openai_api_base,`
`412`	`412`	`)`
`413`		`- if model_config.provider == ModelProvider.ANTHROPIC:`
	`413`	`+ elif model_config.provider == ModelProvider.ANTHROPIC:`
`414`	`414`	`model = AnthropicClientWrapper(`
`415`	`415`	`api_key=settings.anthropic_api_key,`
`416`	`416`	`base_url=settings.anthropic_api_base,`