@ran-sh/dsh-crew 1.10.7 → 1.10.9

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-crew",
-  "version": "1.10.7",
+  "version": "1.10.9",
   "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": "1.10.7",
+  "version": "1.10.9",
   "type": "module",
   "main": "./src/hub/entry.mjs",
   "bin": {

package/src/install/official-frontend-assets.mjs

@@ -1,8 +1,9 @@
 import { createHash, randomUUID } from 'node:crypto';
-import { existsSync, lstatSync, mkdirSync, mkdtempSync, readFileSync, renameSync, rmSync, writeFileSync } from 'node:fs';
+import { existsSync, lstatSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
 import { dirname, join } from 'node:path';
 import { homedir } from 'node:os';
 import { pathToFileURL } from 'node:url';
+import { renameTree } from './tree-move.mjs';
 
 function plan(home, root) {
   const bridge = join(root, 'official-web-bridge');
@@ -61,13 +62,17 @@ export function installOfficialFrontendAssets({ home = homedir(), root } = {}) {
         mkdirSync(dirname(file), { recursive: true });
         writeFileSync(file, bytes);
       }
-      try { renameSync(stage, p.snapshotRoot); stage = null; }
+      // Windows can refuse this rename for a moment when something else holds a
+      // handle under the destination, and the refusal is not about the rename:
+      // the same call succeeds shortly after. Producing the snapshot first in a
+      // concurrent installer is the other outcome this tolerates.
+      try { renameTree(stage, p.snapshotRoot); stage = null; }
       catch (error) { if (!matches(p)) throw error; }
     }
     if (!matches(p)) throw new Error('frontend snapshot conflicts with its revision');
     temp = `${p.overlayFile}.${randomUUID()}.tmp`;
     writeFileSync(temp, p.overlay, { flag: 'wx', mode: 0o600 });
-    renameSync(temp, p.overlayFile); temp = null;
+    renameTree(temp, p.overlayFile); temp = null;
     return { ok: true, changed: true, revision: p.revision, snapshotRoot: p.snapshotRoot, overlayFile: p.overlayFile };
   } catch (error) {
     return { ok: false, code: 'OFFICIAL_FRONTEND_ASSETS_FAILED', error: error.message };

package/src/mcp-runtime.mjs

@@ -222,6 +222,15 @@ export function buildMcpWorkflowRuntime(deps) {
         source,
         preset,
         delivery,
+        // Isolation belongs to this client: it already allocated the workspace
+        // and `spec.cwd` is the result. Saying so keeps the hub executing in that
+        // cwd instead of resolving an isolation of its own and allocating a
+        // second workspace — which put the work somewhere the client was not
+        // looking and captured no candidate for.
+        requested_isolation: 'shared',
+        // The same authorization the client was given, so the hub judges the
+        // workspace evidence against the same task contract.
+        allow_no_changes: spec.allow_no_changes === true,
       });
       spec.onAttemptStarted?.(spawned.id);
 

package/src/runtime-identity.mjs

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

package/src/workflow-runtime.mjs

@@ -38,8 +38,23 @@ function lines(value) {
   return value.split(/\r?\n/).map((l) => l.trim()).filter(Boolean);
 }
 
+// A reviewer writes its verdict as one line, and it decorates that line the way
+// prose is decorated: bolded, bulleted, quoted, or introduced by a label. The
+// contract asks for "approved / needs changes / rejected", and every one of
+// those decorations used to fall through to `inconclusive` — which blocks
+// acceptance, so turning on the automatic reviewer could reject correct work
+// because the reviewer wrote **Approved** instead of Approved.
+function stripVerdictDecoration(value) {
+  return String(value ?? '')
+    .replace(/^[\s>*_`#-]+/, '')
+    .replace(/^\**\s*(?:final\s+)?verdict\s*[:\-—]\s*/i, '')
+    .replace(/^[\s>*_`#-]+/, '')
+    .replace(/[\s*_`]+$/, '')
+    .trim();
+}
+
 export function normalizeReviewVerdict(value) {
-  const s = String(value ?? '').trim().toLowerCase();
+  const s = stripVerdictDecoration(value).toLowerCase();
   if (/^(approve|approved|pass)\b/.test(s) || s === 'pass' || s === 'approve') return 'approve';
   if (/request.*chang|chang.*request|reject|needs changes/i.test(s)) return 'request_changes';
   return 'inconclusive';
@@ -382,6 +397,10 @@ export function createWorkflowRuntime(adapters, {
           cwd: job.execution_cwd,
           effort: job.effort,
           timeout_seconds: job.timeout_seconds,
+          // Passed through so the executor judges the workspace evidence against
+          // the same task contract this workflow was given, instead of against a
+          // default that would refuse an authorization the caller granted.
+          allow_no_changes: job.allow_no_changes === true,
           policy,
           source: job.source,
           model_class_hint: job.model_class_hint,

package/src/workflow.mjs

@@ -81,12 +81,18 @@ const NO_CHANGE_ASSERTION_RE = new RegExp(
 function isNoChangeAssertion(line) {
   const normalized = line
     .replace(/^(?:[-*+]\s+)+/, '')
-    .replace(/[`"'“”‘’]/g, '')
+    .replace(/[`"'“”‘’*_]/g, '')
     .replace(/[.!。！]+$/, '')
     .trim()
     .toLowerCase();
   if (normalized === '') return false;
-  return NO_CHANGE_ASSERTION_RE.test(normalized);
+  // The assertion is usually a whole line, but a report may also end with it —
+  // "**Final state: no files changed.**" is the same declaration stated as a
+  // summary. Testing each sentence and clause is what finds it without matching
+  // a phrase that merely appears inside an unrelated one.
+  return normalized
+    .split(/\s*(?:[.:;,—–()\[\]]|->)\s*/)
+    .some((fragment) => fragment !== '' && NO_CHANGE_ASSERTION_RE.test(fragment.trim()));
 }
 
 // Whether the report claims the workspace holds changes. A report that asserts