Skip to content

feat(cli): add Codex handoff integration#253

Closed
kleinpanic wants to merge 2 commits into
Open-ACP:mainfrom
kleinpanic:feat/codex-handoff-integration
Closed

feat(cli): add Codex handoff integration#253
kleinpanic wants to merge 2 commits into
Open-ACP:mainfrom
kleinpanic:feat/codex-handoff-integration

Conversation

@kleinpanic
Copy link
Copy Markdown

@kleinpanic kleinpanic commented May 8, 2026

Summary

Adds Codex handoff support so existing Codex CLI sessions can be adopted into OpenACP and continued from a configured messaging adapter.

This is session-level continuation through the agent resume/session ID. It does not claim live mirroring of an already-running terminal process or replay of prior terminal scrollback.

Related Issues

None.

Type of Change

  • New feature
  • Bug fix
  • Documentation
  • Tests

Changes Made

  • Adds a Codex integration spec to openacp integrate.
  • Installs Codex handoff hooks and an openacp-handoff skill without replacing unrelated Codex hooks.
  • Adds openacp adopt codex --tmux and --tmux-pane for explicit tmux-assisted discovery of Codex resume IDs.
  • Uses a deterministic adopted-session title based on agent, workspace, and short external session ID.
  • Documents Codex tmux adoption and clarifies that handoff is continuation, not terminal transcript mirroring.

Testing

  • Ran pnpm build
  • Ran pnpm test src/core/sessions/__tests__/adopted-session-title.test.ts src/cli/commands/__tests__/adopt-json.test.ts src/cli/__tests__/integrate.test.ts
  • Ran git diff --check origin/main..HEAD
  • Ran gitleaks git --log-opts origin/main..HEAD --redact --no-banner

Platform Impact

  • CLI
  • All / Core
  • Telegram
  • Discord
  • Slack
  • Web UI

Checklist

  • Code follows the project's TypeScript conventions, including ESM .js imports
  • Relevant documentation was updated
  • Tests were added or updated
  • pnpm build passes
  • Changes are backward compatible with existing config and data formats

Copilot AI review requested due to automatic review settings May 8, 2026 12:28
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@kleinpanic kleinpanic force-pushed the feat/codex-handoff-integration branch from 538d6e5 to d2455d7 Compare May 8, 2026 19:10
@kleinpanic kleinpanic closed this May 8, 2026
@kleinpanic kleinpanic deleted the feat/codex-handoff-integration branch May 8, 2026 19:39
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