feat(agents): add Grok Build as a native ACP runtime#2344
Open
daniel-farina wants to merge 1 commit into
Open
Conversation
Register Grok Build in the managed ACP catalog so Buzz can discover and spawn `grok agent --always-approve stdio` without a separate adapter package. Wire default args, ~/.grok/bin discovery, onboarding visibility, process sweep, docs, and unit coverage. Verified live with `buzz-acp models` against the installed Grok CLI.
4 tasks
Author
|
Tracking issue: #2347 |
9 tasks
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
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
Closes #2347.
id: grok) so Desktop can discover and spawn it like Goose/Claude/Codex.grok agent --always-approve stdio) — no separate*-acpnpm adapter.~/.grok/binto binary discovery, process-sweep namegrok, onboarding visibility, docs, and unit tests for default arg normalization.Why
Grok Build already speaks ACP over stdio (
grok agent stdio). Users currently cannot select it as a first-class Buzz harness without a custom command. This mirrors the Pi/Cursor runtime registration pattern.What changed
KNOWN_ACP_RUNTIMESagent,--always-approve,stdio(also maps bare legacyacp→ runtime default)~/.grok/bingrokto runtime order + readiness CLI pathbuzz-acpREADMEnormalizes_grok_args_to_agent_stdio(buzz-acp + desktop)Validation
cargo test -p buzz-acp --lib normalizes_— 6 passed (incl. Grok)BUZZ_ACP_AGENT_COMMAND=grok buzz-acp modelslists Grok modelsbuzz-acp+ Grok connected as Fizz, presence online, channel reply E2EHow to try
Then @mention the agent from Buzz Desktop (owner-only by default).
Checklist
unwrap()in production code