@sebastianandreasson/pi-autonomous-agents 0.5.1 → 0.6.0

package/README.md

@@ -29,7 +29,7 @@ The package ships a top-level [SETUP.md](./SETUP.md) specifically for that workf
 ## What This Package Owns
 
 - unattended loop orchestration
-- PI adapter integration
+- PI Node SDK integration
 - config loading
 - prompt assembly
 - verification/tester/visual-review handoff
@@ -60,6 +60,10 @@ pi/
 
 Typical scripts:
 
+- `pi:once` / `pi:run` use default `sdk` transport
+- `pi:mock` skips real agent execution
+
+
 ```json
 {
   "scripts": {
@@ -67,7 +71,8 @@ Typical scripts:
     "pi:once": "PI_CONFIG_FILE=pi.config.json pi-harness once",
     "pi:run": "PI_CONFIG_FILE=pi.config.json pi-harness run",
     "pi:report": "PI_CONFIG_FILE=pi.config.json pi-harness report",
-    "pi:visual:once": "PI_CONFIG_FILE=pi.config.json pi-harness visual-once"
+    "pi:visual:once": "PI_CONFIG_FILE=pi.config.json pi-harness visual-once",
+    "pi:visualize": "PI_CONFIG_FILE=pi.config.json pi-harness visualize"
   }
 }
 ```
@@ -82,7 +87,7 @@ pi-harness run
 pi-harness report
 pi-harness clear-history
 pi-harness visual-once
-pi-harness adapter
+pi-harness visualize
 pi-harness visual-review-worker
 ```
 
@@ -115,6 +120,7 @@ The package supports:
 - one default visual-review model via `visualReviewModel`
 - optional per-role overrides via `roleModels`
 - per-model endpoint config in `models`
+- default transport via `transport` (`sdk` or `mock`)
 
 Typical pattern:
 
@@ -173,8 +179,7 @@ Common fields in `pi.config.json`:
 - `taskFile`
 - `developerInstructionsFile`
 - `testerInstructionsFile`
-- `transport`
-- `adapterCommand`
+- `transport` (`sdk` or `mock`)
 - `piModel`
 - `models`
 - `roleModels`
@@ -192,7 +197,7 @@ Common fields in `pi.config.json`:
 
 Key defaults:
 
-- `transport`: `adapter`
+- `transport`: `sdk`
 - `commitMode`: `agent`
 - `promptMode`: `compact`
 - `piTools`: `read,edit,write,find,ls,bash`
@@ -209,7 +214,7 @@ The package is optimized for local models by default:
 - changed-file lists and feedback excerpts are capped
 - prompts prefer `read` for source inspection
 - shell is intended for `git`, tests, and narrow diagnostics
-- the adapter warns on obvious oversized shell-based file reads
+- SDK transport carries forward oversized shell-read warnings and loop/timeout guards
 - the supervisor emits large-file/spec warnings when touched files are getting risky
 
 This is deliberate. Large monolith files, huge e2e specs, and broad TODO items are one of the main causes of local-model drift and retry loops.
@@ -232,6 +237,7 @@ Recent versions of the package isolate each run more aggressively:
 - in-progress iteration state persisted before agent work starts
 - stale run locks recovered when the owning PID is gone
 - timeout cleanup kills the full spawned process group, not only the direct child
+- parent-death watchers shut down orphaned supervisor layers instead of letting them continue under `PPID 1`
 
 That is meant to prevent orphaned timed-out agents or concurrent supervisors from corrupting shared state.
 
@@ -258,6 +264,15 @@ Useful files during a run:
 
 `pi-harness report` summarizes recent telemetry and surfaces things like terminal reasons and large-file warnings.
 
+`pi-harness visualize` starts lightweight local web UI for orchestration flow. By default it listens on `127.0.0.1:4317`. Override with `PI_VISUALIZER_HOST` and `PI_VISUALIZER_PORT`.
+
+Visualizer now includes:
+- run history selector from `.pi-runtime/runs/`
+- orchestration flow strip
+- per-iteration stage graph with retries/rechecks
+- clickable graph nodes and timeline rows that show full event JSON
+- historical run summaries and per-run last output snapshots
+
 ## Visual Review Contract
 
 Visual review is optional and generic. The harness does not know how to navigate your app.
@@ -281,7 +296,7 @@ That clears configured harness runtime/history artifacts and verifies they are g
 - [SETUP.md](./SETUP.md)
   Agent-facing setup instructions for consuming repos.
 - [docs/PI_SUPERVISOR.md](./docs/PI_SUPERVISOR.md)
-  More detailed flow, adapter, and runtime documentation.
+  More detailed flow, transport, telemetry, and runtime documentation.
 - [templates/PROJECT_SETUP.md](./templates/PROJECT_SETUP.md)
   Minimal consuming-repo layout summary.
 

package/SETUP.md

@@ -82,6 +82,9 @@ Minimal example:
 
 5. Add package scripts.
 
+- `pi:once` and `pi:run` should use default `sdk` transport unless the repo has a very specific reason not to.
+- `pi:mock` is for setup validation when real PI execution is not ready yet.
+
 Add these scripts to the consuming repo `package.json`, adapting only if necessary:
 
 ```json
@@ -174,6 +177,8 @@ If the repo is not ready for a real run yet, at minimum run:
 PI_CONFIG_FILE=pi.config.json PI_TRANSPORT=mock PI_TEST_CMD= pi-harness once
 ```
 
+Default transport is `sdk`. Only set `PI_TRANSPORT` when you explicitly want `mock`.
+
 If setup validation fails, fix the config rather than leaving a half-configured repo.
 
 The harness should fail fast if:

package/docs/PI_SUPERVISOR.md

@@ -28,13 +28,15 @@ Main package files:
 
 - `src/pi-supervisor.mjs`: controller
 - `src/pi-client.mjs`: transport layer
-- `src/pi-rpc-adapter.mjs`: built-in adapter from supervisor JSON to `pi --mode rpc`
+- `src/pi-sdk-turn.mjs`: in-process PI SDK turn runner for `transport: "sdk"`
 - `src/pi-config.mjs`: config loader
-- `src/pi-repo.mjs`: repo helpers, verification runner, and optional legacy git finalize step
+- `src/pi-repo.mjs`: repo helpers, verification runner, and git finalize helpers
 - `src/pi-telemetry.mjs`: telemetry writer/reader
 - `src/pi-prompts.mjs`: default prompt builders
 - `src/pi-visual-review.mjs`: multimodal visual-review worker
 - `src/pi-visual-once.mjs`: one-shot manual visual review runner
+- `src/pi-visualizer.mjs`: local web UI for orchestration flow and active stage
+- `src/pi-visualizer-shared.mjs`: flow-state helpers for visualizer
 - `src/pi-report.mjs`: telemetry summary report
 - `templates/DEVELOPER.md`: default developer-role instructions template
 - `templates/TESTER.md`: default tester-role instructions template
@@ -46,10 +48,15 @@ pi-harness once
 pi-harness run
 pi-harness report
 pi-harness visual-once
+pi-harness visualize
 ```
 
 The package reads `PI_CONFIG_FILE` if provided. Otherwise it falls back to the bundled generic `pi.config.json`.
 
+`pi-harness visualize` serves polling web UI over local HTTP. Defaults: `PI_VISUALIZER_HOST=127.0.0.1`, `PI_VISUALIZER_PORT=4317`.
+
+It reads active-run lock, per-run state, per-run iteration summary, per-run last output snapshot, and telemetry to show current stage plus historical runs.
+
 ## Config Contract
 
 Projects typically provide their own `pi.config.json` with fields such as:
@@ -98,67 +105,9 @@ For unattended inner-loop work, `testCommand` should be a bounded smoke gate rat
 The supervisor supports:
 
 - `PI_TRANSPORT=mock`
