@lmzhen/dsh-evolution-skill-history 0.11.2 → 0.12.0

package/README.md

@@ -32,7 +32,7 @@ The row is **inert without a web server** (`ctx.get('webServer')`, not a declare
 
 ## What the panel 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 offers **Restore this version** unless it already IS the live content (the host marks those), and the button turns into an inline confirmation first. After a restore the list is read again, because the restore itself is a new version.
+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 offers **Restore this version** unless it already IS the live content (the host marks those), and the button turns into an inline confirmation first. 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; before it, seeing what another writer recorded in the meantime meant opening the panel again for the list, or clicking the open skill again for its rows (the aside counts are read with the list, not per row).
 
 ## Build
 
@@ -44,7 +44,7 @@ This package adds **no model-visible content**: no prompt section, no tool schem
 
 ## Known limitations and deferred work
 
-- **No rendered spec.** The family's specs are Node-level and this package's client half cannot be rendered there, so the panel is covered by `tsc`, the client bundle build, the pure `tests/client-api.spec.ts` and a live pass on the installed artifact. A rendered spec (jsdom plus a driven fixture runtime) is the follow-up that would catch wiring in CI.
+- **No rendered spec.** The family's specs are Node-level and this package's client half cannot be rendered there, so the panel is covered by `tsc`, the client bundle build, the pure `tests/client-api.spec.ts` and a live pass on the installed artifact. A rendered spec (jsdom plus a driven fixture runtime) is the follow-up that would catch wiring in CI — 0.11.3 is the cost of not having one: the first installed pass restored the content correctly and re-read the rows correctly while the curator's result sentence never appeared, because the reload that follows a restore cleared the note state it had just set. No Node-level spec and no `tsc` can see the order of two state updates; only clicking the panel showed it.
 - **The list shows skills with recorded versions only.** A skill that never wrote through the library has no history to browse, so it does not appear; the tree-wide listing stays with the skill catalog.
 - **No diff view.** The routes return entries, not bodies; a two-version diff would need a second read route and a client-side differ.
 - **Support-file bytes are history only.** They share the index with the body, so they are listed and labelled, but `undo` refuses them by name — the bytes stay in the blob store for a hand copy.

package/lib/client.js

@@ -110,12 +110,19 @@ window.__ModuleLoader__.load({
 		* The panel owns no rule: the host reports each body row with a verdict (`undoable`) computed from
 		* the version facts in evolution-core, and the curator owns the words a refusal uses. This file is
 		* presentation plus local state — it never subscribes to anything and it never sees the context.
+		*
+		* Two state rules, both pinned by a live pass rather than by a test (a browser half has no unit test
+		* seat in this family): a restore's result sentence stays on screen while the rows are read again, and
+		* nothing is read on a poll — the skill list is read on mount and by the refresh button, the rows when
+		* a skill opens, when the refresh button is pressed and after a restore. The width cap keeps a
+		* version's action within reach of the version it acts on.
 		* @module @lmzhen/dsh-evolution-skill-history/client
 		*/
 		const root = {
 			display: "flex",
 			height: "100%",
 			minHeight: "0",
+			maxWidth: "960px",
 			fontSize: "13px",
 			color: "var(--dsw-alias-label-primary)"
 		};
@@ -132,6 +139,12 @@ window.__ModuleLoader__.load({
 			overflowY: "auto",
 			padding: "12px 14px"
 		};
+		const header = {
+			display: "flex",
+			alignItems: "center",
+			gap: "10px",
+			margin: "0 0 4px"
+		};
 		const skillButton = {
 			display: "block",
 			width: "100%",
@@ -149,9 +162,10 @@ window.__ModuleLoader__.load({
 			fontWeight: 600
 		};
 		const heading = {
-			margin: "0 0 4px",
+			margin: "0",
 			fontSize: "13px",
-			fontWeight: 600
+			fontWeight: 600,
+			flex: "1 1 auto"
 		};
 		const hint = {
 			margin: "0 0 10px",
@@ -229,19 +243,42 @@ window.__ModuleLoader__.load({
 			const [payload, setPayload] = (0, react.useState)(void 0);
 			const [noteText, setNoteText] = (0, react.useState)(void 0);
 			const [pending, setPending] = (0, react.useState)(void 0);
+			const readTicket = (0, react.useRef)(0);
+			const failureText = (error) => face.t("error") + ": " + (error instanceof Error ? error.message : String(error));
 			const failed = (error) => {
-				setNoteText(face.t("error") + ": " + (error instanceof Error ? error.message : String(error)));
+				setNoteText(failureText(error));
 			};
-			const refresh = () => {
-				setSkills([]);
+			/** Read the skills again, keeping the sentence: a re-read is not an answer to anything. */
+			const reloadSkills = () => {
 				face.loadSkills().then(setSkills).catch(failed);
 			};
-			const open = (name) => {
+			/**
+			* Show one skill's chains, keeping the sentence. A re-read that FAILS reports through the caller's
+			* own handler, so the failure of a read can never erase the write it was reading after.
+			* @param name - the skill to open.
+			* @param onFailure - what to do with a failed read; the panel shows it as its own sentence.
+			*/
+			const showVersions = (name, onFailure = failed) => {
+				const ticket = readTicket.current + 1;
+				readTicket.current = ticket;
 				setSelected(name);
 				setPayload(void 0);
 				setPending(void 0);
-				setNoteText(void 0);
-				face.loadVersions(name).then(setPayload).catch(failed);
+				face.loadVersions(name).then((next) => {
+					if (readTicket.current === ticket) setPayload(next);
+				}).catch((error) => {
+					if (readTicket.current === ticket) onFailure(error);
+				});
+			};
+			/** Open a skill from the list: another skill's sentence described another skill, so it goes away. */
+			const open = (name) => {
+				if (name !== selected) setNoteText(void 0);
+				showVersions(name);
+			};
+			/** What the refresh button does: both reads again, and the sentence stays. */
+			const refresh = () => {
+				reloadSkills();
+				if (selected !== void 0) showVersions(selected);
 			};
 			const restore = (v) => {
 				if (selected === void 0) return;
@@ -250,12 +287,15 @@ window.__ModuleLoader__.load({
 					return;
 				}
 				setPending(void 0);
-				face.undo(selected, v).then((message) => {
+				const name = selected;
+				face.undo(name, v).then((message) => {
+					showVersions(name, (error) => {
+						setNoteText(message + "\n" + failureText(error));
+					});
 					setNoteText(message);
-					open(selected);
 				}).catch(failed);
 			};
-			(0, react.useEffect)(refresh, []);
+			(0, react.useEffect)(reloadSkills, []);
 			const loaded = payload !== void 0;
 			return (0, react.createElement)("div", { style: root }, (0, react.createElement)("aside", { style: aside }, skills.length === 0 ? (0, react.createElement)("p", { style: hint }, face.t("empty.skills")) : skills.map((skill) => (0, react.createElement)("button", {
 				key: skill.name,
@@ -264,7 +304,11 @@ window.__ModuleLoader__.load({
 				onClick: () => {
 					open(skill.name);
 				}
-			}, skill.name + "  (" + String(skill.versions) + ")"))), (0, react.createElement)("div", { style: main }, (0, react.createElement)("h2", { style: heading }, face.t("title")), (0, react.createElement)("p", { style: hint }, face.t("hint")), noteText === void 0 ? null : (0, react.createElement)("p", { style: note }, noteText), selected === void 0 ? null : loaded ? null : (0, react.createElement)("p", { style: hint }, face.t("loading")), loaded ? (0, react.createElement)("div", null, group({
+			}, skill.name + "  (" + String(skill.versions) + ")"))), (0, react.createElement)("div", { style: main }, (0, react.createElement)("div", { style: header }, (0, react.createElement)("h2", { style: heading }, face.t("title")), (0, react.createElement)("button", {
+				type: "button",
+				style: action,
+				onClick: refresh
+			}, face.t("refresh"))), (0, react.createElement)("p", { style: hint }, face.t("hint")), noteText === void 0 ? null : (0, react.createElement)("p", { style: note }, noteText), selected === void 0 ? null : loaded ? null : (0, react.createElement)("p", { style: hint }, face.t("loading")), loaded ? (0, react.createElement)("div", null, group({
 				...face,
 				restore,
 				confirming: pending

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

@@ -5,6 +5,12 @@
  * The panel owns no rule: the host reports each body row with a verdict (`undoable`) computed from
  * the version facts in evolution-core, and the curator owns the words a refusal uses. This file is
  * presentation plus local state — it never subscribes to anything and it never sees the context.
+ *
+ * Two state rules, both pinned by a live pass rather than by a test (a browser half has no unit test
+ * seat in this family): a restore's result sentence stays on screen while the rows are read again, and
+ * nothing is read on a poll — the skill list is read on mount and by the refresh button, the rows when
+ * a skill opens, when the refresh button is pressed and after a restore. The width cap keeps a
+ * version's action within reach of the version it acts on.
  * @module @lmzhen/dsh-evolution-skill-history/client
  */
 import { type ReactNode } from 'react';

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.11.2",
+  "version": "0.12.0",
   "publishConfig": {
     "access": "public"
   },
@@ -30,21 +30,21 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@lmzhen/dsh-evolution-core": "^0.11.2"
+    "@lmzhen/dsh-evolution-core": "^0.12.0"
   },
   "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.11.2"
+    "@lmzhen/dsh-evolution-curator": "^0.12.0"
   },
   "devDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-host-webserver": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-core": "^0.11.2",
-    "@lmzhen/dsh-evolution-curator": "^0.11.2",
-    "@lmzhen/dsh-evolution-io": "^0.11.2",
-    "@lmzhen/dsh-evolution-io-node": "^0.11.2"
+    "@lmzhen/dsh-evolution-core": "^0.12.0",
+    "@lmzhen/dsh-evolution-curator": "^0.12.0",
+    "@lmzhen/dsh-evolution-io": "^0.12.0",
+    "@lmzhen/dsh-evolution-io-node": "^0.12.0"
   },
   "dsh": {
     "client": {