Add oh-my-pi (omp) agent provider#500
Open
Git-on-my-level wants to merge 4 commits into
Open
Conversation
Adds omp as a fourth built-in agent provider alongside codex, claude-code,
and pi. omp is a divergent fork of vanilla pi with its own auth store,
Bun-first runtime, and RPC protocol, so bb drives the user-installed omp CLI
over its RPC mode via a thin framing bridge — no omp SDK is bundled, so there
is no version drift and omp owns auth/models at ~/.omp/agent.
- Provider registry entry (full-only perms, pi-mirroring caps) + skill-root type
- omp adapter: bb JSON-RPC <-> omp AgentSessionEvent translation (turn lifecycle
+ assistant text streaming; long-tail events route to the unhandled fallback)
- omp bridge: JSON-RPC 2.0 <-> omp {type:...} RPC framing, spawns omp --mode rpc
- UI: OmpIcon (pi glyph), provider-icon wiring, model picker groups by provider
- Host-daemon bundle manifest, server tests, bb-guide template, configuration.md
Verified end-to-end against omp 16.2.2: model/list returns the user's full
catalog and a full turn streams assistant text through agent_end.
… keys - Track per-thread omp sessions with switch_session/new_session and route events through the active thread instead of a single currentThreadId - Respawn omp when cwd or spawn-time options change between threads - Forward reasoning level, instructions, skill paths, and env vars to omp - Use providerKey instead of providerLabel for React group keys in the model picker
Git-on-my-level
force-pushed
the
add-omp-provider
branch
from
a100d58 to
6183ab1
Compare
Collaborator
|
I'm all for adding more providers! Our Pi provider is pretty out of date right now (I don't think anyone uses it). I think omp uses ACP so to simplify the surface area we could just support it via this. This is how our OpenCode integration works as well! It isn't well documented now but you should actually be able to add omp to editor by just adding a line to the ~/.bb/config.json, ask your agent how to do this! I think then we can make a config that auto detects if you have omp installed and auto configures it via acp (again look at how we handle open code) |
SawyerHood
requested changes
Jul 4, 2026
SawyerHood
left a comment
SawyerHood
left a comment
Collaborator
There was a problem hiding this comment.
Try to use ACP as the connector please. Also in the future open an issue before opening a PR
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds omp (oh-my-pi) as a fourth built-in agent provider alongside codex, claude-code, and pi.
omp is a divergent fork of vanilla pi — different auth store (
~/.omp/agent/agent.db, SQLite + multi-credential), Bun-first runtime with Rust native bindings, and its own RPC protocol. bb's existing"pi"provider bundles the pre-rename@mariozechner/pi-coding-agent@0.70.5, which cannot read omp's auth DB (verified:AuthStorage.hasAuth()returns false for every provider against a real omp install). Rather than migrate that dependency (broken at import time by omp's Bun-only APIs), this PR adds omp as a separate provider that drives the user-installedompCLI over its RPC mode via a thin framing bridge — no omp SDK is bundled, so there is no version drift, and omp owns auth/models entirely.Changes
agent-providers):"omp"entry, full-only perms, pi-mirroring caps;AgentRuntimeOmpSkillRoot.omp/adapter.ts): bb JSON-RPC ↔ ompAgentSessionEventtranslation — turn lifecycle + assistant text streaming. Unrecognized lifecycle/UI events are dropped silently (no timeline noise).omp/bridge/bridge.ts): JSON-RPC 2.0 ↔ omp{type:...}RPC framing; spawns the user'somp --mode rpc(no shell); answers ompextension_ui_requestframes so turns don't block; resets child state on exit so the bridge recovers after an omp crash; preserves text alongside attachment input; surfaces prompt/steer dispatch failures instead of hanging.OmpIcon(pi glyph), provider-icon wiring, model picker groups by provider (groupModelOptionsByProvider) when a catalog spans 2+provider/prefixes.configuration.md(BB_OMP_BINARY).Verified
omp 16.2.2:model/listreturns the user's full catalog (157 models across cursor/openai-codex/xai-oauth/zai), and a full turn streams assistant text and completes cleanly (turn/completed, zeroprovider/unhandled).Test plan
bb provider models omp→ lists the user's omp models grouped by provider.bb thread spawn --provider omp --model <provider/model> ...→ turn runs and completes.ompthen/login); override binary viaBB_OMP_BINARY.Known limitations (follow-ups)
fullonly (omp has no bb readonly/workspace-write analog).Review update
Additional validation
pnpm exec turbo run test --filter=@bb/agent-runtimepnpm exec turbo run typecheck --filter=@bb/agent-runtime