-- `PI_TRANSPORT=adapter`
-
-The built-in adapter command is typically:
-
-```bash
-pi-harness adapter
-```
-
-When using `adapter`, set `PI_ADAPTER_COMMAND` to a command that:
-
-1. Reads one JSON request from `stdin`
-2. Talks to PI RPC or your own PI wrapper
-3. Writes one JSON response to `stdout`
-4. Exits with code `0` on success
-
-Request shape:
-
-```json
-{
-  "sessionId": "existing-or-empty",
-  "sessionFile": "/absolute/path/to/session.jsonl",
-  "prompt": "controller prompt",
-  "cwd": "/absolute/repo/path",
-  "taskFile": "/absolute/repo/path/TODOS.md",
-  "instructionsFile": "/absolute/repo/path/pi/DEVELOPER.md",
-  "runtimeDir": "/absolute/repo/path/.pi-runtime",
-  "piCli": "pi",
-  "model": "local/model-name",
-  "tools": "read,edit,write,find,ls,bash",
-  "thinking": "",
-  "noExtensions": false,
-  "noSkills": false,
-  "noPromptTemplates": false,
-  "noThemes": true,
-  "metadata": {
-    "iteration": 1,
-    "retryCount": 0,
-    "reason": "main_workflow"
-  }
-}
-```
-
-Response shape:
-
-```json
-{
-  "sessionId": "stable-session-id",
-  "sessionFile": "/absolute/path/to/session.jsonl",
-  "status": "success",
-  "output": "agent output text",
-  "notes": "short controller note"
-}
-```
-
-Allowed response `status` values:
+- `PI_TRANSPORT=sdk` (default)
 
-- `success`
-- `stalled`
-- `timed_out`
-- `failed`
-- `canceled`
+`sdk` runs PI in-process via Node SDK. `mock` preserves harness flow but skips real agent work.
 
 ## Git Finalization
 
@@ -215,15 +164,16 @@ The capture command must write a JSON manifest at `PI_VISUAL_MANIFEST_FILE` with
 
 ## Loop Mitigation
 
-The built-in adapter mitigates obvious local loops by watching PI RPC tool events:
+SDK transport mitigates obvious local loops by watching agent and tool events:
 
 - repeated identical tool calls are aborted
 - repeated same-path churn is aborted
 - a soft `continue` can be sent after inactivity
 - a separate tool-aware watchdog can tolerate long-running `bash` or browser work without treating the turn as dead
 - a hard no-event timeout aborts a wedged turn instead of hanging indefinitely
+- parent-loss shutdown tears down owned supervisor child work instead of allowing orphaned background runs
 
-Important: terminal streaming does not reset the heartbeat by itself. The watchdog keys off PI RPC events and active tool state, not raw shell output.
+Important: terminal streaming does not reset the heartbeat by itself. The watchdog keys off SDK agent events and active tool state, not raw shell output.
 
 ## Telemetry
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@sebastianandreasson/pi-autonomous-agents",
   "private": false,
-  "version": "0.5.1",
+  "version": "0.6.0",
   "type": "module",
   "description": "Portable unattended PI harness for developer/tester/visual-review loops.",
   "license": "MIT",
@@ -15,9 +15,12 @@
   "bin": {
     "pi-harness": "./src/cli.mjs"
   },
