Skip to content

fix: update Pi runtime for GPT-6 models #193

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.

Sign up for GitHub

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

Closed
QAyong wants to merge 1 commit into from
Closed

fix: update Pi runtime for GPT-6 models #193

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions apps/buddy/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,8 @@
},
"dependencies": {
"@anthropic-ai/sandbox-runtime": "0.0.77",
"@earendil-works/pi-ai": "^0.86.1",
"@earendil-works/pi-coding-agent": "^0.86.1",
"@earendil-works/pi-ai": "^0.87.1",
"@earendil-works/pi-coding-agent": "^0.87.1",
"@js-temporal/polyfill": "^0.5.1",
"@modelcontextprotocol/client": "2.0.0",
"@mozilla/readability": "^0.6.0",
Expand Down
3 changes: 3 additions & 0 deletions apps/buddy/shared/conversation/modelSelection.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,9 @@ const OPENAI_FAST_MODE_MODEL_IDS = new Set([
'gpt-5.6-luna',
'gpt-5.6-sol',
'gpt-5.6-terra',
'gpt-6-astra',
'gpt-6-luna',
'gpt-6-sol',
])

export interface BuddyServiceTierOption {
Expand Down
35 changes: 35 additions & 0 deletions patches/@earendil-works__pi-ai@0.87.1.patch
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
diff --git a/dist/api/openai-completions.js b/dist/api/openai-completions.js
--- a/dist/api/openai-completions.js
+++ b/dist/api/openai-completions.js
@@ -1141,6 +1141,14 @@ export function convertMessages(model, context, compat, options) {
}
lastRole = msg.role;
}
+ if (params.some((message) => message.role === "assistant" &&
+ typeof message.reasoning_content === "string")) {
+ for (const message of params) {
+ if (message.role === "assistant" && message.reasoning_content === undefined) {
+ message.reasoning_content = "";
+ }
+ }
+ }
return params;
}
function convertTools(tools, compat) {
diff --git a/dist/api/openai-responses-shared.js b/dist/api/openai-responses-shared.js
index a2049d5a1f633d7a96aa3cf080ded6310df8f97f..cd1f08e0f8300d04cae27662d8815731cd0a7083 100644
--- a/dist/api/openai-responses-shared.js
+++ b/dist/api/openai-responses-shared.js
@@ -355,7 +355,11 @@ export async function processResponsesStream(openaiStream, output, stream, model
}
if (item.type === "message") {
applyMessagePhaseStopReason(item);
- const block = { type: "text", text: "" };
+ const block = {
+ type: "text",
+ text: "",
+ textSignature: encodeTextSignatureV1(item.id, item.phase ?? undefined),
+ };
output.content.push(block);
const slot = { type: "text", block, contentIndex: output.content.length - 1 };
outputSlots.set(outputIndex, slot);
Loading