@opencode-ai/ai 0.0.0-dev-18263 → 0.0.0-dev-18266

package/dist/protocols/open-responses.d.ts

@@ -550,7 +550,6 @@ export declare const Event: Schema.StructWithRest<Schema.Struct<{
     readonly headers: Schema.optional<Schema.Unknown>;
 }>, readonly [Schema.$Record<Schema.String, Schema.Unknown>]>;
 export type Event = Schema.Schema.Type<typeof Event>;
-export type ItemKind = "message" | "reasoning" | "function-call" | "hosted-tool";
 export interface Extension {
     readonly id: string;
     readonly name: string;
@@ -560,7 +559,6 @@ export interface Extension {
         readonly request: LLMRequest;
     }) => MediaInput | undefined;
     readonly lowerHostedToolItem?: (item: unknown) => ExtendedHostedToolItem | undefined;
-    readonly acceptsItemID?: (kind: ItemKind, id: string) => boolean;
 }
 export interface ParserState {
     readonly id: string;

package/dist/protocols/open-responses.js

@@ -281,41 +281,36 @@ export const lowerToolChoice = (protocolName, toolChoice) => ProviderShared.matc
     required: () => "required",
     tool: (toolName) => ({ type: "function", name: toolName }),
 });
-// Servers validate item ids on replayed history, and a malformed or oversized
-// id can fail an otherwise valid request. Only server-issued tokens are worth
-// resending; anything else is treated as absent so the item is resent without
-// an id (or skipped, for items that cannot be expressed without one).
-const ITEM_ID_PATTERN = /^[A-Za-z0-9_-]{1,64}$/;
+// Server-issued item ids need a nonempty prefix and suffix, but the prefix is
+// provider-defined and does not necessarily identify the item's semantic type.
 const itemID = (providerMetadata, providerMetadataKey) => {
     const metadata = providerMetadata?.[providerMetadataKey];
-    return ProviderShared.isRecord(metadata) &&
-        typeof metadata.itemId === "string" &&
-        ITEM_ID_PATTERN.test(metadata.itemId)
-        ? metadata.itemId
-        : undefined;
+    if (!ProviderShared.isRecord(metadata) || typeof metadata.itemId !== "string")
+        return undefined;
+    const separator = metadata.itemId.indexOf("_");
+    return separator > 0 && separator < metadata.itemId.length - 1 ? metadata.itemId : undefined;
 };
