feat(skills): install grill-me for both CLIs alongside the speckit set - #30
Merged
Merged
Conversation
Install the third-party grill-me skill (mattpocock/skills) for both Claude and Codex everywhere the kit installs skills. The step lives in install.sh.tpl, so it is baked into the runtime-home render for runners and runs on workstation installs (install-agents.sh delegates to install.sh). The skill executes inside agent context, so it is pinned to a reviewed upstream ref (mattpocock/skills#v1.1.0, commit d574778f) and the skills CLI is version-pinned (skills@1.5.16); bumping either is a renovate/review event. The install is idempotent (the skills CLI converges on re-run) and loud-skips with a doctor warning when npx or the network is unavailable rather than failing the whole install, matching the MCP fleet fallback posture. The skills CLI's Claude agent name is claude-code (not claude); the Claude branch passes claude-code while the installer keeps its own INSTALL_CLAUDE gating and CLAUDE_HOME/SKILLS_DIR variable names. Registers grill-me in manifest.yaml (skill entry + provenance pin) and adds a render-agent-kit.py --doctor check so --check/--doctor cover it. Records the pin in skills-source.lock, THIRD_PARTY_NOTICES.md, and renovate.json.
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
1 participant
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.
Closes #27.
Makes the grill-me skill part of the standard skill set installed for both agent CLIs, wherever the kit installs skills — the runtime-home render (so runner images and boot-time overlays get it) and install-agents.sh for workstations (which delegates to the same install.sh, mirroring how council and speckit are wired).
install_grill_meruns for both CLIs, gated on INSTALL_CLAUDE/INSTALL_CODEX, invokingnpx --yes skills@1.5.16 add mattpocock/skills#v1.1.0 --skill=grill-me --agent <agent> --yes --global. The skills CLI's registry key for Claude isclaude-code(notclaude); passing the wrong name would have made the CLI reject it and silently skip — cross-provider review caught this, and the Claude call now passesclaude-codewhile the installer keeps its own INSTALL_CLAUDE/CLAUDE_HOME names.Verified: render --check and --doctor (6 ok/0 warn/0 fail), validate_manifest passes, bash -n clean, 18 manifest tests pass. Reviewed cross-provider (codex): one BLOCK finding (the claude-code agent name) fixed and re-verified. Rebased onto main after the manifest-sha fix (#28).