Skip to content

fix(llm): omit incompatible generation parameters - #43

Merged
Warma10032 merged 2 commits into
mainfrom
codex/fix-openai-gpt5-parameters
Aug 30, 2026
Merged

fix(llm): omit incompatible generation parameters#43
Warma10032 merged 2 commits into
mainfrom
codex/fix-openai-gpt5-parameters

Conversation

@Warma10032

@Warma10032 Warma10032 commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • OpenAI 兼容请求仅发送模型和消息。
  • 所有 LLM 调用路径均省略 temperature。
  • Anthropic 请求保留必填的 max_tokens。
  • 新增三条调用路径的请求参数回归测试。

Motivation

修复 GPT-5 调用失败问题,并避免新版模型因可选生成参数不兼容而拒绝请求。

Changes

  • src/services/llm/providers.ts 移除 OpenAI 兼容请求中的 max_tokens、max_completion_tokens 和 temperature。
  • src/services/llm/providers.ts 移除 Anthropic 与原生自定义 fetch 请求中的 temperature。
  • src/services/llm/types.ts、src/services/ai.ts 和 src/background.ts 移除不再使用的 temperature 负载字段。

Testing

  • npm test -- --run
  • npm run build

Risk

  • Medium risk. OpenAI 兼容服务使用服务端默认输出长度,应用不再传递输出上限。
  • Low risk. temperature 使用各服务的默认采样策略。

Notes

Fixes #42

@vercel

vercel Bot commented Aug 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
video-ad-guard Ready Ready Preview Aug 30, 2026 1:08pm

@Warma10032 Warma10032 changed the title fix(llm): support GPT-5 request parameters fix(llm): omit incompatible generation parameters Aug 30, 2026
@Warma10032
Warma10032 merged commit 8ea9718 into main Aug 30, 2026
5 checks passed
@Warma10032
Warma10032 deleted the codex/fix-openai-gpt5-parameters branch August 30, 2026 13:16
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.

对于使用OpenAI API时的gpt-5.6-luna模型,提示当前接口或模型不支持该请求参数

1 participant