-const acceptsItemID = (extension, kind, id) => id !== undefined && (extension.acceptsItemID?.(kind, id) ?? true);
-const lowerToolCall = (part, providerMetadataKey, extension) => {
+const lowerToolCall = (part, providerMetadataKey) => {
     const id = itemID(part.providerMetadata, providerMetadataKey);
     return {
         type: "function_call",
-        ...(acceptsItemID(extension, "function-call", id) ? { id } : {}),
+        ...(id === undefined ? {} : { id }),
         call_id: part.id,
         name: part.name,
         arguments: ProviderShared.encodeJson(part.input),
     };
 };
-const lowerReasoning = (part, providerMetadataKey, extension) => {
+const lowerReasoning = (part, providerMetadataKey) => {
     const metadata = part.providerMetadata?.[providerMetadataKey];
-    const id = itemID(part.providerMetadata, providerMetadataKey);
-    if (!ProviderShared.isRecord(metadata) || !acceptsItemID(extension, "reasoning", id))
+    if (!ProviderShared.isRecord(metadata))
         return undefined;
+    const id = itemID(part.providerMetadata, providerMetadataKey);
     const encryptedContent = typeof metadata.reasoningEncryptedContent === "string" || metadata.reasoningEncryptedContent === null
         ? metadata.reasoningEncryptedContent
         : undefined;
     return {
         type: "reasoning",
-        id,
+        ...(id === undefined ? {} : { id }),
         summary: part.text.length > 0 ? [{ type: "summary_text", text: part.text }] : [],
         encrypted_content: encryptedContent,
     };
@@ -400,8 +395,7 @@ const lowerMessages = Effect.fn("OpenResponses.lowerMessages")(function* (reques
                     return;
                 const groups = content.reduce((groups, part) => {
                     const metadata = part.providerMetadata?.[providerMetadataKey];
-                    const rawID = itemID(part.providerMetadata, providerMetadataKey);
-                    const id = acceptsItemID(extension, "message", rawID) ? rawID : undefined;
+                    const id = itemID(part.providerMetadata, providerMetadataKey);
                     const phase = ProviderShared.isRecord(metadata) ? messagePhase(metadata.phase) : undefined;
                     const group = groups.at(-1);
                     if (group && group.id === id && group.phase === phase)
@@ -426,17 +420,18 @@ const lowerMessages = Effect.fn("OpenResponses.lowerMessages")(function* (reques
                 }
                 if (part.type === "reasoning") {
                     flushText();
-                    const reasoning = lowerReasoning(part, providerMetadataKey, extension);
+                    const reasoning = lowerReasoning(part, providerMetadataKey);
                     if (!reasoning)
                         continue;
-                    const existing = reasoningItems[reasoning.id];
+                    const existing = reasoning.id === undefined ? undefined : reasoningItems[reasoning.id];
                     if (existing) {
                         existing.summary.push(...reasoning.summary);
                         if (typeof reasoning.encrypted_content === "string")
                             existing.encrypted_content = reasoning.encrypted_content;
                         continue;
                     }
-                    reasoningItems[reasoning.id] = reasoning;
+                    if (reasoning.id !== undefined)
+                        reasoningItems[reasoning.id] = reasoning;
                     input.push(reasoning);
                     continue;
                 }
@@ -444,7 +439,7 @@ const lowerMessages = Effect.fn("OpenResponses.lowerMessages")(function* (reques
                     flushText();
                     if (part.providerExecuted === true)
                         continue;
-                    input.push(lowerToolCall(part, providerMetadataKey, extension));
+                    input.push(lowerToolCall(part, providerMetadataKey));
                     continue;
                 }
                 if (part.type === "tool-result" && part.providerExecuted === true) {
@@ -455,7 +450,7 @@ const lowerMessages = Effect.fn("OpenResponses.lowerMessages")(function* (reques
                         : Schema.is(HostedToolItem)(part.result.value)
                             ? part.result.value
                             : extension.lowerHostedToolItem?.(part.result.value);
-                    if (acceptsItemID(extension, "hosted-tool", id) && hosted?.id === id) {
+                    if (id !== undefined && hosted?.id === id) {
                         if (!hostedToolItems.has(id)) {
                             input.push(hosted);
                             hostedToolItems.add(id);

package/dist/protocols/openai-responses.js

@@ -68,27 +68,10 @@ const OpenAIResponsesBody = Schema.Struct({
     ...OpenAIResponsesCoreFields,
     stream: Schema.Literal(true),
 });
-// Replayed items are paired with stored server state by id, so a foreign or
-// synthetic token can fail request validation even when `call_id` pairing is
-// intact. Only resend ids in each item kind's own grammar; hosted tool
-// items keep generic validation because every hosted tool mints its own
-// prefix. The same allowlist approach codex uses before resending history
-// (codex-rs core/src/client.rs, `prepare_response_items_for_request`).
-const ITEM_ID_PREFIXES = {
-    message: ["msg_"],
-    reasoning: ["rs_"],
-    "function-call": ["fc_"],
-    // Every hosted tool mints its own id prefix, so items keep generic validation.
-    "hosted-tool": [],
-};
 const extension = {
     id: ADAPTER,
     name: NAME,
     lowerHostedToolItem: (item) => (Schema.is(OpenAIResponsesHostedToolItem)(item) ? item : undefined),
-    acceptsItemID: (kind, id) => {
-        const prefixes = ITEM_ID_PREFIXES[kind];
-        return prefixes.length === 0 || prefixes.some((prefix) => id.startsWith(prefix));
-    },
 };
 const nativeImageToolInput = (tool) => {
     const native = tool.native?.openai;

package/package.json

@@ -1,6 +1,6 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
-  "version": "0.0.0-dev-18263",
+  "version": "0.0.0-dev-18266",
   "name": "@opencode-ai/ai",
   "type": "module",
   "license": "MIT",
@@ -30,7 +30,7 @@
   "devDependencies": {
     "@clack/prompts": "1.0.0-alpha.1",
     "@effect/platform-node": "4.0.0-rc.111",
-    "@opencode-ai/http-recorder": "0.0.0-dev-18263",
+    "@opencode-ai/http-recorder": "0.0.0-dev-18266",
     "@tsconfig/bun": "1.0.9",
     "@types/bun": "1.3.13",
     "@typescript/native-preview": "7.0.0-dev.20251207.1",
@@ -39,7 +39,7 @@
   "dependencies": {
     "@smithy/eventstream-codec": "4.2.14",
     "@smithy/util-utf8": "4.2.2",
-    "@opencode-ai/schema": "0.0.0-dev-18263",
+    "@opencode-ai/schema": "0.0.0-dev-18266",
     "aws4fetch": "1.0.20",
     "effect": "4.0.0-rc.111",
     "google-auth-library": "10.5.0"