Skip to content

perf(cli): replace status polling with turn-intent push - #565

Merged
Harry19081 merged 3 commits into
developfrom
junyu/cli-runner-intent-propagation
Jul 30, 2026
Merged

perf(cli): replace status polling with turn-intent push#565
Harry19081 merged 3 commits into
developfrom
junyu/cli-runner-intent-propagation

Conversation

@beruro

@beruro beruro commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • propagate persisted turn-intent IDs through the CLI runner and websocket status path
  • expose batched status RPCs for focus/reconnect reconciliation
  • replace per-session reconciliation polling with a bounded push coordinator
  • preserve account-scoped resume state and terminal ordering across transports

Stack

Depends on #512, which now contains only the persisted turn-intent batch projection. This PR is the end-to-end CLI transport slice; frontend queue/intervention semantics will follow separately.

Validation

  • targeted Vitest: 8 passed
  • pnpm typecheck and changed-file ESLint
  • cargo test -p org2 agent_sessions::cli: 280 passed, 1 ignored
  • cargo check -p org2
  • rustfmt check on all changed Rust files
  • git diff --check junyu/pr512-core-rebuild...HEAD

Architecture boundary

Covered persistence adapter, RPC schema/procedure, CLI runner, websocket transport, and reconciliation projection. Generic async consumers, caches/scans, diagnostics, profiling, and Team Inbox are excluded.

@beruro
beruro force-pushed the junyu/cli-runner-intent-propagation branch from 5a075ab to cac6149 Compare July 30, 2026 07:07
@beruro
beruro force-pushed the junyu/rpc-performance-closure branch from c42f777 to cb96c41 Compare July 30, 2026 07:09
@beruro
beruro force-pushed the junyu/cli-runner-intent-propagation branch from cac6149 to 952201d Compare July 30, 2026 07:50
@beruro
beruro force-pushed the junyu/cli-runner-intent-propagation branch from 952201d to e746dd4 Compare July 30, 2026 08:05
@beruro
beruro changed the base branch from junyu/rpc-performance-closure to develop July 30, 2026 08:22
@Harry19081
Harry19081 merged commit df08817 into develop Jul 30, 2026
1 check passed
@Harry19081
Harry19081 self-requested a review July 30, 2026 08:45
Harry19081 added a commit to Ethereallllllll1/ORG2 that referenced this pull request Jul 30, 2026
PR org2AI#565 papered over two `too_many_arguments` hits in the CLI run commands
with `#[allow]`. Group the arguments into `CliRunRequest` /
`CliMessageRequest` instead, matching the `request: T` command convention
already used by `benchmark_*`, `human_session_*`, and pagination.

`turn_intent_id` + `client_message_id` move into a `TurnIdentity` pair so
the "adopt the client's ids or mint fresh ones" rule lives in one place
rather than being duplicated across the two entry points.
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.

2 participants