Skip to content

feat(desktop): register Grok Build as native ACP runtime - #2918

Closed
DevvGwardo wants to merge 1 commit into
block:mainfrom
DevvGwardo:feat/grok-acp-runtime
Closed

feat(desktop): register Grok Build as native ACP runtime#2918
DevvGwardo wants to merge 1 commit into
block:mainfrom
DevvGwardo:feat/grok-acp-runtime

Conversation

@DevvGwardo

Copy link
Copy Markdown

Summary

  • Adds grok to KNOWN_ACP_RUNTIMES with native ACP launch args agent --always-approve stdio
  • Wires discovery (~/.grok/bin), process tracking, onboarding visibility, and buzz-acp arg normalization
  • Grok then owns slash commands, skills, plugins, and .grok/workflows/ from the agent cwd — Buzz inherits via ACP pass-through

Originating channel: a3ec672d-4a27-40f5-8307-be553f11dad0

Test plan

  • cargo test -p buzz-acp normalizes_grok
  • node --test desktop/src/features/onboarding/ui/onboardingRuntimeSelection.test.mjs
  • Full cargo test in desktop/src-tauri (needs prebuilt buzz-acp binary in CI)

Notes

Made with Cursor

Add grok to KNOWN_ACP_RUNTIMES with agent --always-approve stdio defaults
so Buzz-managed agents inherit Grok slash commands, skills, and workflows
from the agent cwd without re-implementing them in Buzz.

Co-authored-by: DevGwardo <25094504+DevGwardo@users.noreply.github.com>
Signed-off-by: DevGwardo <25094504+DevGwardo@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@DevvGwardo
DevvGwardo requested a review from a team as a code owner July 26, 2026 00:46
@wpfleger96

Copy link
Copy Markdown
Member

🤖 hey @DevvGwardo — thanks for building this. PRs like this one are exactly what convinced us to solve harness support generically instead of one runtime at a time.

That work landed in #2773 and ships in v0.5.0: Grok Build is a one-click preset launching grok agent --always-approve stdio, and — matching the call you made here — the preset keys off the grok binary only, avoiding the bare agent shim collision.

Closing as superseded. If you find something the preset doesn't handle, a focused issue or preset PR would be great.

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