@lmzhen/dsh-evolution-commands 0.27.0 → 0.27.1

package/lib/index.js

@@ -1602,40 +1602,53 @@ function legacyLine(row) {
 	const more = row.pending.length > 6 ? ` (+${row.pending.length - 6} more)` : "";
 	return `legacy settings: ${row.pending.length} family value(s) still on ${row.source ?? "(unknown)"}: ${shown}${more} — /evolution migrate moves them`;
 }
+/**
+* W8-S7: the doctor's text face, VIEW-FIRST.
+*
+* The frozen DiagnosticsView (7 facts + 8 runtime rows) is the single text source for everything
+* it carries — install form, platform services, runs, channel/preset/host/pairing. The legacy
+* report now contributes only the facts the view does not represent yet: cross-profile bundles,
+* the family service inference, capability absence, the scoped-row probe, the legacy settings
+* document, pending/executing audit records, preset freshness, and the conflict/env/budget/
+* parameter/session-search findings + next steps. This is the W8-S7 rewrite branch: the text face
+* has real consumers (the `/evolution doctor` command, README/INSTALL, doctor.spec), so the legacy
+* rendering was rewritten instead of deleted — but every fact the view owns renders from the view
+* exactly once (the legacy install-form line is gone).
+* @param report - the assembled report (view + legacy findings).
+* @returns the text face.
+*/
 function renderDoctorText(report) {
-	const lines = [
-		`Evolution doctor — install form: ${report.installForm} (deployment: ${report.deploymentForm}; from ${report.formSource === "platform" ? "this runtime's bundle surface" : "the cross-profile aggregate"})`,
-		report.runtimeBundles === null ? "runtime bundles: unknown — the platform bundle surface (pluginManager.listBundles) is not mounted here" : `runtime bundles (this profile): ${report.runtimeBundles.length > 0 ? report.runtimeBundles.join(", ") : "(none)"}`,
-		report.dormantBundles.length === 0 ? null : `installed but not selected: ${report.dormantBundles.join(", ")} — the Plugins page lists them; this runtime mounts neither`,
-		`bundles (all profiles): ${report.bundles.length > 0 ? report.bundles.join(", ") : "(none)"}`,
-		`services: review=${report.services.review} (${report.serviceSource === "platform" ? "the live row, this runtime" : "inferred from bundles, all profiles"}) curator=${report.services.curator} approval=${report.services.approval} skillUsage=${report.services.skillUsage} io=${report.services.io}`,
-		report.capabilityAbsence.length === 0 ? null : `capability absence (${report.capabilityAbsence.length}): ${report.capabilityAbsence.map((entry) => `${entry.service} — ${entry.feature}`).join("; ")}`,
-		scopedProbeLine(report.scopedProbe),
-		legacyLine(report.legacy),
-		`pending: ${report.pendingCount === null ? "unknown" : report.pendingCount}`,
-		`executing: ${report.executingCount === null ? "unknown" : report.executingCount}`
-	];
+	const extras = [];
+	extras.push(report.runtimeBundles === null ? "runtime bundles: unknown — the platform bundle surface (pluginManager.listBundles) is not mounted here" : `runtime bundles (this profile): ${report.runtimeBundles.length > 0 ? report.runtimeBundles.join(", ") : "(none)"}`);
+	if (report.dormantBundles.length > 0) extras.push(`installed but not selected: ${report.dormantBundles.join(", ")} — the Plugins page lists them; this runtime mounts neither`);
+	extras.push(`bundles (all profiles): ${report.bundles.length > 0 ? report.bundles.join(", ") : "(none)"}`);
+	extras.push(`services: review=${report.services.review} (${report.serviceSource === "platform" ? "the live row, this runtime" : "inferred from bundles, all profiles"}) curator=${report.services.curator} approval=${report.services.approval} skillUsage=${report.services.skillUsage} io=${report.services.io}`);
+	if (report.capabilityAbsence.length > 0) extras.push(`capability absence (${report.capabilityAbsence.length}): ${report.capabilityAbsence.map((entry) => `${entry.service} — ${entry.feature}`).join("; ")}`);
+	extras.push(scopedProbeLine(report.scopedProbe));
+	extras.push(legacyLine(report.legacy));
+	extras.push(`pending: ${report.pendingCount === null ? "unknown" : report.pendingCount}`);
+	extras.push(`executing: ${report.executingCount === null ? "unknown" : report.executingCount}`);
 	for (const row of report.presetFreshness) {
 		if (row.status === "absent") continue;
 		const verdict = row.status === "fresh" ? "fresh — a fresh generation matches this file byte-for-byte" : row.status === "differs" ? "DIFFERS from a fresh generation — it is an install-time snapshot; re-run the installer (/evolution preset install) to regenerate" : `unknown — a fresh generation could not be recomputed (${row.detail ?? "reason not recorded"})`;
-		lines.push(`preset:   ${row.base} → ${row.destination}  ${verdict}`);
+		extras.push(`preset:   ${row.base} → ${row.destination}  ${verdict}`);
 	}
-	if (report.presetIssues.length > 0) lines.push("agent presets:", ...report.presetIssues.map((line) => `  ! ${line}`));
-	if (report.foreignProfiles.length > 0) lines.push("profiles-foreign: " + report.foreignProfiles.join(", ") + " — not part of this plane: /evolution preset install ignores them, while the bundles (all profiles) line above counts them");
-	if (report.conflicts.length > 0) lines.push("conflicts:", ...report.conflicts.map((line) => `  ! ${line}`));
-	if (report.envIssues.length > 0) lines.push("env:", ...report.envIssues.map((line) => `  ! ${line}`));
-	if (report.budgetIssues.length > 0) lines.push("memory budget:", ...report.budgetIssues.map((line) => `  ! ${line}`));
-	if (report.paramIssues.length > 0) lines.push("parameters:", ...report.paramIssues.map((line) => `  ! ${line}`));
-	if (report.queryIssues.length > 0) lines.push("session search:", ...report.queryIssues.map((line) => `  ! ${line}`));
-	if (report.actions.length > 0) lines.push("next steps:", ...report.actions.map((line) => `  → ${line}`));
-	return lines.filter((line) => line !== null).join("\n");
+	if (report.presetIssues.length > 0) extras.push("agent presets:", ...report.presetIssues.map((line) => `  ! ${line}`));
+	if (report.foreignProfiles.length > 0) extras.push("profiles-foreign: " + report.foreignProfiles.join(", ") + " — not part of this plane: /evolution preset install ignores them, while the bundles (all profiles) line above counts them");
+	if (report.conflicts.length > 0) extras.push("conflicts:", ...report.conflicts.map((line) => `  ! ${line}`));
+	if (report.envIssues.length > 0) extras.push("env:", ...report.envIssues.map((line) => `  ! ${line}`));
+	if (report.budgetIssues.length > 0) extras.push("memory budget:", ...report.budgetIssues.map((line) => `  ! ${line}`));
+	if (report.paramIssues.length > 0) extras.push("parameters:", ...report.paramIssues.map((line) => `  ! ${line}`));
+	if (report.queryIssues.length > 0) extras.push("session search:", ...report.queryIssues.map((line) => `  ! ${line}`));
+	if (report.actions.length > 0) extras.push("next steps:", ...report.actions.map((line) => `  → ${line}`));
+	return renderDiagnosticsText(report.view).trimEnd() + "\n\n补充检查（view 未覆盖的运行时事实）：\n" + extras.join("\n");
 }
 /**
 * G6/W8 tail: render the frozen DiagnosticsView as the doctor's diagnostics section.
 *
 * This is the ONE text face of the 7+8 rows: the same object `--json` and the diagnostics route
-* answer, so the three readers cannot disagree. Legacy install/conflict sections keep their own
-* renderer until the remaining W8-S7 rewrite lands.
+* answer, so the three readers cannot disagree. The legacy report only adds the facts this frozen
+* view does not carry yet (see renderDoctorText).
 * @param view - the assembled view.
 * @returns the section text (ends with a newline).
 */
