chore(ci): repin own-CI reviewer to claude-sonnet-4-6 (cost) - #82
Merged
Conversation
Partial walk-back of the PR #79 'fable throughout' directive, scoped to review.yml only. Per run, Fable 5 reviews at above-Opus pricing with a ~30% tokenizer surcharge and always-on thinking; the repo's own history shows Sonnet at ~70-90s/PR with modest spend vs a single 5m37s Fable run that plausibly drained the CI key's credits. Sonnet 4.6 built the PR #64-#78 verdict-gate track record - proven at this gate. Scaffolded template defaults (model: fable) deliberately unchanged: those bill the end user's own plan/key. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
tigers1997
added a commit
that referenced
this pull request
Jun 12, 2026
…p with #82) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
5 tasks
tigers1997
added a commit
that referenced
this pull request
Jun 18, 2026
…tchet + Fable 5 default (#83) Bundle release of PRs #59–#82, spanning 2026-05-24 → 2026-06-17. Key tracks: - AGPL-3.0 relicense with the discipline-skills MIT carve-out - discipline-skills module (curated 7-skill fork of obra/superpowers) - Contributor-governance ratchet (branch protection, CLA, AI-review verdict gate) - Adjacent-project dogfood cascade (hook-group union merge, persona-drift [ NOTICE ], --modules +x/-y, retrofit-byproduct gitignore, manifest written_by_sha + --whats-new) - Doc-cleared hook-payload batch + block-dangerous-bash $HOME hardening - CC 2.1.155–2.1.175 compat surveys (incl. the agent schema-status correction; subagent-nesting wording held on docs contradiction — follow-up PR resolves once 2.1.181 reconciliation lands in repo) - Fable 5 as the default model throughout (scaffolded settings, intake field, test-runner agent; own-CI reviewer briefly pinned then walked back to claude-sonnet-4-6 on cost in #82) Also restores the [2.6.0] section header accidentally deleted in PR #67 merge (it had folded 2.6.0 entries into Unreleased, over-reporting in --whats-new) and adapts test-whats-new.sh case C to the empty-Unreleased state a fresh release produces. CC_VERSION 2.6.0 → 2.7.0. Claude Code compat 2.1.116–2.1.150. See CHANGELOG.md § [2.7.0] for the full bullet detail. Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
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.
Summary
One-line model repin for the CI review gate, per maintainer decision (2026-06-12):
review.yml's--model claude-fable-5→claude-sonnet-4-6. Partial walk-back of the PR #79 "fable throughout" directive, scoped to own-CI only — scaffolded template defaults ("model": "fable", test-runner agent) are deliberately unchanged, since those bill the end user's own plan/key and the// model: "sonnet"downgrade stub is already documented insettings.local.json.example.Why
is_error/$0 (credit exhaustion suspected — see PR chore(compat): CC 2.1.172-2.1.175 survey - agent key unblocked; nesting wording held on docs contradiction #81 thread).issue_comment-triggered runs execute the workflow from the default branch.Files
.github/workflows/review.yml— the pin (one line).config_schema.py— compat comment records the Fable pin window (2026-06-09 → 2026-06-12) and the rationale.CHANGELOG.md— Unreleased entry.Verification
python3 configure.py --check✓ (imports the editedconfig_schema.py; 12 modules, 84 template files).CI expectations on this PR
This PR modifies
review.yml, so GitHub's workflow-validation safety blocks the action itself — expectai-reviewto either graceful-skip or hard-fail on OIDC (both previously observed; the dead API key makes a real review impossible right now anyway).verdict-gate's self-bootstrap escape detects the workflow file in the diff and soft-passes by design (PR #79 precedent), so the required set (check+verdict-gate+license/cla) can go green and this is mergeable without a working reviewer.🤖 Generated with Claude Code