dsh-plugin-effort-declare 0.1.0 → 0.1.2

package/lib/client.js

@@ -12,9 +12,11 @@ window.__ModuleLoader__.load({
 		*
 		* Levels match `@deepseek-ai/dsh-llm-pi-ai` catalog.ts `THINKING_LEVELS`.
 		* Formats match `SUPPORTED_THINKING_FORMATS` in the same file (rc.8).
-		* Runtime UI prefers the live settings schema union; these lists are the
-		* fallback plus a test pin so a silent drift is a failing test, not a
-		* second hand-maintained copy nobody notices.
+		* Tests pin these lists against a checked-in schema fixture
+		* (`tests/fixtures/pi-ai-thinking-format-union.ts`) and the local level
+		* whitelist. The settings page never offers the handwritten thinkingFormat
+		* list as writable choices — only the live schema union, plus a stored
+		* value that the union omitted.
 		*/
 		/** Selectable reasoning levels, in pi-ai escalation order. */
 		const THINKING_LEVELS = [
@@ -28,22 +30,6 @@ window.__ModuleLoader__.load({
 		];
 		/** Thinking levels other than Off; Off has its own tri-state control. */
 		const THINKING_LEVELS_WITHOUT_OFF = THINKING_LEVELS.filter((level) => level !== "off");
-		/**
-		* openai-completions thinkingFormat values from llm-pi-ai catalog.ts rc.8.
-		* Tests pin this list; the settings page prefers schema union choices.
-		*/
-		const FALLBACK_THINKING_FORMATS = [
-			"openai",
-			"deepseek",
-			"openrouter",
-			"together",
-			"zai",
-			"qwen",
-			"chat-template",
-			"qwen-chat-template",
-			"string-thinking",
-			"ant-ling"
-		];
 		/** Wire protocol this plugin's v1 editor supports. */
 		const OPENAI_COMPLETIONS = "openai-completions";
 		/** Settings namespace this plugin writes. */
@@ -172,6 +158,14 @@ window.__ModuleLoader__.load({
 		* Route-card draft merge: user-layer slices, namespace revision, dirty preserve.
 		* No React — settings UI and tests share these helpers.
 		*/
+		/** JSON-stable equality matching pathOps (key order included). */
+		function sliceEqual(left, right) {
+			return JSON.stringify(left) === JSON.stringify(right);
+		}
+		/** Whether two settings slices differ. */
+		function sliceChanged(before, after) {
+			return !sliceEqual(before, after);
+		}
 		/** Build a draft from the stored user subtree (never from effective `value`). */
 		function routeDraftFromUserProfile(args) {
 			const { provider, displayName, settingsPath, revision, userProfile } = args;
@@ -226,9 +220,103 @@ window.__ModuleLoader__.load({
 				});
 			});
 		}
+		function modelRowId(row) {
+			return String(row.id);
+		}
+		function indexById(rows) {
+			const map = /* @__PURE__ */ new Map();
+			for (const row of rows) {
+				const id = modelRowId(row);
+				if (!map.has(id)) map.set(id, row);
+			}
+			return map;
+		}
+		function effortsPresence(row) {
+			if (row === void 0 || !Object.hasOwn(row, "reasoningEfforts")) return {
+				present: false,
+				value: void 0
+			};
+			return {
+				present: true,
+				value: row.reasoningEfforts
+			};
+		}
+		function effortsEqual(left, right) {
+			if (left.present !== right.present) return false;
+			if (!left.present) return true;
+			return sliceEqual(left.value, right.value);
+		}
+		function overlayLocalEfforts(incomingRow, prevRow) {
+			const next = structuredClone(incomingRow);
+			if (Object.hasOwn(prevRow, "reasoningEfforts")) next.reasoningEfforts = structuredClone(prevRow.reasoningEfforts);
+			else delete next.reasoningEfforts;
+			return next;
+		}
+		function objectKeyChanged(left, right, key) {
+			const leftHas = Object.hasOwn(left, key);
+			if (leftHas !== Object.hasOwn(right, key)) return true;
+			if (!leftHas) return false;
+			return sliceChanged(left[key], right[key]);
+		}
 		/**
-		* Apply a freshly loaded table. Dirty cards keep models/compat; originals and
-		* revision follow the incoming snapshot so a later save is against the new user layer.
+		* Membership follows the latest user-layer models list (Models page add/delete).
+		* Local unsaved `reasoningEfforts` (including a cleared key) overlay by id.
+		*/
+		function mergeModelsById(args) {
+			const prevById = indexById(args.prevModels);
+			const prevOrigById = indexById(args.prevOriginal);
+			const incomingOrigById = indexById(args.incomingOriginal);
+			const incomingIds = new Set(args.incomingModels.map(modelRowId));
+			let conflicted = false;
+			const models = args.incomingModels.map((incomingRow) => {
+				const id = modelRowId(incomingRow);
+				const prevRow = prevById.get(id);
+				if (prevRow === void 0) return structuredClone(incomingRow);
+				const prevOrig = prevOrigById.get(id);
+				if (!!effortsEqual(effortsPresence(prevRow), effortsPresence(prevOrig))) return structuredClone(incomingRow);
+				const incomingOrig = incomingOrigById.get(id);
+				if (!effortsEqual(effortsPresence(prevOrig), effortsPresence(incomingOrig))) conflicted = true;
+				return overlayLocalEfforts(incomingRow, prevRow);
+			});
+			for (const [id, prevRow] of prevById) {
+				if (incomingIds.has(id)) continue;
+				const prevOrig = prevOrigById.get(id);
+				if (effortsEqual(effortsPresence(prevRow), effortsPresence(prevOrig))) continue;
+				if (!effortsEqual(effortsPresence(prevOrig), effortsPresence(incomingOrigById.get(id)))) conflicted = true;
+			}
+			return {
+				models,
+				conflicted
+			};
+		}
+		/**
+		* Three-way compat merge: locally changed keys stay local; everything else
+		* follows incoming. Conflict only when a locally dirty key also moved in originals.
+		*/
+		function mergeCompat(args) {
+			if (!sliceChanged(args.prev, args.prevOriginal)) return {
+				compat: cloneObject(args.incoming),
+				conflicted: false
+			};
+			const compat = cloneObject(args.incoming);
+			let conflicted = false;
+			const keys = /* @__PURE__ */ new Set([...Object.keys(args.prev), ...Object.keys(args.prevOriginal)]);
+			for (const key of keys) {
+				if (!objectKeyChanged(args.prev, args.prevOriginal, key)) continue;
+				if (objectKeyChanged(args.prevOriginal, args.incomingOriginal, key)) conflicted = true;
+				if (Object.hasOwn(args.prev, key)) compat[key] = structuredClone(args.prev[key]);
+				else delete compat[key];
+			}
+			return {
+				compat,
+				conflicted
+			};
+		}
+		/**
+		* Apply a freshly loaded table. Membership and metadata follow incoming;
+		* unsaved reasoningEfforts / dirty compat keys overlay by id. Conflict only
+		* when a locally dirty field also changed in originals (revision-only bumps
+		* and sibling-card saves do not warn).
 		*/
 		function mergeLoadedDrafts(current, incoming, options) {
 			const currentByProvider = new Map(current.map((draft) => [draft.provider, draft]));
@@ -237,13 +325,27 @@ window.__ModuleLoader__.load({
 				drafts: incoming.map((next) => {
 					const prev = currentByProvider.get(next.provider);
 					if (prev === void 0 || !options.preserveDirty || !draftDirty(prev)) return next;
-					conflicted.push(next.provider);
+					const modelsMerge = mergeModelsById({
+						prevModels: prev.models,
+						prevOriginal: prev.originalModels,
+						incomingModels: next.models,
+						incomingOriginal: next.originalModels
+					});
+					const compatMerge = mergeCompat({
+						prev: prev.compat,
+						prevOriginal: prev.originalCompat,
+						incoming: next.compat,
+						incomingOriginal: next.originalCompat
+					});
+					if (modelsMerge.conflicted || compatMerge.conflicted) conflicted.push(next.provider);
 					return {
-						...prev,
+						provider: next.provider,
 						displayName: next.displayName,
 						settingsPath: next.settingsPath,
 						revision: next.revision,
+						models: modelsMerge.models,
 						originalModels: cloneModels(next.originalModels),
+						compat: compatMerge.compat,
 						originalCompat: cloneObject(next.originalCompat),
 						compatPresent: next.compatPresent
 					};
@@ -434,25 +536,28 @@ window.__ModuleLoader__.load({
 		/**
 		* First paint: `ensure()` (reads only from idle). Never treat ensure as refresh.
 		* Callers that must not apply a stale settlement compare generation themselves.
+		*
+		* `formats` is the live schema union only. Empty means the dropdown has no
+		* writable choices (stored values stay visible via `thinkingFormatChoices`).
 		*/
 		async function loadDrafts(api, describe, schema) {
 			await describe.ensure();
 			const mirrored = describe.getSnapshot();
 			if (mirrored.view === void 0) return {
 				writable: false,
-				formats: [...FALLBACK_THINKING_FORMATS],
+				formats: [],
 				drafts: [],
 				error: mirrored.error ?? void 0
 			};
 			const providersResponse = await api.llm.providers({});
 			if (!providersResponse.result.ok) return {
 				writable: mirrored.view.writable,
-				formats: [...FALLBACK_THINKING_FORMATS],
+				formats: [],
 				drafts: [],
 				error: providersResponse.result.error.message
 			};
 			const pi = new Map(mirrored.view.namespaces.map((view) => [view.ns, view])).get(LLM_PI_AI_NS);
-			let formats = [...FALLBACK_THINKING_FORMATS];
+			let formats = [];
 			let schemaDefaultApi;
 			if (pi !== void 0) try {
 				const root = schema.rehydrate(pi.schema);
@@ -488,6 +593,36 @@ window.__ModuleLoader__.load({
 			};
 		}
 		//#endregion
+		//#region src/client/schema-ops.ts
+		/** Wrap a live settingsSchema service as plain callbacks. */
+		function bindSchema(service) {
+			return {
+				rehydrate: (serialized) => service.rehydrate(serialized),
+				nodeAtPath: (root, path) => service.nodeAtPath(root, path),
+				getPath: (value, path) => service.getPath(value, path),
+				hasPath: (value, path) => service.hasPath(value, path),
+				validate: (node, draft) => service.validate(node, draft)
+			};
+		}
+		/**
+		* Pre-mutate schema check used by the settings page.
+		* A returned string means do not call `settings.mutate`.
+		*/
+		function validateSaveDraft(schema, root, settingsPath, afterModels, afterCompat, willWriteCompat) {
+			const modelsNode = schema.nodeAtPath(root, [...settingsPath, "models"]);
+			if (modelsNode !== void 0) {
+				const error = schema.validate(modelsNode, afterModels);
+				if (error !== void 0) return error;
+			}
+			if (willWriteCompat) {
+				const compatNode = schema.nodeAtPath(root, [...settingsPath, "compat"]);
+				if (compatNode !== void 0) {
+					const error = schema.validate(compatNode, afterCompat);
+					if (error !== void 0) return error;
+				}
+			}
+		}
+		//#endregion
 		//#region \0dsh-css:C:\Users\zimo\AppData\Roaming\io.github.hairyf.deepseek-harness-desktop\data\dsh\临时目录\dsh-plugin-effort-declare\src\client\effort-declare.module.css.mjs
 		const cssText = ".hYEBUa_section{max-width:720px;color:var(--dsw-alias-label-primary);flex-direction:column;gap:12px;display:flex}.hYEBUa_title{color:var(--dsw-alias-label-primary);margin:0;font-size:16px;font-weight:500;line-height:24px}.hYEBUa_intro{color:var(--dsw-alias-label-tertiary);margin:0;font-size:14px;line-height:22px}.hYEBUa_notice{color:var(--dsw-alias-state-warn-label);margin:0;font-size:12px;line-height:18px}.hYEBUa_savedNotice{color:var(--dsw-alias-state-success-primary);margin:0;font-size:12px;line-height:18px}.hYEBUa_error{color:var(--dsw-alias-state-error-primary);margin:0;font-size:12px;line-height:18px}.hYEBUa_rows{flex-direction:column;gap:8px;margin:12px 0 0;padding:0;list-style:none;display:flex}.hYEBUa_rowCard{border:1px solid var(--dsw-alias-border-l2);border-radius:12px;flex-direction:column;gap:12px;padding:12px 14px;display:flex}.hYEBUa_rowHead{align-items:baseline;gap:8px;display:flex}.hYEBUa_rowName{color:var(--dsw-alias-label-primary);font-size:14px;font-weight:500;line-height:22px}.hYEBUa_rowTag{border:1px solid var(--dsw-alias-border-l3);color:var(--dsw-alias-label-secondary);border-radius:4px;flex:none;padding:1px 6px;font-size:11px;line-height:16px}.hYEBUa_compatSummary{color:var(--dsw-alias-label-tertiary);margin:0;font-size:12px;line-height:18px}.hYEBUa_presetRow,.hYEBUa_actions{flex-wrap:wrap;align-items:center;gap:8px;display:flex}.hYEBUa_fieldLabel{color:var(--dsw-alias-label-secondary);font-size:12px;font-weight:500;line-height:18px}.hYEBUa_primaryButton,.hYEBUa_secondaryButton{box-sizing:border-box;height:36px;font:inherit;cursor:pointer;border-radius:18px;justify-content:center;align-items:center;padding:0 14px;font-size:14px;line-height:22px;display:inline-flex}.hYEBUa_primaryButton{background:var(--dsw-alias-button-primary-fill);color:var(--dsw-alias-label-primary-foreground);border:none}.hYEBUa_primaryButton:hover:not(:disabled){background:var(--dsw-alias-button-primary-hover)}.hYEBUa_secondaryButton{border:1px solid var(--dsw-alias-border-l2);color:var(--dsw-alias-label-primary);background:0 0}.hYEBUa_secondaryButton:hover:not(:disabled){background:var(--dsw-alias-interactive-bg-hover-solid)}.hYEBUa_primaryButton:disabled,.hYEBUa_secondaryButton:disabled,.hYEBUa_linkButton:disabled,.hYEBUa_input:disabled{opacity:.4;cursor:default}.hYEBUa_primaryButton:focus-visible,.hYEBUa_secondaryButton:focus-visible,.hYEBUa_linkButton:focus-visible,.hYEBUa_input:focus-visible{box-shadow:0 0 0 2px var(--dsw-alias-border-l3);outline:none}.hYEBUa_linkButton{box-sizing:border-box;height:28px;color:var(--dsw-alias-label-tertiary);font:inherit;cursor:pointer;background:0 0;border:none;border-radius:14px;align-items:center;padding:0 10px;font-size:12px;line-height:18px;display:inline-flex}.hYEBUa_linkButton:hover:not(:disabled){background:var(--dsw-alias-interactive-bg-hover);color:var(--dsw-alias-label-secondary)}.hYEBUa_modelEntry{border-top:1px solid var(--dsw-alias-border-l2);flex-direction:column;gap:8px;padding:10px 0;display:flex}.hYEBUa_modelHead{flex-wrap:wrap;align-items:baseline;gap:8px;display:flex}.hYEBUa_modelId{font-size:13px;font-weight:500;line-height:20px}.hYEBUa_modelName{color:var(--dsw-alias-label-tertiary);font-size:12px;line-height:18px}.hYEBUa_levels{flex-wrap:wrap;gap:10px 14px;display:flex}.hYEBUa_level{align-items:center;gap:6px;font-size:12px;line-height:18px;display:inline-flex}.hYEBUa_wireRow{flex-wrap:wrap;align-items:center;gap:8px;display:flex}.hYEBUa_input,.hYEBUa_selectInput{box-sizing:border-box;border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-module-platform);height:32px;color:var(--dsw-alias-label-primary);font:inherit;border-radius:8px;padding:0 10px;font-size:13px}.hYEBUa_wireInput{width:7em}.hYEBUa_offGroup{flex-direction:column;gap:6px;display:flex}.hYEBUa_advanced{border-top:1px solid var(--dsw-alias-border-l2);padding-top:8px}.hYEBUa_advanced summary{cursor:pointer;color:var(--dsw-alias-label-secondary);font-size:12px;line-height:18px}.hYEBUa_advancedBody{flex-direction:column;gap:10px;padding-top:10px;display:flex}.hYEBUa_check{align-items:flex-start;gap:8px;font-size:12px;line-height:18px;display:flex}";
 		const cssTagId = "dsh-plugin-effort-declare/effort-declare.module.css";
@@ -653,10 +788,10 @@ window.__ModuleLoader__.load({
 			});
 		}
 		function RouteCard(props) {
-			const { draft, writable, busy, reloading, t, onChange } = props;
+			const { draft, writable, busy, saveLocked, t, onChange } = props;
 			const noModels = draft.models.length === 0;
 			const editDisabled = !writable || busy || noModels;
-			const saveLocked = editDisabled || reloading;
+			const saveDisabled = saveLocked || noModels;
 			const formats = thinkingFormatChoices(props.formats, draft.compat.thinkingFormat);
 			const summary = compatSummary(draft.compat);
 			const sameWire = draft.compat.supportsReasoningEffort === false;
@@ -840,7 +975,7 @@ window.__ModuleLoader__.load({
 						}), /* @__PURE__ */ (0, react_jsx_runtime.jsx)("button", {
 							type: "button",
 							className: effort_declare_module_css_default.primaryButton,
-							disabled: saveLocked || !dirty || clientError !== void 0,
+							disabled: saveDisabled || !dirty || clientError !== void 0,
 							onClick: () => {
 								props.onSave(draft);
 							},
@@ -872,12 +1007,12 @@ window.__ModuleLoader__.load({
 					return;
 				}
 				const generation = nextGeneration(generationRef);
-				setStatus("loading");
+				if (draftsRef.current.length === 0) setStatus("loading");
 				setError("");
 				loadDrafts(api, describe, schema).then((result) => {
 					if (!generationIsCurrent(generationRef, generation)) return;
 					setWritable(result.writable);
-					if (result.formats.length > 0) setFormats(result.formats);
+					setFormats(result.formats);
 					if (result.error !== void 0) {
 						setStatus("error");
 						setError(result.error);
@@ -885,10 +1020,14 @@ window.__ModuleLoader__.load({
 					}
 					const merged = mergeLoadedDrafts(draftsRef.current, result.drafts, { preserveDirty });
 					setDrafts(merged.drafts);
-					if (merged.conflicted.length > 0) setNotices(Object.fromEntries(merged.conflicted.map((provider) => [provider, {
-						kind: "conflict",
-						text: t("dirtyConflict")
-					}])));
+					if (merged.conflicted.length > 0) setNotices((current) => {
+						const next = { ...current };
+						for (const provider of merged.conflicted) next[provider] = {
+							kind: "conflict",
+							text: t("dirtyConflict")
+						};
+						return next;
+					});
 					setStatus("ready");
 				}, (failure) => {
 					if (!generationIsCurrent(generationRef, generation)) return;
@@ -906,23 +1045,46 @@ window.__ModuleLoader__.load({
 			}, [reload]);
 			(0, react.useEffect)(() => {
 				if (props.subscribeInvalidate === void 0) return void 0;
-				return props.subscribeInvalidate(() => {
-					reload(true);
+				return props.subscribeInvalidate((source) => {
+					if (source === "writable") {
+						const view = describe?.getSnapshot().view;
+						if (view !== void 0) setWritable(view.writable);
+						return;
+					}
+					if (source === "settings" || source === "directory") reload(true);
+				});
+			}, [
+				describe,
+				props.subscribeInvalidate,
+				reload
+			]);
+			const patchNotice = (provider, notice) => {
+				setNotices((current) => {
+					const copy = { ...current };
+					if (notice === void 0) delete copy[provider];
+					else copy[provider] = notice;
+					return copy;
 				});
-			}, [props.subscribeInvalidate, reload]);
+			};
 			const save = async (draft) => {
-				if (api === void 0 || describe === void 0) return;
-				if (status === "loading" || busyRoute !== null) return;
+				if (api === void 0 || describe === void 0 || schema === void 0) return;
+				if (status === "loading" || busyRoute !== null) {
+					patchNotice(draft.provider, {
+						kind: "error",
+						text: t("saveBusy")
+					});
+					return;
+				}
 				const blocking = draft.models.map((row) => errorText(modelEffortError(row), t)).find((text) => text !== void 0);
 				if (blocking !== void 0) {
-					setNotices({ [draft.provider]: {
+					patchNotice(draft.provider, {
 						kind: "error",
 						text: blocking
-					} });
+					});
 					return;
 				}
 				setBusyRoute(draft.provider);
-				setNotices({});
+				patchNotice(draft.provider, void 0);
 				try {
 					const ops = buildSaveOps({
 						settingsPath: draft.settingsPath,
@@ -935,16 +1097,38 @@ window.__ModuleLoader__.load({
 						setDrafts((current) => current.map((row) => row.provider === draft.provider ? alignDraft(row) : row));
 						return;
 					}
+					const willWriteCompat = ops.some((op) => op.path.length > draft.settingsPath.length && op.path[draft.settingsPath.length] === "compat");
+					const pi = describe.getSnapshot().view?.namespaces.find((view) => view.ns === LLM_PI_AI_NS);
+					if (pi !== void 0) {
+						let root;
+						try {
+							root = schema.rehydrate(pi.schema);
+						} catch {
+							root = void 0;
+						}
+						if (root !== void 0) {
+							const schemaError = validateSaveDraft(schema, root, draft.settingsPath, draft.models, draft.compat, willWriteCompat);
+							if (schemaError !== void 0) {
+								patchNotice(draft.provider, {
+									kind: "error",
+									text: schemaError
+								});
+								return;
+							}
+						}
+					}
 					const response = await api.settings.mutate({
 						ns: LLM_PI_AI_NS,
 						ops,
 						expectedRevision: draft.revision
 					});
 					if (!response.result.ok) {
-						setNotices({ [draft.provider]: {
-							kind: "error",
-							text: response.result.error.code === "settings-conflict" ? t("conflict") : response.result.error.message
-						} });
+						const conflict = response.result.error.code === "settings-conflict";
+						patchNotice(draft.provider, {
+							kind: conflict ? "conflict" : "error",
+							text: conflict ? t("conflict") : response.result.error.message
+						});
+						if (conflict) reload(true);
 						return;
 					}
 					const view = response.result.value;
@@ -953,15 +1137,15 @@ window.__ModuleLoader__.load({
 						user: view.user ?? {},
 						revision: view.revision
 					}));
-					setNotices({ [draft.provider]: {
+					patchNotice(draft.provider, {
 						kind: "saved",
 						text: t("saved")
-					} });
+					});
 				} catch (failure) {
-					setNotices({ [draft.provider]: {
+					patchNotice(draft.provider, {
 						kind: "error",
 						text: failure instanceof Error ? failure.message : t("loadError")
-					} });
+					});
 				} finally {
 					setBusyRoute(null);
 				}
@@ -969,6 +1153,8 @@ window.__ModuleLoader__.load({
 			const showLoading = status === "loading" && drafts.length === 0;
 			const showEmpty = status === "ready" && drafts.length === 0;
 			const showList = drafts.length > 0;
+			const hasCardFailure = Object.values(notices).some((notice) => notice.kind === "conflict" || notice.kind === "error");
+			const showReload = status === "error" || hasCardFailure || showEmpty || !writable || status === "loading" || showList;
 			return /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("div", {
 				className: effort_declare_module_css_default.section,
 				children: [
@@ -992,7 +1178,7 @@ window.__ModuleLoader__.load({
 						className: effort_declare_module_css_default.error,
 						children: error
 					}) : null,
-					status === "error" ? /* @__PURE__ */ (0, react_jsx_runtime.jsx)("button", {
+					showReload ? /* @__PURE__ */ (0, react_jsx_runtime.jsx)("button", {
 						type: "button",
 						className: effort_declare_module_css_default.secondaryButton,
 						onClick: () => {
@@ -1014,26 +1200,18 @@ window.__ModuleLoader__.load({
 							formats: formats.length > 0 ? formats : [],
 							writable,
 							busy: busyRoute === draft.provider,
-							reloading: status === "loading",
+							saveLocked: !writable || busyRoute !== null || status === "loading",
 							notice: notices[draft.provider],
 							t,
 							onChange: (next) => {
-								setNotices((current) => {
-									const copy = { ...current };
-									delete copy[next.provider];
-									return copy;
-								});
+								patchNotice(next.provider, void 0);
 								setDrafts((current) => current.map((row) => row.provider === next.provider ? next : row));
 							},
 							onSave: (next) => {
 								save(next);
 							},
 							onCancel: (next) => {
-								setNotices((current) => {
-									const copy = { ...current };
-									delete copy[next.provider];
-									return copy;
-								});
+								patchNotice(next.provider, void 0);
 								setDrafts((current) => current.map((row) => row.provider === next.provider ? {
 									...row,
 									models: cloneModels(row.originalModels),
@@ -1046,17 +1224,6 @@ window.__ModuleLoader__.load({
 			});
 		}
 		//#endregion
-		//#region src/client/schema-ops.ts
-		/** Wrap a live settingsSchema service as plain callbacks. */
-		function bindSchema(service) {
-			return {
-				rehydrate: (serialized) => service.rehydrate(serialized),
-				nodeAtPath: (root, path) => service.nodeAtPath(root, path),
-				getPath: (value, path) => service.getPath(value, path),
-				hasPath: (value, path) => service.hasPath(value, path)
-			};
-		}
-		//#endregion
 		//#region src/client/locales.ts
 		const NS = "plugin-effort-declare";
 		const zh = {
@@ -1070,6 +1237,7 @@ window.__ModuleLoader__.load({
 			readOnly: "当前设置为只读，无法保存。",
 			save: "保存",
 			saving: "保存中…",
+			saveBusy: "另有路由正在保存，请稍候。",
 			cancel: "取消",
 			saved: "已保存。对话选择器会按新的能力声明显示 Effort 行。",
 			conflict: "设置已被其他地方改过，请重新加载后再保存。",
@@ -1090,7 +1258,7 @@ window.__ModuleLoader__.load({
 			clear: "清除本模型声明",
 			advanced: "高级：协议方言",
 			thinkingFormat: "thinkingFormat",
-			thinkingFormatDefault: "默认（openai）",
+			thinkingFormatDefault: "默认（省略该键）",
 			supportsDeveloperRole: "系统提示走 system 而不是 developer（supportsDeveloperRole: false）",
 			supportsReasoningEffort: "不发 reasoning_effort，只发开关（supportsReasoningEffort: false）",
 			developerTrueHint: "当前文档是 supportsDeveloperRole: true。v1 只能强制 false 或缺席；勾选会写成 false，取消勾选会删除该键。",
@@ -1112,6 +1280,7 @@ window.__ModuleLoader__.load({
 			readOnly: "Settings are read-only; saving is disabled.",
 			save: "Save",
 			saving: "Saving…",
+			saveBusy: "Another route is saving. Wait, then save this card.",
 			cancel: "Cancel",
 			saved: "Saved. The composer Effort row follows this capability declaration.",
 			conflict: "Settings changed elsewhere. Reload, then save again.",
@@ -1132,7 +1301,7 @@ window.__ModuleLoader__.load({
 			clear: "Clear this model’s declaration",
 			advanced: "Advanced: protocol dialect",
 			thinkingFormat: "thinkingFormat",
-			thinkingFormatDefault: "Default (openai)",
+			thinkingFormatDefault: "Default (omit the key)",
 			supportsDeveloperRole: "Send system prompts as system, not developer (supportsDeveloperRole: false)",
 			supportsReasoningEffort: "Do not send reasoning_effort; switch only (supportsReasoningEffort: false)",
 			developerTrueHint: "The document has supportsDeveloperRole: true. v1 can only force false or omit the key; checking writes false, unchecking deletes the key.",
@@ -1181,7 +1350,8 @@ window.__ModuleLoader__.load({
 				rehydrate: (serialized) => settingsSchema.rehydrate(serialized),
 				nodeAtPath: (root, path) => settingsSchema.nodeAtPath(root, path),
 				getPath: (value, path) => settingsSchema.getPath(value, path),
-				hasPath: (value, path) => settingsSchema.hasPath(value, path)
+				hasPath: (value, path) => settingsSchema.hasPath(value, path),
+				validate: (node, draft) => settingsSchema.validate(node, draft)
 			});
 			const t = ctx.locale.bind(NS);
 			const describe = ctx.settingsScope.describe();
@@ -1191,6 +1361,9 @@ window.__ModuleLoader__.load({
 					for (const listener of invalidation) listener(source);
 				};
 				const disposers = [
+					describe.subscribe(() => {
+						emit("writable");
+					}),
 					ctx.remote.$on("settings/document-updated", (ns) => {
 						if (ns !== "llm-pi-ai") return;
 						emit("settings");
@@ -1215,14 +1388,13 @@ window.__ModuleLoader__.load({
 			ctx.slots.inject("settings.section", () => ctx.slots.register({
 				name: "settings.section",
 				id: "effort-declare",
-				order: 15,
+				order: 12,
 				label: () => t("nav"),
 				locale: NS,
 				inject: () => ({
 					api: connection.api,
 					describe,
 					schema,
-					t,
 					subscribeInvalidate
 				})
 			}, EffortDeclareSection));