Oh My Pi (omp) as a built-in ACP provider #10883
Closed
omnificate
started this conversation in
Ideas
Replies: 1 comment
|
Duplicate of the existing Ideas thread: #6814 Please fold the implementation details / PR offer from here into that discussion so the request stays in one place. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
T3 Code has no plugin surface, and omp (can1357/oh-my-pi, 30k stars) ships a native stdio ACP server (
omp acp), so the cheapest correct integration is a seventh built-in driver on the existing generic ACP client stack - exactly the shape of the Grok/Cursor drivers.Prior art: PR #9038 implemented this and was closed unmerged in the Sept cleanup pass; issue #8985 was closed NOT_PLANNED with a pointer to Ideas. Opening this discussion per CONTRIBUTING.md before re-proposing the code.
What the driver does:
omp acpwith RuntimeMode-mapped approval flags (Supervised --approval-mode=always-ask, Auto-accept edits --approval-mode=write, Auto --auto-approve, Full access --approval-mode=yolo)Validation done on a local build: full server/contracts/web suites green (~8.5k tests incl. ~30 omp adapter tests for streaming, approvals, model-switch-without-respawn, cancel, subagent projection), plus a real end-to-end run against omp 18.1.15 (11.9k-model catalog, streamed turn in the built UI). Happy to open the PR rebased on current main if the direction is wanted.
All reactions