@lmzhen/dsh-evolution-skill-history 0.13.0 → 0.13.2

package/README.md

@@ -35,11 +35,11 @@ The row is **inert without a web server** (`ctx.get('webServer')`, not a declare
 
 **The left column** lists the skills that recorded versions, one row per skill: its name, how many versions it has, its own one-line description (the listing already carried it) and the management state the marker probe reported. A **search box** filters that list by name or description — client state only, no route behind it.
 
-**The right pane** shows the selected skill's versions as two groups: **Body versions** (restorable) and **File versions** (history only, because a restore rewrites `SKILL.md`). A body row reads like a sentence rather than a dump — `v3  targeted edit  2026-09-27 18:20 · 3 hours ago  (+1363 chars)` — with a *current* capsule on the live content, **Restore this version** (an inline two-click confirmation) on the others, and **Diff** expanding that version against its predecessor (lines added and removed, plus the changed region, marked truncated when it is long). After a restore the rows are read again — the restore itself is a new version — while the curator's result sentence stays on screen. **Refresh** re-reads the skill list and the open skill.
+**The right pane** shows the selected skill's versions as two groups: **Body versions** (restorable) and **File versions** (history only, because a restore rewrites `SKILL.md`). A body row reads like a sentence rather than a dump — `v3  targeted edit  2026-09-27 18:20 · 3 hours ago  (+1363 chars)` — with a *current* capsule on the live content, **Restore this version** (an inline two-click confirmation, with a **Cancel** beside it once it is armed) on the others, and **Diff** expanding that version against its predecessor (lines added and removed, plus the changed region, marked truncated when it is long). After a restore the rows are read again — the restore itself is a new version — while the curator's result sentence stays on screen. **Refresh** re-reads the skill list and the open skill.
 
 **The panel owns no rule and no arithmetic.** The host reports every row with its whole verdict — `undoable` (evolution-core's artifact classification against the live bytes), `actionKind` (a closed vocabulary key), `age` (a bucket plus a count), `charsDelta`, `summary` — and this half substitutes words from its locale dictionary and renders. The time, the action words and the sentences are therefore localized; the facts are computed once, host-side, so the panel and the slash commands cannot drift.
 
-**Appearance follows the platform's tokens, not its modules.** The panel carries its own stylesheet (`src/client/styles.ts`, injected once behind `data-plugin-css`) and copies its metrics from the platform's own components — the command card's `<pre>` for the diff block, the sidebar row's hover and focus ring for the controls — while every size is a `--dsh-content-font-size*` token or a `calc()` over one, so the panel follows the deployment's content size the way the platform's panels do. Importing the platform's control package is NOT possible from here without splitting this package's compiler faces: the specifier is path-mapped to source, and `tsc -b` then refuses the foreign project (host/client face split, `react` devDependencies and a CI graph change would all be needed). The roll call of what the panel deliberately does not do is in `## Known limitations and deferred work`.
+**Appearance follows the platform's tokens, not its modules.** The panel carries its own stylesheet (`src/client/styles.ts`, injected once behind `data-plugin-css`) and copies its metrics from the platform's own components — the command card's `<pre>` for the diff block, the sidebar row's hover and focus ring for the controls, and the theme's semantic state pair (`--dsw-alias-state-success-primary` / `--dsw-alias-state-error-primary`) for the diff's added and removed lines — while every size is a `--dsh-content-font-size*` token or a `calc()` over one, so the panel follows the deployment's content size the way the platform's panels do. Importing the platform's control package is NOT possible from here without splitting this package's compiler faces: the specifier is path-mapped to source, and `tsc -b` then refuses the foreign project (host/client face split, `react` devDependencies and a CI graph change would all be needed). The roll call of what the panel deliberately does not do is in `## Known limitations and deferred work`.
 
 ## Build
 

package/lib/client.js

@@ -68,8 +68,7 @@ window.__ModuleLoader__.load({
 		const zh = {
 			"entry.label": "技能历史",
 			title: "技能历史",
-			hint: "每次技能写入留下的内容版本。回退只改 SKILL.md 正文，标记、使用计数与策展状态不变。",
-			"hint.current": "内容相同的版本会同时标「当前」。",
+			hint: "每次技能写入留下的内容版本。回退只改 SKILL.md 正文，标记、使用计数与策展状态不变。内容相同的版本会同时标「当前」。",
 			search: "按名字或描述筛选",
 			"search.none": "没有匹配的技能。",
 			"group.content": "正文版本",
@@ -130,8 +129,7 @@ window.__ModuleLoader__.load({
 		const en = {
 			"entry.label": "Skill history",
 			title: "Skill history",
-			hint: "The content versions every skill write leaves behind. Restoring changes the SKILL.md body only: markers, usage counts and curation state stay as they are.",
-			"hint.current": "Versions holding the same content are all marked “current”.",
+			hint: "The content versions every skill write leaves behind. Restoring changes the SKILL.md body only: markers, usage counts and curation state stay as they are. Versions holding the same content are all marked “current”.",
 			search: "Filter by name or description",
 			"search.none": "No skill matches.",
 			"group.content": "Body versions",
@@ -242,7 +240,7 @@ window.__ModuleLoader__.load({
 			".evo-hist-skill-line{display:flex;align-items:baseline;gap:6px}",
 			".evo-hist-skill-name{min-width:0;overflow:hidden;text-overflow:ellipsis;white-space:nowrap}",
 			".evo-hist-skill-count{flex:none;margin-left:auto;color:var(--dsw-alias-label-tertiary)}",
-			".evo-hist-skill-desc{margin-top:2px;color:var(--dsw-alias-label-tertiary);overflow:hidden;text-overflow:ellipsis;white-space:nowrap}",
+			".evo-hist-skill-desc{display:block;margin-top:2px;color:var(--dsw-alias-label-tertiary);overflow:hidden;text-overflow:ellipsis;white-space:nowrap}",
 			".evo-hist-main{flex:1 1 auto;min-width:0;min-height:0;overflow-y:auto;padding:12px 14px}",
 			".evo-hist-head{display:flex;align-items:center;gap:10px;margin:0 0 4px}",
 			".evo-hist-title{flex:1 1 auto;margin:0;font-size:inherit;font-weight:600}",
@@ -262,7 +260,7 @@ window.__ModuleLoader__.load({
 			".evo-hist-button:focus-visible{outline:2px solid var(--dsw-alias-label-primary);outline-offset:-2px}",
 			".evo-hist-button[data-tone=\"primary\"]{border-color:var(--dsw-alias-brand-primary);color:var(--dsw-alias-brand-primary)}",
 			".evo-hist-pre{margin:6px 0 2px;padding:12px 16px;max-height:260px;overflow:auto;border:.5px solid var(--dsw-alias-border-l1);border-radius:12px;background:var(--dsw-alias-markdown-code-block);color:var(--dsw-alias-label-primary);font:var(--dsw-font-markdown-code-block-small);white-space:pre-wrap}",
-			".evo-hist-pre-add{color:var(--dsw-alias-brand-primary)}",
+			".evo-hist-pre-add{color:var(--dsw-alias-state-success-primary)}",
 			".evo-hist-pre-del{color:var(--dsw-alias-state-error-primary)}"
 		].join("\n");
 		//#endregion
@@ -333,7 +331,7 @@ window.__ModuleLoader__.load({
 			return (0, react.createElement)("pre", { className: "evo-hist-pre" }, rows.length === 0 ? "±" : rows);
 		}
 		/** One version row: what happened, when, how big, and what it is against. */
-		function versionRow(face, row, confirming, openDiff, diff, restore, toggleDiff) {
+		function versionRow(face, row, confirming, openDiff, diff, restore, cancelRestore, toggleDiff) {
 			const parts = [
 				(0, react.createElement)("span", {
 					key: "v",
@@ -355,15 +353,25 @@ window.__ModuleLoader__.load({
 				key: "current",
 				className: "evo-hist-capsule"
 			}, face.t("current")));
-			if (row.undoable === true) actions.push((0, react.createElement)("button", {
-				key: "undo",
-				type: "button",
-				className: "evo-hist-button",
-				"data-tone": confirming === row.v ? "primary" : void 0,
-				onClick: () => {
-					restore(row.v);
-				}
-			}, confirming === row.v ? face.t("undo.confirm") : face.t("undo")));
+			if (row.undoable === true) {
+				if (confirming === row.v) actions.push((0, react.createElement)("button", {
+					key: "cancel",
+					type: "button",
+					className: "evo-hist-button",
+					onClick: () => {
+						cancelRestore();
+					}
+				}, face.t("undo.no")));
+				actions.push((0, react.createElement)("button", {
+					key: "undo",
+					type: "button",
+					className: "evo-hist-button",
+					"data-tone": confirming === row.v ? "primary" : void 0,
+					onClick: () => {
+						restore(row.v);
+					}
+				}, confirming === row.v ? face.t("undo.confirm") : face.t("undo")));
+			}
 			if (row.undoable === true || row.charsDelta !== void 0) actions.push((0, react.createElement)("button", {
 				key: "diff",
 				type: "button",
@@ -389,7 +397,7 @@ window.__ModuleLoader__.load({
 		/** One group: a heading, an optional note, and its rows. */
 		function group(face, title, entries, noteText, state) {
 			if (entries.length === 0) return null;
-			return (0, react.createElement)("section", { key: title }, (0, react.createElement)("h3", { className: "evo-hist-group" }, title), noteText === void 0 ? null : (0, react.createElement)("p", { className: "evo-hist-hint" }, noteText), ...entries.map((entry) => versionRow(face, entry, state.confirming, state.openDiff, state.diffs.get(entry.v), state.restore, state.toggleDiff)));
+			return (0, react.createElement)("section", { key: title }, (0, react.createElement)("h3", { className: "evo-hist-group" }, title), noteText === void 0 ? null : (0, react.createElement)("p", { className: "evo-hist-hint" }, noteText), ...entries.map((entry) => versionRow(face, entry, state.confirming, state.openDiff, state.diffs.get(entry.v), state.restore, state.cancelRestore, state.toggleDiff)));
 		}
 		/**
 		* The panel body.
@@ -499,11 +507,16 @@ window.__ModuleLoader__.load({
 			const needle = query.trim().toLowerCase();
 			const visible = (0, react.useMemo)(() => needle === "" ? skills : skills.filter((skill) => skill.name.toLowerCase().includes(needle) || skill.description.toLowerCase().includes(needle)), [skills, needle]);
 			const loaded = payload !== void 0;
+			/** Drop the pending confirmation: the button that opened it must not be the only way out. */
+			const cancelRestore = () => {
+				setPending(void 0);
+			};
 			const rowState = {
 				confirming: pending,
 				openDiff,
 				diffs,
 				restore,
+				cancelRestore,
 				toggleDiff
 			};
 			return (0, react.createElement)("div", { className: "evo-hist-root" }, (0, react.createElement)("aside", { className: "evo-hist-aside" }, (0, react.createElement)("div", { className: "evo-hist-search" }, (0, react.createElement)("input", {
@@ -526,7 +539,7 @@ window.__ModuleLoader__.load({
 				type: "button",
 				className: "evo-hist-button",
 				onClick: refresh
-			}, face.t("refresh"))), (0, react.createElement)("p", { className: "evo-hist-hint" }, face.t("hint") + " " + face.t("hint.current")), noteText === void 0 ? null : (0, react.createElement)("p", {
+			}, face.t("refresh"))), (0, react.createElement)("p", { className: "evo-hist-hint" }, face.t("hint")), noteText === void 0 ? null : (0, react.createElement)("p", {
 				className: "evo-hist-note",
 				"data-error": noteError ? "true" : void 0
 			}, noteText), selected === void 0 ? null : loaded ? null : (0, react.createElement)("p", { className: "evo-hist-hint" }, face.t("loading")), loaded ? (0, react.createElement)("div", null, group(face, face.t("group.content"), payload.content, void 0, rowState), group(face, face.t("group.support"), payload.support, face.t("group.support.note"), rowState), payload.content.length === 0 && payload.support.length === 0 ? (0, react.createElement)("p", { className: "evo-hist-hint" }, face.t("empty.versions")) : null) : null));
@@ -551,18 +564,23 @@ window.__ModuleLoader__.load({
 		/** Row order inside the global panel list (beside the platform's own rows). */
 		const PANEL_ORDER = 35;
 		/**
-		* The row's icon: a platform-style line glyph rather than an emoji.
+		* The row's icon: a platform-style line glyph, sized by the SIDEBAR's own contract.
 		*
-		* An emoji cannot follow `currentColor`, cannot match the 16/18px the sidebar hands its glyph slot,
-		* and renders differently on every platform font — the sidebar's other rows are 1.5px-stroke line
-		* icons, and this one now is too. The label carries the accessible name, so the glyph stays
-		* decoration.
+		* The platform hands every `sidebar.panellist` occupant `{ size, active }` (`SidebarRoot.tsx`
+		* `renderSlot('sidebar.panellist', { size: wide ? 16 : 18, active }, …)`) and its own icons render at
+		* that size (`IconGlobeOutline14 size={size}` in the sidebar's own spec). The neighbouring rows in a
+		* shipped GUI take the OTHER route — they inject their entry into the sidebar's DOM with their own
+		* `svg { width: 18px; height: 18px }` CSS — so 18px is the size a row glyph is expected to occupy.
+		* A glyph that sizes itself from CSS (`width: 100%`) resolves against an unsized flex box and fills
+		* the column: the size must come from the prop, with the neighbours' 18 as the fallback. The label
+		* carries the accessible name, so the glyph stays decoration.
 		*/
-		function PanelIcon() {
+		function PanelIcon(props = {}) {
+			const size = typeof props.size === "number" && props.size > 0 ? props.size : 18;
 			return (0, react.createElement)("svg", {
 				"aria-hidden": "true",
-				width: "100%",
-				height: "100%",
+				width: String(size),
+				height: String(size),
 				viewBox: "0 0 16 16",
 				fill: "none",
 				stroke: "currentColor",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-skill-history",
   "description": "Web surface for skill content history: loopback host routes plus the sidebar panel (community build)",
-  "version": "0.13.0",
+  "version": "0.13.2",
   "publishConfig": {
     "access": "public"
   },
@@ -30,21 +30,21 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@lmzhen/dsh-evolution-core": "^0.13.0"
+    "@lmzhen/dsh-evolution-core": "^0.13.2"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-host-webserver": "^0.1.5-rc.2",
     "react": "^18.2.0",
-    "@lmzhen/dsh-evolution-curator": "^0.13.0"
+    "@lmzhen/dsh-evolution-curator": "^0.13.2"
   },
   "devDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-host-webserver": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-core": "^0.13.0",
-    "@lmzhen/dsh-evolution-curator": "^0.13.0",
-    "@lmzhen/dsh-evolution-io": "^0.13.0",
-    "@lmzhen/dsh-evolution-io-node": "^0.13.0"
+    "@lmzhen/dsh-evolution-core": "^0.13.2",
+    "@lmzhen/dsh-evolution-curator": "^0.13.2",
+    "@lmzhen/dsh-evolution-io": "^0.13.2",
+    "@lmzhen/dsh-evolution-io-node": "^0.13.2"
   },
   "dsh": {
     "client": {