+  "dependencies": {
+    "@mariozechner/pi-coding-agent": "^0.66.1"
+  },
   "scripts": {
-    "check": "node --check src/cli.mjs && node --check src/pi-clear-history.mjs && node --check src/pi-client.mjs && node --check src/pi-config.mjs && node --check src/pi-flow.mjs && node --check src/pi-heartbeat.mjs && node --check src/pi-history.mjs && node --check src/pi-preflight.mjs && node --check src/pi-prompts.mjs && node --check src/pi-repo.mjs && node --check src/pi-report.mjs && node --check src/pi-rpc-adapter.mjs && node --check src/pi-supervisor.mjs && node --check src/pi-telemetry.mjs && node --check src/pi-visual-once.mjs && node --check src/pi-visual-review.mjs && node --check src/index.mjs && node --check test/pi-heartbeat.test.mjs && node --check test/pi-role-models.test.mjs && node --check test/pi-flow.test.mjs && node --check test/pi-history.test.mjs && node --check test/pi-prompts.test.mjs && node --check test/pi-preflight.test.mjs && node --check test/pi-repo.test.mjs && node --check test/pi-telemetry.test.mjs",
-    "test": "node --test test/pi-heartbeat.test.mjs test/pi-role-models.test.mjs test/pi-flow.test.mjs test/pi-history.test.mjs test/pi-prompts.test.mjs test/pi-preflight.test.mjs test/pi-repo.test.mjs test/pi-telemetry.test.mjs"
+    "check": "node --check src/cli.mjs && node --check src/pi-clear-history.mjs && node --check src/pi-client.mjs && node --check src/pi-config.mjs && node --check src/pi-flow.mjs && node --check src/pi-heartbeat.mjs && node --check src/pi-history.mjs && node --check src/pi-preflight.mjs && node --check src/pi-prompts.mjs && node --check src/pi-repo.mjs && node --check src/pi-report.mjs && node --check src/pi-sdk-turn.mjs && node --check src/pi-supervisor.mjs && node --check src/pi-telemetry.mjs && node --check src/pi-visual-once.mjs && node --check src/pi-visual-review.mjs && node --check src/pi-visualizer.mjs && node --check src/pi-visualizer-shared.mjs && node --check src/index.mjs && node --check test/pi-heartbeat.test.mjs && node --check test/pi-lifecycle.test.mjs && node --check test/pi-role-models.test.mjs && node --check test/pi-flow.test.mjs && node --check test/pi-history.test.mjs && node --check test/pi-prompts.test.mjs && node --check test/pi-preflight.test.mjs && node --check test/pi-repo.test.mjs && node --check test/pi-sdk-supervisor.test.mjs && node --check test/pi-sdk-turn.test.mjs && node --check test/pi-telemetry.test.mjs && node --check test/pi-visualizer-shared.test.mjs && node --check test/fixtures/fake-pi.mjs && node --check test/fixtures/fake-pi-sdk.mjs",
+    "test": "node --test test/pi-heartbeat.test.mjs test/pi-lifecycle.test.mjs test/pi-role-models.test.mjs test/pi-flow.test.mjs test/pi-history.test.mjs test/pi-prompts.test.mjs test/pi-preflight.test.mjs test/pi-repo.test.mjs test/pi-sdk-supervisor.test.mjs test/pi-sdk-turn.test.mjs test/pi-telemetry.test.mjs test/pi-visualizer-shared.test.mjs"
   },
   "files": [
     "src",

package/pi.config.json

@@ -1,6 +1,5 @@
 {
-  "transport": "adapter",
-  "adapterCommand": "pi-harness adapter",
+  "transport": "sdk",
   "instructionsFile": "",
   "taskFile": "TODOS.md",
   "commitMode": "agent",

package/src/cli.mjs

@@ -4,6 +4,11 @@ import path from 'node:path'
 import { spawn } from 'node:child_process'
 import process from 'node:process'
 import { fileURLToPath } from 'node:url'
+import {
+  registerOwnedChildProcess,
+  signalChildProcess,
+  watchParentProcess,
+} from './pi-repo.mjs'
 
 const scriptDir = path.dirname(fileURLToPath(import.meta.url))
 
@@ -13,7 +18,7 @@ const COMMANDS = new Map([
   ['report', 'pi-report.mjs'],
   ['clear-history', 'pi-clear-history.mjs'],
   ['visual-once', 'pi-visual-once.mjs'],
-  ['adapter', 'pi-rpc-adapter.mjs'],
+  ['visualize', 'pi-visualizer.mjs'],
   ['visual-review-worker', 'pi-visual-review.mjs'],
 ])
 
@@ -36,10 +41,53 @@ function main() {
     env: process.env,
     stdio: 'inherit',
   })
+  registerOwnedChildProcess(child)
+
+  let shuttingDown = false
+  let forceKillTimer = null
+  const stopWatchingParent = watchParentProcess(() => {
+    shutdown({
+      signal: 'SIGTERM',
+      exitCode: 1,
+    })
+  })
+
+  function shutdown({
+    signal,
+    exitCode,
+  }) {
+    if (shuttingDown) {
+      return
+    }
+
+    shuttingDown = true
+    stopWatchingParent()
+    signalChildProcess(child.pid, signal)
+    forceKillTimer = setTimeout(() => {
+      signalChildProcess(child.pid, 'SIGKILL')
+    }, 1000)
+    if (typeof forceKillTimer.unref === 'function') {
+      forceKillTimer.unref()
+    }
+    process.exitCode = exitCode
+  }
+
+  for (const signal of ['SIGINT', 'SIGTERM', 'SIGHUP']) {
+    process.on(signal, () => {
+      shutdown({
+        signal,
+        exitCode: 128,
+      })
+    })
+  }
 
   child.on('exit', (code, signal) => {
+    stopWatchingParent()
+    if (forceKillTimer) {
+      clearTimeout(forceKillTimer)
+    }
     if (signal) {
-      process.kill(process.pid, signal)
+      process.exitCode = 128
       return
     }
     process.exitCode = code ?? 1

package/src/index.mjs

@@ -12,3 +12,5 @@ export {
 export { clearHarnessHistory, collectHistoryTargets } from './pi-history.mjs'
 export { collectLargeFileWarnings } from './pi-repo.mjs'
 export { runAgentTurn } from './pi-client.mjs'
+export { createSdkSession, createTools, normalizeToolNames, resolveModel, runSdkTurn, runSdkTurnWithPi, splitModelSpec } from './pi-sdk-turn.mjs'
+export { deriveCurrentIteration, deriveFlowSnapshot, deriveStageGraph, formatActiveLabel, getFlowSteps, getLabelForKind, getStepKeyForActiveRun, getStepKeyForKind } from './pi-visualizer-shared.mjs'

package/src/pi-client.mjs

@@ -1,9 +1,9 @@
 import { randomUUID } from 'node:crypto'
 import {
   appendLog,
-  runShellCommand,
   writeTextFile,
 } from './pi-repo.mjs'
+import { runSdkTurn } from './pi-sdk-turn.mjs'
 
 function truncateForNotes(text) {
   const trimmed = text.trim()
@@ -30,6 +30,13 @@ function formatLastAgentOutput(response) {
   return `${sections.join('\n')}\n`
 }
 
+async function writeAgentOutputSnapshot(config, content) {
+  await writeTextFile(config.lastAgentOutputFile, content)
+  if (config.runLastAgentOutputFile && config.runLastAgentOutputFile !== config.lastAgentOutputFile) {
+    await writeTextFile(config.runLastAgentOutputFile, content)
+  }
+}
+
 async function runMockTurn({ config, sessionId, sessionFile, prompt, reason }) {
   const nextSessionId = sessionId || `mock-${randomUUID()}`
   const nextSessionFile = sessionFile || `${config.piRuntimeDir}/mock-${nextSessionId}.jsonl`
@@ -40,10 +47,10 @@ async function runMockTurn({ config, sessionId, sessionFile, prompt, reason }) {
     'Prompt preview:',
     prompt,
     '',
-    'Mock mode does not edit files. Point PI_TRANSPORT=adapter at a real adapter to enable unattended work.',
+    'Mock mode does not edit files. Use default sdk transport for real unattended work.',
   ].join('\n')
 
-  await writeTextFile(config.lastAgentOutputFile, `${output}\n`)
+  await writeAgentOutputSnapshot(config, `${output}\n`)
   await appendLog(config.logFile, `Mock agent turn completed for session ${nextSessionId}`)
   if (config.streamTerminal) {
     process.stderr.write(`[PI mock] ${reason}\n`)
@@ -71,111 +78,60 @@ async function runMockTurn({ config, sessionId, sessionFile, prompt, reason }) {
   }
 }
 
-function parseAdapterResponse(stdout) {
-  const trimmed = stdout.trim()
-  if (trimmed === '') {
-    throw new Error('Adapter returned no JSON on stdout.')
-  }
-
-  try {
-    return JSON.parse(trimmed)
-  } catch {
-    const lines = trimmed.split('\n').map((line) => line.trim()).filter(Boolean)
-    const lastLine = lines.at(-1)
-    if (!lastLine) {
-      throw new Error('Adapter returned no parseable JSON on stdout.')
-    }
-    return JSON.parse(lastLine)
-  }
-}
-
-async function runAdapterTurn({ config, model, sessionId, sessionFile, prompt, iteration, retryCount, reason }) {
-  if (config.adapterCommand.trim() === '') {
-    throw new Error('PI_TRANSPORT=adapter requires PI_ADAPTER_COMMAND to be set.')
-  }
-
-  const request = {
-    sessionId,
-    sessionFile,
-    prompt,
-    cwd: config.cwd,
-    taskFile: config.taskFile,
-    instructionsFile: config.instructionsFile,
-    developerInstructionsFile: config.developerInstructionsFile,
-    testerInstructionsFile: config.testerInstructionsFile,
-    runtimeDir: config.runRuntimeDir || config.piRuntimeDir,
-    piCli: config.piCli,
-    model: model ?? config.piModel,
-    tools: config.piTools,
-    thinking: config.piThinking,
-    noExtensions: config.piNoExtensions,
-    noSkills: config.piNoSkills,
-    noPromptTemplates: config.piNoPromptTemplates,
-    noThemes: config.piNoThemes,
-    streamTerminal: config.streamTerminal,
-    loopRepeatThreshold: config.loopRepeatThreshold,
-    samePathRepeatThreshold: config.samePathRepeatThreshold,
-    continueAfterSeconds: config.continueAfterSeconds,
-    toolContinueAfterSeconds: config.toolContinueAfterSeconds,
-    continueMessage: config.continueMessage,
-    noEventTimeoutSeconds: config.noEventTimeoutSeconds,
-    toolNoEventTimeoutSeconds: config.toolNoEventTimeoutSeconds,
-    metadata: {
-      iteration,
-      retryCount,
-      reason,
-    },
-  }
-
+async function runSdkTransportTurn({ config, model, sessionId, sessionFile, prompt, iteration, retryCount, reason }) {
   await appendLog(
     config.logFile,
-    `Starting adapter turn via: ${config.adapterCommand} iteration=${iteration} retry=${retryCount} reason=${reason}`
+    `Starting SDK turn iteration=${iteration} retry=${retryCount} reason=${reason}`
   )
-  const result = await runShellCommand({
-    cwd: config.cwd,
-    command: config.adapterCommand,
-    timeoutSeconds: config.agentTimeoutSeconds,
-    stdinText: `${JSON.stringify(request)}\n`,
-    streamStderrToParent: config.streamTerminal,
-  })
-
-  await writeTextFile(config.lastAgentOutputFile, result.combinedOutput)
 
-  if (result.timedOut) {
-    await appendLog(config.logFile, 'Adapter turn timed out')
-    return {
-      sessionId: sessionId || '',
-      sessionFile: sessionFile || '',
-      status: 'timed_out',
-      exitCode: result.exitCode,
-      timedOut: true,
-      durationSeconds: result.durationSeconds,
-      output: result.combinedOutput,
-      notes: 'Adapter process exceeded the configured timeout.',
-      role: '',
+  const startedAt = Date.now()
+  let response
+  try {
+    response = await runSdkTurn({
+      sessionId,
+      sessionFile,
+      prompt,
+      cwd: config.cwd,
+      taskFile: config.taskFile,
+      instructionsFile: config.instructionsFile,
+      developerInstructionsFile: config.developerInstructionsFile,
+      testerInstructionsFile: config.testerInstructionsFile,
+      runtimeDir: config.runRuntimeDir || config.piRuntimeDir,
+      piCli: config.piCli,
       model: model ?? config.piModel,
-      toolCalls: 0,
-      toolErrors: 0,
-      messageUpdates: 0,
-      stopReason: '',
-      loopDetected: false,
-      loopSignature: '',
-      terminalReason: 'agent_timeout',
-    }
-  }
-
-  if (result.exitCode !== 0) {
-    await appendLog(config.logFile, `Adapter turn failed with exit code ${result.exitCode}`)
-    await writeTextFile(config.lastAgentOutputFile, result.combinedOutput)
+      tools: config.piTools,
+      thinking: config.piThinking,
+      noExtensions: config.piNoExtensions,
+      noSkills: config.piNoSkills,
+      noPromptTemplates: config.piNoPromptTemplates,
+      noThemes: config.piNoThemes,
+      streamTerminal: config.streamTerminal,
+      loopRepeatThreshold: config.loopRepeatThreshold,
+      samePathRepeatThreshold: config.samePathRepeatThreshold,
+      continueAfterSeconds: config.continueAfterSeconds,
+      toolContinueAfterSeconds: config.toolContinueAfterSeconds,
+      continueMessage: config.continueMessage,
+      noEventTimeoutSeconds: config.noEventTimeoutSeconds,
+      toolNoEventTimeoutSeconds: config.toolNoEventTimeoutSeconds,
+      metadata: {
+        iteration,
+        retryCount,
+        reason,
+      },
+    })
+  } catch (error) {
+    const notes = error instanceof Error ? error.message : String(error)
+    await writeAgentOutputSnapshot(config, `${notes}\n`)
+    await appendLog(config.logFile, `SDK turn failed: ${notes}`)
     return {
       sessionId: sessionId || '',
       sessionFile: sessionFile || '',
       status: 'failed',
-      exitCode: result.exitCode,
+      exitCode: 1,
       timedOut: false,
-      durationSeconds: result.durationSeconds,
-      output: result.combinedOutput,
-      notes: truncateForNotes(result.combinedOutput) || 'Adapter exited non-zero.',
+      durationSeconds: Math.max(0, Math.round((Date.now() - startedAt) / 1000)),
+      output: '',
+      notes,
       role: '',
       model: model ?? config.piModel,
       toolCalls: 0,
@@ -184,29 +140,22 @@ async function runAdapterTurn({ config, model, sessionId, sessionFile, prompt, i
       stopReason: '',
       loopDetected: false,
       loopSignature: '',
-      terminalReason: 'adapter_failed',
+      terminalReason: 'sdk_failed',
     }
   }
 
-  const response = parseAdapterResponse(result.stdout)
-  await writeTextFile(config.lastAgentOutputFile, formatLastAgentOutput(response))
-  const nextSessionId = String(response.sessionId ?? sessionId ?? '')
-  const nextSessionFile = String(response.sessionFile ?? sessionFile ?? '')
-  const status = String(response.status ?? 'success')
-  const output = String(response.output ?? result.combinedOutput)
-  const notes = String(response.notes ?? truncateForNotes(output))
-
-  await appendLog(config.logFile, `Adapter turn completed with status ${status}`)
+  await writeAgentOutputSnapshot(config, formatLastAgentOutput(response))
+  await appendLog(config.logFile, `SDK turn completed with status ${String(response.status ?? 'success')}`)
 
   return {
-    sessionId: nextSessionId,
-    sessionFile: nextSessionFile,
-    status,
-    exitCode: result.exitCode,
-    timedOut: false,
-    durationSeconds: result.durationSeconds,
-    output,
-    notes,
+    sessionId: String(response.sessionId ?? sessionId ?? ''),
+    sessionFile: String(response.sessionFile ?? sessionFile ?? ''),
+    status: String(response.status ?? 'success'),
+    exitCode: 0,
+    timedOut: String(response.status ?? '') === 'timed_out',
+    durationSeconds: Math.max(0, Math.round((Date.now() - startedAt) / 1000)),
+    output: String(response.output ?? ''),
+    notes: String(response.notes ?? ''),
     role: String(response.role ?? ''),
     model: String(response.model ?? model ?? config.piModel ?? ''),
     toolCalls: Number.isFinite(Number(response.toolCalls)) ? Number(response.toolCalls) : 0,
@@ -224,9 +173,9 @@ export async function runAgentTurn(args) {
     return await runMockTurn(args)
   }
 
-  if (args.config.transport === 'adapter') {
-    return await runAdapterTurn(args)
+  if (args.config.transport === 'sdk') {
+    return await runSdkTransportTurn(args)
   }
 
-  throw new Error(`Unsupported PI transport "${args.config.transport}". Expected "mock" or "adapter".`)
+  throw new Error(`Unsupported PI transport "${args.config.transport}". Expected "mock" or "sdk".`)
 }

package/src/pi-config.mjs

@@ -196,7 +196,6 @@ export function loadConfig(mode = 'once') {
   const cwd = process.cwd()
   const repoConfig = readRepoConfig(cwd)
   const file = repoConfig.values
-  const bundledAdapterCommand = 'pi-harness adapter'
   const bundledDeveloperInstructionsFile = path.join(packageRoot, 'templates', 'DEVELOPER.md')
   const bundledTesterInstructionsFile = path.join(packageRoot, 'templates', 'TESTER.md')
   const modelProfiles = readObject('models', file.models, {})
@@ -222,13 +221,14 @@ export function loadConfig(mode = 'once') {
       : bundledTesterInstructionsFile
   )
 
+  const transport = readString('PI_TRANSPORT', file.transport, 'sdk')
+
   return {
     cwd,
     configFile: repoConfig.configFile,
     mode: mode === 'run' ? 'run' : 'once',
-    transport: readString('PI_TRANSPORT', file.transport, 'adapter'),
+    transport,
     agentName: readString('PI_AGENT_NAME', file.agentName, 'PI'),
-    adapterCommand: readString('PI_ADAPTER_COMMAND', file.adapterCommand, bundledAdapterCommand),
     taskFile: resolveFromCwd(cwd, 'PI_TASK_FILE', file.taskFile, 'TODOS.md'),
     instructionsFile: resolveInstructionsFile(cwd, 'PI_INSTRUCTIONS_FILE', file.instructionsFile, bundledDeveloperInstructionsFile),
     developerInstructionsFile,