dsh-provider-qoder 0.2.0 → 0.2.1

package/lib/index.d.ts

@@ -44,6 +44,7 @@ interface Config {
   /** @deprecated Migrated to modelsByRegion for the selected region. */
   models?: QoderCatalogModel[];
   streamIdleTimeoutMs?: number;
+  preserveThinking?: boolean;
 }
 declare const Config: z<Config>;
 //#endregion

package/lib/index.js

@@ -823,10 +823,7 @@ function validateMessageShapes(messages) {
 				if (message.role !== "assistant") throw unsupported("Qoder tool calls are valid only in assistant messages.");
 				continue;
 			}
-			if (block.type === "reasoning") {
-				if (message.role !== "assistant") throw unsupported("Qoder historical reasoning is valid only in assistant messages.");
-				continue;
-			}
+			if (block.type === "reasoning") continue;
 			throw unsupported(`Qoder transport encountered unsupported block type: ${String(block.type)}`);
 		}
 	}
@@ -881,6 +878,7 @@ async function validateAndTranslateMessages(messages, systemPrompt, attachments,
 		uploader: pipeline?.uploader,
 		credentials: pipeline?.credentials
 	};
+	const preserveThinking = pipeline?.preserveThinking ?? true;
 	const output = [];
 	if (typeof systemPrompt === "string" && systemPrompt.trim().length > 0) output.push({
 		role: "system",
@@ -911,6 +909,7 @@ async function validateAndTranslateMessages(messages, systemPrompt, attachments,
 			continue;
 		}
 		let text = "";
+		let reasoningText = "";
 		const userContent = [];
 		const pendingImages = [];
 		let hasImage = false;
@@ -944,14 +943,19 @@ async function validateAndTranslateMessages(messages, systemPrompt, attachments,
 				});
 				continue;
 			}
-			if (block.type === "reasoning") continue;
+			if (block.type === "reasoning") {
+				if (message.role === "assistant") reasoningText += block.text;
+				continue;
+			}
 		}
 		if (message.role === "assistant") {
-			if (!text && toolCalls.length === 0) continue;
+			const hasReasoning = preserveThinking && reasoningText.length > 0;
+			if (!text && toolCalls.length === 0 && !hasReasoning) continue;
 			output.push({
 				role: "assistant",
 				content: text || " ",
-				...toolCalls.length === 0 ? {} : { tool_calls: toolCalls }
+				...toolCalls.length === 0 ? {} : { tool_calls: toolCalls },
+				...hasReasoning ? { reasoning_content: reasoningText } : {}
 			});
 			continue;
 		}
@@ -2002,16 +2006,22 @@ async function* parseQoderSse(stream, options = {}) {
 								};
 								toolCalls.set(upstreamIndex, state);
 							}