@@ -3174,7 +3187,7 @@ function apply(ctx, rawConfig = {}) {
 					...mounted
 				});
 				if (input === "doctor --json") return ok(JSON.stringify(report.view, null, 2));
-				return ok(renderDoctorText(report) + "\n" + renderDiagnosticsText(report.view));
+				return ok(renderDoctorText(report));
 			}
 			if (input === "" || input === "help") return ok(`自进化：记忆、技能、会话回顾、技能整理——状态、待批准写入与维护。\n${renderHelpText()}`);
 			return err(`/evolution ${input} 不是本构建能应答的子命令（已知子命令但缺参数也会落到这里）。\n${renderHint()}`);

package/lib/types/doctor.d.ts

@@ -159,13 +159,28 @@ export declare function diagnose(ctx: {
     home?: string;
     mounted?: MountedProfile;
 }): Promise<DoctorReport>;
+/**
+ * W8-S7: the doctor's text face, VIEW-FIRST.
+ *
+ * The frozen DiagnosticsView (7 facts + 8 runtime rows) is the single text source for everything
+ * it carries — install form, platform services, runs, channel/preset/host/pairing. The legacy
+ * report now contributes only the facts the view does not represent yet: cross-profile bundles,
+ * the family service inference, capability absence, the scoped-row probe, the legacy settings
+ * document, pending/executing audit records, preset freshness, and the conflict/env/budget/
+ * parameter/session-search findings + next steps. This is the W8-S7 rewrite branch: the text face
+ * has real consumers (the `/evolution doctor` command, README/INSTALL, doctor.spec), so the legacy
+ * rendering was rewritten instead of deleted — but every fact the view owns renders from the view
+ * exactly once (the legacy install-form line is gone).
+ * @param report - the assembled report (view + legacy findings).
+ * @returns the text face.
+ */
 export declare function renderDoctorText(report: DoctorReport): string;
 /**
  * G6/W8 tail: render the frozen DiagnosticsView as the doctor's diagnostics section.
  *
  * This is the ONE text face of the 7+8 rows: the same object `--json` and the diagnostics route
- * answer, so the three readers cannot disagree. Legacy install/conflict sections keep their own
- * renderer until the remaining W8-S7 rewrite lands.
+ * answer, so the three readers cannot disagree. The legacy report only adds the facts this frozen
+ * view does not carry yet (see renderDoctorText).
  * @param view - the assembled view.
  * @returns the section text (ends with a newline).
  */

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-commands",
   "description": "Human commands for the evolution family (/evolution pending|curator|maintain|doctor) (community build)",
-  "version": "0.27.0",
+  "version": "0.27.1",
   "publishConfig": {
     "access": "public"
   },
@@ -27,12 +27,12 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.5",
-    "@lmzhen/dsh-evolution-approval": "^0.27.0",
-    "@lmzhen/dsh-evolution-core": "^0.27.0",
-    "@lmzhen/dsh-evolution-maintenance": "^0.27.0"
+    "@lmzhen/dsh-evolution-approval": "^0.27.1",
+    "@lmzhen/dsh-evolution-core": "^0.27.1",
+    "@lmzhen/dsh-evolution-maintenance": "^0.27.1"
   },
   "optionalDependencies": {
-    "@lmzhen/dsh-evolution-agent-preset": "^0.27.0"
+    "@lmzhen/dsh-evolution-agent-preset": "^0.27.1"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.5",