Skip to content

Grok Build: supported after live CLI smoke; ask only in prompting modes - #1

Merged
tristanroth1 merged 2 commits into
mainfrom
grok-live-smoke
Oct 8, 2026
Merged

tristanroth1 merged 2 commits into
mainfrom
grok-live-smoke

Conversation

@tristanroth1

Copy link
Copy Markdown
Contributor

Live Grok 1.0.46 smoke in an isolated GROK_HOME/HOME with only the clickfix-guard hooks (compat.claude/cursor hooks off):

  • allow: plain curl api | head runs, pre and post hooks fire silently
  • block: curl ... | bash denied with the native deny shape (bypass and default modes)
  • tracker: curl -o setup.sh then bash setup.sh denied
  • allowlisted installer: under always-approve the guard returned ask and Grok ran the command without a prompt. Fixed: Grok asks only in default/acceptEdits/plan, denies otherwise. Re-tested live: denied. Unit tests added (186 pass).

README: Grok drops "experimental", tested-version row, allowlist and ask mode wording say where Grok can ask.

Owner: founder
Next: independent review, then merge

🤖 Generated with Claude Code

…ow, tracker)

Generated with founder-ops coding agent (claude)
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…k ask)

Live Grok 1.0.46 smoke: an allowlisted installer got permissionDecision=ask
under always-approve and Grok ran it without a prompt. supports_ask now
requires default/acceptEdits/plan for Grok; tests cover both modes.

Generated with founder-ops coding agent (claude)
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@tristanroth1

Copy link
Copy Markdown
Contributor Author

Review: author claude (founder-ops coding agent), reviewer Codex (codex exec, read-only), 2026-10-08. Round 1 FIX-FIRST (Grok ask path unverified) -> live-tested, fixed in d1f7cd3. Round 2: SHIP.

@tristanroth1
tristanroth1 merged commit ac1a615 into main Oct 8, 2026
4 checks passed
@tristanroth1
tristanroth1 deleted the grok-live-smoke branch October 8, 2026 09:34
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