Skip to content

Route ronin workflows by task profile - #11

Closed
drrius wants to merge 7 commits into
mainfrom
codex/task-profile-routing
Closed

drrius wants to merge 7 commits into
mainfrom
codex/task-profile-routing

Conversation

@drrius

@drrius drrius commented Aug 14, 2026 •

Copy link
Copy Markdown
Owner

ronin should trust distinct jobs and reproducible evidence, not model plurality.

This change:

  • defines explore, implement, judge, explain, and verify as behavioral task profiles
  • makes model and reasoning-effort routing optional capability, latency, cost, or tool-fit choices
  • reports fresh context, model relationship, and provider relationship independently, with honest joint provenance tiers
  • updates delegated workflows, reviewer prompts, playbooks, setup, adapters, and guides to use the profile contract
  • adds a machine-readable profile contract, installed readback, routing fixtures, semantic guards, and strict typechecking for repository tests
  • tightens the README opening and removes obsolete caveat and Cursor-coexistence sections carried from the original branch

Single-model hosts remain complete supported environments. Cross-provider review stays available as an explicit escalation, but reviewer count or model agreement never substitutes for evidence or acceptance verification.

Validation:

  • bun run check
  • 37 profile and routing tests, 127 assertions
  • 58 tooling tests, 219 assertions
  • strict repository-test and tooling typechecks
  • deterministic build and isolated-home install, reinstall, readback, and exact uninstall
  • zero-vulnerability Bun audit
  • git diff --check

Review:

  • fresh-context Codex judge: approved with no findings
  • Claude Opus 5 at xhigh effort: cross-provider approval with no findings

Note

High Risk
This changes core delegation contracts, skill names, install path, and release verification across the whole distribution; consumers and CI must adopt the new ronin-* commands and skills-CLI install model.

Overview
Ronin stops routing on models and host adapters. Delegation is defined by five task profiles (explore, implement, judge, explain, verify) in skills/ronin-core/task-profiles.json, with subagents inheriting the active model and reviews reported as self-review or fresh-context-review. HOST_CONTRACT.md and workflow skills are updated to match; docs and guides drop /setup-ronin, models.yaml, and cross-provider review framing.

Public surface is curated and namespaced. Retained skills move to the ronin-* prefix (43 skills); upstream arena, automate-me, and setup-pstack are removed. ronin-architect replaces parallel arena-style runs with explicitly named structural directions, per-direction explore workers, and a judge synthesis step.

Distribution and verification simplify. The adapters/ tree and Bun global install/uninstall paths go away; install is documented via npx skills, and build ships skills + docs only. verify.mjs adds profile/host-contract semantic checks, expects 43 skills, bans legacy model-role and “manufactured diversity” copy in key docs, and removes isolated-home install lifecycle tests from the gate.

Reviewed by Cursor Bugbot for commit 89d6305. Bugbot is set up for automated code reviews on this repo. Configure here.

drrius added 4 commits August 14, 2026 20:17
It claimed daily pstack use and no Cursor in consecutive sentences.
pstack is a Cursor plugin, so both cannot be true.
The renamed entry points already keep the two projects apart, so the
section told readers to route around a problem the names solved.
Watcher and Azure-testing detail is release-note depth, not README
depth. The upstream line moves up next to the divergence pointer.
@drrius
drrius marked this pull request as ready for review August 14, 2026 19:51

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 35be11e889

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md Outdated
```

Three commands carry their own names so you can always tell which is which. `ronin-mode`, `setup-ronin`, `ronin-review`. Upstream calls them `poteto-mode`, `setup-pstack`, `show-me-your-work`.
Three commands carry their own names. `ronin-mode`, `setup-ronin`, `ronin-review`. Upstream calls them `poteto-mode`, `setup-pstack`, `show-me-your-work`, so both sets can sit side by side and you always know which one you invoked.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Restore the Cursor coexistence warning

When Cursor's pstack plugin and a global ronin installation are both visible to Cursor, only these three entry points are renamed; PORTING.md:27 confirms that the other leaves retain their upstream names and are indistinguishable in Cursor's picker. Commands such as /how, /arena, and /swarm can therefore still collide, so claiming both sets can sit side by side while deleting the host-targeted installation guidance can cause users to invoke the wrong implementation. Restore that guidance or qualify the claim.

Useful? React with 👍 / 👎.

Comment thread skills/setup-ronin/SKILL.md Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using high effort and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 35be11e. Configure here.

Comment thread skills/ronin-reflect/references/synthesizer.md
Comment thread skills/ronin-mode/playbooks/shipping.md Outdated
@drrius

drrius commented Aug 14, 2026

Copy link
Copy Markdown
Owner Author

Superseded by a smaller product boundary. The replacement PR keeps the principles, namespaces every retained skill, and removes the adapter and model-routing layers.

@drrius drrius closed this Aug 14, 2026
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.

1 participant