Skip to content

Add agent-agnostic peer discovery and durable task communication - #644

Open
parsakhaz wants to merge 5 commits into
mainfrom
codex/agent-communication
Open

parsakhaz wants to merge 5 commits into
mainfrom
codex/agent-communication

Conversation

@parsakhaz

@parsakhaz parsakhaz commented Sep 14, 2026 •

Copy link
Copy Markdown
Member

Pane already has a local socket daemon, but an idle terminal cannot identify which dispatched task was consumed or completed. This adds a common task protocol while preserving the existing terminal and worktree orchestration.

  • Add durable, addressed messages in sessions.db with stable request IDs, atomic inbox claims, immutable terminal replies and revision-based waits.
  • Add the same peers discovery/send/inbox/reply/wait commands to npm and Python CLIs. Managed guidance and environment hints expose them to arbitrary agent runtimes.
  • Bundle an optional Pi extension using native message delivery. It never treats turn_end or agent_settled as task completion and never automatically replays an uncertain claim.
  • Add watch --quiet-panel for correlated work, retaining blockers, held input and exits. Terminal wakes remain explicit, guarded and unconfirmed until a receipt.
  • Document recovery, capability limits, the Pi source investigation and an 18-scenario token simulation using the shipping watcher cadence.

For two false-ready pauses per task, simulated savings range from 3.0% to 62.3%, depending on extra receiver model turns. Clean readiness can make added messaging more expensive. These are synthetic coordination estimates, not measured billing.

Verified on dec7ea2: all 13 checks passed, including six npm/Python wrapper configurations across Linux/macOS/Windows, main-process suites on all three systems, consolidated lint, workspace types, the main/Pi bundle build, frontend tests and Electron/functional smoke tests. CI evidence.

Both automated review findings were fixed and resolved: terminal wakes now claim the requested message id, and quiet readiness still cancels pending blockers before output is suppressed. Targeted regression tests cover both. The wrapper fixture also handles expected client-disconnect races exposed by macOS CI.

A live authenticated Pi model session was not exercised; the adapter is verified against the pinned upstream API contract and automated delivery/recovery lifecycle tests. The documentation states this integration limit.

Canonical Codex/Claude orchestrator guidance: https://github.com/dcouple/skills/pull/94

Closes #638.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-14T11:22:09.041408Z 08986b0 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 08986b0894

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread main/src/ipc/runpanePeers.ts Outdated
Comment thread main/src/services/workspaceJournal.ts Outdated
@parsakhaz
parsakhaz force-pushed the codex/agent-communication branch from dec7ea2 to 5f8c668 Compare October 3, 2026 05:40

This branch has not been deployed

No deployments
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.

Unify agent discovery and communication with durable task-correlated messages

1 participant