Skip to content

feat(cli): --llm-provider default follows --harness — --generate is not always claude#99

Merged
krimvp merged 1 commit into
mainfrom
claude/harness-generation-config-ymxsxb
Jul 9, 2026
Merged

feat(cli): --llm-provider default follows --harness — --generate is not always claude#99
krimvp merged 1 commit into
mainfrom
claude/harness-generation-config-ymxsxb

Conversation

@krimvp

@krimvp krimvp commented Jul 9, 2026

Copy link
Copy Markdown
Owner

The compiler that authors a --generate bar (and the judge/approver) ran on
--llm-provider, which defaulted to 'claude' unconditionally: picking
--harness codex still required the claude CLI on PATH just to generate the
verification. The default now FOLLOWS the harness (defaultLlmProvider:
claude/codex/droid/pi -> the same CLI, goaly-code -> openai on its required
endpoint, the test-only fake -> claude), so one installed tool is enough; an
explicit --llm-provider (CLI flag or config file) still overrides.

A --resume that adopts the resumed run's recorded harness re-derives a
non-explicit provider from the adopted harness, so the preflight and the LLM
steps track the harness the run will actually use. The independence warnings
already surface the worker/judge/approver model collapse this default can
create, unchanged.

Co-Authored-By: Claude Fable 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_017wrUw8x68sdrFGuFSqMZ4G

…ot always claude

The compiler that authors a --generate bar (and the judge/approver) ran on
--llm-provider, which defaulted to 'claude' unconditionally: picking
--harness codex still required the claude CLI on PATH just to generate the
verification. The default now FOLLOWS the harness (defaultLlmProvider:
claude/codex/droid/pi -> the same CLI, goaly-code -> openai on its required
endpoint, the test-only fake -> claude), so one installed tool is enough; an
explicit --llm-provider (CLI flag or config file) still overrides.

A --resume that adopts the resumed run's recorded harness re-derives a
non-explicit provider from the adopted harness, so the preflight and the LLM
steps track the harness the run will actually use. The independence warnings
already surface the worker/judge/approver model collapse this default can
create, unchanged.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017wrUw8x68sdrFGuFSqMZ4G
@krimvp
krimvp enabled auto-merge July 9, 2026 13:30
@krimvp
krimvp merged commit 608d4a3 into main Jul 9, 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.

2 participants