Skip to content

feat: deepseek llm provider - #41

Merged
vinodvx merged 3 commits into
agenticenv:mainfrom
virinchiv:feat/deepseek-llm-provider
Jul 8, 2026
Merged

feat: deepseek llm provider#41
vinodvx merged 3 commits into
agenticenv:mainfrom
virinchiv:feat/deepseek-llm-provider

Conversation

@virinchiv

Copy link
Copy Markdown
Collaborator

Description

Adds DeepSeek as a built-in LLM provider (pkg/llm/deepseek) implementing interfaces.LLMClient, alongside the existing OpenAI, Anthropic, and Gemini clients.
DeepSeek exposes an OpenAI-compatible Chat Completions API, so the client uses the openai-go SDK purely as transport (no new dependency) but implements the interface independently.

  • Defaults the base URL to https://api.deepseek.com (overridable via llm.WithBaseURL).
  • Sends system prompts with the system role (DeepSeek rejects OpenAI's developer role).
  • Maps deepseek-reasoner's reasoning_content to LLMStreamChunk.ThinkingDelta (streaming) and LLMResponse.Metadata["reasoning_content"] (non-streaming).
  • Maps ResponseFormatJSON to json_object (DeepSeek doesn't support OpenAI's json_schema type; any Schema is ignored).

Registers the LLMProviderDeepSeek constant and wires the provider into the cmd and examples config factories. Supports deepseek-chat and deepseek-reasoner models.

Type of change

New feature (non-breaking change which adds functionality)

Related issues

Fixes #37

Checklist

  • [ Y ] I have run make check
  • [ Y ] I have run task examples:all
  • [ Y ] I have run make tidy if I added or removed dependencies
  • [ Y ] Commit messages follow conventional commits (e.g. feat:, fix:, docs:)
  • [ Y ] I have added/updated tests for my changes
  • [ Y ] Documentation is updated if needed

@virinchiv
virinchiv requested a review from vinodvx as a code owner July 7, 2026 20:54
@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 67.00508% with 65 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/llm/deepseek/client.go 67.00% 61 Missing and 4 partials ⚠️

📢 Thoughts on this report? Let us know!

@vinodvx

vinodvx commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Thanks for the contribution, this looks solid. Merging now.

@vinodvx
vinodvx merged commit c5e3081 into agenticenv:main Jul 8, 2026
4 checks passed
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.

[Feature]: Add DeepSeek as a built-in LLM provider

2 participants