singularity/singularity-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #1365 from jbrahy/fix/openai-reasoning-encrypted-content-always-include

Browse source 
fix: always include reasoning.encrypted_content for OpenAI reasoning models
This commit is contained in:
TÂCHES 2026-03-19 16:38:24 -06:00 • committed by GitHub
commit ad01d0ada4
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
.gitignore vendored
View file

@ -1,6 +1,6 @@
# ── GSD project state (development-only, lives in worktree branches) ──
package-lock.json
.claude/
RELEASE-GUIDE.md
*.tgz

2
packages/pi-ai/src/providers/openai-responses.ts
View file

@ -156,13 +156,13 @@ function buildParams(model: Model<"openai-responses">, context: Context, options
}
if (model.reasoning) {
params.include = ["reasoning.encrypted_content"];
if (options?.reasoningEffort || options?.reasoningSummary) {
const effort = clampReasoningForModel(model.name, options?.reasoningEffort || "medium") as typeof options.reasoningEffort;
params.reasoning = {
effort: effort || "medium",
summary: options?.reasoningSummary || "auto",
};
params.include = ["reasoning.encrypted_content"];
} else {
if (model.name.startsWith("gpt-5")) {
// Jesus Christ, see https://community.openai.com/t/need-reasoning-false-option-for-gpt-5/1351588/7