[aryateja2106]

Adds cursor-agent (Cursor's Composer 2 REPL) as a first-class `omc team` worker provider alongside claude/codex/gemini.

Why

cursor-agent is a very capable interactive coding agent that many users (including myself) already keep a running Cursor Pro/Max plan for. Making it a native OMC worker lets those users burn Cursor credits for throughput work while the Claude orchestrator retains the plan/verify role — without leaving the `omc team` mental model.

The user also gets the full cmux-pane visibility: they can watch the cursor-agent REPL live, interrupt by typing directly into the split, and the orchestrator continues to coordinate via the standard filesystem A2A at `.omc/state/team//`.

Changes

File	Change
`src/team/model-contract.ts`	Extend `CliAgentType` with `'cursor'`; add `CONTRACTS.cursor` (binary `cursor-agent`, `supportsPromptMode: false`, minimal launch args)
`src/team/types.ts`	Extend `WorkerBackend` with `'tmux-cursor'`; widen `WorkerInfo.worker_cli` to include `'cursor'`
`src/team/capabilities.ts`	Map `tmux-cursor` to `['code-edit', 'refactoring', 'general']`
`src/team/cli-detection.ts`	Detect `cursor-agent` in `detectAllClis()`
`src/team/worker-bootstrap.ts`	Add `case 'cursor':` to `agentTypeGuidance` with REPL-aware instructions (stay in session, claim/transition tasks via team-api, do not auto-exit)
`src/team/cli-worker-contract.ts`	Skip `shouldInjectContract` for cursor, matching the claude carve-out
`src/cli/team.ts`	Add `'cursor'` to `VALID_CLI_AGENT_TYPES`; update help text

Executor-only: why

Codex and Gemini use `supportsPromptMode: true` because they're one-shot CLIs (run a prompt → exit). The CLI-worker output contract (`CONTRACT_ROLES` = critic/code-reviewer/security-reviewer/test-engineer) depends on exit-on-complete so the verdict JSON can be read by the leader's worker-completion handler.

cursor-agent is an interactive REPL — it does not exit after answering. Routing a reviewer role to a cursor worker would hang the team because the verdict file would never be written. This patch:

1. Sets `supportsPromptMode: false` on the cursor contract.
2. Extends `shouldInjectContract` to skip cursor (alongside claude).
3. Documents the constraint in the in-prompt `agentTypeGuidance`.

Mailbox / inbox / sentinel / heartbeat flow is unchanged — cursor workers participate in the standard filesystem A2A the same way claude workers do.

Verification

```
$ npx tsc --noEmit

all production sources pass

$ cursor-agent --version # confirms binary detection

→ version output

```

End-to-end test (pending once this PR lands):

```
$ omc team 1:cursor:executor "implement hello.py and ack on leader-fixed"

cursor pane spawns in cmux split, receives trigger message,

claims task, edits file, transitions task status, waits.

```

Known follow-up (flagged honestly)

One residual TS error remains in `src/tests/runtime-guidance-plan-ralph.test.ts` at line 87 — it uses a hard-coded literal-union helper signature `(agentType: "claude" | "codex" | "gemini") => agentType is "claude"` that should be widened to `(agentType: CliAgentType) => agentType is "claude"` to accept the new provider. My local agent harness currently guards `*.test.ts` edits (it's a generic "don't silence tests" hook that mis-fires on type-widening), so rather than bypass it I'm surfacing this for a one-line amendment in-PR. Happy to paste the exact diff in a comment — it's simply replacing the literal union with the existing `CliAgentType` import.

Compatibility

• Existing claude/codex/gemini worker types and behaviors are unchanged.
• No bridge/MCP changes — cursor uses the existing tmux-pane path.
• The `BridgeConfig.provider` literal `'codex' | 'gemini'` is intentionally NOT widened: cursor does not go through the bridge runtime, so that narrower type correctly excludes it.

Relationship to cmux

This PR pairs well with two upstream cmux fixes I just opened to make `omc team` work inside cmux at all:

• fix(tmux-compat): expand single-char format codes (#S/#I/#W/#P/...) — fixes #2750 manaflow-ai/cmux#3031 — `__tmux-compat display-message` short-form format expansion
• fix(surface): accept surface_ref and surface aliases in send/read RPCs (related #2045) manaflow-ai/cmux#3032 — `surface.send_text/read_text/send_key` RPC accepts `surface_ref`/`surface` aliases

Once all three land, `cmux omc team 1:cursor,1:codex,1:gemini "..."` works end-to-end with each agent in its own cmux split, driveable both by the user (direct keyboard input) and the orchestrator (via the filesystem mailbox).

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1a02a5f834

ℹ️ 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".

[chatgpt-codex-connector]

Add cursor to the active omc team parser

Adding cursor in src/cli/team.ts does not make the shipped omc team command accept it, because the real entrypoint wires teamCommand from src/cli/commands/team.ts (see src/cli/index.ts) and that parser still uses VALID_TEAM_CLI_AGENT_TYPES = ['claude','codex','gemini']. In that code path, 1:cursor is treated as a role token rather than an agent type, so users cannot actually launch cursor workers via the primary CLI surface.

Useful? React with 👍 / 👎.

[chatgpt-codex-connector]

Include cursor in v1 CLI-worker shutdown detection

Widening CliAgentType to include cursor allows v1 teams to persist cursor agent types, but shutdownTeam() in src/team/runtime.ts still classifies CLI-worker teams using only claude|codex|gemini. When OMC_RUNTIME_V2 is disabled, cursor teams are misclassified as bridge teams and the shutdown path waits for shutdown-ack.json files that tmux CLI workers never write, causing avoidable timeout delays (30s by default).

Useful? React with 👍 / 👎.

[Yeachan-Heo]

Thanks for the contribution. We're closing this PR because it targets main; this repo takes contribution PRs against dev.

Please retarget the change onto dev and reopen as a fresh PR if you want us to review it there.

—
[repo owner's gaebal-gajae (clawdbot) 🦞]