chore: code improvements

Author: fbuechler

packages/types/src/provider-settings.ts

@@ -296,6 +296,7 @@ const openAiNativeSchema = apiModelIdProviderModelSchema.extend({
 	// OpenAI Responses API service tier for openai-native provider only.
 	// UI should only expose this when the selected model supports flex/priority.
 	openAiNativeServiceTier: serviceTierSchema.optional(),
+	// Disable features that require identity verification.
 	openAiNativeUnverifiedOrg: z.boolean().optional(),
 })
 

src/api/providers/__tests__/openai-native.spec.ts

@@ -1954,4 +1954,51 @@ describe("Unverified org gating behavior", () => {
 		expect(body.reasoning?.effort).toBeDefined()
 		expect(body.reasoning?.summary).toBeUndefined()
 	})
+
+	it("include reasoning.summary in createMessage request when unverified org is false", async () => {
+		// Arrange
+		const handler = new OpenAiNativeHandler({
+			apiModelId: "gpt-5-2025-08-07",
+			openAiNativeApiKey: "test-api-key",
+			openAiNativeUnverifiedOrg: false, // => stream=true, and summary should be included
+		})
+
+		// Mock SDK to return a proper async iterable for streaming
+		const createMockStream = (chunks: any[]) => {
+			return {
+				async *[Symbol.asyncIterator]() {
+					for (const chunk of chunks) {
+						yield chunk
+					}
+				},
+			}
+		}
+
+		mockResponsesCreate.mockResolvedValueOnce(
+			createMockStream([
+				{ type: "response.text.delta", delta: "Test" },
+				{
+					type: "response.done",
+					response: { id: "resp_stream_1", usage: { prompt_tokens: 10, completion_tokens: 1 } },
+				},
+			]),
+		)
+
+		// Act
+		const systemPrompt = "You are a helpful assistant."
+		const messages: Anthropic.Messages.MessageParam[] = [{ role: "user", content: "Hello!" }]
+		const stream = handler.createMessage(systemPrompt, messages)
+		for await (const _ of stream) {
+			// drain
+		}
+
+		// Assert
+		expect(mockResponsesCreate).toHaveBeenCalledTimes(1)
+		const body = mockResponsesCreate.mock.calls[0][0]
+		expect(body.model).toBe("gpt-5-2025-08-07")
+		expect(body.stream).toBe(true)
+		// GPT-5 includes reasoning effort and summary should be "auto" when unverified org is false
+		expect(body.reasoning?.effort).toBeDefined()
+		expect(body.reasoning?.summary).toBe("auto")
+	})
 })

src/api/providers/openai-native.ts

@@ -221,11 +221,11 @@ export class OpenAiNativeHandler extends BaseProvider implements SingleCompletio
 		yield* this.executeRequest(requestBody, model, metadata, systemPrompt, messages)
 	}
 
-	private isStreamingAllowed(): boolean {
+	private canOrganizationUseStreaming(): boolean {
 		return !this.options.openAiNativeUnverifiedOrg
 	}
 
-	private isReasoningSummaryAllowed(): boolean {
+	private canOrganizationUseGpt5ReasoningSummary(): boolean {
 		return !!(this.options.enableGpt5ReasoningSummary && !this.options.openAiNativeUnverifiedOrg)
 	}
 
@@ -260,8 +260,8 @@ export class OpenAiNativeHandler extends BaseProvider implements SingleCompletio
 		const allowedTierNames = new Set(model.info.tiers?.map((t) => t.name).filter(Boolean) || [])
 
 		// Centralized gating for unverified organizations
-		const stream = this.isStreamingAllowed()
-		const enableGpt5ReasoningSummary = this.isReasoningSummaryAllowed()
+		const stream = this.canOrganizationUseStreaming()
+		const enableGpt5ReasoningSummary = this.canOrganizationUseGpt5ReasoningSummary()
 
 		const body: Gpt5RequestBody = {
 			model: model.id,
@@ -1427,7 +1427,7 @@ export class OpenAiNativeHandler extends BaseProvider implements SingleCompletio
 			if (reasoningEffort) {
 				requestBody.reasoning = {
 					effort: reasoningEffort,
-					...(this.isReasoningSummaryAllowed() ? { summary: "auto" as const } : {}),
+					...(this.canOrganizationUseGpt5ReasoningSummary() ? { summary: "auto" as const } : {}),
 				}
 			}