Skip to content

fix(agents): correct prompt hook docs, split modules, and fix install script#1384

Merged
joshua-mo-143 merged 2 commits into0xPlaygrounds:mainfrom
illegalcall:add-rig-skill
Feb 16, 2026
Merged

fix(agents): correct prompt hook docs, split modules, and fix install script#1384
joshua-mo-143 merged 2 commits into0xPlaygrounds:mainfrom
illegalcall:add-rig-skill

Conversation

@illegalcall
Copy link
Copy Markdown
Contributor

@illegalcall illegalcall commented Feb 12, 2026

Summary

  • Split rig-quality-gates module into rig-prompt-hooks (hook API docs) + rig-quality-gates (coding style & quality standards), fixing prompt hook content leaking into unrelated adapter files (e.g. vector-store)
  • Fixed incorrect API references in AGENTS.md: CancelSignal::cancel()ToolCallHookAction::Terminate/HookAction::Terminate, removed fictional dual-trait (PromptHook + StreamingPromptHook) section, added #[cfg] attributes to WasmCompat traits, expanded Capabilities example with Capable<T>/Nothing
  • Fixed install-rig-skill.sh: install_gemini now appends instead of overwriting, SKILL_SOURCE detection no longer changes cwd
  • Fixed grep pattern in rig-migrate skill and Self::NAME usage in tools.md
  • Set executable bit (100755) on all three shell scripts — fixes CI "Permission denied" on sync_agent_instruction_files.sh
  • Regenerated all 27 adapter files via sync script; validation passes

Test plan

  • scripts/sync_agent_instruction_files.sh runs without errors
  • scripts/validate_agent_instruction_files.sh confirms all adapters in sync
  • Spot-check Claude Code skills (/rig-agent-scaffold, /rig-provider-checklist, /rig-review-gates)
  • Verify install-rig-skill.sh --agent gemini appends correctly on a clean target

Fixes #1343

@illegalcall illegalcall changed the title fix(agents): correct prompt hook docs, split modules, and fix install… fix(agents): correct prompt hook docs, split modules, and fix install script Feb 12, 2026
Copy link
Copy Markdown
Contributor

@joshua-mo-143 joshua-mo-143 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks mostly OK, see comments

Comment thread scripts/install-rig-skill.sh
Comment thread .claude/skills/rig-agent-scaffold/SKILL.md Outdated
Comment thread scripts/sync_agent_instruction_files.sh
Comment thread .claude/commands/rig-agent-scaffold.md
Copy link
Copy Markdown
Contributor

@joshua-mo-143 joshua-mo-143 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I think we will need to add some documentation on the docsite to match this.

@joshua-mo-143 joshua-mo-143 added the ready to merge PR is ready to merge. label Feb 16, 2026
@joshua-mo-143 joshua-mo-143 added this pull request to the merge queue Feb 16, 2026
Merged via the queue into 0xPlaygrounds:main with commit 58968e8 Feb 16, 2026
6 checks passed
@github-actions github-actions Bot mentioned this pull request Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: Rig skill

2 participants