Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add deepseek support #446

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

satoru2727
Copy link

#443
Added DeepSeek model support. Since I'm new to this project, please feel free to point out any issues. Context caching for DeepSeek (as discussed in #443 (comment)) is not implemented yet.

Add DeepSeekEngine to support DeepSeek API.  This
includes a new DeepSeekConfig interface and updates to
the engine selection logic.
Adds support for the DeepSeek AI provider.  Updates the README,
config validation, and model list to include DeepSeek.  This
allows users to utilize DeepSeek models with the OpenCommit
tool.
@satoru2727
Copy link
Author

According to DeepSeek, I found that we don't need any special prompt for context caching

@satoru2727 satoru2727 marked this pull request as ready for review January 30, 2025 14:08
@satoru2727 satoru2727 changed the base branch from master to dev January 31, 2025 08:55
@satoru2727 satoru2727 changed the base branch from dev to master January 31, 2025 08:58
The DeepSeekEngine constructor is refactored to use the
spread syntax for better readability and maintainability
when merging config parameters.  The baseURL is now
explicitly set within the constructor.
…pSeekEngineeekConfig and fix typo"

This reverts commit f492367.
src/engine/deepseek.ts Show resolved Hide resolved
♻️ refactor(engine): improve OpenAiEngine and create a
    new DeepSeekEngine class to handle DeepSeek API
    requests.  The DeepSeekEngine class inherits from
    OpenAiEngine and overrides the
    generateCommitMessage method to use the DeepSeek
    API.  This change improves code organization and
    maintainability.
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