Merge branch 'feat/dte-5-orchestrator' of https://github.com/easonLiangWorldedtech/Zoo-Code into feat/dte-trial-all

Author: easonliang28

src/core/tools/NewTaskTool.ts

@@ -76,11 +76,16 @@ export class NewTaskTool extends BaseTool<"new_task"> {
 			const modelCapabilities = task.api.getModel().info.supportsReasoningEffort
 			// "disable" stays in the element type: capability arrays may carry it (it is a
 			// settings off-switch, not a start level) and is filtered where levels are listed.
+			// Registry capability arrays are never trusted blindly — normalize to the known
+			// level set so an unknown string cannot reach the ask payload or the child effort.
 			const supportedLevels: readonly (ReasoningEffortExtended | "disable")[] =
 				modelCapabilities === true
 					? NEW_TASK_EFFORT_LEVELS
 					: Array.isArray(modelCapabilities)
-						? modelCapabilities
+						? modelCapabilities.filter(
+								(level): level is ReasoningEffortExtended | "disable" =>
+									level === "disable" || isNewTaskEffortLevel(level),
+							)
 						: []
 			// "disable" is a settings off-switch, never a start level: filtering it can
 			// leave an empty list (a model whose only capability is "disable"). In that

src/core/tools/__tests__/newTaskThinkingEffort.spec.ts

@@ -279,6 +279,44 @@ describe("new_task thinking_effort validation (DTE series 5/5)", () => {
 		expect(delegateParentAndOpenChild).not.toHaveBeenCalled()
 	})
 
+	it("filters unknown capability strings out of the supported levels (DTE series 5/5)", async () => {
+		const { task } = makeTask({
+			supportsReasoningEffort: ["turbo", "low", "disable"],
+			askEffort: "low",
+		})
+		const callbacks = makeCallbacks()
+
+		await runNewTask(task, { thinking_effort: "low" }, callbacks)
+
+		// Registry data is normalized to the known level set: the unknown "turbo"
+		// and the settings off-switch "disable" never reach the ask payload.
+		expect(callbacks.askApproval).toHaveBeenCalledTimes(1)
+		const [askType, toolMessage] = vi.mocked(callbacks.askApproval).mock.calls[0]
+		expect(askType).toBe("tool")
+		const payload = JSON.parse(toolMessage as string) as {
+			tool: string
+			supportedThinkingEfforts?: string[]
+		}
+		expect(payload.tool).toBe("newTask")
+		expect(payload.supportedThinkingEfforts).toEqual(["low"])
+	})
+
+	it("treats a capability array of only unknown values as unsupported (DTE series 5/5)", async () => {
+		const { task, delegateParentAndOpenChild } = makeTask({
+			supportsReasoningEffort: ["turbo"],
+		})
+		const callbacks = makeCallbacks()
+
+		await runNewTask(task, { thinking_effort: "low" }, callbacks)
+
+		// Normalization drops the unknown value, leaving no start level: the
+		// unsupported-model wording is the accurate hint.
+		expect(callbacks.pushToolResult).toHaveBeenCalledWith(
+			expect.stringContaining("does not support thinking_effort"),
+		)
+		expect(delegateParentAndOpenChild).not.toHaveBeenCalled()
+	})
+
 	it("pre-fills the ask payload with the effort and the supported levels, filtering 'disable'", async () => {
 		const { task } = makeTask({
 			supportsReasoningEffort: ["disable", "low", "medium"],