Install
$ agentstack add skill-sfourdrinier-grok-skills-review ✓ scanned · ✓ verified — works with Claude Code, Cursor, and more.
Security review
✓ PassedNo issues found. Passed automated security review. · v0.1.0 How review works →
- ✓ Prompt-injection patterns
- ✓ Secret / credential exfiltration
- ✓ Dangerous shell & filesystem operations
- ✓ Untrusted network calls
- ✓ Known-malicious package signatures
What it can access
- ✓ Network access No
- ✓ Filesystem access No
- ✓ Shell / process execution No
- ✓ Environment & secrets No
- ✓ Dynamic code execution No
From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.
About
How to run (transparent)
- Take the absolute Base directory for this skill from the Skill tool
(the folder that contains this skill's SKILL.md and run.mjs).
- Set
SKILL_BASEto that path. Do not invent versioned cache paths. - Always invoke the companion only through this skill's runner:
SKILL_BASE=''
# Required for completion notifications (see plugin/references/execution-context.md):
export GROK_COMPANION_EXECUTION_CONTEXT=foreground # or background
node "$SKILL_BASE/run.mjs" [args...]
run.mjs finds the plugin install from its own location and runs scripts/grok-companion.mjs. No CLAUDE_PLUGIN_ROOT / PLUGIN_ROOT required.
If the host already exported CLAUDE_PLUGIN_ROOT or PLUGIN_ROOT, you may call node "$CLAUDE_PLUGIN_ROOT/scripts/grok-companion.mjs" instead; prefer "$SKILL_BASE/run.mjs" whenever the Skill tool loaded this skill.
Return companion stdout verbatim. Never put free-text in --task "..."; use --task-file - with a single-quoted heredoc.
Run a Grok review through the hardened wrapper and relay its result envelope.
Raw slash-command arguments: $ARGUMENTS
Core constraint:
- This command is review-only. Do not fix issues, apply patches, or suggest you
are about to make changes. Your only job is to run the review and return the wrapper's stdout envelope VERBATIM.
Required wrapper flags (copy exactly, substitute only placeholder values):
--targetdefaults to.(repo root / working tree) when omitted.--baseframes a branch review against that base (comparison text
only). It does not force worktree isolation.
--isolated(opt-in) runs the review in an owned external worktree at HEAD
with tracked dirty applied. Use when you need a clean snapshot without live checkout noise. Setup failures fail closed as isolation-unavailable (no silent fallback to the live tree). Default is live checkout.
- Exactly one of
--taskor--task-fileis required. Prefer
--task-file for anything beyond a short one-line prompt.
- Preserve the user's arguments exactly. Do not strip, add, or reorder flags.
Do not invent a flag that is not in the argument-hint.
- Shell-injection safety for
--task: the task is free text you must NEVER
place in a shell-evaluated position. $(...)/backticks inside a double-quoted --task "..." run locally BEFORE the wrapper validates them. When the arguments carry a --task , deliver that text on STDIN with --task-file - and a SINGLE-QUOTED heredoc so the shell passes it byte-for-byte; the companion stages it into a temp file for the wrapper.
- Shell-injection safety for flag VALUES (
--target, a--task-file,
--schema, --model, --timeout, --max-turns, and EVERY other value you substitute from $ARGUMENTS): wrap each substituted value in SINGLE quotes, for example --target ''. Single quotes stop the shell from evaluating $(...)/backticks, so a hostile value reaches the companion as one literal argv token and the wrapper validates it (target path resolution + escape guards). An unquoted OR double-quoted value would be command-substituted locally BEFORE the wrapper ever sees it -- the same injection class as an unsafe --task "...". The bare --web flag carries no value to quote.
--web passthrough:
- Web tools are OFF by default (review determinism). Pass
--webonly when the
review genuinely depends on current external practices, current library or software versions, or living external documentation the repo's own rules and code cannot answer (for example "does this match the latest stable API for library X"). Do not add --web otherwise.
Execution mode (foreground vs background):
- If the raw arguments include
--wait, run in the foreground (do not ask). - If the raw arguments include
--background, run in a Claude background task
(do not ask).
--waitand--backgroundare Claude Code execution flags. Do NOT forward
them to the companion or the wrapper; strip them from the wrapper argv.
- Otherwise, estimate the review size first:
- Run
git status --short --untracked-files=all. - Run
git diff --shortstatfor the working tree. - Recommend foreground only when the change is clearly tiny (roughly 1-2
files). In every other case, including unclear size, recommend background.
- Then use
AskUserQuestionexactly once, with the recommended option first and
its label suffixed with (Recommended). The two options are:
Wait for resultsRun in background
Foreground flow (one Bash call, then relay verbatim). When the arguments carry a --task , route that text through STDIN so it is never shell-evaluated:
export GROK_COMPANION_EXECUTION_CONTEXT=foreground
node "$SKILL_BASE/run.mjs" review --target '' [other non-task flags from $ARGUMENTS, each substituted value single-quoted] --task-file -
GROK_TASK
When the arguments already use --task-file , drop the heredoc and pass every flag as single-quoted argv tokens:
export GROK_COMPANION_EXECUTION_CONTEXT=foreground
node "$SKILL_BASE/run.mjs" review --target '' --task-file '' [other non-task flags from $ARGUMENTS, each substituted value single-quoted]
- Return the command stdout envelope VERBATIM. Do not paraphrase, summarize, or
add commentary before or after it. Preserve the exit status. Do not fix any issue the review reports.
Background flow:
- Set
export GROK_COMPANION_EXECUTION_CONTEXT=background(canonical pattern in
plugin/references/execution-context.md).
- Launch the same command with
Bash(run_in_background: true). - Do not wait for completion or read its output this turn.
- Tell the user: "Grok review started in the background. Run `/grok:status
--run-id to read the result envelope (the run id is printed when the run finishes). If notifications are enabled (setup --notification-mode auto`), a completion signal may also fire when the job ends."
Foreground flow must set export GROK_COMPANION_EXECUTION_CONTEXT=foreground before node "$SKILL_BASE/run.mjs".
If the companion prints an actionable "could not locate the Grok wrapper" message instead of an envelope, tell the user to run /grok:setup.
Source & license
This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.
- Author: sfourdrinier
- Source: sfourdrinier/grok-skills
- License: MIT
Install and usage instructions live in the source repository linked above.
Reviews
No reviews yet — be the first.
Write a review
Versions
- v0.1.0 Imported from the upstream source.