Commit 41520bc
Bump skill turn caps after hitting review limit on v0.24.0 (#789)
PR #788 (Update stacklok/toolhive to v0.24.0) failed the run
when skill_review hit the 30-turn cap mid-edit. Details from run
24786157448:
- skill_gen: 67 turns / $4.72 (under 500 cap) -- succeeded,
committed `42267da Document local vMCP CLI mode`
- skill_review: 31 turns / $1.86 -- exited with subtype
`error_max_turns`, cascading workflow to `failure`, which
skipped autofix and PR body augmentation
My initial cap of 30 for review was sized against silent/no-
changes releases (4-6 turns baseline). For a real multi-file
content review the editorial pass walks each edited file and
makes tightening edits; ~30-100 turns is legitimate working
range.
Changes:
- skill_review: 30 -> 200 (2x-6x headroom over working range)
- skill_gen: 500 -> 1000 (defensive doubling; 500 was
never hit in production, but 1000 keeps us well clear of
the 397-turn v3-test anomaly)
Hitting either cap still produces a loud failure; raise
deliberately if a release genuinely needs more.
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>1 parent bafb24a commit 41520bc
1 file changed
Lines changed: 18 additions & 10 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
548 | 548 | | |
549 | 549 | | |
550 | 550 | | |
551 | | - | |
552 | | - | |
553 | | - | |
554 | | - | |
555 | | - | |
| 551 | + | |
| 552 | + | |
| 553 | + | |
| 554 | + | |
| 555 | + | |
| 556 | + | |
| 557 | + | |
556 | 558 | | |
557 | 559 | | |
558 | | - | |
| 560 | + | |
559 | 561 | | |
560 | 562 | | |
561 | 563 | | |
| |||
738 | 740 | | |
739 | 741 | | |
740 | 742 | | |
741 | | - | |
742 | | - | |
743 | | - | |
| 743 | + | |
| 744 | + | |
| 745 | + | |
| 746 | + | |
| 747 | + | |
| 748 | + | |
| 749 | + | |
| 750 | + | |
| 751 | + | |
744 | 752 | | |
745 | 753 | | |
746 | | - | |
| 754 | + | |
747 | 755 | | |
748 | 756 | | |
749 | 757 | | |
| |||
0 commit comments