@ran-sh/dsh-crew 1.10.9 → 2.0.0

package/.claude-plugin/plugin.json

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

package/src/hub/index.mjs

@@ -22,6 +22,7 @@ import { readHarnessModelCatalog } from '../model-catalog.mjs';
 import { appendDeliveryInstructions, prependJobIdentity, parseDeliveryReport, formatDeliveryMetadata } from '../delivery.mjs';
 import { captureWorkspaceBaseline, captureWorkspaceDiff, NOT_A_GIT_REPOSITORY } from '../workspace-audit.mjs';
 import { applyWorkspaceEvidence, buildOutcome, JOB_PHASES } from '../workflow.mjs';
+import { normalizeReviewVerdict } from '../workflow-runtime.mjs';
 import { boundedMachineCodeFromError } from '../structured-error-code.mjs';
 import { createCanonicalJobEvent, projectWorkflowView } from '../job-contracts.mjs';
 import { getHubRuntimeIdentity } from '../runtime-identity.mjs';
@@ -1096,12 +1097,12 @@ export class WorkerRegistry {  constructor(ctx) {
         job.delivery_missing = parsed.missing;
         job.delivery_metadata = formatDeliveryMetadata(parsed);
         if (job.role === 'reviewer') {
-          const verdictText = String(parsed.sections?.Verdict ?? '').trim().toLowerCase();
-          const verdict = /^(approve|approved|pass)\b/.test(verdictText)
-            ? 'approve'
-            : /request.*chang|chang.*request|reject|needs changes/i.test(verdictText)
-              ? 'request_changes'
-              : 'inconclusive';
+          // The rule for reading a verdict lives in one place. This block used to
+          // carry its own copy, and the two copies drifted: the shared one learned
+          // to see through `**Approved**`, this one did not, so the Hub recorded
+          // `inconclusive` for a reviewer whose report the workflow accepted —
+          // which also kept the readiness matrix from seeing a real review.
+          const verdict = normalizeReviewVerdict(parsed.sections?.Verdict ?? job.result ?? '');
           const lines = (value) => String(value ?? '').split(/\r?\n/).map((line) => line.trim()).filter(Boolean).slice(0, 40);
           job.review = {
             verdict, status: job.status, delivery_complete: parsed.complete === true,

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.9';
+export const RUNTIME_VERSION = '2.0.0';
 export const HUB_PROTOCOL_VERSION = 1;
 
 export const HUB_CAPABILITIES = Object.freeze([