@klarkxy/dsh-model-hub 0.1.8 → 0.2.1

package/lib/index.js

@@ -1,261 +1,26 @@
-import { mkdir, writeFile } from "node:fs/promises";
-import { dirname, join } from "node:path";
 import Schema from "@deepseek-ai/schemastery";
 import { registerHostRpc } from "@klarkxy/dsh-plugin-kit/host-rpc";
-import { modelRouteRoleSchemastery, modelRouteSchemastery } from "@klarkxy/dsh-model-route/schemastery";
-import { detectModelFields, modelRouteOverride, normalizeModelRoute } from "@klarkxy/dsh-model-route";
-import { collectModelSettings, notifyModelSettingsChanged } from "@klarkxy/dsh-plugin-kit/model-settings";
-import { isSubagentSession, modelSettingsChangesIdentity } from "@klarkxy/dsh-plugin-kit/contracts";
-import { createModelSettingsChanges } from "@klarkxy/dsh-plugin-kit/model-settings-changes";
+import { notifyModelSettingsChanged } from "@klarkxy/dsh-plugin-kit/model-settings";
 import { mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from "node:fs";
-//#region src/diagnostics.ts
-/** Invalidation is constant-time. Only startup and explicit reads request a
-* diagnostic; a burst shares one deferred run. Work never reschedules itself. */
-function createProbeDiagnostics(run, onError) {
-	let dirty = true;
-	let stopped = false;
-	let timer;
-	let running;
-	return {
-		invalidate() {
-			if (!stopped) dirty = true;
-		},
-		request() {
-			if (stopped || !dirty || timer !== void 0 || running) return;
-			timer = setTimeout(() => {
-				timer = void 0;
-				if (stopped || !dirty) return;
-				dirty = false;
-				running = Promise.resolve().then(run).catch(onError).finally(() => {
-					running = void 0;
-				});
-			}, 0);
-		},
-		async dispose() {
-			stopped = true;
-			if (timer !== void 0) clearTimeout(timer);
-			timer = void 0;
-			await running;
-		}
-	};
-}
-//#endregion
-//#region src/probe.ts
-/**
-* Pure probe-report builder for the model hub. Takes the host's
-* `settings.describe()` output and reports every detected model-route field,
-* plus whether this plugin's own self-test markers survived projection.
-*/
-const SELF_TEST_EXTRA_KEY_PATH = "selfTestMeta";
-const SELF_TEST_ROLE_PATH = "selfTestRole";
-function channelOf(fields, firstSegment) {
-	const hit = fields.find((field) => field.path[0] === firstSegment);
-	return hit ? hit.via : "absent";
-}
-function buildProbeReport(descriptors, now = /* @__PURE__ */ new Date()) {
-	const entries = listModelFields(descriptors).map(({ ns, fields }) => ({
-		ns,
-		fields
-	}));
-	let ownSchema;
-	for (const descriptor of descriptors) if (descriptor.ns.includes("model-hub")) ownSchema = descriptor.schema;
-	const own = entries.find((entry) => entry.ns.includes("model-hub"));
-	return {
-		generatedAt: now.toISOString(),
-		totalNamespaces: descriptors.length,
-		namespacesWithModelFields: entries.length,
-		entries,
-		selfTest: {
-			extraKey: own ? channelOf(own.fields, SELF_TEST_EXTRA_KEY_PATH) : "absent",
-			role: own ? channelOf(own.fields, SELF_TEST_ROLE_PATH) : "absent",
-			ownSchema
-		}
-	};
-}
-/** The field matrix the hub page edits: every detected model-route field. */
-function listModelFields(descriptors) {
-	const entries = [];
-	for (const descriptor of descriptors) {
-		const fields = detectModelFields(descriptor.schema, { value: descriptor.value });
-		if (!fields.length) continue;
-		entries.push({
-			ns: descriptor.ns,
-			revision: typeof descriptor.revision === "number" ? descriptor.revision : 0,
-			fields: fields.map((field) => ({
-				path: [...field.path],
-				via: field.via,
-				...field.label ? { label: field.label } : {},
-				...field.description ? { description: field.description } : {},
-				...field.supportsReasoning === false ? { supportsReasoning: false } : {},
-				...field.rootKeys ? { rootKeys: [...field.rootKeys] } : {},
-				...field.writeKeys ? { writeKeys: [...field.writeKeys] } : {},
-				...field.preserveSiblings ? { preserveSiblings: true } : {},
-				...field.marker ? { marker: field.marker } : {},
-				...field.current ? { current: field.current } : {},
-				...field.editable === false ? { editable: false } : {},
-				...field.arrayItem ? { arrayItem: true } : {}
-			}))
-		});
-	}
-	return entries;
-}
-/**
-* Validate a hub-page write against a fresh detection pass: the target must be
-* a currently detected model-route field, so the page can never write a
-* non-model key. An empty route unsets the field back to its schema default
-* ("follow the host default"); a concrete route is set normalized.
-*/
-function planFieldWrite(entries, payload) {
-	const row = payload && typeof payload === "object" && !Array.isArray(payload) ? payload : void 0;
-	const ns = typeof row?.ns === "string" ? row.ns : "";
-	const path = Array.isArray(row?.path) && row.path.every((s) => typeof s === "string" && s) ? row.path : void 0;
-	if (!ns || !path) throw new Error("Invalid field reference");
-	const entry = entries.find((item) => item.ns === ns);
-	const field = entry?.fields.find((item) => item.path.length === path.length && item.path.every((segment, index) => segment === path[index]));
-	if (!entry || !field) throw new Error("Not a detected model field");
-	if (field.editable === false) throw new Error("No collection item to edit");
-	const override = modelRouteOverride(normalizeModelRoute(row?.route));
-	if (row?.expectedRevision !== void 0 && (!Number.isSafeInteger(row.expectedRevision) || row.expectedRevision < 0)) throw new Error("Invalid revision");
-	if (typeof row?.expectedRevision !== "number") throw new Error("A valid expected revision is required; refresh and retry");
-	const expectedRevision = row.expectedRevision;
-	if (!path.length || field.preserveSiblings) return {
-		ns,
-		expectedRevision,
-		ops: (field.rootKeys ?? field.writeKeys ?? ["provider", "model"]).map((key) => {
-			const value = (override ?? (field.arrayItem ? {
-				provider: "",
-				model: ""
-			} : void 0))?.[key];
-			return value === void 0 ? {
-				op: "unset",
-				path: [...path, key]
-			} : {
-				op: "set",
-				path: [...path, key],
-				value
-			};
-		})
-	};
-	return {
-		ns,
-		expectedRevision,
-		ops: override || field.arrayItem ? [{
-			op: "set",
-			path,
-			value: override ?? {
-				provider: "",
-				model: ""
-			}
-		}] : [{
-			op: "unset",
-			path
-		}]
-	};
-}
-//#endregion
-//#region src/owners.ts
-/** Owner contributions supersede the same native namespace, never its storage. */
-function createHubSettings(events, settings, ownNamespace = "@klarkxy/dsh-model-hub") {
-	const readOwner = async (owner, signal) => {
-		try {
-			const snapshot = await owner.read(signal);
-			if (!Number.isSafeInteger(snapshot.revision) || snapshot.revision < 0) throw new Error("Invalid owner revision");
-			return {
-				ns: owner.ns,
-				title: owner.title,
-				source: "owner",
-				ownerId: owner.instanceId,
-				revision: snapshot.revision,
-				fields: snapshot.fields.map((field) => ({
-					...field,
-					path: [...field.path],
-					via: "contribution"
-				})),
-				...snapshot.catalog !== void 0 ? { catalog: snapshot.catalog } : {},
-				...snapshot.catalogError ? { catalogError: snapshot.catalogError } : {}
-			};
-		} catch (error) {
-			signal.throwIfAborted();
-			return {
-				ns: owner.ns,
-				title: owner.title,
-				source: "owner",
-				ownerId: owner.instanceId,
-				revision: 0,
-				fields: [],
-				error: error instanceof Error ? error.message : String(error)
-			};
-		}
-	};
-	const list = async (signal) => {
-		signal.throwIfAborted();
-		const owners = await collectModelSettings(events);
-		const namespaces = new Set(owners.map((owner) => owner.ns));
-		const native = listModelFields(settings.describe({ redactSecrets: true })).filter((entry) => !namespaces.has(entry.ns)).map((entry) => ({
-			...entry,
-			fields: entry.fields.filter((field) => !(entry.ns === ownNamespace && ["selfTestMeta", "selfTestRole"].includes(field.path[0] ?? "")))
-		})).filter((entry) => entry.fields.length > 0);
-		const entries = await Promise.all(owners.map((owner) => readOwner(owner, signal)));
-		signal.throwIfAborted();
-		return [...native, ...entries].sort((a, b) => a.ns.localeCompare(b.ns));
-	};
-	const set = async (payload, signal) => {
-		signal.throwIfAborted();
-		const row = payload && typeof payload === "object" && !Array.isArray(payload) ? payload : void 0;
-		if (!row || typeof row.ns !== "string" || !Array.isArray(row.path) || !row.path.every((segment) => typeof segment === "string" && segment)) throw new Error("Invalid field reference");
-		if (!Number.isSafeInteger(row.expectedRevision) || row.expectedRevision < 0) throw new Error("A valid expected revision is required; refresh and retry");
-		const input = row.route && typeof row.route === "object" && !Array.isArray(row.route) ? row.route : void 0;
-		if (!input || [
-			"provider",
-			"model",
-			"reasoningEffort"
-		].some((key) => input[key] !== void 0 && typeof input[key] !== "string")) throw new Error("Invalid model route");
-		const normalized = normalizeModelRoute(input);
-		if ((Boolean(String(input.provider ?? "").trim()) || Boolean(String(input.model ?? "").trim())) && (!normalized.provider || !normalized.model)) throw new Error("Invalid model route");
-		const owner = (await collectModelSettings(events)).find((item) => item.ns === row.ns);
-		if (owner) {
-			if (row.ownerId !== owner.instanceId) throw new Error("Model settings owner changed; refresh and retry");
-			const entry = await readOwner(owner, signal);
-			if (entry.error) throw new Error(typeof entry.error === "string" ? entry.error : entry.error.en);
-			if (entry.revision !== row.expectedRevision) throw new Error("Settings changed; refresh and retry");
-			const field = entry.fields.find((field) => JSON.stringify(field.path) === JSON.stringify(row.path));
-			if (!field || field.editable === false) throw new Error("Not an editable model field");
-			const route = normalized;
-			if (field.allowFollow === false && (!route.provider || !route.model)) throw new Error("This model setting requires a concrete model");
-			await owner.write(field.path, route, row.expectedRevision, signal);
-		} else {
-			if (row.ownerId !== void 0) throw new Error("Model settings owner unloaded; refresh and retry");
-			if (row.ns === ownNamespace) throw new Error("Self-test fields are diagnostic only");
-			const plan = planFieldWrite(listModelFields(settings.describe({ redactSecrets: true })), payload);
-			await settings.mutate(plan.ns, plan.ops, plan.expectedRevision);
-		}
-		return {
-			committed: true,
-			ns: row.ns
-		};
-	};
-	return {
-		list,
-		set
-	};
-}
-//#endregion
-//#region src/changes.ts
-/** Model Hub forwards every settings namespace; feature plugins filter their own. */
-function registerHubModelSettingsChanges(ctx) {
-	createModelSettingsChanges(ctx, modelSettingsChangesIdentity("@klarkxy/dsh-model-hub"));
-}
-//#endregion
+import { dirname, join } from "node:path";
+import { modelSettingsChangesIdentity } from "@klarkxy/dsh-plugin-kit/contracts";
+import { createModelSettingsChanges } from "@klarkxy/dsh-plugin-kit/model-settings-changes";
 //#region src/cast.ts
 /**
 * dsh-model-hub — subagent cast (host half).
 *
 * Edits the host-native `subagent-model-selection-settings` namespace
 * (`enabled` + `allowedModels` routes) through the official settings CAS
-* path, and keeps one user-annotated "task tendency" per route in
-* `<profile>/data/model-hub/cast.json`. Tendencies are advisory hints for
-* the system-prompt injection in index.ts; the main agent still decides
-* which model a delegated subagent receives.
+* path, and keeps the per-route user preferences in
+* `<profile>/data/model-hub/cast.json`: the free-text "task tendency" note
+* and the reasoning effort.
+*
+* The native AllowedModelRoute schema carries provider and model only — the
+* host strips a reasoning effort from the settings value — so the effort is a
+* plugin-side preference keyed by provider/model and saved with the file's own
+* CAS. Tendencies and efforts are advisory hints for the system-prompt
+* injection in index.ts; the main agent still decides which model a delegated
+* subagent receives.
 */
 /** Native namespace owning the subagent model-selection preference. */
 const CAST_NAMESPACE = "subagent-model-selection-settings";
@@ -264,12 +29,13 @@ const CAST_TENDENCIES_NAMESPACE = "@klarkxy/dsh-model-hub/cast-tendencies";
 /** Tendency ids are limited like host route ids; notes stay short prompt lines. */
 const CAST_LIMITS = {
 	id: 250,
-	tendency: 500
+	tendency: 500,
+	effort: 80
 };
 function isRecord(value) {
 	return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-/** Tendency store key for one provider/model pair. */
+/** Store key for one provider/model pair; it keys tendencies and efforts. */
 function tendencyKey(provider, model) {
 	return JSON.stringify([provider, model]);
 }
@@ -279,13 +45,12 @@ function parseAllowedModels(value) {
 	const routes = [];
 	list.forEach((entry, index) => {
 		if (!isRecord(entry)) return;
-		const { provider, model, reasoningEffort } = entry;
+		const { provider, model } = entry;
 		if (typeof provider !== "string" || !provider || typeof model !== "string" || !model) return;
 		routes.push({
 			index,
 			provider,
-			model,
-			...typeof reasoningEffort === "string" && reasoningEffort ? { reasoningEffort } : {}
+			model
 		});
 	});
 	return {
@@ -293,22 +58,17 @@ function parseAllowedModels(value) {
 		length: list.length
 	};
 }
+/**
+* Normalize a native route write. The native schema owns provider and model
+* only: a reasoning effort is a plugin-side preference and never travels
+* through the settings CAS.
+*/
 function normalizeRouteInput(input) {
 	const row = isRecord(input) ? input : {};
 	const provider = typeof row.provider === "string" ? row.provider.trim() : "";
 	const model = typeof row.model === "string" ? row.model.trim() : "";
 	if (!provider || !model) throw new Error("A route requires non-empty provider and model ids");
 	if (provider.length > CAST_LIMITS.id || model.length > CAST_LIMITS.id) throw new Error(`Route provider and model ids are limited to ${CAST_LIMITS.id} characters`);
-	if (row.reasoningEffort !== void 0) {
-		const effort = typeof row.reasoningEffort === "string" ? row.reasoningEffort.trim() : "";
-		if (!effort) throw new Error("A route reasoning effort must be a non-empty string");
-		if (effort.length > CAST_LIMITS.id) throw new Error(`Route reasoning effort is limited to ${CAST_LIMITS.id} characters`);
-		return {
-			provider,
-			model,
-			reasoningEffort: effort
-		};
-	}
 	return {
 		provider,
 		model
@@ -324,12 +84,13 @@ function validateIndex(value) {
 }
 /**
 * Render the injected guidance. Pure: callers resolve enabled state and the
-* tendency-merged routes. Returns '' when the injection should not appear —
+* preference-merged routes. Returns '' when the injection should not appear —
 * the host prompt renderer drops empty sections.
 */
 function buildCastPrompt(enabled, routes) {
 	if (!enabled || !routes.length) return "";
 	const annotated = routes.filter((route) => typeof route.tendency === "string" && route.tendency.trim());
+	const withEffort = routes.filter((route) => typeof route.reasoningEffort === "string" && route.reasoningEffort.trim());
 	const guidance = [
 		"Subagent model selection:",
 		"When delegating an independent task, actively choose a suitable model instead of silently inheriting your own. If list_subagent_models and a delegation tool with provider/model fields are available, inspect the available routes before the first model-selectable delegation in this task, reuse those results for later delegations, and supply provider and model together explicitly.",
@@ -341,17 +102,27 @@ function buildCastPrompt(enabled, routes) {
 		provider: route.provider,
 		model: route.model
 	})}${route.reasoningEffort ? ` (reasoning effort: ${route.reasoningEffort})` : ""}: ${route.tendency}`));
+	if (withEffort.length) guidance.push("Configured reasoning effort (advisory reference configuration, applied when a subagent starts; it does not change subagents that are already running): pass reasoning_effort only when the selected route supports that value and omit it otherwise. When a route's provider/model changes, an effort the new route does not support is dropped.", ...withEffort.map((route) => `- ${JSON.stringify({
+		provider: route.provider,
+		model: route.model
+	})}: reasoning effort "${route.reasoningEffort}"`));
 	return guidance.join("\n");
 }
 function createCast(deps) {
 	const castFile = () => {
-		if (!deps.profileDir) throw new Error("No profile directory; cast tendencies cannot be stored");
+		if (!deps.profileDir) throw new Error("No profile directory; cast preferences cannot be stored");
 		return join(deps.profileDir, "data", "model-hub", "cast.json");
 	};
-	const readTendencies = () => {
+	/**
+	* Always read the durable revision: another tab or plugin incarnation may
+	* have committed since the last snapshot. Failed reads never become a cache.
+	* v1 and v2 stores hold tendencies only; v3 adds the effort map.
+	*/
+	const readPreferences = () => {
 		if (!deps.profileDir) return {
 			revision: 0,
-			tendencies: {}
+			tendencies: {},
+			efforts: {}
 		};
 		let raw;
 		try {
@@ -359,42 +130,48 @@ function createCast(deps) {
 		} catch (error) {
 			if (error.code === "ENOENT") return {
 				revision: 0,
-				tendencies: {}
+				tendencies: {},
+				efforts: {}
 			};
-			throw new Error("Cast tendency store cannot be read; restore access to cast.json and refresh", { cause: error });
+			throw new Error("Cast preference store cannot be read; restore access to cast.json and refresh", { cause: error });
 		}
 		let parsed;
 		try {
 			parsed = JSON.parse(raw);
 		} catch (error) {
-			throw new Error("Cast tendency store is damaged; repair cast.json and refresh", { cause: error });
+			throw new Error("Cast preference store is damaged; repair cast.json and refresh", { cause: error });
 		}
-		if (isRecord(parsed) && parsed.schemaVersion !== 1 && parsed.schemaVersion !== 2) throw new Error("Cast tendency store version is unsupported; use a compatible plugin before saving");
+		const version = isRecord(parsed) ? parsed.schemaVersion : void 0;
+		if (version !== 1 && version !== 2 && version !== 3) throw new Error("Cast preference store version is unsupported; use a compatible plugin before saving");
 		try {
-			if (!isRecord(parsed) || !isRecord(parsed.tendencies)) throw new Error("Invalid tendency store schema");
-			if (parsed.schemaVersion === 2 && (!Number.isSafeInteger(parsed.revision) || parsed.revision < 0)) throw new Error("Invalid tendency store revision");
-			for (const [key, value] of Object.entries(parsed.tendencies)) {
+			if (!isRecord(parsed) || !isRecord(parsed.tendencies)) throw new Error("Invalid preference store schema");
+			if (version !== 1 && (!Number.isSafeInteger(parsed.revision) || parsed.revision < 0)) throw new Error("Invalid preference store revision");
+			const efforts = version === 3 ? parsed.efforts : {};
+			if (!isRecord(efforts)) throw new Error("Invalid preference store schema");
+			for (const map of [parsed.tendencies, efforts]) for (const [key, value] of Object.entries(map)) {
 				const ids = JSON.parse(key);
-				if (!Array.isArray(ids) || ids.length !== 2 || !ids.every((id) => typeof id === "string" && id.trim()) || typeof value !== "string") throw new Error("Invalid tendency store entry");
+				if (!Array.isArray(ids) || ids.length !== 2 || !ids.every((id) => typeof id === "string" && id.trim()) || typeof value !== "string") throw new Error("Invalid preference store entry");
 			}
 			return {
-				revision: parsed.schemaVersion === 1 ? 0 : parsed.revision,
-				tendencies: parsed.tendencies
+				revision: version === 1 ? 0 : parsed.revision,
+				tendencies: parsed.tendencies,
+				efforts
 			};
 		} catch (error) {
-			throw new Error("Cast tendency store schema is invalid; repair cast.json and refresh", { cause: error });
+			throw new Error("Cast preference store schema is invalid; repair cast.json and refresh", { cause: error });
 		}
 	};
 	/** Atomic store write: temp file plus rename, so a crash never halves JSON. */
-	const writeTendencies = (next, revision) => {
+	const writePreferences = (next, revision) => {
 		const file = castFile();
 		mkdirSync(dirname(file), { recursive: true });
 		const temp = `${file}.${process.pid}.tmp`;
 		try {
 			writeFileSync(temp, JSON.stringify({
-				schemaVersion: 2,
+				schemaVersion: 3,
 				revision,
-				tendencies: next
+				tendencies: next.tendencies,
+				efforts: next.efforts
 			}, null, 2));
 			renameSync(temp, file);
 		} catch (error) {
@@ -416,24 +193,28 @@ function createCast(deps) {
 			length: parsed.length
 		};
 	};
-	const mergeTendencies = (routes, stored) => {
+	/** Merge the stored preferences into the native routes. */
+	const mergePreferences = (routes, stored) => {
 		return routes.map((route) => {
-			const tendency = stored[tendencyKey(route.provider, route.model)];
-			return tendency ? {
+			const key = tendencyKey(route.provider, route.model);
+			const tendency = stored.tendencies[key];
+			const effort = stored.efforts[key];
+			return {
 				...route,
-				tendency
-			} : { ...route };
+				...tendency ? { tendency } : {},
+				...effort ? { reasoningEffort: effort } : {}
+			};
 		});
 	};
 	const getCast = () => {
 		const raw = readRaw();
 		try {
-			const stored = readTendencies();
+			const stored = readPreferences();
 			return {
 				enabled: raw.enabled,
 				revision: raw.revision,
 				tendencyRevision: stored.revision,
-				routes: mergeTendencies(raw.routes, stored.tendencies)
+				routes: mergePreferences(raw.routes, stored)
 			};
 		} catch (error) {
 			return {
@@ -507,24 +288,41 @@ function createCast(deps) {
 			return ops;
 		});
 	};
-	/** Synchronous read/CAS/rename in one host turn; no native settings write. */
-	const setTendency = (provider, model, tendency, expectedRevision, signal) => {
-		if (typeof provider !== "string" || !provider.trim() || typeof model !== "string" || !model.trim()) throw new Error("A tendency requires non-empty provider and model ids");
-		if (typeof tendency !== "string") throw new Error("A tendency must be a string");
-		const text = tendency.trim();
-		if (text.length > CAST_LIMITS.tendency) throw new Error(`A tendency is limited to ${CAST_LIMITS.tendency} characters`);
-		if (!deps.profileDir) throw new Error("No profile directory; cast tendencies cannot be stored");
+	/**
+	* Save one route's plugin-side preferences with the file's own CAS. The
+	* returned state is the committed one, so the page never waits for a later
+	* refresh that could still carry the previous value.
+	*/
+	const setRoutePreferences = (provider, model, preferences, expectedRevision, signal) => {
+		if (typeof provider !== "string" || !provider.trim() || typeof model !== "string" || !model.trim()) throw new Error("A route preference requires non-empty provider and model ids");
+		if (!deps.profileDir) throw new Error("No profile directory; cast preferences cannot be stored");
 		const revision = validateRevision(expectedRevision);
-		const stored = readTendencies();
-		if (stored.revision !== revision) throw new Error("Cast tendencies changed; refresh and retry");
-		if (revision === Number.MAX_SAFE_INTEGER) throw new Error("Cast tendency revision limit reached");
-		const raw = readRaw();
-		const next = { ...stored.tendencies };
+		const stored = readPreferences();
+		if (stored.revision !== revision) throw new Error("Cast preferences changed; refresh and retry");
+		if (revision === Number.MAX_SAFE_INTEGER) throw new Error("Cast preference revision limit reached");
+		const patch = isRecord(preferences) ? preferences : {};
+		const next = {
+			tendencies: { ...stored.tendencies },
+			efforts: { ...stored.efforts }
+		};
 		const key = tendencyKey(provider.trim(), model.trim());
-		if (text) next[key] = text;
-		else Reflect.deleteProperty(next, key);
+		if ("tendency" in patch) {
+			if (typeof patch.tendency !== "string") throw new Error("A tendency must be a string");
+			const text = patch.tendency.trim();
+			if (text.length > CAST_LIMITS.tendency) throw new Error(`A tendency is limited to ${CAST_LIMITS.tendency} characters`);
+			if (text) next.tendencies[key] = text;
+			else Reflect.deleteProperty(next.tendencies, key);
+		}
+		if ("reasoningEffort" in patch) {
+			if (typeof patch.reasoningEffort !== "string") throw new Error("A reasoning effort must be a string");
+			const effort = patch.reasoningEffort.trim();
+			if (effort.length > CAST_LIMITS.effort) throw new Error(`A reasoning effort is limited to ${CAST_LIMITS.effort} characters`);
+			if (effort) next.efforts[key] = effort;
+			else Reflect.deleteProperty(next.efforts, key);
+		}
+		const raw = readRaw();
 		signal?.throwIfAborted();
-		writeTendencies(next, revision + 1);
+		writePreferences(next, revision + 1);
 		try {
 			deps.notifyTendenciesChanged?.(revision + 1);
 		} catch {}
@@ -532,9 +330,14 @@ function createCast(deps) {
 			enabled: raw.enabled,
 			revision: raw.revision,
 			tendencyRevision: revision + 1,
-			routes: mergeTendencies(raw.routes, next)
+			routes: mergePreferences(raw.routes, next)
 		};
 	};
+	/** Synchronous read/CAS/rename in one host turn; no native settings write. */
+	const setTendency = (provider, model, tendency, expectedRevision, signal) => {
+		if (typeof tendency !== "string") throw new Error("A tendency must be a string");
+		return setRoutePreferences(provider, model, { tendency }, expectedRevision, signal);
+	};
 	/**
 	* Section text for the system-prompt injection. The native
 	* `subagentModelSelection` service is authoritative when present; a missing
@@ -562,9 +365,9 @@ function createCast(deps) {
 			return "";
 		}
 		try {
-			return buildCastPrompt(snapshot.enabled, mergeTendencies(snapshot.routes, readTendencies().tendencies));
+			return buildCastPrompt(snapshot.enabled, mergePreferences(snapshot.routes, readPreferences()));
 		} catch (error) {
-			deps.logger?.warn("model-hub: cast tendencies unavailable: %s", error.message);
+			deps.logger?.warn("model-hub: cast preferences unavailable: %s", error.message);
 			return buildCastPrompt(snapshot.enabled, snapshot.routes);
 		}
 	};
@@ -575,145 +378,18 @@ function createCast(deps) {
 		setRoute,
 		removeRoute,
 		setTendency,
+		setRoutePreferences,
 		promptText
 	};
 }
 //#endregion
-//#region src/running.ts
-/**
-* dsh-model-hub — running subagent routes (host half).
-*
-* Read-only live view answering "which model and reasoning effort is each
-* running subagent actually using". `subagent/start` and `subagent/end`
-* maintain the live set; each child's effective route is read from its
-* session epoch header (`session.requestHeader()`) and kept current through
-* the session's own `request/header` events (reason `change` covers mid-run
-* model switches). Children already running when this plugin (re)loads are
-* recovered from the live agent registry via `isSubagentSession`.
-*
-* The effective route is session data recorded by the host; this tracker
-* never writes to sessions or settings. Display-only invalidations reuse the
-* model-settings change channel with RUNNING_NAMESPACE so open pages refetch.
-*/
-/** Invalidation namespace for the running-subagent view; carries no settings. */
-const RUNNING_NAMESPACE = "@klarkxy/dsh-model-hub/running-subagents";
-function text(value) {
-	return typeof value === "string" ? value.trim() : "";
-}
-/** Normalize one epoch header into the display route plus the adapter-default flag. */
-function routeFromHeader(header) {
-	const config = header?.config;
-	const provider = text(config?.provider);
-	const model = text(config?.model);
-	if (!provider || !model) return {};
-	const effort = text(config?.reasoningEffort);
-	return {
-		route: effort ? {
-			provider,
-			model,
-			reasoningEffort: effort
-		} : {
-			provider,
-			model
-		},
-		...effort && header?.adapterDefaults?.reasoningEffort === true ? { effortIsAdapterDefault: true } : {}
-	};
-}
-function sameRoute(left, right) {
-	return left.route?.provider === right.route?.provider && left.route?.model === right.route?.model && left.route?.reasoningEffort === right.route?.reasoningEffort && left.effortIsAdapterDefault === right.effortIsAdapterDefault;
-}
-function createRunningTracker(deps) {
-	const entries = /* @__PURE__ */ new Map();
-	let revision = 0;
-	const bump = () => deps.notify(++revision);
-	const add = (id, info) => {
-		const session = (deps.agents?.get(id))?.session;
-		const next = routeFromHeader(session?.requestHeader?.());
-		const existing = entries.get(id);
-		if (existing) {
-			const providerName = info?.provider ?? existing.providerName;
-			const local = info?.local ?? existing.local;
-			const changed = existing.providerName !== providerName || existing.local !== local || !sameRoute(existing, next);
-			existing.providerName = providerName;
-			existing.local = local;
-			existing.route = next.route;
-			existing.effortIsAdapterDefault = next.effortIsAdapterDefault;
-			if (changed) bump();
-			return;
-		}
-		entries.set(id, {
-			id,
-			local: info?.local !== false,
-			...info?.provider ? { providerName: info.provider } : {},
-			...typeof session?.header?.parentSession === "string" && session.header.parentSession ? { parentSessionId: session.header.parentSession } : {},
-			...typeof session?.header?.createdAt === "number" ? { startedAt: session.header.createdAt } : {},
-			route: next.route,
-			effortIsAdapterDefault: next.effortIsAdapterDefault
-		});
-		bump();
-	};
-	const remove = (id) => {
-		if (entries.delete(id)) bump();
-	};
-	try {
-		for (const agent of deps.agents?.list() ?? []) if (agent?.id && isSubagentSession(agent.session)) add(agent.id);
-	} catch {}
-	deps.events.on("subagent/start", (info) => {
-		if (typeof info?.id === "string" && info.id) add(info.id, info);
-	});
-	deps.events.on("subagent/end", (info) => {
-		if (typeof info?.id === "string") remove(info.id);
-	});
-	deps.events.on("agent/disposed", (payload) => {
-		if (typeof payload?.agent?.id === "string") remove(payload.agent.id);
-	});
-	deps.events.on("session/event", (session, event) => {
-		if (event?.type !== "request/header") return;
-		const id = typeof session?.id === "string" ? session.id : void 0;
-		const entry = id ? entries.get(id) : void 0;
-		if (!entry) return;
-		const next = routeFromHeader(event.data?.header);
-		if (sameRoute(entry, next)) return;
-		entry.route = next.route;
-		entry.effortIsAdapterDefault = next.effortIsAdapterDefault;
-		bump();
-	});
-	return { async list(signal) {
-		const pendingParents = /* @__PURE__ */ new Set();
-		for (const entry of entries.values()) if (entry.label === void 0 && entry.parentSessionId) pendingParents.add(entry.parentSessionId);
-		if (deps.subagents && pendingParents.size) await Promise.allSettled([...pendingParents].map(async (parent) => {
-			const children = await deps.subagents.listChildren(parent, signal);
-			const labels = new Map(children.map((child) => [child.id, child.label]));
-			for (const entry of entries.values()) {
-				if (entry.parentSessionId !== parent || entry.label !== void 0) continue;
-				const label = labels.get(entry.id);
-				if (typeof label === "string" && label.trim()) entry.label = label.trim();
-			}
-		}));
-		const snapshot = [...entries.values()].map((entry) => ({
-			...entry,
-			route: entry.route ? { ...entry.route } : void 0
-		})).sort((a, b) => (a.startedAt ?? 0) - (b.startedAt ?? 0) || a.id.localeCompare(b.id));
-		return {
-			revision,
-			entries: snapshot
-		};
-	} };
+//#region src/changes.ts
+/** Model Hub forwards every settings namespace; feature plugins filter their own. */
+function registerHubModelSettingsChanges(ctx) {
+	createModelSettingsChanges(ctx, modelSettingsChangesIdentity("@klarkxy/dsh-model-hub"));
 }
 //#endregion
 //#region src/index.ts
-/**
-* dsh-model-hub — cross-plugin model settings hub.
-*
-* Host half: reads every plugin's settings descriptor through the host
-* `settings` service, runs model-route field detection, and serves the field
-* matrix to the browser half over an authenticated RPC channel. Writes go
-* through `settings.mutate` — the official settings page's own path — with
-* the descriptor revision as CAS. The boot probe report is kept: it is logged
-* and written to `<profile>/data/model-hub/probe-report.json`. The subagent
-* cast (cast.ts) edits the native subagent model-selection settings the same
-* way and injects per-route task tendencies into the system prompt.
-*/
 const name = "@klarkxy/dsh-model-hub";
 const inject = [
 	"settings",
@@ -722,94 +398,26 @@ const inject = [
 	"systemPrompt"
 ];
 const CHANNEL = "/dsh-model-hub";
-const Config = Schema.object({
-	selfTestMeta: modelRouteSchemastery({
-		purpose: "self-test-meta",
-		label: {
-			zh: "自检字段（自定义键通道）",
-			en: "Self-test field (custom key)"
-		}
-	}).volatile(),
-	selfTestRole: modelRouteRoleSchemastery({
-		purpose: "self-test-role",
-		label: {
-			zh: "自检字段（role 通道）",
-			en: "Self-test field (role)"
-		}
-	}).volatile()
-});
+const Config = Schema.object({});
 function apply(ctx) {
 	registerHubModelSettingsChanges(ctx);
 	const profile = ctx.get("profileContext");
 	const settings = ctx.settings;
-	const describe = () => settings.describe({ redactSecrets: true });
-	const ownNamespace = ctx.fiber.entry?.options?.id ?? "@klarkxy/dsh-model-hub";
-	const hub = createHubSettings(ctx, {
-		describe,
-		mutate: (ns, ops, revision) => settings.mutate(ns, ops, revision)
-	}, ownNamespace);
 	const cast = createCast({
-		describe,
-		mutate: (ns, ops, revision) => settings.mutate(ns, ops, revision),
+		describe: () => settings.describe({ redactSecrets: true }),
+		mutate: (ns, ops, revision) => settings.mutate(ns, [...ops], revision),
 		profileDir: profile?.dir,
 		logger: ctx.logger,
 		notifyTendenciesChanged: (revision) => notifyModelSettingsChanged(ctx, CAST_TENDENCIES_NAMESPACE, revision)
 	});
-	const running = createRunningTracker({
-		events: ctx,
-		agents: ctx.get("agents"),
-		subagents: ctx.get("subagents"),
-		notify: (revision) => notifyModelSettingsChanged(ctx, RUNNING_NAMESPACE, revision)
-	});
 	ctx.systemPrompt.section({
 		name: "model-hub:subagent-cast",
 		order: ctx.systemPrompt.getSectionOrder("TOOL_SUBAGENT") + 1,
 		interpolate: false,
 		text: () => cast.promptText(() => ctx.get("subagentModelSelection")?.current())
 	});
-	const run = async () => {
-		const reason = "requested";
-		let report;
-		try {
-			report = buildProbeReport(describe());
-		} catch (error) {
-			ctx.logger.warn("model-hub: probe failed (%s): %s", reason, error instanceof Error ? error.message : String(error));
-			return;
-		}
-		ctx.logger.info("model-hub: probe (%s): %d namespaces, %d with model fields; self-test extra-key=%s role=%s", reason, report.totalNamespaces, report.namespacesWithModelFields, report.selfTest.extraKey, report.selfTest.role);
-		if (profile?.dir) try {
-			const dir = join(profile.dir, "data", "model-hub");
-			await mkdir(dir, { recursive: true });
-			await writeFile(join(dir, "probe-report.json"), JSON.stringify(report, null, 2));
-		} catch (error) {
-			ctx.logger.warn("model-hub: could not write probe report: %s", error instanceof Error ? error.message : String(error));
-		}
-	};
-	const diagnostics = createProbeDiagnostics(run, (error) => {
-		ctx.logger.warn("model-hub: diagnostics failed: %s", String(error));
-	});
-	ctx.effect(() => {
-		const off = ctx.on("settings/document-updated", () => diagnostics.invalidate());
-		diagnostics.request();
-		return async () => {
-			off();
-			await diagnostics.dispose();
-		};
-	});
 	ctx.effect(() => registerHostRpc(ctx, CHANNEL, async (endpoint, payload, signal) => {
 		try {
-			if (endpoint === "fields.list") {
-				const value = await hub.list(signal);
-				diagnostics.request();
-				return {
-					ok: true,
-					value
-				};
-			}
-			if (endpoint === "fields.set") return {
-				ok: true,
-				value: await hub.set(payload, signal)
-			};
 			if (endpoint.startsWith("cast.")) {
 				const row = payload && typeof payload === "object" && !Array.isArray(payload) ? payload : {};
 				if (endpoint === "cast.get") return {
@@ -836,9 +444,9 @@ function apply(ctx) {
 					ok: true,
 					value: cast.setTendency(row.provider, row.model, row.tendency, row.expectedRevision, signal)
 				};
-				if (endpoint === "cast.running") return {
+				if (endpoint === "cast.setPreferences") return {
 					ok: true,
-					value: await running.list(signal)
+					value: cast.setRoutePreferences(row.provider, row.model, row.preferences, row.expectedRevision, signal)
 				};
 			}
 			throw new Error(`Unknown endpoint: ${endpoint}`);