Skip to content

Conversation

@tangbiubiu
Copy link

The deepseek provider has been added.
And the corresponding modifications have been made to the .env.example and README.

@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @tangbiubiu, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request introduces support for DeepSeek models by integrating a new DeepSeekProvider. This enhancement allows users to leverage DeepSeek's capabilities for both orchestrator and cypher model roles within the application. The changes also include necessary updates to the configuration examples and documentation to guide users on how to set up and utilize the new provider.

Highlights

  • DeepSeek Provider Integration: A new DeepSeekProvider class has been implemented, allowing the system to utilize DeepSeek models for both orchestration and cypher tasks.
  • Configuration Updates: The .env.example file now includes commented-out examples for configuring DeepSeek as both an orchestrator and cypher provider, complete with API key and model suggestions.
  • Documentation Enhancement: The README.md has been updated to reflect the new DeepSeek provider, including a dedicated section for 'All Deepseek Models' configuration and extending the list of supported providers for both ORCHESTRATOR_PROVIDER and CYPHER_PROVIDER variables.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces support for the DeepSeek provider, including configuration examples in .env.example and README.md, and the necessary provider implementation in codebase_rag/providers/base.py. The changes are well-structured. My feedback focuses on improving documentation consistency, ensuring type consistency in the new provider implementation, and encouraging the addition of unit tests for the new functionality to maintain code quality.

# CYPHER_MODEL=gemini-2.5-flash
# CYPHER_API_KEY=your-google-api-key

# Example 6: All DeepSeek
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

The example numbering here is inconsistent with the README.md file. This section is labeled "Example 6", but in the README.md it is introduced as "Option 4". To avoid user confusion, it would be best to align the numbering and naming convention across both files.

# Example 4: All DeepSeek

@tangbiubiu tangbiubiu marked this pull request as draft December 4, 2025 09:10
@vitali87
Copy link
Owner

vitali87 commented Dec 5, 2025

Thanks for this!

PR #161 adds LiteLLM support (coming soon), which handles DeepSeek along with 100+ other providers - see https://docs.litellm.ai/docs/providers/deepseek

We'd suggest closing this PR to avoid maintaining separate provider classes for each cloud service.

If you think there's value in having a dedicated DeepSeek provider that LiteLLM doesn't cover, let us know - happy to hear your thoughts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants