@ran-sh/dsh-crew 0.5.2 → 0.5.3

package/src/server.mjs

@@ -26,8 +26,9 @@ import { buildReviewTask } from './information-flow.mjs';
 import { projectWorkflowView } from './job-contracts.mjs';
 import { loadRoleProfiles, resolveRoleProfile } from './role-profiles.mjs';
 import { loadWorkspaceContexts, resolveWorkspaceContext, buildWorkspaceTask, addContextReferences, isSafeBranchName } from './workspace-context.mjs';
-import { buildExtensionContract } from './extension-contract.mjs';
-import { assessWorkspaceReadiness } from './workspace-readiness.mjs';
+import { buildExtensionContract } from './extension-contract.mjs';
+import { assessWorkspaceReadiness } from './workspace-readiness.mjs';
+import { detectOrchestrator } from './orchestrator.mjs';
 
 const server = new McpServer({ name: 'dsh-crew', version: RUNTIME_VERSION });
 
@@ -44,10 +45,11 @@ const workspaceSchema = z.object({
   branch: z.string().refine(isSafeBranchName, 'invalid git branch name').optional(),
   worktree: z.enum(['auto', 'existing', 'none']).optional(),
 }).optional().describe('Per-job workspace overrides. auto isolates coding Workers; existing/none use the supplied workspace.');
-const constraintsSchema = z.object({
-  timeout_seconds: z.number().int().positive().max(7200).optional(),
-  allow_fallback: z.boolean().optional(),
-}).optional().describe('Per-job constraints override profile and session defaults.');
+const constraintsSchema = z.object({
+  timeout_seconds: z.number().int().positive().max(7200).optional(),
+  allow_fallback: z.boolean().optional(),
+  allow_no_changes: z.boolean().optional().describe('Explicitly allow a verified read-only or analysis job to succeed with zero workspace changes.'),
+}).optional().describe('Per-job constraints override profile and session defaults.');
 
 // Session-level configuration. This MCP server process lives exactly as long
 // as one Claude Code / Codex session, so plain memory IS session scope.
@@ -123,19 +125,7 @@ function text(obj) {
   return { content: [{ type: 'text', text: typeof payload === 'string' ? payload : JSON.stringify(payload, null, 2) }] };
 }
 
-function detectOrchestrator() {
-  if (process.env.CLAUDECODE || process.env.CLAUDE_CODE_ENTRYPOINT) return 'claude-code';
-  try {
-    const { execSync } = require('node:child_process');
-    const comm = execSync(`ps -o comm= -p ${process.ppid}`, { encoding: 'utf8' }).trim().toLowerCase();
-    if (comm.includes('claude')) return 'claude-code';
-    if (comm.includes('codex')) return 'codex';
-    return comm.split('/').pop() || 'unknown';
-  } catch { return 'unknown'; }
-}
-import { createRequire } from 'node:module';
-const require = createRequire(import.meta.url);
-const ORCHESTRATOR = detectOrchestrator();
+const ORCHESTRATOR = detectOrchestrator();
 
 function policyRejection(decision) {
   return text({
@@ -232,9 +222,10 @@ function prepareDispatch({ task, role, tier, legacy_tier, effort, cwd, timeout_s
       profile_id: resolvedProfile.profile_id,
       requested_isolation: requestedIsolation,
       workspace_branch: workspaceOverride?.branch ?? withRefs.context?.default_branch ?? null,
-      timeout_seconds: effectiveTimeout,
-      allow_fallback: constraints?.allow_fallback ?? profileValue.fallback,
-      routing: profileValue.routing,
+      timeout_seconds: effectiveTimeout,
+      allow_fallback: constraints?.allow_fallback ?? profileValue.fallback,
+      allow_no_changes: constraints?.allow_no_changes === true,
+      routing: profileValue.routing,
       review_strictness: profileValue.review_strictness,
       workspace_context: withRefs.context,
     },
@@ -333,10 +324,12 @@ async function buildConfigReport() {
   const hubCompatibility = await hubStatus({ force: true });
   let effectiveWorkerProvider = null;
   let effectiveWorkerSelection = { flash: null, pro: null };
-  let providerResolutionError;
-  let providerCatalogChecked = false;
-  let providerCatalogBody = null;
-  const workerProviderMode = globalConfig.worker_provider_mode ?? 'deepseek-official';
+  let providerResolutionError;
+  let providerCatalogChecked = false;
+  let providerCatalogBody = null;
+  let hubJobsChecked = false;
+  let hubJobsBody = null;
+  const workerProviderMode = globalConfig.worker_provider_mode ?? 'deepseek-official';
   if (workerProviderMode === 'deepseek-official') {
     effectiveWorkerSelection = {
       flash: { provider: 'deepseek-official', model: 'deepseek-v4-flash', source: 'legacy-strict' },
@@ -366,16 +359,27 @@ async function buildConfigReport() {
       providerCatalogChecked = true;
       providerResolutionError = err?.message ?? String(err);
     }
-  } else if (hubCompatibility.reachable) {
-    providerResolutionError = hubCompatibilityMessage(hubCompatibility);
-  }
-  effectiveWorkerProvider = effectiveWorkerSelection.flash?.provider ?? null;
-  const readinessMatrix = buildConfigReadinessMatrix({
+  } else if (hubCompatibility.reachable) {
+    providerResolutionError = hubCompatibilityMessage(hubCompatibility);
+  }
+  if (hubCompatibility.compatible) {
+    try {
+      hubJobsChecked = true;
+      const jobsRes = await fetch(`${globalConfig.hub_url}/_dsh/dsh-crew/jobs`, { signal: AbortSignal.timeout(800) });
+      hubJobsBody = await jobsRes.json();
+    } catch {
+      hubJobsChecked = true;
+    }
+  }
+  effectiveWorkerProvider = effectiveWorkerSelection.flash?.provider ?? null;
+  const readinessMatrix = buildConfigReadinessMatrix({
     hubCompatibility,
     workerProviderMode,
-    providerCatalogChecked,
-    providerCatalogBody,
-  });
+    providerCatalogChecked,
+    providerCatalogBody,
+    hubJobsChecked,
+    hubJobsBody,
+  });
   const roleProfiles = loadRoleProfiles();
   const workspaceReadiness = await assessWorkspaceReadiness({ cwd: process.cwd() });
   const extensionContract = buildExtensionContract({

package/src/workflow-runtime.mjs

@@ -15,7 +15,7 @@
 // appear here.
 
 import { resolveModelPolicy, shouldAutoReview, getRoleState } from './policy.mjs';
-import { buildOutcome, decideNextStep, JOB_PHASES, canTransition } from './workflow.mjs';
+import { applyWorkspaceEvidence, buildOutcome, decideNextStep, JOB_PHASES, canTransition } from './workflow.mjs';
 import { parseDeliveryReport } from './delivery.mjs';
 import { classifyFailure } from './failure-classification.mjs';
 import { createCanonicalJobEvent } from './job-contracts.mjs';
@@ -70,17 +70,6 @@ function mutationDetected(before, after) {
   return after.fingerprint != null && before.fingerprint !== after.fingerprint;
 }
 
-function deliveryClaimsChanges(outcome) {
-  return Array.isArray(outcome?.changes) && outcome.changes.length > 0;
-}
-
-function workspaceEvidenceOK(outcome, candidate) {
-  if (!candidate) return true;
-  const hasChanges = Array.isArray(candidate.changed_files) && candidate.changed_files.length > 0;
-  if (outcome?.execution_status !== 'completed') return true;
-  return !deliveryClaimsChanges(outcome) || hasChanges;
-}
-
 function sumUsage(attempts) {
   const tokens = { input: 0, output: 0, reasoning: 0 };
   for (const a of attempts) {
@@ -181,9 +170,10 @@ export function createWorkflowRuntime(adapters, {
       requested_isolation: spec.requested_isolation ?? null,
       workspace_branch: spec.workspace_branch ?? null,
       timeout_seconds: spec.timeout_seconds ?? null,
-      profile_id: spec.profile_id ?? null,
-      allow_fallback: spec.allow_fallback !== false,
-      routing: spec.routing ?? 'auto',
+      profile_id: spec.profile_id ?? null,
+      allow_fallback: spec.allow_fallback !== false,
+      allow_no_changes: spec.allow_no_changes === true,
+      routing: spec.routing ?? 'auto',
       review_strictness: spec.review_strictness ?? null,
       workspace_context: spec.workspace_context ? { ...spec.workspace_context } : null,
       effort: spec.effort ?? 'max',
@@ -385,14 +375,17 @@ export function createWorkflowRuntime(adapters, {
           failJob(job, Object.assign(new Error(ar.error ?? 'infrastructure failure'), { code: WORKFLOW_ERROR_CODES.ATTEMPT_INFRA_FAILURE }));
           return;
         }
-        const outcome = ar.outcome ?? buildOutcome({
+        let outcome = ar.outcome ?? buildOutcome({
           result: ar.result ?? '',
           stopReason: ar.stopReason,
           executionStatus: ar.status === 'done' ? 'completed' : 'failed',
         });
         transition(job, JOB_PHASES.VERIFYING, `attempt ${attempt} complete`);
 
-        // Capture the latest candidate after every attempt. Capture failure is
+        let workspaceEvidenceAvailable = false;
+        let workspaceHasChanges = false;
+
+        // Capture the latest candidate after every attempt. Capture failure is
         // not allowed to delete the only recoverable state: retain the worktree
         // and preserve the worker business result, while surfacing the warning.
         if (alloc?.ok && alloc.isolation === 'worktree') {
@@ -402,17 +395,23 @@ export function createWorkflowRuntime(adapters, {
             job.retain_workspace = true;
             job.candidate = null;
             job.events.push({ at: clock(), phase: job.phase, type: 'candidate/failed', attempt, message: 'candidate capture failed; worktree retained for recovery' });
-          } else {
-            job.candidate = candidate;
-            attemptView.candidate_fingerprint = candidate.fingerprint ?? null;
-            outcome.workspace_evidence_ok = workspaceEvidenceOK(outcome, candidate);
-            if (candidate.complete === false || candidate.replayable === false) {
+          } else {
+            job.candidate = candidate;
+            attemptView.candidate_fingerprint = candidate.fingerprint ?? null;
+            workspaceEvidenceAvailable = true;
+            workspaceHasChanges = Array.isArray(candidate.changed_files) && candidate.changed_files.length > 0;
+            if (candidate.complete === false || candidate.replayable === false) {
               job.retain_workspace = true;
               job.events.push({ at: clock(), phase: job.phase, type: 'candidate/incomplete', attempt, message: 'candidate is not fully replayable; worktree retained' });
             }
-          }
-        }
-        job.outcome = outcome;
+          }
+        }
+        outcome = applyWorkspaceEvidence(outcome, {
+          evidenceAvailable: workspaceEvidenceAvailable,
+          hasChanges: workspaceHasChanges,
+          allowNoChanges: job.allow_no_changes,
+        });
+        job.outcome = outcome;
 
         const reviewRequested = !isReviewJob && shouldAutoReview(config);
         const reviewerAuto = getRoleState(config, 'reviewer') !== 'disabled';
@@ -580,8 +579,9 @@ export function createWorkflowRuntime(adapters, {
       current_model: job.attempts[job.attempts.length - 1]?.model ?? null,
       model_class_hint: job.model_class_hint,
       source: job.source,
-      profile_id: job.profile_id,
-      routing: job.routing,
+      profile_id: job.profile_id,
+      allow_no_changes: job.allow_no_changes,
+      routing: job.routing,
       workspace_context: job.workspace_context ? { ...job.workspace_context } : null,
       workspace_branch: job.workspace_branch,
       timeout_seconds: job.timeout_seconds,

package/src/workflow.mjs

@@ -58,10 +58,68 @@ export function canTransition(from, to) {
   return allowed.includes(to);
 }
 
-function splitSection(value) {
-  if (typeof value !== 'string' || value.trim() === '') return [];
-  return value.split(/\r?\n/).map((l) => l.trim()).filter(Boolean);
-}
+function splitSection(value) {
+  if (typeof value !== 'string' || value.trim() === '') return [];
+  return value.split(/\r?\n/).map((l) => l.trim()).filter(Boolean);
+}
+
+const NO_CHANGE_SENTINELS = new Set([
+  'no files changed',
+  'no file changed',
+  'no changes',
+  '无文件变更',
+  '没有文件变更',
+  '未更改任何文件',
+  '无变更',
+]);
+
+function deliveryClaimsChanges(outcome) {
+  return Array.isArray(outcome?.changes) && outcome.changes.length > 0;
+}
+
+export function applyWorkspaceEvidence(outcome, {
+  evidenceAvailable = false,
+  hasChanges = false,
+  allowNoChanges = false,
+  requireNoChangeAuthorization = true,
+} = {}) {
+  const next = { ...outcome };
+  const claimsChanges = deliveryClaimsChanges(next);
+  if (evidenceAvailable && next.execution_status === 'completed') {
+    next.workspace_evidence_ok = claimsChanges === hasChanges;
+  }
+  const tests = Array.isArray(next.tests) ? next.tests : [];
+  const verifiedNoChange = requireNoChangeAuthorization === true
+    && evidenceAvailable === true
+    && allowNoChanges === true
+    && hasChanges === false
+    && claimsChanges === false
+    && next.execution_status === 'completed'
+    && next.workspace_evidence_ok === true
+    && next.delivery?.complete === true
+    && tests.some((test) => test.status === 'PASS')
+    && !tests.some((test) => test.status === 'FAIL');
+  if (verifiedNoChange) {
+    next.task_status = 'success';
+    next.no_change_verified = true;
+  } else if (requireNoChangeAuthorization === true && claimsChanges === false && next.task_status === 'success') {
+    next.task_status = 'partial';
+    delete next.no_change_verified;
+  }
+  return next;
+}
+
+function parseChanges(section) {
+  return splitSection(section).filter((line) => {
+    const normalized = line
+      .replace(/^(?:[-*+]\s+)+/, '')
+      .replace(/[`"'“”‘’]/g, '')
+      .replace(/[.!。！]+$/g, '')
+      .trim()
+      .toLowerCase();
+    return !NO_CHANGE_SENTINELS.has(normalized);
+  });
+}
 
 function parseTests(section) {
   return splitSection(section).map((line) => {
@@ -106,7 +164,7 @@ export function buildOutcome({ result = '', deliveryMeta, executionStatus, stopR
     }),
     confidence: null,
     needs_escalation: false,
-    changes: splitSection(parsed.sections.Diff),
+    changes: parseChanges(parsed.sections.Diff),
     tests,
     tests_status: testsStatus ?? null,
     risks: splitSection(parsed.sections.Risks),

package/windows/start-dsh-crew.cmd

@@ -2,54 +2,56 @@
 setlocal EnableExtensions
 title DSH Crew Launcher
 
-set "CREW_HOME=%USERPROFILE%\.config\dsh-crew\harness"
-set "OFFICIAL_HOME=%USERPROFILE%\.dsh"
-set "DSH_CLI=%CREW_HOME%\runtime\node_modules\.bin\dsh.cmd"
-set "CREW_URL=http://127.0.0.1:3210"
-set "UI_URL=http://127.0.0.1:3080"
+set "LAUNCH_REQUEST=%*"
+set "LAUNCH_MODE=open"
+set "LAUNCH_DIR=%~dp0"
+
+if "%~1"=="" goto :run
+if /i "%~1"=="--background" (
+  set "LAUNCH_MODE=background"
+  shift
+  goto :validate
+)
+if /i "%~1"=="--open" (
+  set "LAUNCH_MODE=open"
+  shift
+  goto :validate
+)
+if /i "%~1"=="--watch" (
+  set "LAUNCH_MODE=watch"
+  shift
+  goto :validate
+)
+if /i "%~1"=="--help" goto :help
+goto :invalid_argument
+
+:validate
+if not "%~1"=="" goto :invalid_argument
+
+:run
+set "LAUNCH_HELPER=%LAUNCH_DIR%start-dsh-crew.ps1"
 set "LAUNCH_LOG=%TEMP%\dsh-crew-launcher.log"
-
-if not exist "%DSH_CLI%" goto :not_installed
-if not exist "%CREW_HOME%\profiles\dsh-crew\package.json" goto :not_installed
-if not exist "%OFFICIAL_HOME%\profiles\web\package.json" goto :official_missing
-
-call :ensure_service 3210 dsh-crew "%CREW_HOME%" "%CREW_URL%"
-if errorlevel 1 goto :failed
-call :ensure_service 3080 web "%OFFICIAL_HOME%" "%UI_URL%"
-if errorlevel 1 goto :failed
+if not exist "%LAUNCH_HELPER%" (
+  >>"%LAUNCH_LOG%" echo [%date% %time%] ERROR Managed launcher helper is missing: %LAUNCH_HELPER%
+  echo ERROR: DSH Crew launcher helper is missing.
+  echo Repair it with: dsh-crew update
+  if /i "%LAUNCH_MODE%"=="open" pause
+  exit /b 1
+)
+
+powershell.exe -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Bypass -File "%LAUNCH_HELPER%" -Mode "%LAUNCH_MODE%"
+set "LAUNCH_EXIT=%ERRORLEVEL%"
+if not "%LAUNCH_EXIT%"=="0" if /i "%LAUNCH_MODE%"=="open" pause
+exit /b %LAUNCH_EXIT%
+
+:invalid_argument
+echo ERROR: Unsupported launcher arguments: %LAUNCH_REQUEST%
+echo Use --open, --background, or --watch.
+exit /b 64
+
+:help
+echo Usage: %~nx0 [--open ^| --background ^| --watch]
+echo   --open        Start both services and open http://127.0.0.1:3080/.
+echo   --background  Start both services silently without opening a browser.
+echo   --watch       Keep both services healthy and restart them after an exit.
 exit /b 0
-
-:ensure_service
-set "LAUNCH_PORT=%~1"
-set "LAUNCH_PROFILE=%~2"
-set "LAUNCH_HOME=%~3"
-set "LAUNCH_URL=%~4"
-
-call :health_check "%LAUNCH_URL%"
-if not errorlevel 1 exit /b 0
-
-powershell.exe -NoLogo -NoProfile -NonInteractive -Command "$listener=Get-NetTCPConnection -State Listen -LocalPort $env:LAUNCH_PORT -ErrorAction SilentlyContinue; if ($listener) { exit 0 } else { exit 1 }" >nul 2>&1
-if not errorlevel 1 exit /b 1
-
-powershell.exe -NoLogo -NoProfile -NonInteractive -WindowStyle Hidden -Command "try { $env:DSH_HOME=$env:LAUNCH_HOME; Start-Process -FilePath $env:DSH_CLI -ArgumentList @('--profile',$env:LAUNCH_PROFILE,'--host','127.0.0.1','--port',$env:LAUNCH_PORT,'--no-open') -WindowStyle Hidden -ErrorAction Stop } catch { ('['+(Get-Date -Format s)+'] Failed to start '+$env:LAUNCH_PROFILE+': '+$_.Exception.Message) | Add-Content -LiteralPath $env:LAUNCH_LOG; exit 1 }" >nul 2>&1
-if errorlevel 1 exit /b 1
-
-powershell.exe -NoLogo -NoProfile -NonInteractive -Command "$lastError=$null; $deadline=(Get-Date).AddSeconds(90); do { try { $r=Invoke-RestMethod -Uri ($env:LAUNCH_URL+'/_dsh/dsh-crew/extension') -TimeoutSec 2; if ($r.ok -eq $true -and $r.extension.runtime.runtime_version) { exit 0 } } catch { $lastError=$_.Exception.Message }; Start-Sleep -Milliseconds 500 } while ((Get-Date) -lt $deadline); if (-not $lastError) { $lastError='No healthy response before the startup deadline.' }; ('['+(Get-Date -Format s)+'] '+$env:LAUNCH_PROFILE+' health check failed: '+$lastError) | Add-Content -LiteralPath $env:LAUNCH_LOG; exit 1" >nul 2>&1
-exit /b %ERRORLEVEL%
-
-:health_check
-set "HEALTH_URL=%~1"
-powershell.exe -NoLogo -NoProfile -NonInteractive -Command "try { $r=Invoke-RestMethod -Uri ($env:HEALTH_URL+'/_dsh/dsh-crew/extension') -TimeoutSec 2; if ($r.ok -eq $true -and $r.extension.runtime.runtime_version) { exit 0 } } catch {}; exit 1" >nul 2>&1
-exit /b %ERRORLEVEL%
-
-:not_installed
-echo [%date% %time%] DSH Crew is not installed completely.>>"%LAUNCH_LOG%"
-exit /b 1
-
-:official_missing
-echo [%date% %time%] Official DeepSeek Harness web profile was not found.>>"%LAUNCH_LOG%"
-exit /b 1
-
-:failed
-echo [%date% %time%] DSH Crew startup failed.>>"%LAUNCH_LOG%"
-exit /b 1