Skip to content

Relocate reviewer-pinning out of the shared model-tiers block (v2.24.1) - #5

Merged
codxse merged 1 commit into
masterfrom
refactor/model-tiers-reviewer-pin-relocation
Jul 20, 2026
Merged

Relocate reviewer-pinning out of the shared model-tiers block (v2.24.1)#5
codxse merged 1 commit into
masterfrom
refactor/model-tiers-reviewer-pin-relocation

Conversation

@codxse

@codxse codxse commented Jul 20, 2026

Copy link
Copy Markdown
Owner

Problem

shared/model-tiers.md bundled two things with different audiences and inlined the whole bundle into all five skills:

  • Tier classification — needed by every skill (each classifies its own model), neutral voice. ✓
  • Reviewer pinning by host — consumed only by /evaluate, written as /evaluate ("/evaluate's request-changes path must…").

So /case, /refine, /solve, and /orchestrate each carried ~30 lines of /evaluate-voiced pinning instructions they never reference — dead weight addressed to the wrong skill.

Change

  • shared/model-tiers.md → Tier classification only. Still synced into all five skills by model-tiers-sync.sh.
  • Reviewer pinning by host moved into evaluate/SKILL.md as native prose — its sole consumer, and its step 4b.1 already points at it.
  • Dropped the redundant host table in favor of the capability-keyed branch list it duplicated (aligns with CLAUDE.md's "keys off host capability, not a host list").
  • Fixed the now-stale "in the Model Tiers section above" cross-reference.

No behavior change — the pinning logic is identical, just relocated and de-duplicated. Intra-/evaluate redundancy between the native map and step 4b.1's operational detail was reviewed and intentionally left (local redundancy on the execution hard path).

Bumps

Plugin/marketplace 2.24.0 → 2.24.1 (all four manifests); /case 2.10.1, /refine 1.9.1, /orchestrate 1.5.1, /solve 1.7.1, /evaluate 1.14.1.

Verification

  • model-tiers-sync.sh + rubrics-sync.sh: no drift
  • All four manifests: valid JSON
  • model-guard.sh not run — no Model Guard prose touched

🤖 Generated with Claude Code

The shared model-tiers map bundled two things with different audiences and
inlined the whole bundle into all five skills: Tier classification (needed by
every skill, which each classify their own model) and Reviewer pinning by host
(consumed only by /evaluate). That put ~30 lines of /evaluate-voiced pinning
instructions into /case, /refine, /solve, and /orchestrate — dead weight
addressed to the wrong skill.

Shrink shared/model-tiers.md to Tier classification only (still synced into all
five by model-tiers-sync.sh). Move Reviewer pinning by host into
evaluate/SKILL.md as native prose — its sole consumer, and its step 4b.1 already
points at it — and drop the redundant host table in favor of the capability-keyed
branch list it duplicated. Fix the now-stale "in the Model Tiers section above"
cross-reference. No behavior change: the pinning logic is identical, just
relocated and de-duplicated.

Bumps: plugin/marketplace 2.24.0 -> 2.24.1 (all four manifests); /case 2.10.1,
/refine 1.9.1, /orchestrate 1.5.1, /solve 1.7.1, /evaluate 1.14.1.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@codxse
codxse merged commit ab29bdd into master Jul 20, 2026
2 checks passed
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