Skip to content

Add Oh My Pi (omp) with built-in ACP - #572

Open
Gyarados4157 wants to merge 1 commit into
agentclientprotocol:mainfrom
Gyarados4157:add-omp-agent-18.1.9
Open

Add Oh My Pi (omp) with built-in ACP#572
Gyarados4157 wants to merge 1 commit into
agentclientprotocol:mainfrom
Gyarados4157:add-omp-agent-18.1.9

Conversation

@Gyarados4157

Copy link
Copy Markdown

Adds Oh My Pi (omp) to the ACP registry so Zed and other ACP clients can install it from the registry instead of a hand-written custom agent.

This is not a new adapter. Oh My Pi already ships a built-in ACP server:

omp acp

which is equivalent to:

npx -y @oh-my-pi/pi-coding-agent@18.1.9 acp

Why this PR

  • Registry currently has pi-acp (upstream Pi via a third-party adapter). It does not list Oh My Pi.
  • Open PRs #301 (omp-acp, 15.9.0) and #502 (omp, 17.3.3) are stale. This one pins the current npm release 18.1.9 and uses the upstream built-in ACP command.

Distribution

Once merged, hourly version updates can pick up later npm releases.

Authentication

initialize advertises Agent Auth:

{
  "id": "agent",
  "name": "Use existing local credentials",
  "description": "Authenticate via the provider keys/OAuth state already configured under ~/.omp."
}

Verified locally against the installed omp/18.1.9 binary:

initialize -> protocolVersion 1
agentInfo: { "name": "oh-my-pi", "title": "Oh My Pi", "version": "18.1.9" }
authMethods: [{ "id": "agent", "name": "Use existing local credentials", ... }]

This matches AUTHENTICATION.md (Agent Auth, type defaults to agent).

Validation

SKIP_URL_VALIDATION=1 uv run --with jsonschema .github/workflows/build_registry.py --dry-run

passed and included Added agent: omp v18.1.9. Icon is a 16x16 monochrome currentColor π mark (upstream favicon uses a gradient, which the registry rejects).

After merge, Zed users can install Oh My Pi from zed: acp registry and start an external-agent thread without editing settings.json.

Registers the upstream Oh My Pi CLI as an ACP agent. omp 18.1.9
already speaks ACP over stdio via `omp acp` / npx
`@oh-my-pi/pi-coding-agent@18.1.9 acp`, so Zed and other ACP
clients can install it from the registry instead of a custom
agent_servers entry.

This is the current built-in path, not the older standalone
omp-acp adapter. Existing open PRs agentclientprotocol#301 and agentclientprotocol#502 target stale
15.x / 17.x pins.
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.

1 participant