-
Notifications
You must be signed in to change notification settings - Fork 81
fix: 修复 Deepseek API 消息格式兼容性问题 #44
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
base: main
Are you sure you want to change the base?
fix: 修复 Deepseek API 消息格式兼容性问题 #44
Conversation
- 合并连续的用户消息(user role)为单条消息 - 确保符合 Deepseek API 的消息顺序要求
WalkthroughThe change consolidates two separate message entries in the Changes
Sequence Diagram(s)sequenceDiagram
participant Handler as process_summary
participant Diff as generate_diff_summary
Handler->>Handler: Combine FINAL_SUMMARY_PROMPT and final_summaries_content (separated by newlines)
Handler->>Diff: Send consolidated summary message
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🔇 Additional comments (1)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
当消息队列中出现多个连续的 user role 消息时,Deepseek API 会返回 400 Bad Request 错误,具体报错如下:
Summary by CodeRabbit