feat(sdk/python): expose provider-agnostic harness variants - #1024
feat(sdk/python): expose provider-agnostic harness variants#1024ddbaron wants to merge 1 commit into
Conversation
510351d to
39e9014
Compare
santoshkumarradha
left a comment
There was a problem hiding this comment.
Focused Python harness tests passed locally for me, and the public variant plumbing looks clean. I also approved the fork-gated workflows so the full matrix is running now.
Performance
✓ No regressions detected |
📊 Coverage gateThresholds from
✅ Gate passedNo surface regressed past the allowed threshold and the aggregate stayed above the floor. |
📐 Patch coverage gateThreshold: 80% on lines this PR touches vs
✅ Patch gate passedEvery surface whose lines were touched by this PR has patch coverage at or above the threshold. |
|
This still looks good from my side, but |
|
@ddbaron thanks for this ! @AbirAbbas lets enhance this with tsx and go support as well. |
39e9014 to
fc015da
Compare
Summary
varianttoHarnessConfig,HarnessRunner.run, andAgent.harness.#variantmodel suffix where the selected provider supports variants.This change is independently applicable to
main. The OpenCode configuration and recursion-isolation work is kept in the related OpenCode pull request rather than made a prerequisite here.Validation
uv run --extra dev pytest -q tests/test_harness_agent_wiring.py tests/test_harness_runner.py tests/test_harness_types.pygit diff --checkThe focused tests pass against the current
mainbranch.Related to the original proposal: #960
Related to, but independently applicable from, the OpenCode change: #1023
The fork preview remains available at ddbaron#4.