Skip to content

docs(codex-tools): add named agent dispatch mapping for Codex#900

Merged
arittr merged 7 commits intodevfrom
drew/codex-named-agent-dispatch
Mar 24, 2026
Merged

docs(codex-tools): add named agent dispatch mapping for Codex#900
arittr merged 7 commits intodevfrom
drew/codex-named-agent-dispatch

Conversation

@arittr
Copy link
Collaborator

@arittr arittr commented Mar 24, 2026

Summary

  • Adds a "Named agent dispatch" section to codex-tools.md explaining how Codex should handle skills that reference named agent types (e.g., superpowers:code-reviewer)
  • Maps Task tool (superpowers:code-reviewer)spawn_agent(agent_type="worker", message=...) with the agent's prompt content inlined
  • Includes message framing guidance (task-delegation framing, XML tags, execution directive) since Codex's message parameter is user-level input, not a system prompt
  • Documents when this workaround can be removed (RawPluginManifest gains an agents field)

Why

Codex's spawn_agent requires pre-registered TOML agent roles — it can't dispatch named agent types defined by agents/*.md files. Skills like requesting-code-review reference superpowers:code-reviewer which doesn't exist in Codex. The worker built-in + inlined prompt approach avoids duplicating agent definitions across formats.

Test plan

  • Manual verification: Codex user invokes requesting-code-review, review runs as worker subagent
  • Verify Claude Code path is unaffected (no skill files changed)

Closes #647

🤖 Generated with Claude Code

Drew Ritter and others added 6 commits March 23, 2026 12:30
Design for making using-git-worktrees, finishing-a-development-branch,
and subagent-driven-development skills work in the Codex App's sandboxed
worktree environment. Read-only environment detection via git-dir vs
git-common-dir comparison, ~48 lines across 4 files, zero breaking changes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Fix three Important issues from spec review:
- Clarify Step 1.5 placement relative to existing Steps 2/3
- Re-derive environment state at cleanup time instead of relying on
  earlier skill output
- Acknowledge pre-existing Step 5 cleanup inconsistency

Also: precise step references, exact codex-tools.md content, clearer
Integration section update instructions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add commit SHA + data loss warning to handoff payload (HIGH)
- Add explicit commit step before handoff (HIGH)
- Remove misleading "mark as externally managed" from Path B
- Add executing-plans 1-line edit (was missing)
- Add branch name derivation rules
- Add conditional UI language for non-App environments
- Add sandbox fallback for permission errors
- Add STOP directive after Step 0 reporting

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…spec

Both tests address real risk scenarios:
- #5: cleanup guard bug would delete Codex App's own worktree (data loss)
- #10: Local thread sandbox fallback needs manual Codex App validation

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
8 tasks covering: environment detection in using-git-worktrees,
Step 1.5 + cleanup guard in finishing-a-development-branch,
Integration line updates, codex-tools.md docs, automated tests,
and final verification.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@arittr arittr changed the base branch from dev to drew/pri-823-codex-app-compat March 24, 2026 00:06
@arittr
Copy link
Collaborator Author

arittr commented Mar 24, 2026

@arittr arittr changed the base branch from drew/pri-823-codex-app-compat to dev March 24, 2026 00:36
@arittr arittr force-pushed the drew/codex-named-agent-dispatch branch from 72ab835 to 458269a Compare March 24, 2026 00:36
@arittr arittr merged commit 7820adc into dev Mar 24, 2026
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.

Codex: requesting-code-review assumes Task/subagent dispatch that native skill install does not provide

2 participants