Repository navigation
Make /ronin say ronin - #14
Merged
Merged
Conversation
drrius
marked this pull request as ready for review
August 14, 2026 21:23
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.
Claude Code exposed the old Poteto name in
/ronin's picker. The command moved. Its metadata did not.This replaces the public description, renames the shared worker prompt to
ronin-agent, and rejects Poteto branding in skill metadata. Upstream provenance keeps its original names.Existing copied installs will not update themselves. Reinstall ronin after this merges.
Verify:
bun run checkdist/ronin/skills/ronin/SKILL.mdNote
Low Risk
Documentation, metadata, persona rename, and test/verify updates only; no runtime auth or data-path changes.
Overview
Fixes Claude Code’s
/roninpicker showing the old Poteto name by aligning public skill metadata and worker prompts with ronin branding.The main
roninskill description is rewritten so it no longer references Poteto.ronin-coreandHOST_CONTRACT.mdpoint delegates atpersonas/ronin-agent.md(renamed frompoteto-agent.md) with updated frontmatter and prose.skills/ronin/SKILL.md,references/plan.md, and test fixtures (delegation.json,routing.json) follow the same naming.scripts/verify.mjsnow fails if any skillnameordescriptionstill matchespoteto/potato, and the persona routing check targetsronin-agent.md.orch.test.tsexpects branch names likeronin/u1instead ofpoteto/u1.Reviewed by Cursor Bugbot for commit 68707fd. Bugbot is set up for automated code reviews on this repo. Configure here.