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

package/lib/index.js

@@ -247,23 +247,28 @@ function makeSkillHistoryRoutes(services) {
 * @module @lmzhen/dsh-evolution-skill-history
 */
 const name = "evolution-skill-history";
-/** No declared service: the web server is OPTIONAL (a headless profile has none) and the curator is
-* resolved per request, so the row activates wherever it is installed and answers honestly where a
-* service is missing — the platform rule for optional services is ctx.get, not a waited-on inject. */
+/** No declared service: a headless profile must not leave a pending row, and the curator is resolved
+* per request. The web server is waited on INSIDE apply (see below), not declared here. */
 const inject = [];
 /**
-* Mount the routes as ONE effect: the disposers run on unload and on HMR.
-* @param ctx - the host context carrying webServer and the curator.
+* Wait for the web server, then mount the routes as ONE effect (disposers run on unload and on HMR).
+*
+* **Why `ctx.inject` and not `ctx.get`**: profile rows apply in file order, so at this row's apply time
+* the web server may not exist yet — `ctx.get('webServer')` returning undefined is a TIMING fact, not a
+* verdict, and acting on it (0.11.1 shipped that way) left the routes permanently unregistered while the
+* panel rendered and every call answered HTTP 404. Waiting on the service mounts the routes whenever it
+* arrives, and a profile that never gets one simply never runs this callback — inert, never pending.
+* @param ctx - the host context; the web server and curator arrive through it.
 */
 function apply(ctx) {
-	const webServer = ctx.get("webServer");
-	if (webServer === void 0) return;
-	ctx.effect(() => {
-		const disposers = makeSkillHistoryRoutes({ getCurator: () => ctx.get("evolutionCurator") }).map((route) => webServer.register(route));
-		return () => {
-			for (const dispose of disposers) dispose();
-		};
-	}, "evolution-skill-history: routes");
+	ctx.inject(["webServer"], (scope) => {
+		scope.effect(() => {
+			const disposers = makeSkillHistoryRoutes({ getCurator: () => scope.get("evolutionCurator") }).map((route) => scope.webServer.register(route));
+			return () => {
+				for (const dispose of disposers) dispose();
+			};
+		}, "evolution-skill-history: routes");
+	});
 }
 //#endregion
 export { apply, inject, name };

package/lib/types/index.d.ts

@@ -9,13 +9,18 @@
  */
 import type { Context } from '@deepseek-ai/cordis';
 export declare const name = "evolution-skill-history";
-/** No declared service: the web server is OPTIONAL (a headless profile has none) and the curator is
- * resolved per request, so the row activates wherever it is installed and answers honestly where a
- * service is missing — the platform rule for optional services is ctx.get, not a waited-on inject. */
+/** No declared service: a headless profile must not leave a pending row, and the curator is resolved
+ * per request. The web server is waited on INSIDE apply (see below), not declared here. */
 export declare const inject: string[];
 /**
- * Mount the routes as ONE effect: the disposers run on unload and on HMR.
- * @param ctx - the host context carrying webServer and the curator.
+ * Wait for the web server, then mount the routes as ONE effect (disposers run on unload and on HMR).
+ *
+ * **Why `ctx.inject` and not `ctx.get`**: profile rows apply in file order, so at this row's apply time
+ * the web server may not exist yet — `ctx.get('webServer')` returning undefined is a TIMING fact, not a
+ * verdict, and acting on it (0.11.1 shipped that way) left the routes permanently unregistered while the
+ * panel rendered and every call answered HTTP 404. Waiting on the service mounts the routes whenever it
+ * arrives, and a profile that never gets one simply never runs this callback — inert, never pending.
+ * @param ctx - the host context; the web server and curator arrive through it.
  */
 export declare function apply(ctx: Context): void;
 //# sourceMappingURL=index.d.ts.map

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