Skip to content

feat(cua-driver): add internal action truth records - #2711

Merged
f-trycua merged 1 commit into
codex/cua-driver-checked-statefrom
codex/cua-driver-action-truth
Jul 31, 2026
Merged

f-trycua merged 1 commit into
codex/cua-driver-checked-statefrom
codex/cua-driver-action-truth

Conversation

@f-trycua

@f-trycua f-trycua commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a rich, non-serialized ActionExecutionRecord at the canonical dispatch seam
  • normalize the existing macOS, Windows, Linux, and browser action payloads into five stable route families without changing MCP bytes
  • retain requested and actual delivery separately and downgrade legacy confirmations without trusted readback
  • add internal action truth to opt-in trajectory artifacts while keeping request targets, coordinates, scope, and debug-only evidence out of the stable projection

This is intentionally stacked on #2705 and has been replayed onto its certified head e52ff35fdc2756194659d442ce03841dbfd51162. It should be rebased to main after #2705 merges. The public ActionResult contract and SDK accessors remain a separate atomic breaking PR.

Compatibility

  • ToolResult.action_record is #[serde(skip)]
  • legacy structuredContent is returned unchanged
  • contract manifest and generated SDK bindings remain unchanged
  • recording action.json gains only an additive action_truth field

Verification

Freshly rerun on rebased head 68b6e7b776ffec5fd8302ff77b585437f28a6ac2:

  • cargo test -p cua-driver-core --lib --locked (448 passed)
  • cargo test -p platform-macos --lib --locked (259 passed)
  • cargo check -p cua-driver --locked
  • cargo test -p cua-driver-contract --lib --locked (21 passed)
  • cargo run --locked -p cua-driver-contract --bin cua-contract-gen -- all --check
  • cargo fmt --all -- --check
  • git diff --check

Architecture was reviewed with Claude Code Fable against the exact #2705 head. Its key recommendations incorporated here are the core/contract boundary, constructive no-echo projection, browser-action inclusion in the breaking bump, and removal of the window-scope validation gap in the follow-up.

@f-trycua
f-trycua marked this pull request as ready for review July 31, 2026 12:46
@f-trycua
f-trycua requested a review from injaneity as a code owner July 31, 2026 12:46
@f-trycua
f-trycua force-pushed the codex/cua-driver-checked-state branch from 8573e5a to e52ff35 Compare July 31, 2026 14:50
@f-trycua
f-trycua force-pushed the codex/cua-driver-action-truth branch from 3e81d57 to 68b6e7b Compare July 31, 2026 15:58
@f-trycua

Copy link
Copy Markdown
Collaborator Author

Fresh exact-head CI completed for 68b6e7b776ffec5fd8302ff77b585437f28a6ac2: all 28 checks completed with 26 successes, 2 intentional skips, and 0 failures. This includes Linux and Windows Rust builds, portable contract parity on Linux/macOS/Windows, generated bindings, docs, release metadata, packaging, and the full Nix matrix. The PR remains stacked behind #2705 and will be rebased to main after that prerequisite merges.

@f-trycua
f-trycua merged commit d180422 into codex/cua-driver-checked-state Jul 31, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant