Connect Grok Build as a first-class coding agent - #344
Open
shawnyeager wants to merge 2 commits into
Open
Conversation
Register Grok in the agent harness, add hey setup grok, and copy the HEY skill into $GROK_HOME/skills/hey. Include Grok in skill install, refresh, doctor, setup agents, and --remove. No Grok plugin yet.
Sensitive Change Detection (shadow mode)This PR modifies control-plane files:
|
Contributor
There was a problem hiding this comment.
Pull request overview
Adds Grok Build as a first-class coding-agent integration alongside Claude Code and Codex.
Changes:
- Adds Grok detection, skill installation, health checks, refresh, and removal.
- Integrates Grok into setup commands, environment selection, documentation, and installer behavior.
- Expands tests, though binary-only Grok detection remains untested.
Tip
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.
Reviewed changes
Copilot reviewed 20 out of 20 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.surface |
Adds the Grok setup command. |
AGENTS.md |
Documents Grok architecture. |
README.md |
Documents Grok setup and selection. |
scripts/install.ps1 |
Documents the Grok selector. |
scripts/install.sh |
Documents the Grok selector. |
tests/e2e/installer.bats |
Extends installer compatibility coverage. |
internal/harness/grok.go |
Implements Grok detection and health checks. |
internal/harness/grok_test.go |
Tests Grok home and skill health. |
internal/harness/agent.go |
Updates agent metadata examples. |
internal/harness/agent_test.go |
Verifies Grok registration. |
internal/cmd/setup_agent.go |
Implements hey setup grok. |
internal/cmd/setup_agents.go |
Adds Grok selection and aggregation. |
internal/cmd/setup_agents_test.go |
Tests Grok setup and removal flows. |
internal/cmd/setup_agents_remove.go |
Removes managed Grok skills. |
internal/cmd/setup_test.go |
Covers Grok command registration. |
internal/cmd/skill_install.go |
Installs skills for detected Grok. |
internal/cmd/skill_install_test.go |
Tests Grok ownership protections. |
internal/cmd/skill_refresh.go |
Refreshes Grok skill copies. |
internal/cmd/skill_refresh_test.go |
Tests Grok refresh behavior. |
internal/cmd/help_topics.go |
Documents Grok environment selection. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Grok Build can now be connected the same way Codex is.
hey setup grokcopies the HEY skill into$GROK_HOME/skills/hey(default~/.grok/skills/hey).hey skill install, skill refresh,hey doctor,hey setup agents, and--removeall include Grok. There is no Grok plugin. Health is skill presence only, matching Codex.HEY_SETUP_AGENTacceptsgrok.allattempts every registered agent, including Grok. Detection uses$GROK_HOMEor~/.grok, or agrokbinary on PATH, matching Codex.Session-settled decisions carried from planning: Grok matches Codex, not Claude — no
.grok-plugin(user-directed); detect via$GROK_HOME/~/.grokor the grok binary (user-approved);HEY_SETUP_AGENT=alluses AllAgents() so Grok is always attempted (user-approved).Tests
make testpassed locally (go test ./internal/...)Summary by cubic
Adds Grok Build as a first-class coding agent, connected the same way Codex is.
hey setup grokcopies the HEY skill into$GROK_HOME/skills/hey(default~/.grok/skills/hey).hey skill install, skill refresh,hey doctor,hey setup agents, and--removeall include Grok.HEY_SETUP_AGENTacceptsgrok, andallnow attempts every registered agent including Grok.$GROK_HOME/~/.grokor agrokbinary on PATH.Written for commit b5b1102. Summary will update on new commits.