Skip to content

Add Oh My Pi ACP (built-in)#301

Open
jiwangyihao wants to merge 3 commits into
agentclientprotocol:mainfrom
jiwangyihao:add-omp-acp
Open

Add Oh My Pi ACP (built-in)#301
jiwangyihao wants to merge 3 commits into
agentclientprotocol:mainfrom
jiwangyihao:add-omp-acp

Conversation

@jiwangyihao
Copy link
Copy Markdown

@jiwangyihao jiwangyihao commented May 12, 2026

Adds the built-in ACP server mode for Oh My Pi.

Distribution:

  • npm/npx: @oh-my-pi/pi-coding-agent@15.9.0 with acp

Authentication:

  • ACP initialize advertises the built-in agent auth method, which satisfies Registry auth validation.
  • The built-in ACP path also supports terminal auth when the client advertises ACP terminal auth capability.

Local validation performed:

  • SKIP_URL_VALIDATION=1 uv run --with jsonschema .github/workflows/build_registry.py --dry-run passed and included omp-acp v15.9.0.
  • Manual auth handshake against npx --prefix <tmp> --yes @oh-my-pi/pi-coding-agent@15.9.0 acp returned agentInfo: oh-my-pi v15.9.0 and authMethods: [{ id: "agent" }].

Notes:

  • This entry now points at the upstream Oh My Pi package instead of the standalone omp-acp adapter package.
  • Host-native verify_agents.py --auth-check --agent omp-acp --type npx on Windows/MSYS still uses bare npx, which is a local host quirk; GitHub Actions runs the registry checks on ubuntu-latest.

@gabrielmoreira
Copy link
Copy Markdown

@jiwangyihao omp has built-in ACP support: https://omp.sh/docs/acp

@jiwangyihao
Copy link
Copy Markdown
Author

@jiwangyihao omp has built-in ACP support: https://omp.sh/docs/acp

I recently discovered this (OMP has built-in ACP support, though I found some blocking-related issues during testing) and submitted some fixes to the OMP repository via PRs. Those PRs have now been merged. After the next OMP release is published and tested, I will update this PR to use OMP’s built-in ACP support.

@jiwangyihao jiwangyihao changed the title Add Oh My Pi ACP Add Oh My Pi ACP (built-in) Jun 4, 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.

2 participants