dsh-llm-verifier 0.2.0 → 0.2.1

package/lib/client.js

@@ -111,6 +111,7 @@ window.__ModuleLoader__.load({
 			"settings.catalogFailures": "部分模型目录读取失败",
 			"settings.saved": "设置已保存。",
 			"settings.reload": "重新载入",
+			"settings.unsaved": "当前页面有未保存的修改：点击「保存」后才会写入设置；「重新载入」会丢弃这些修改。",
 			"settings.save": "保存设置",
 			"settings.saving": "保存中…",
 			"stats.pageTitle": "Verifier 工具统计",
@@ -291,6 +292,7 @@ window.__ModuleLoader__.load({
 			"settings.catalogFailures": "Some model catalogs failed to load",
 			"settings.saved": "Settings saved.",
 			"settings.reload": "Reload",
+			"settings.unsaved": "This page has unsaved changes: click Save to write them; Reload discards them.",
 			"settings.save": "Save Settings",
 			"settings.saving": "Saving…",
 			"stats.pageTitle": "Verifier Tool Statistics",
@@ -447,6 +449,55 @@ window.__ModuleLoader__.load({
 			if (typeof draft === "string" && draft.trim()) return draft.trim();
 			if (typeof previous === "string" && previous.trim()) return previous.trim();
 		}
+		/**
+		* Deep equality for JSON-shaped settings values, insensitive to object key
+		* order and treating a missing key as `undefined`. The settings page compares a
+		* draft against a resolved view that was built from a different object shape,
+		* so identity comparison is never enough.
+		*/
+		function sameSettingValue(left, right) {
+			if (left === right) return true;
+			if (Array.isArray(left) && Array.isArray(right)) {
+				if (left.length !== right.length) return false;
+				for (let index = 0; index < left.length; index += 1) if (!sameSettingValue(left[index], right[index])) return false;
+				return true;
+			}
+			if (Array.isArray(left) || Array.isArray(right)) return false;
+			if (typeof left !== "object" || typeof right !== "object" || left === null || right === null) return false;
+			const a = left;
+			const b = right;
+			for (const key of /* @__PURE__ */ new Set([...Object.keys(a), ...Object.keys(b)])) if (!sameSettingValue(a[key], b[key])) return false;
+			return true;
+		}
+		/**
+		* Build the user layer for one settings save.
+		*
+		* Keys this client does not own survive untouched and every draft field is
+		* written, but a field that only repeats the base composition is *dropped*
+		* rather than pinned. Pinning is what makes a shipped default change
+		* unreachable: the budget fields once defaulted to 48/160, and a section that
+		* stored those numbers kept overriding 64/240 and then 96/240 forever. `base`
+		* is the descriptor's composition base (the resolved plugin config), so the
+		* stored section keeps exactly what the user really overrode.
+		*
+		* An explicit `undefined` draft value means "clear this override": it deletes
+		* the stored key instead of pinning a valueless one, which is how the form
+		* removes a judge label, a reasoning effort or a cache directory.
+		*
+		* A host that reports no `base` cannot be pruned against and keeps the old
+		* write-everything behavior.
+		*/
+		function sectionForSave(user, draft, base) {
+			const section = { ...user };
+			for (const [key, value] of Object.entries(draft)) {
+				if (value === void 0 || base !== void 0 && sameSettingValue(value, base[key])) {
+					delete section[key];
+					continue;
+				}
+				section[key] = value;
+			}
+			return section;
+		}
 		/** An eight-candidate select: ring + pivot rounds (18 pairs) x three criteria, one repeat. */
 		const WORST_CASE_ROUTE_CALLS_PER_JUDGE = 54;
 		/** Final acceptance: three criteria x the default two repeats (one per A/B position). */
@@ -836,6 +887,50 @@ window.__ModuleLoader__.load({
 			(0, react.useEffect)(() => {
 				load();
 			}, []);
+			const dirty = (0, react.useMemo)(() => loaded !== null && draft !== null && !sameSettingValue(draft, values(loaded.settings)), [loaded, draft]);
+			const latest = (0, react.useRef)({
+				loaded,
+				draft
+			});
+			latest.current = {
+				loaded,
+				draft
+			};
+			(0, react.useEffect)(() => {
+				const sync = () => {
+					if (document.visibilityState === "hidden") return;
+					const { loaded: at, draft: local } = latest.current;
+					if (!at || !local) return;
+					if (!sameSettingValue(local, values(at.settings))) return;
+					(async () => {
+						try {
+							const s = await remote.settings.describe();
+							if (!s.ok) return;
+							const view = s.value.namespaces.find((x) => x.ns === NS);
+							if (!view || view.revision === at.settings.revision) return;
+							const next = {
+								...at,
+								settings: view,
+								writable: s.value.writable
+							};
+							latest.current = {
+								loaded: next,
+								draft: values(view)
+							};
+							setLoaded(next);
+							setDraft(values(view));
+							setEditing({});
+							setSaved(false);
+						} catch {}
+					})();
+				};
+				window.addEventListener("focus", sync);
+				document.addEventListener("visibilitychange", sync);
+				return () => {
+					window.removeEventListener("focus", sync);
+					document.removeEventListener("visibilitychange", sync);
+				};
+			}, [remote]);
 			const models = (0, react.useMemo)(() => loaded?.groups.find((g) => g.id === draft?.provider)?.models ?? [], [loaded, draft?.provider]);
 			const efforts = models.find((m) => m.id === draft?.model)?.reasoning?.efforts ?? [];
 			const patch = (key, value) => {
@@ -894,17 +989,15 @@ window.__ModuleLoader__.load({
 				setSaved(false);
 				setError(null);
 				try {
-					const section = {
-						...record(loaded.settings.user),
+					const editable = {
 						...draft,
-						extraJudges: serializeExtraJudges(draft.extraJudges)
+						extraJudges: serializeExtraJudges(draft.extraJudges),
+						reasoningEffort: draft.reasoningEffort || void 0,
+						label: draft.label?.trim() || void 0,
+						cacheDir: resolveCacheDirOnSave(draft.cacheDir, values(loaded.settings).cacheDir)
 					};
-					if (!draft.reasoningEffort) delete section.reasoningEffort;
-					if (!draft.label || !draft.label.trim()) delete section.label;
-					else section.label = draft.label.trim();
-					const resolvedCacheDir = resolveCacheDirOnSave(draft.cacheDir, values(loaded.settings).cacheDir);
-					if (!resolvedCacheDir) delete section.cacheDir;
-					else section.cacheDir = resolvedCacheDir;
+					const base = loaded.settings.base === void 0 ? void 0 : record(loaded.settings.base);
+					const section = sectionForSave(record(loaded.settings.user), editable, base);
 					const res = await remote.settings.update(NS, section, loaded.settings.revision);
 					if (!res.ok) throw new Error(res.error.message);
 					setLoaded((v) => v ? {
@@ -1629,7 +1722,7 @@ window.__ModuleLoader__.load({
 						},
 						children: error
 					}),
-					saved && /* @__PURE__ */ (0, react_jsx_runtime.jsx)("p", {
+					saved && !dirty && /* @__PURE__ */ (0, react_jsx_runtime.jsx)("p", {
 						style: {
 							margin: 0,
 							fontSize: 12,
@@ -1638,6 +1731,15 @@ window.__ModuleLoader__.load({
 						},
 						children: t["settings.saved"]
 					}),
+					dirty && /* @__PURE__ */ (0, react_jsx_runtime.jsx)("p", {
+						style: {
+							margin: 0,
+							fontSize: 12,
+							lineHeight: "18px",
+							color: "var(--dsw-alias-state-warn-label)"
+						},
+						children: t["settings.unsaved"]
+					}),
 					/* @__PURE__ */ (0, react_jsx_runtime.jsxs)("div", {
 						style: {
 							display: "flex",
@@ -2633,6 +2735,8 @@ window.__ModuleLoader__.load({
 		exports.isVerdictFailed = isVerdictFailed;
 		exports.money = money;
 		exports.resolveCacheDirOnSave = resolveCacheDirOnSave;
+		exports.sameSettingValue = sameSettingValue;
+		exports.sectionForSave = sectionForSave;
 		exports.tFormat = tFormat;
 		exports.toolLabels = toolLabels;
 		exports.useLanguage = useLanguage;