-							if (rawCall.id !== void 0) {
-								if (typeof rawCall.id !== "string" || !rawCall.id) throw malformed("Qoder tool call has an invalid id.");
+							if (rawCall.id !== void 0) if (rawCall.id === "" || rawCall.id === null) {} else if (typeof rawCall.id !== "string") throw malformed("Qoder tool call has an invalid id.");
+							else {
 								if (state.id && state.id !== rawCall.id) throw malformed("Qoder changed a streamed tool-call id.");
 								state.id = rawCall.id;
 							}
+							let hasNewName = false;
 							if (rawCall.function?.name !== void 0) {
 								const name = rawCall.function.name;
-								if (typeof name !== "string" || !name) throw malformed("Qoder tool call has an invalid name.");
-								if (state.name && state.name !== name) throw malformed("Qoder changed a streamed tool-call name.");
-								state.name = name;
+								if (name === "" || name === null) {} else if (typeof name !== "string") throw malformed("Qoder tool call has an invalid name.");
+								else {
+									if (state.name && state.name !== name) throw malformed("Qoder changed a streamed tool-call name.");
+									if (state.name !== name) {
+										state.name = name;
+										hasNewName = true;
+									}
+								}
 							}
 							if (rawCall.function?.arguments !== void 0) {
 								if (typeof rawCall.function.arguments !== "string") throw malformed("Qoder tool-call arguments delta is not a string.");
@@ -2029,7 +2039,7 @@ async function* parseQoderSse(stream, options = {}) {
 									...state.name ? { name: state.name } : {},
 									argumentsDelta
 								};
-							} else if (wasOpen && state.blockIndex !== void 0 && rawCall.function?.name !== void 0) yield {
+							} else if (wasOpen && state.blockIndex !== void 0 && hasNewName) yield {
 								type: "tool-call-delta",
 								index: state.blockIndex,
 								id: ToolCallId(state.id),
@@ -2199,6 +2209,7 @@ var DefaultQoderTransport = class {
 	modelFlights = new SingleFlight();
 	attachments;
 	imageUploader;
+	preserveThinking;
 	constructor(options) {
 		this.region = options.region;
 		this.resolvePat = options.resolvePat;
@@ -2208,6 +2219,7 @@ var DefaultQoderTransport = class {
 		this.responseHeaderTimeoutMs = options.responseHeaderTimeoutMs ?? 3e4;
 		this.metadataTimeoutMs = options.metadataTimeoutMs;
 		this.attachments = options.attachments;
+		this.preserveThinking = options.preserveThinking;
 		this.auth = new QoderAuthService({
 			fetch: this.fetchImpl,
 			logger: this.logger,
@@ -2272,7 +2284,8 @@ var DefaultQoderTransport = class {
 		const credentials = await this.auth.getCredentials(pat, options.signal);
 		const messages = await translateQoderMessages(options, this.attachments, {
 			uploader: this.imageUploader,
-			credentials
+			credentials,
+			preserveThinking: this.preserveThinking
 		});
 		yield* streamQoderChat(options, model, await this.auth.getCredentials(pat, options.signal), messages, {
 			fetch: this.fetchImpl,
@@ -2319,7 +2332,8 @@ const Config = z.object({
 	region: z.union(["global", "china"]).default("global"),
 	modelsByRegion: modelsByRegionSchema.default({}),
 	models: z.array(catalogModel),
-	streamIdleTimeoutMs: z.number().step(1).min(1).default(defaultStreamIdleTimeoutMs)
+	streamIdleTimeoutMs: z.number().step(1).min(1).default(defaultStreamIdleTimeoutMs),
+	preserveThinking: z.boolean().default(true)
 });
 function resolveModels(models) {
 	const resolved = models ?? defaultModels;
@@ -2461,7 +2475,8 @@ function apply(ctx, config = {}) {
 		region: initialRegion,
 		modelsByRegion: { ...config.modelsByRegion },
 		...hasConfiguredModels ? { models: resolveModels(config.models) } : {},
-		streamIdleTimeoutMs: config.streamIdleTimeoutMs ?? 3e5
+		streamIdleTimeoutMs: config.streamIdleTimeoutMs ?? 3e5,
+		preserveThinking: config.preserveThinking ?? true
 	};
 	let current = () => baseConfig;
 	let legacyModelsRegion = initialRegion;
@@ -2470,12 +2485,13 @@ function apply(ctx, config = {}) {
 		global: [],
 		china: []
 	};
-	const createTransport = (region, streamIdleTimeoutMs, resolvePat = () => resolveManagedQoderPat(ctx.credentials)) => createQoderTransport({
+	const createTransport = (region, streamIdleTimeoutMs, preserveThinking, resolvePat = () => resolveManagedQoderPat(ctx.credentials)) => createQoderTransport({
 		region,
 		resolvePat,
 		logger,
 		streamIdleTimeoutMs,
-		attachments: ctx.attachments
+		attachments: ctx.attachments,
+		preserveThinking
 	});
 	const resolveConfig = () => {
 		const value = current();
@@ -2483,14 +2499,16 @@ function apply(ctx, config = {}) {
 		return {
 			region,
 			models: mergeQoderDiscoveryMetadata(modelsFor(value, region, legacyModelsRegion), discoveredCatalogs[region]),
-			streamIdleTimeoutMs: value.streamIdleTimeoutMs ?? 3e5
+			streamIdleTimeoutMs: value.streamIdleTimeoutMs ?? 3e5,
+			preserveThinking: value.preserveThinking ?? true
 		};
 	};
 	const initial = resolveConfig();
-	let activeTransport = createTransport(initial.region, initial.streamIdleTimeoutMs);
+	let activeTransport = createTransport(initial.region, initial.streamIdleTimeoutMs, initial.preserveThinking);
 	let activeTransportConfig = {
 		region: initial.region,
-		streamIdleTimeoutMs: initial.streamIdleTimeoutMs
+		streamIdleTimeoutMs: initial.streamIdleTimeoutMs,
+		preserveThinking: initial.preserveThinking
 	};
 	const adapter = new QoderAdapter({
 		resolveTransport: () => activeTransport,
@@ -2501,11 +2519,12 @@ function apply(ctx, config = {}) {
 	const registration = ctx.llm.registerAdapter([providerQoder], adapter);
 	const refreshAdapter = () => {
 		const next = resolveConfig();
-		if (next.region !== activeTransportConfig.region || next.streamIdleTimeoutMs !== activeTransportConfig.streamIdleTimeoutMs) {
-			activeTransport = createTransport(next.region, next.streamIdleTimeoutMs);
+		if (next.region !== activeTransportConfig.region || next.streamIdleTimeoutMs !== activeTransportConfig.streamIdleTimeoutMs || next.preserveThinking !== activeTransportConfig.preserveThinking) {
+			activeTransport = createTransport(next.region, next.streamIdleTimeoutMs, next.preserveThinking);
 			activeTransportConfig = {
 				region: next.region,
-				streamIdleTimeoutMs: next.streamIdleTimeoutMs
+				streamIdleTimeoutMs: next.streamIdleTimeoutMs,
+				preserveThinking: next.preserveThinking
 			};
 		}
 		adapter.replaceModels(next.models);
@@ -2559,7 +2578,7 @@ function apply(ctx, config = {}) {
 	const discoverModels = async (signal, suppliedPat) => {
 		const snapshot = resolveConfig();
 		const normalizedPat = suppliedPat?.trim();
-		const models = await (normalizedPat ? createTransport(snapshot.region, snapshot.streamIdleTimeoutMs, () => Promise.resolve(normalizedPat)) : activeTransport).discoverModels(signal);
+		const models = await (normalizedPat ? createTransport(snapshot.region, snapshot.streamIdleTimeoutMs, snapshot.preserveThinking, () => Promise.resolve(normalizedPat)) : activeTransport).discoverModels(signal);
 		discoveredCatalogs[snapshot.region] = models;
 		refreshAdapter();
 		await persistDiscoveredModels(snapshot.region);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "dsh-provider-qoder",
   "description": "Global Qoder subscription adapter for the DeepSeek Harness LLM seam",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "publishConfig": {
     "access": "public"
   },