Skip to content

feat(harness): route code and fix to sonnet by default (POC) - #877

Closed
guyoron1 wants to merge 2 commits into
fullsend-ai:mainfrom
guyoron1:feat/cheaper-harness-defaults
Closed

feat(harness): route code and fix to sonnet by default (POC)#877
guyoron1 wants to merge 2 commits into
fullsend-ai:mainfrom
guyoron1:feat/cheaper-harness-defaults

Conversation

@guyoron1

Copy link
Copy Markdown

Draft / POC for the 2026-08-19 contributors meeting — not ready for review.

Cost win: code → sonnet, fix → sonnet (harness/*.yaml + agents/*.md, 8 lines). Triage and review keep opus.

Why: triage needs opus for planning judgment, review needs opus to catch real issues rather than rubber-stamp diffs — those two stay put. Code and fix are implementation-following-a-plan work, where sonnet is the cheaper tier without giving up the judgment calls that matter. See fullsend ADR 0090 (draft PR in fullsend core) for the full cost-aware routing plan; this is its "now" step.

hack/lint-agent-docs passes. Open question for the meeting: whether code/fix on sonnet holds implementation quality — worth an A/B on the next 30 issues before un-drafting.

July-12 30-issue benchmark on the full pipeline: triage and review are
where Opus is least needed (triage 52.6% cheaper with RTK+Ponytail alone;
review is diff-bound). Code and fix keep opus. See fullsend ADR 0090.

Signed-off-by: guy oron <goron@redhat.com>
…t code/fix

Previous commit inverted the routing: triage needs opus for planning
judgment, review needs opus to catch real issues, not just diff-bound
mechanical checks. Code and fix are the phases with headroom — sonnet
is cheaper implementation without the planning/judgment load.

triage: haiku -> opus
review: sonnet -> opus
code:   opus -> sonnet
fix:    opus -> sonnet

Signed-off-by: guy oron <goron@redhat.com>
@github-actions

Copy link
Copy Markdown

Functional tests did not run

Functional tests run automatically for org/repo members and collaborators on pull requests.

For other contributors, a maintainer must add the ok-to-test label after the latest push.

@guyoron1

guyoron1 commented Sep 1, 2026

Copy link
Copy Markdown
Author

Closing this stale POC draft — superseded by the split-out review-stack PRs (#1004#1009 line). Will re-raise anything still relevant as a fresh PR against current main.

@guyoron1 guyoron1 closed this Sep 1, 2026
@fullsend-ai-retro

fullsend-ai-retro Bot commented Sep 1, 2026

Copy link
Copy Markdown

🤖 Finished Retro · ❌ Failure (creating sandbox: sandbox creation failed after 3 attempts: sandbox create failed: exit status 1 (output: Error: × code: 'The system is not in a state required for the operation's │ execution', messag…) · Started 11:49 AM UTC · Completed 11:49 AM UTC

Commit: 2aeb6db · View workflow run →

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