dsh-mnemon-source-skills 0.5.2 → 0.5.4

package/lib/contracts.d.ts

@@ -239,6 +239,30 @@ interface RunTarget {
   draftId?: string;
   versionId?: string;
 }
+/** One Provider and model, as a user picked it; never carries a reasoning effort. */
+interface ModelRoute {
+  provider: string;
+  model: string;
+}
+/**
+ * The models the user picked for skill work, kept beside the ledger. Every
+ * entry is optional: absent means "follow the task Agent's own route".
+ * `suggestionModels` is keyed by suggestion id, `skillRunModels` by skill name.
+ */
+interface DraftPreferences {
+  version: 1;
+  draftModel?: ModelRoute;
+  runModel?: ModelRoute;
+  suggestionModels: Record<string, ModelRoute>;
+  skillRunModels: Record<string, ModelRoute>;
+}
+/** What the pickers read: the stored picks, named the way the pages use them. */
+interface DraftPreferencesView {
+  model?: ModelRoute;
+  runModel?: ModelRoute;
+  suggestionModels: Record<string, ModelRoute>;
+  skillRunModels: Record<string, ModelRoute>;
+}
 interface CaseResult {
   caseId: string;
   title: string;
@@ -298,6 +322,10 @@ interface ReviewLogEntry {
 }
 /** How the Source admits a review's suggestions; the Host sends its settings with each review. */
 interface SkillsPolicy {
+  /** Whether a review may propose new skills; omitted by older Hosts, default true. */
+  allowCreate?: boolean;
+  /** Whether a review may improve existing skills; omitted by older Hosts, default true. */
+  allowUpdate?: boolean;
   minConfidence: number;
   expiryDays: number;
   maxPendingPerSession: number;
@@ -592,4 +620,4 @@ interface MarketInstall {
   installedAt: string;
 }
 //#endregion
-export { BacktestRun, CaseResult, CatalogView, DEFAULT_SKILLS_POLICY, type DetectedSource, DraftStatus, DraftView, EVOLUTION_SIGNALS, EvalCase, EvalCheck, EvidencePack, EvidenceReference, EvolutionSignal, LocalizedText, MarketAudit, MarketEntry, MarketInstall, MarketInstallState, MarketItem, MarketPreview, MarketRegistry, MarketRegistryFile, MarketSearchResult, MarketSource, type MarketSourceKind, MarketView, ReviewLogEntry, ReviewRecordInput, ReviewRecordResult, ReviewScore, RunStatus, RunTarget, RunTargetKind, SKILL_ROOT_KINDS, SKILL_ROOT_RANK, SUGGESTION_STATUSES, SkillCatalog, SkillDocument, SkillDraft, SkillInvocation, SkillProblem, SkillRoot, SkillRootKind, SkillSuggestion, SkillSummary, SkillVersion, SkillVersionReason, SkillsOverview, SkillsPolicy, SuggestionStatus, SuggestionView, Trajectory, TrajectoryAnnotation, TrajectoryStats, TrajectoryStep, TrajectoryStepKind, TrajectorySummary, TrashEntry, TrashRestoreInput, TrashRestorePreview };
+export { BacktestRun, CaseResult, CatalogView, DEFAULT_SKILLS_POLICY, type DetectedSource, DraftPreferences, DraftPreferencesView, DraftStatus, DraftView, EVOLUTION_SIGNALS, EvalCase, EvalCheck, EvidencePack, EvidenceReference, EvolutionSignal, LocalizedText, MarketAudit, MarketEntry, MarketInstall, MarketInstallState, MarketItem, MarketPreview, MarketRegistry, MarketRegistryFile, MarketSearchResult, MarketSource, type MarketSourceKind, MarketView, ModelRoute, ReviewLogEntry, ReviewRecordInput, ReviewRecordResult, ReviewScore, RunStatus, RunTarget, RunTargetKind, SKILL_ROOT_KINDS, SKILL_ROOT_RANK, SUGGESTION_STATUSES, SkillCatalog, SkillDocument, SkillDraft, SkillInvocation, SkillProblem, SkillRoot, SkillRootKind, SkillSuggestion, SkillSummary, SkillVersion, SkillVersionReason, SkillsOverview, SkillsPolicy, SuggestionStatus, SuggestionView, Trajectory, TrajectoryAnnotation, TrajectoryStats, TrajectoryStep, TrajectoryStepKind, TrajectorySummary, TrashEntry, TrashRestoreInput, TrashRestorePreview };

package/lib/contracts.js

@@ -31,6 +31,8 @@ const SUGGESTION_STATUSES = [
 	"invalidated"
 ];
 const DEFAULT_SKILLS_POLICY = Object.freeze({
+	allowCreate: true,
+	allowUpdate: true,
 	minConfidence: .6,
 	expiryDays: 7,
 	maxPendingPerSession: 5

package/lib/index.js

@@ -489,6 +489,43 @@ var SkillsStore = class {
 			installs: array(value.installs)
 		};
 	}
+	/**
+	* What the user picked for draft and backtest runs, kept out of `fingerprint()`
+	* on purpose: changing a picker must not invalidate an open page's revision.
+	*/
+	preferences() {
+		const stored = object$1(this.readJson("preferences.json")) ?? {};
+		const route = (value) => {
+			const picked = object$1(value);
+			const provider = typeof picked?.provider === "string" ? picked.provider : void 0;
+			const model = typeof picked?.model === "string" ? picked.model : void 0;
+			return provider !== void 0 && model !== void 0 ? {
+				provider,
+				model
+			} : void 0;
+		};
+		const routes = (value) => {
+			const result = {};
+			const map = object$1(value);
+			if (map !== void 0) for (const [key, item] of Object.entries(map)) {
+				const picked = route(item);
+				if (picked !== void 0) result[key] = picked;
+			}
+			return result;
+		};
+		const draft = route(stored.draftModel);
+		const run = route(stored.runModel);
+		return {
+			version: 1,
+			...draft === void 0 ? {} : { draftModel: draft },
+			...run === void 0 ? {} : { runModel: run },
+			suggestionModels: routes(stored.suggestionModels),
+			skillRunModels: routes(stored.skillRunModels)
+		};
+	}
+	savePreferences(preferences) {
+		this.writeJson("preferences.json", preferences);
+	}
 	saveLedger(ledger) {
 		ledger.generation += 1;
 		ledger.reviews = ledger.reviews.slice(0, 200);
@@ -732,13 +769,33 @@ function clip$1(value, maximum) {
 	const text = value.trim();
 	return text.length <= maximum ? text : text.slice(0, maximum - 1) + "…";
 }
-/** Move pending suggestions past their expiry to `expired`; true when anything changed. */
+/**
+* `drafting` was set before this process started, so no live writer owns it any
+* more. A draft write is cancelled only by the Host going away, and the catch
+* that would put the suggestion back never runs then, so the row would sit in
+* "drafting" forever and refuse both a retry and a delete.
+*/
+const PROCESS_STARTED_AT = Date.now() - Math.round(process.uptime() * 1e3);
+const ORPHAN_DRAFT_ERROR = "the host restarted while this draft was being written";
+function orphanedDraft(suggestion) {
+	const at = Date.parse(suggestion.updatedAt);
+	return !Number.isFinite(at) || at < PROCESS_STARTED_AT;
+}
+/**
+* Move pending suggestions past their expiry to `expired`, and suggestions whose
+* draft was abandoned by a dead process back to `pending`; true when anything changed.
+*/
 function expireSuggestions(ledger, now) {
 	let changed = false;
 	for (const suggestion of ledger.suggestions) if (suggestion.status === "pending" && Date.parse(suggestion.expiresAt) < now.getTime()) {
 		suggestion.status = "expired";
 		suggestion.updatedAt = now.toISOString();
 		changed = true;
+	} else if (suggestion.status === "drafting" && orphanedDraft(suggestion)) {
+		suggestion.status = "pending";
+		suggestion.updatedAt = now.toISOString();
+		suggestion.lastError = ORPHAN_DRAFT_ERROR;
+		changed = true;
 	}
 	return changed;
 }
@@ -766,7 +823,14 @@ function policyFrom(value) {
 		const candidate = value?.[key];
 		return typeof candidate === "number" && Number.isFinite(candidate) && candidate >= minimum && candidate <= maximum ? candidate : DEFAULT_SKILLS_POLICY[key];
 	};
+	const allowed = (key) => {
+		const candidate = value?.[key];
+		if (candidate !== void 0 && typeof candidate !== "boolean") throw new Error(`skills policy ${key} must be a boolean`);
+		return candidate ?? true;
+	};
 	return {
+		allowCreate: allowed("allowCreate"),
+		allowUpdate: allowed("allowUpdate"),
 		minConfidence: pick("minConfidence", 0, 1),
 		expiryDays: Math.round(pick("expiryDays", 1, 90)),
 		maxPendingPerSession: Math.round(pick("maxPendingPerSession", 1, 20))
@@ -785,6 +849,7 @@ function admitReview(context, input) {
 	const stepIndexes = new Set(input.trajectory.steps.map((step) => step.index));
 	const steps = (values) => [...new Set(values.filter((value) => Number.isInteger(value) && stepIndexes.has(value)))].sort((a, b) => a - b);
 	const skipped = [];
+	const allowed = (action) => (action === "create" ? policy.allowCreate : policy.allowUpdate) !== false;
 	let inserted = 0;
 	let updated = 0;
 	let casesAdded = 0;
@@ -826,6 +891,14 @@ function admitReview(context, input) {
 		}
 		const existing = activeSkill(catalog, name);
 		const action = existing !== void 0 ? "update" : "create";
+		if (!allowed(action)) {
+			skipped.push({
+				skillName: name,
+				action,
+				reason: "suggestion_type_disabled"
+			});
+			continue;
+		}
 		if (raw.action === "update" && existing === void 0) {
 			skipped.push({
 				skillName: name,
@@ -940,6 +1013,13 @@ function admitReview(context, input) {
 			});
 			continue;
 		}
+		if (!allowed(activeSkill(catalog, name) === void 0 ? "create" : "update")) {
+			skipped.push({
+				skillName: name,
+				reason: "suggestion_type_disabled"
+			});
+			continue;
+		}
 		if (similarCase(evals.cases, name, title, prompt) !== void 0) {
 			skipped.push({
 				skillName: name,
@@ -3387,6 +3467,39 @@ var SkillsService = class {
 			return { purged: entries.length };
 		});
 	}
+	/** The model the user picked for drafts and backtests, if any. */
+	draftPreferences() {
+		const stored = this.store.preferences();
+		return {
+			...stored.draftModel === void 0 ? {} : { model: stored.draftModel },
+			...stored.runModel === void 0 ? {} : { runModel: stored.runModel },
+			suggestionModels: stored.suggestionModels,
+			skillRunModels: stored.skillRunModels
+		};
+	}
+	/**
+	* Record one picker change. `suggestionId` targets a single suggestion,
+	* `skillName` with `slot: 'run'` a single skill, and a null model clears the
+	* entry so the page falls back to following the task Agent again.
+	*/
+	setDraftPreference(input) {
+		return this.write(() => {
+			const preferences = this.store.preferences();
+			if (input.slot === "run") {
+				if (input.skillName !== void 0) {
+					if (input.model === null) delete preferences.skillRunModels[input.skillName];
+					else preferences.skillRunModels[input.skillName] = input.model;
+				} else if (input.model === null) delete preferences.runModel;
+				else preferences.runModel = input.model;
+			} else if (input.suggestionId !== void 0) {
+				if (input.model === null) delete preferences.suggestionModels[input.suggestionId];
+				else preferences.suggestionModels[input.suggestionId] = input.model;
+			} else if (input.model === null) delete preferences.draftModel;
+			else preferences.draftModel = input.model;
+			this.store.savePreferences(preferences);
+			return this.draftPreferences();
+		});
+	}
 	restoreVersion(workspace, input) {
 		return this.write(() => {
 			const skill = findSkill(this.catalog(workspace), input.key);
@@ -3426,6 +3539,25 @@ var SkillsService = class {
 			return suggestion;
 		});
 	}
+	/** Remove a suggestion for good, together with any draft it produced and its model pick. */
+	deleteSuggestion(id) {
+		return this.write(() => {
+			const ledger = this.store.ledger();
+			expireSuggestions(ledger, this.clock());
+			const index = ledger.suggestions.findIndex((candidate) => candidate.id === id);
+			if (index < 0) throw new Error("suggestion not found: " + id);
+			if (ledger.suggestions[index].status === "drafting") throw new Error("a draft is being written for this suggestion; wait for it to finish");
+			ledger.suggestions.splice(index, 1);
+			ledger.drafts = ledger.drafts.filter((draft) => draft.suggestionId !== id);
+			const preferences = this.store.preferences();
+			if (preferences.suggestionModels[id] !== void 0) {
+				delete preferences.suggestionModels[id];
+				this.store.savePreferences(preferences);
+			}
+			this.store.saveLedger(ledger);
+			return { deleted: id };
+		});
+	}
 	/** The Host marks a suggestion `drafting` before a task Agent writes, and back to `pending` if it fails. */
 	suggestionStatus(input) {
 		return this.withSuggestion(input.id, (suggestion) => {
@@ -3675,6 +3807,40 @@ var SkillsService = class {
 			return draft;
 		});
 	}
+	/** Remove a draft for good; its suggestion goes back to `pending` so it can be drafted again. */
+	deleteDraft(id) {
+		return this.write(() => {
+			const ledger = this.store.ledger();
+			const index = ledger.drafts.findIndex((candidate) => candidate.id === id);
+			if (index < 0) throw new Error("draft not found: " + id);
+			const draft = ledger.drafts[index];
+			ledger.drafts.splice(index, 1);
+			if (draft.suggestionId !== void 0) {
+				const suggestion = ledger.suggestions.find((candidate) => candidate.id === draft.suggestionId);
+				if (suggestion?.draftId === draft.id) {
+					delete suggestion.draftId;
+					if (suggestion.status === "review") suggestion.status = "pending";
+					suggestion.updatedAt = this.clock().toISOString();
+				}
+			}
+			this.store.saveLedger(ledger);
+			return { deleted: id };
+		});
+	}
+	/**
+	* Put suggestions whose draft write died with the previous Host back to
+	* `pending`, so the page stops showing them as "drafting" forever.
+	*/
+	reclaimOrphanDrafts() {
+		return this.write(() => {
+			const ledger = this.store.ledger();
+			const before = ledger.suggestions.filter((suggestion) => suggestion.status === "drafting").length;
+			if (!expireSuggestions(ledger, this.clock())) return { reclaimed: 0 };
+			this.store.saveLedger(ledger);
+			const after = ledger.suggestions.filter((suggestion) => suggestion.status === "drafting").length;
+			return { reclaimed: Math.max(0, before - after) };
+		});
+	}
 	/** The content a run tests, resolved from its target. */
 	targetContent(workspace, skillName, target) {
 		if (target.kind === "none") return { target: { kind: "none" } };
@@ -4268,6 +4434,16 @@ function rootKind(value) {
 	if (typeof value !== "string" || !SKILL_ROOT_KINDS.includes(value)) throw new Error("root must be one of " + SKILL_ROOT_KINDS.join(", "));
 	return value;
 }
+/** A picked model, or null to clear the pick and follow the task Agent again. */
+function draftModelInput(value) {
+	if (value === void 0 || value === null) return null;
+	if (typeof value !== "object" || Array.isArray(value)) throw new Error("model must be an object or null");
+	const picked = value;
+	return {
+		provider: memoryInputText(picked.provider, "model.provider", 100),
+		model: memoryInputText(picked.model, "model.model", 200)
+	};
+}
 function caseInput(value, skillName) {
 	const input = memoryInputRecord(value, "case");
 	const checks = input.checks;
@@ -4362,7 +4538,14 @@ function step(value) {
 function policyInput(value) {
 	const input = memoryInputRecord(value, "policy");
 	const number = (key) => typeof input[key] === "number" && Number.isFinite(input[key]) ? { [key]: input[key] } : {};
+	const allowed = (key) => {
+		if (input[key] === void 0) return {};
+		if (typeof input[key] !== "boolean") throw new Error(`skills policy ${key} must be a boolean`);
+		return { [key]: input[key] };
+	};
 	return {
+		...allowed("allowCreate"),
+		...allowed("allowUpdate"),
 		...number("minConfidence"),
 		...number("expiryDays"),
 		...number("maxPendingPerSession")
@@ -4485,6 +4668,7 @@ function createSkillsMemorySource(config = {}) {
 				...effective.dshHome === void 0 ? {} : { dshHome: effective.dshHome },
 				...effective.agentsHome === void 0 ? {} : { agentsHome: effective.agentsHome }
 			});
+			service.reclaimOrphanDrafts().catch(() => void 0);
 			const facts = (scope, management) => management ? {
 				sourceInstanceKey: context.sourceInstanceKey,
 				sourceTypeId: SKILLS_SOURCE_TYPE_ID,
@@ -4542,6 +4726,7 @@ function createSkillsMemorySource(config = {}) {
 									...optionalText(fields.skill, "skill", 64) === void 0 ? {} : { skill: optionalText(fields.skill, "skill", 64) }
 								});
 								case "draft": return service.draft(root, memoryInputText(fields.id, "id", 100));
+								case "draft-preferences": return service.draftPreferences();
 								case "reviews": return service.reviews({
 									...typeof fields.sessionId === "string" ? { sessionId: fields.sessionId } : {},
 									...typeof fields.limit === "number" ? { limit: fields.limit } : {}
@@ -4629,6 +4814,9 @@ function createSkillsMemorySource(config = {}) {
 						case "suggestion-dismiss":
 							value = await service.dismissSuggestion(memoryInputText(fields.id, "id", 100));
 							break;
+						case "suggestion-delete":
+							value = await service.deleteSuggestion(memoryInputText(fields.id, "id", 100));
+							break;
 						case "suggestion-status": {
 							const status = fields.status;
 							if (status !== "drafting" && status !== "pending") throw new Error("status must be drafting or pending");
@@ -4685,6 +4873,22 @@ function createSkillsMemorySource(config = {}) {
 						case "draft-discard":
 							value = await service.discardDraft(memoryInputText(fields.id, "id", 100));
 							break;
+						case "draft-delete":
+							value = await service.deleteDraft(memoryInputText(fields.id, "id", 100));
+							break;
+						case "draft-preference": {
+							const slot = optionalText(fields.slot, "slot", 10);
+							if (slot !== void 0 && slot !== "draft" && slot !== "run") throw new Error("slot must be draft or run");
+							const suggestionId = optionalText(fields.suggestionId, "suggestionId", 100);
+							if (suggestionId !== void 0 && slot === "run") throw new Error("a suggestion pick cannot be a backtest pick");
+							value = await service.setDraftPreference({
+								...slot === void 0 ? {} : { slot },
+								...suggestionId === void 0 ? {} : { suggestionId },
+								...optionalText(fields.skillName, "skillName", 64) === void 0 ? {} : { skillName: optionalText(fields.skillName, "skillName", 64) },
+								model: draftModelInput(fields.model)
+							});
+							break;
+						}
 						case "run-start":
 							value = await service.startRun({
 								skillName: memoryInputText(fields.skillName, "skillName", 64),

package/lib/types/client/api.d.ts

@@ -1,5 +1,5 @@
-import type { MnemonSourceManagementClient } from 'dsh-mnemon/client';
-import type { BacktestRun, CatalogView, DraftView, EvalCase, MarketPreview, MarketRegistry, MarketSearchResult, MarketView, EvidencePack, ReviewLogEntry, ReviewRecordResult, RunTarget, SkillDocument, SkillDraft, SkillRootKind, SkillSuggestion, SkillSummary, SkillVersion, SkillsOverview, SuggestionStatus, SuggestionView, Trajectory, TrajectorySummary, TrashEntry, TrashRestoreInput, TrashRestorePreview } from '../contracts.ts';
+import type { MnemonSourceManagementClient, TaskAgentModelCatalog } from 'dsh-mnemon/client';
+import type { BacktestRun, CatalogView, DraftPreferencesView, DraftView, EvalCase, MarketPreview, MarketRegistry, MarketSearchResult, MarketView, EvidencePack, ModelRoute, ReviewLogEntry, ReviewRecordResult, RunTarget, SkillDocument, SkillDraft, SkillRootKind, SkillSuggestion, SkillSummary, SkillVersion, SkillsOverview, SuggestionStatus, SuggestionView, Trajectory, TrajectorySummary, TrashEntry, TrashRestoreInput, TrashRestorePreview } from '../contracts.ts';
 export interface CaseDraft {
     id?: string;
     skillName: string;
@@ -75,6 +75,14 @@ export interface SkillsPageClient {
     }>;
     restoreVersion(key: string, versionId: string, expectedHash: string): Promise<SkillSummary>;
     dismissSuggestion(id: string): Promise<SkillSuggestion>;
+    /** Delete a waiting suggestion and any draft written for it. */
+    deleteSuggestion(id: string): Promise<{
+        deleted: string;
+    }>;
+    /** Delete a draft; its suggestion goes back to waiting. */
+    deleteDraft(id: string): Promise<{
+        deleted: string;
+    }>;
     pinTrajectory(id: string, pinned: boolean): Promise<TrajectorySummary>;
     deleteTrajectory(id: string): Promise<{
         deleted: string;
@@ -92,6 +100,18 @@ export interface SkillsPageClient {
     discardDraft(id: string): Promise<SkillDraft>;
     /** Attach a baseline run to the draft it is compared with. */
     linkRun(draftId: string, runId: string): Promise<void>;
+    /** The model each piece of model work follows: the two global picks and their per-item overrides. */
+    draftPreferences(): Promise<DraftPreferencesView>;
+    /** Pick the model for drafts, for backtests, or for one suggestion or skill; `null` follows the task Agent again. */
+    saveDraftModel(model: ModelRoute | null): Promise<DraftPreferencesView>;
+    saveSuggestionModel(id: string, model: ModelRoute | null): Promise<DraftPreferencesView>;
+    saveRunModel(model: ModelRoute | null): Promise<DraftPreferencesView>;
+    saveSkillRunModel(skillName: string, model: ModelRoute | null): Promise<DraftPreferencesView>;
+    /**
+     * The routes and model directory a page may offer. Rejects when the Host
+     * offers none, which is also when the page hides its pickers.
+     */
+    models(): Promise<TaskAgentModelCatalog>;
     market(): Promise<MarketView>;
     marketPreview(registryId: string, name: string): Promise<MarketPreview>;
     /** Search the search sources; the page filters the listed ones itself. */
@@ -118,6 +138,8 @@ export interface SkillsPageClient {
         skill?: string;
         guidance?: string;
         targetRoot?: SkillRootKind;
+        provider?: string;
+        model?: string;
     }): Promise<{
         started: true;
     }>;
@@ -125,6 +147,8 @@ export interface SkillsPageClient {
         skillName: string;
         target: RunTarget;
         caseIds?: string[];
+        provider?: string;
+        model?: string;
     }): Promise<{
         runId: string;
     }>;

package/lib/types/client/model-choice.d.ts

@@ -0,0 +1,45 @@
+import type { FieldOption, TaskAgentModelCatalog } from 'dsh-mnemon/client';
+import type { DraftPreferencesView, ModelRoute } from '../contracts.ts';
+import { type SkillsTranslate } from './presentation.ts';
+import type { SkillsPageContext } from './ui.tsx';
+/**
+ * The model pickers the Evolution and Evaluation pages share: which model
+ * drafts and backtests run on, globally and per item, offered from the same
+ * directory the task Agent resolves its own route against.
+ *
+ * A pick is stored as `provider|model`, so one menu value names one route, and
+ * an absent pick is `auto`: the page follows the task Agent, as it did before
+ * the user picked anything.
+ */
+/** The menu value of a pick; `auto` asks for no pick at all. */
+export declare function modelOptionValue(model: ModelRoute | null | undefined): string;
+/** The pick a menu value names; `auto` and a malformed value both pick nothing. */
+export declare function modelOptionPick(value: string): ModelRoute | null;
+/**
+ * Follow-default first, then every Provider's models. The first entry exists
+ * before the directory arrives, so the toolbar holds its place instead of
+ * appearing a round trip later than the rest of the page.
+ */
+export declare function modelOptionsFrom(models: TaskAgentModelCatalog | null, t: SkillsTranslate, autoKey: 'tests.modelAuto' | 'evolution.modelAuto'): FieldOption<string>[];
+/** The same list for one card: follow-default, then the directory without its own default entry. */
+export declare function modelCardOptionsFrom(modelOptions: readonly FieldOption<string>[], t: SkillsTranslate, autoKey: 'tests.modelAuto' | 'evolution.modelAuto'): FieldOption<string>[];
+/** What the pickers read and write, and the directory behind them. */
+export interface ModelChoice {
+    /** `null` until the directory arrives; the pickers are disabled meanwhile. */
+    models: TaskAgentModelCatalog | null;
+    preferences: DraftPreferencesView;
+    /** Every routable model, follow-default first. */
+    options: FieldOption<string>[];
+    /** The same for one card's own pick. */
+    cardOptions: FieldOption<string>[];
+    saveDraftModel(model: ModelRoute | null): void;
+    saveSuggestionModel(id: string, model: ModelRoute | null): void;
+    saveRunModel(model: ModelRoute | null): void;
+    saveSkillRunModel(skillName: string, model: ModelRoute | null): void;
+}
+/**
+ * Read the directory and the stored picks alongside the page's own data, and
+ * write one pick at a time. A write shows locally first so the menu answers at
+ * once, then takes the Host's answer, which is also what a second window sees.
+ */
+export declare function useModelChoice(context: SkillsPageContext, autoKey: 'tests.modelAuto' | 'evolution.modelAuto'): ModelChoice;

package/lib/types/client/skill-tests.d.ts

@@ -1,5 +1,5 @@
 import { type JSX } from 'react';
-import type { BacktestRun } from '../contracts.ts';
+import type { BacktestRun, ModelRoute } from '../contracts.ts';
 import type { SkillsPageContext } from './ui.tsx';
 /**
  * A skill's test cases and its backtests: the cases with a switch each, one
@@ -9,6 +9,7 @@ import type { SkillsPageContext } from './ui.tsx';
 export declare function SkillTests(props: {
     context: SkillsPageContext;
     skillName: string;
+    runModel?: ModelRoute | null;
 }): JSX.Element;
 /** One run's cases: the verdict, each check with the judge's note, and what the simulated run did and answered. */
 export declare function RunDetail(props: {

package/lib/types/contracts.d.ts

@@ -221,6 +221,30 @@ export interface RunTarget {
     draftId?: string;
     versionId?: string;
 }
+/** One Provider and model, as a user picked it; never carries a reasoning effort. */
+export interface ModelRoute {
+    provider: string;
+    model: string;
+}
+/**
+ * The models the user picked for skill work, kept beside the ledger. Every
+ * entry is optional: absent means "follow the task Agent's own route".
+ * `suggestionModels` is keyed by suggestion id, `skillRunModels` by skill name.
+ */
+export interface DraftPreferences {
+    version: 1;
+    draftModel?: ModelRoute;
+    runModel?: ModelRoute;
+    suggestionModels: Record<string, ModelRoute>;
+    skillRunModels: Record<string, ModelRoute>;
+}
+/** What the pickers read: the stored picks, named the way the pages use them. */
+export interface DraftPreferencesView {
+    model?: ModelRoute;
+    runModel?: ModelRoute;
+    suggestionModels: Record<string, ModelRoute>;
+    skillRunModels: Record<string, ModelRoute>;
+}
 export interface CaseResult {
     caseId: string;
     title: string;
@@ -280,6 +304,10 @@ export interface ReviewLogEntry {
 }
 /** How the Source admits a review's suggestions; the Host sends its settings with each review. */
 export interface SkillsPolicy {
+    /** Whether a review may propose new skills; omitted by older Hosts, default true. */
+    allowCreate?: boolean;
+    /** Whether a review may improve existing skills; omitted by older Hosts, default true. */
+    allowUpdate?: boolean;
     minConfidence: number;
     expiryDays: number;
     maxPendingPerSession: number;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-mnemon-source-skills",
-  "version": "0.5.2",
+  "version": "0.5.4",
   "description": "Skills Source for Composable View Memory: DSH skill management, evolution suggestions, evidence trajectories and eval sets.",
   "license": "MIT",
   "type": "module",
@@ -53,7 +53,7 @@
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
-    "dsh-mnemon": "^0.5.33",
+    "dsh-mnemon": "^0.5.37",
     "react": "^18.2.0"
   },
   "devDependencies": {
@@ -62,7 +62,7 @@
     "@types/node": "^22.0.0",
     "@types/react": "^18.3.0",
     "@types/react-dom": "^18.3.0",
-    "dsh-mnemon": "^0.5.35",
+    "dsh-mnemon": "^0.5.37",
     "jsdom": "^26.0.0",
     "react": "^18.2.0",
     "react-dom": "^18.2.0",

package/presentation/locales.json

@@ -335,7 +335,17 @@
     "trash.original": "原位置",
     "trash.chooseHint": "可选择恢复位置或改名。同名替换需要明确确认，现有技能会保留在回收站。",
     "trash.replace": "替换同名技能，并将现有版本保留在回收站",
-    "trash.confirmName": "输入 {name} 确认彻底删除"
+    "trash.confirmName": "输入 {name} 确认彻底删除",
+    "evolution.model": "全局模型",
+    "evolution.modelCard": "所选草案模型",
+    "evolution.modelAuto": "跟随默认",
+    "evolution.modelAutoHint": "使用当前任务 Agent 的模型",
+    "evolution.deleteTitle": "删除这条建议？",
+    "evolution.deleteBody": "它生成的草案和证据引用会一起删掉，无法恢复。",
+    "evolution.deleted": "已删除",
+    "tests.model": "全局模型",
+    "tests.modelCard": "所选回测模型",
+    "tests.modelAuto": "跟随默认"
   },
   "en": {
     "page.title": "Skills",
@@ -673,6 +683,16 @@
     "trash.original": "Original location",
     "trash.chooseHint": "Choose a location or a different name. Replacing a skill requires confirmation and keeps its current version in the Trash.",
     "trash.replace": "Replace the existing skill and keep its current version in the Trash",
-    "trash.confirmName": "Type {name} to confirm permanent deletion"
+    "trash.confirmName": "Type {name} to confirm permanent deletion",
+    "evolution.model": "Global model",
+    "evolution.modelCard": "Selected model",
+    "evolution.modelAuto": "Follow default",
+    "evolution.modelAutoHint": "Use the model of the current task Agent",
+    "evolution.deleteTitle": "Delete this suggestion?",
+    "evolution.deleteBody": "Its draft and the evidence references it cites go with it, and this cannot be undone.",
+    "evolution.deleted": "Deleted",
+    "tests.model": "Global model",
+    "tests.modelCard": "Selected model",
+    "tests.modelAuto": "Follow default"
   }
 }

package/presentation/page.module.css

@@ -180,6 +180,12 @@
 .skList { display: flex; flex-direction: column; overflow: hidden; padding-bottom: 6px; }
 .skPane { padding: clamp(16px, 2vw, 24px); overflow: hidden; }
 
+/* Keep the evolution inbox beside its detail while browsing long lists. */
+.skEvolutionSplit { height: max(320px, calc(100dvh - 320px)); min-height: 0; }
+.skEvolutionSplit > .skList,
+.skEvolutionSplit > .skPane { min-height: 0; overflow-y: auto; overscroll-behavior-y: contain; }
+.skEvolutionSplit > .skList > * { flex-shrink: 0; }
+
 .skListBar { display: flex; align-items: center; gap: 8px; padding: 10px; border-bottom: .5px solid var(--mn-line); }
 .skListBar > :first-child { min-width: 0; flex: 1; }
 .skListNote { margin: 0; padding: 18px 16px; color: var(--mn-faint); font-size: 12px; line-height: 18px; text-align: center; }
@@ -574,6 +580,8 @@
 @media (max-width: 860px) {
   .skSplit { grid-template-columns: minmax(0, 1fr); }
   .skList { max-height: 360px; overflow-y: auto; }
+  .skEvolutionSplit { height: auto; }
+  .skEvolutionSplit > .skPane { overflow: visible; overscroll-behavior-y: auto; }
 }
 
 @media (max-width: 560px) {