diff --git a/doc/RUNNER-INTEGRATION-PLAYBOOK.md b/doc/RUNNER-INTEGRATION-PLAYBOOK.md index d3f39a9418a..50a6714b835 100644 --- a/doc/RUNNER-INTEGRATION-PLAYBOOK.md +++ b/doc/RUNNER-INTEGRATION-PLAYBOOK.md @@ -156,7 +156,7 @@ the Rust decoder or actual packaged sidecar. | Canonical provider event inventory | [provider-events.ts](../packages/paperclip-runner/src/provider-events.ts), [protocol schemas](../packages/paperclip-runner/protocol/) | | ACPX capability and driver declarations | [driver-profile.ts](../packages/paperclip-runner/src/drivers/acpx/driver-profile.ts), [capability-profiles.ts](../packages/paperclip-runner/src/drivers/acpx/capability-profiles.ts) | | Provider-specific requests and activity | [profile-extensions.ts](../packages/paperclip-runner/src/drivers/acpx/profile-extensions.ts), [profile-activity.ts](../packages/paperclip-runner/src/drivers/acpx/profile-activity.ts) | -| Actual ACPX manager and provider interaction | [codex-runtime-adapter.ts](../packages/paperclip-runner/src/drivers/acpx/codex-runtime-adapter.ts), [codex-acpx-driver.ts](../packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.ts) | +| Actual ACPX manager and provider interaction | [acpx-runtime-adapter.ts](../packages/paperclip-runner/src/drivers/acpx/acpx-runtime-adapter.ts), [codex-acpx-driver.ts](../packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.ts) | | Model/mode and session identity | [model-verification.ts](../packages/paperclip-runner/src/drivers/acpx/model-verification.ts), [provider-mode.ts](../packages/paperclip-runner/src/drivers/acpx/provider-mode.ts), [recovery-identity.ts](../packages/paperclip-runner/src/drivers/acpx/recovery-identity.ts) | | TypeScript/Rust sidecar boundary | [sidecar schema](../packages/paperclip-runner/protocol/provider-schemas/acpx-sidecar.schema.json), [sidecar-protocol.ts](../packages/paperclip-runner/src/drivers/acpx/sidecar-protocol.ts), [sidecar CLI](../packages/paperclip-runner/src/cli/), [Rust core](../packages/paperclip-runner/runner/crates/runner-core/src/) | | Release identity | [acpx-profiles.json](../packages/paperclip-runner/acpx-profiles.json), [profile generator](../packages/paperclip-runner/scripts/generate-acpx-profiles.mjs), [qualified-profiles.ts](../packages/paperclip-runner/src/drivers/acpx/qualified-profiles.ts) | diff --git a/doc/architecture/runner-cursor-capabilities.md b/doc/architecture/runner-cursor-capabilities.md index b361f40ffa5..7c11c993909 100644 --- a/doc/architecture/runner-cursor-capabilities.md +++ b/doc/architecture/runner-cursor-capabilities.md @@ -309,7 +309,7 @@ This source audit follows pinned native update → ACPX runtime → `canonicalPr | Native field/interface | Preserved boundary | Current projection / reason for partial use | Follow-up | |---|---|---|---| -| Prompt image blocks | Pinned native initialize advertises image input | Shared `AcpxRuntimeTurnInput` has only `text`; `codex-runtime-adapter.ts` passes that text to ACPX, so no image blocks reach the wrapper from a runner turn | P1: add validated, bounded attachment inputs through the runner/ACPX contract, then prove exact-model behavior; native advertisement alone is not implementation evidence | +| Prompt image blocks | Pinned native initialize advertises image input | Shared `AcpxRuntimeTurnInput` has only `text`; `acpx-runtime-adapter.ts` passes that text to ACPX, so no image blocks reach the wrapper from a runner turn | P1: add validated, bounded attachment inputs through the runner/ACPX contract, then prove exact-model behavior; native advertisement alone is not implementation evidence | | Parent tool `rawInput` | ACPX forwards the body | Canonical tool schema exposes only `inputUpdated`; full native arguments are not displayed | P1: bounded/redacted tool-argument presentation with company and path checks | | Parent tool `content[]` diff `path`, `oldText`, `newText` | ACPX forwards structured content; native permission requests also use it for proposed changes | Canonical tool mapper uses `rawOutput`, so completed provider-reported diff blocks have no dedicated output surface; separate workspace evidence is not equivalent | P1: validate physical workspace locations and expose attributed diff artifacts | | Parent tool `content[]` images/resource blocks | ACPX forwards blocks and produces a summary | Structured media is unused by canonical tool output; safe registration/location semantics must precede ingestion | P1: validated artifact reference projection with explicit provenance | diff --git a/doc/architecture/runner-rich-acp-capabilities.md b/doc/architecture/runner-rich-acp-capabilities.md index 7d6ef088f0e..63bb5883d46 100644 --- a/doc/architecture/runner-rich-acp-capabilities.md +++ b/doc/architecture/runner-rich-acp-capabilities.md @@ -840,7 +840,7 @@ deterministic evidence are mapped separately after the comparison. | Shared capability | User-visible surface | Inspectable implementation / deterministic evidence | | --- | --- | --- | -| Text, tools and child activity | Task transcript; bounded activity details preserve provider/session/tool attribution | `src/drivers/acpx/codex-runtime-adapter.test.ts`, `runner/crates/runner-core/tests/acpx_rich_events.rs`, `ui/src/components/task-chat/TaskChatProtocolActivityRow.test.tsx` | +| Text, tools and child activity | Task transcript; bounded activity details preserve provider/session/tool attribution | `src/drivers/acpx/acpx-runtime-adapter.test.ts`, `runner/crates/runner-core/tests/acpx_rich_events.rs`, `ui/src/components/task-chat/TaskChatProtocolActivityRow.test.tsx` | | Native questions and permissions | Existing question/confirmation cards; only offered decisions can be submitted | `src/protocol/permission-request.test.ts`, `src/drivers/acpx/acp-permission-adapter.test.ts`, `server/src/services/native-runtime/native-question-bridge.test.ts`, `ui/src/components/task-chat/TaskChatProtocolCard.test.tsx` | | Native plan decisions | Full plan description followed by a revision-bound decision; no implicit plan acceptance | Cursor provider fixtures; `ui/storybook/fixtures/evidence/rich-acp-browser-proof.darwin-arm64.json` records the real renderer with a canonical fixture | | Active steering and queued follow-up | Existing active-turn control where a bound method is negotiated; no native Pi queue selector yet | `src/drivers/acpx/turn-controls.test.ts`; provider reports state which wire method is absent or not surfaced | diff --git a/packages/paperclip-runner/scripts/grok-native-smoke.mjs b/packages/paperclip-runner/scripts/grok-native-smoke.mjs index 7bb761ab9a0..a37471757a4 100644 --- a/packages/paperclip-runner/scripts/grok-native-smoke.mjs +++ b/packages/paperclip-runner/scripts/grok-native-smoke.mjs @@ -7,7 +7,7 @@ import { basename, dirname, join, resolve } from 'node:path'; import { parseArgs } from 'node:util'; import { createRequire } from 'node:module'; import { AcpxRuntimeHost } from '../dist/drivers/acpx/runtime-host.js'; -import { openQualifiedAcpxRuntime } from '../dist/drivers/acpx/codex-runtime-adapter.js'; +import { openQualifiedAcpxRuntime } from '../dist/drivers/acpx/acpx-runtime-adapter.js'; import { acpxProviderSessionIdentity } from '../dist/drivers/acpx/recovery-identity.js'; import { QUALIFIED_ACPX_PROFILES } from '../dist/drivers/acpx/qualified-profiles.js'; diff --git a/packages/paperclip-runner/scripts/vitest-lane-durations.json b/packages/paperclip-runner/scripts/vitest-lane-durations.json index f5bb05a609f..f4cdc8ef912 100644 --- a/packages/paperclip-runner/scripts/vitest-lane-durations.json +++ b/packages/paperclip-runner/scripts/vitest-lane-durations.json @@ -91,7 +91,7 @@ "packages/paperclip-runner/src/drivers/acpx/claude-status-banners.test.ts": 100, "packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.test.ts": 893, "packages/paperclip-runner/src/drivers/acpx/codex-credentials.test.ts": 6558, - "packages/paperclip-runner/src/drivers/acpx/codex-runtime-adapter.test.ts": 2923, + "packages/paperclip-runner/src/drivers/acpx/acpx-runtime-adapter.test.ts": 2923, "packages/paperclip-runner/src/drivers/acpx/command-lease-owner.test.ts": 100, "packages/paperclip-runner/src/drivers/acpx/copilot-events.test.ts": 100, "packages/paperclip-runner/src/drivers/acpx/copilot-evidence.test.ts": 100, diff --git a/packages/paperclip-runner/src/cli/acpx-runtime-sidecar.ts b/packages/paperclip-runner/src/cli/acpx-runtime-sidecar.ts index d9c17ddfc73..b850c3a1284 100644 --- a/packages/paperclip-runner/src/cli/acpx-runtime-sidecar.ts +++ b/packages/paperclip-runner/src/cli/acpx-runtime-sidecar.ts @@ -49,7 +49,7 @@ import { normalizeAcpFormElicitation, type NormalizedAcpForm, } from "../drivers/acpx/acp-question-adapter.js"; -import { openCodexAcpxRuntime, AcpxProviderProcessLostError } from "../drivers/acpx/codex-runtime-adapter.js"; +import { openCodexAcpxRuntime, AcpxProviderProcessLostError } from "../drivers/acpx/acpx-runtime-adapter.js"; import { AcpxApprovalRequiredError } from "../drivers/acpx/permission-policy.js"; import { normalizeAcpxPermission, type NormalizedAcpxPermission } from "../drivers/acpx/acp-permission-adapter.js"; import { acpxGoalProjection } from "../drivers/acpx/session-goals.js"; diff --git a/packages/paperclip-runner/src/drivers/acpx/abandoned-bootstrap-record.test.ts b/packages/paperclip-runner/src/drivers/acpx/abandoned-bootstrap-record.test.ts index dd501d90213..0ec2f8e9918 100644 --- a/packages/paperclip-runner/src/drivers/acpx/abandoned-bootstrap-record.test.ts +++ b/packages/paperclip-runner/src/drivers/acpx/abandoned-bootstrap-record.test.ts @@ -4,7 +4,7 @@ import { tmpdir } from "node:os"; import { join } from "node:path"; import { createAcpRuntime, createAgentRegistry, createRuntimeStore } from "acpx/runtime"; import { describe, expect, it } from "vitest"; -import { resetAbandonedBootstrapRecord } from "./codex-runtime-adapter.js"; +import { resetAbandonedBootstrapRecord } from "./acpx-runtime-adapter.js"; // A controlled ACP peer that behaves like Claude: a session that never received // a prompt was never saved, so session/load (and session/resume) of it fails. diff --git a/packages/paperclip-runner/src/drivers/acpx/codex-runtime-adapter.test.ts b/packages/paperclip-runner/src/drivers/acpx/acpx-runtime-adapter.test.ts similarity index 99% rename from packages/paperclip-runner/src/drivers/acpx/codex-runtime-adapter.test.ts rename to packages/paperclip-runner/src/drivers/acpx/acpx-runtime-adapter.test.ts index 653995bb09f..2cde8e51642 100644 --- a/packages/paperclip-runner/src/drivers/acpx/codex-runtime-adapter.test.ts +++ b/packages/paperclip-runner/src/drivers/acpx/acpx-runtime-adapter.test.ts @@ -19,7 +19,7 @@ import { decodeAcpxRuntimeHandleState } from "acpx/runtime"; import { describe, expect, it, vi } from "vitest"; import type { VerifiedAcpxCommandLease } from "./installation-integrity.js"; -import { acpxSessionHandshakeTimeoutMs, claudeExitPlanModeProposal, openCodexAcpxRuntime, resetAbandonedBootstrapRecord } from "./codex-runtime-adapter.js"; +import { acpxSessionHandshakeTimeoutMs, claudeExitPlanModeProposal, openCodexAcpxRuntime, resetAbandonedBootstrapRecord } from "./acpx-runtime-adapter.js"; import { cursorInstructionBinding } from "./cursor-instructions.js"; import { persistedAcpxTurnUsage } from "./usage-accounting.js"; import { createAcpxCommandLeaseOwner } from "./command-lease-owner.js"; diff --git a/packages/paperclip-runner/src/drivers/acpx/codex-runtime-adapter.ts b/packages/paperclip-runner/src/drivers/acpx/acpx-runtime-adapter.ts similarity index 100% rename from packages/paperclip-runner/src/drivers/acpx/codex-runtime-adapter.ts rename to packages/paperclip-runner/src/drivers/acpx/acpx-runtime-adapter.ts diff --git a/packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.test.ts b/packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.test.ts index 0c1873a57ae..83c65e173c8 100644 --- a/packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.test.ts +++ b/packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.test.ts @@ -3,7 +3,7 @@ import { readFileSync } from "node:fs"; import { createHash } from "node:crypto"; import { cursorToolIdentity } from "./cursor-plan-tool-identity.js"; import { describe, expect, it, vi, type Mock } from "vitest"; -import { AcpxProviderProcessLostError } from "./codex-runtime-adapter.js"; +import { AcpxProviderProcessLostError } from "./acpx-runtime-adapter.js"; import { QUALIFIED_ACPX_PROFILES } from "./qualified-profiles.js"; import type { AcpRuntimeEvent } from "acpx/runtime"; diff --git a/packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.ts b/packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.ts index 1efcacd594d..020bfdebade 100644 --- a/packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.ts +++ b/packages/paperclip-runner/src/drivers/acpx/codex-acpx-driver.ts @@ -66,7 +66,7 @@ import { DEFAULT_CODEX_ACPX_RUNTIME_SHUTDOWN_BOUND_MS, AcpxProviderProcessLostError, openCodexAcpxRuntime, -} from "./codex-runtime-adapter.js"; +} from "./acpx-runtime-adapter.js"; import { normalizeAcpxPermission } from "./acp-permission-adapter.js"; import { normalizeAcpFormElicitation, diff --git a/packages/paperclip-runner/src/drivers/acpx/native-distribution-integrity.test.ts b/packages/paperclip-runner/src/drivers/acpx/native-distribution-integrity.test.ts index 61cd382955b..7c4e66acd9b 100644 --- a/packages/paperclip-runner/src/drivers/acpx/native-distribution-integrity.test.ts +++ b/packages/paperclip-runner/src/drivers/acpx/native-distribution-integrity.test.ts @@ -11,7 +11,7 @@ import { afterEach, describe, expect, it, vi } from "vitest"; import { copyOwnedNodeRuntime } from "../../test-support/owned-node-runtime.js"; import { awaitVerifiedAcpxProviderExit, awaitVerifiedAcpxProviderOwnership, verifyNativeAcpxInstallation } from "./installation-integrity.js"; import { createNativeAcpxDistributionSnapshot, readNativeAcpxDistributionEntries, parseNativeAcpxDistributionEntries, type NativeAcpxDistributionInput, type NativeAcpxDistributionEntry } from "./native-distribution-integrity.js"; -import { openCodexAcpxRuntime } from "./codex-runtime-adapter.js"; +import { openCodexAcpxRuntime } from "./acpx-runtime-adapter.js"; import { createAcpxCommandLeaseOwner } from "./command-lease-owner.js"; import { resolveQualifiedAcpxProfile } from "./qualified-profiles.js"; diff --git a/packages/paperclip-runner/src/drivers/acpx/runtime-host.test.ts b/packages/paperclip-runner/src/drivers/acpx/runtime-host.test.ts index 15b1ebb780f..7a6b650da9f 100644 --- a/packages/paperclip-runner/src/drivers/acpx/runtime-host.test.ts +++ b/packages/paperclip-runner/src/drivers/acpx/runtime-host.test.ts @@ -20,7 +20,7 @@ import { type NativeRuntimeContextSnapshot, } from "../../contracts/runtime-context.js"; import { releaseMaterializedNativeRuntimeSkills } from "../runtime-context-materializer.js"; -import { openCodexAcpxRuntime } from "./codex-runtime-adapter.js"; +import { openCodexAcpxRuntime } from "./acpx-runtime-adapter.js"; import { acpxSidecarErrorCode } from "../../cli/acpx-sidecar-input.js"; import { stageManagedCodexCredential } from "./codex-credentials.js"; import type {