@ran-sh/dsh-crew 2.0.4 → 2.0.5

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-crew",
-  "version": "2.0.4",
+  "version": "2.0.5",
   "description": "Dispatch subtasks to DeepSeek Harness (DSH) agents as native subagents with live progress",
   "author": {
     "name": "ZSeven-W"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ran-sh/dsh-crew",
-  "version": "2.0.4",
+  "version": "2.0.5",
   "type": "module",
   "main": "./src/hub/entry.mjs",
   "bin": {

package/src/install/install-legacy.mjs

@@ -600,7 +600,20 @@ export async function installClaudeCode({ home = homedir(), statusline = false,
   } catch (err) {
     actions.push(`cli: plugin install failed — run manually: claude plugin install ${PLUGIN_KEY} (${String(err?.message ?? err).slice(0, 120)})`);
   }
-  return { ok: true, actions };
+  // Report the state that resulted, not the step that was attempted. The CLI is
+  // best-effort — a machine without `claude` is a supported install, and its
+  // settings alone are correct — but a CLI that is present and slow, timed out,
+  // or failed leaves the snapshot exactly as stale as it was, and it is the
+  // snapshot that `installStatus` reads. Saying so here is what lets the caller
+  // stop printing a checkmark for a state it never verified.
+  if (claudeSnapshotReady(home, root)) return { ok: true, actions };
+  return {
+    ok: true,
+    degraded: true,
+    code: 'CLAUDE_PLUGIN_SNAPSHOT_STALE',
+    reason: `Claude Code has not loaded the plugin; run: claude plugin install ${PLUGIN_KEY}`,
+    actions,
+  };
 }
 
 export function installCodex({ home = homedir(), scope, root = ROOT, env = process.env } = {}) {

package/src/install/npx-lifecycle.mjs

@@ -2626,6 +2626,25 @@ function currentInstallationHealth({ home }) {
   return { installed: true, healthy: validated.ok && registered, validated, registered, pointer };
 }
 
+/**
+ * What the operator is told about the Claude Code integration.
+ *
+ * A missing `claude` CLI is a supported install, so `installClaudeCode` stays
+ * best-effort and keeps `ok: true`. A CLI that is present and fails or times out
+ * is a different thing: it leaves Claude Code without the plugin snapshot while
+ * the settings still name it, which is the state that later reads as "needs
+ * repair". This line used to be `${ok === false ? '✗' : '✓'}`, and since `ok` was
+ * never false, the failure branch could not fire and a broken integration was
+ * reported as a checkmark.
+ */
+export function claudeIntegrationLine(result) {
+  if (result?.ok === false) return '✗ Claude Code integration failed';
+  if (result?.degraded === true) {
+    return `- Claude Code integration registered, but not loaded: ${result.reason ?? 'plugin snapshot not refreshed'}`;
+  }
+  return '✓ Claude Code integration';
+}
+
 async function activateRelease({ home, releaseDir, manifest, log, installer, supervisorRoot = runningPackageRoot() }) {
   const registration = ensureCrewPluginRegistration({ home, root: releaseDir, name: manifest.name });
   if (!registration.ok) {
@@ -2668,10 +2687,10 @@ async function activateRelease({ home, releaseDir, manifest, log, installer, sup
 
   const claude = await installer.installClaudeCode({ home, root: integrationRoot });
   if (claude.ok === false) {
-    log(`✗ Claude Code integration failed`);
+    log(claudeIntegrationLine(claude));
     return false;
   }
-  log('✓ Claude Code integration');
+  log(claudeIntegrationLine(claude));
 
   // The official ~/.dsh/profiles/web tree is read-only: activation never
   // repairs or mutates the official bridge. The isolated 3210 Crew backend

package/src/runtime-identity.mjs

@@ -36,7 +36,7 @@ export {
 // included in the identity contract.
 const RUNTIME_ID = randomUUID();
 
-export const RUNTIME_VERSION = '2.0.4';
+export const RUNTIME_VERSION = '2.0.5';
 export const HUB_PROTOCOL_VERSION = 1;
 
 export const HUB_CAPABILITIES = Object.freeze([