Summary
./scripts/test-regression.sh fails at the coverage gate on main (4589b90): the subagentRunnerHandoff wrapper methods in cmd/harnessd/runtime_container.go:121-161 (StartRun, GetRun, Subscribe, CancelRun, RunPrompt, RunPromptWithAllowedTools, SteerRun, ParentRunID) have 0.0% coverage. Introduced by #795.
Impact
The merge gate is red on main; all branches fail verify-and-merge's pre-merge regression run.
Expected vs actual
Expected: gate green on main (no zero-coverage functions). Actual: 8 zero-coverage functions.
Proposed fix direction
Unit test in cmd/harnessd/runtime_container_test.go exercising all 8 wrappers against a real *harness.Runner built with internal/fakeprovider, asserting delegation behavior (including ParentRunID with/without a ParentContextHandoff).
Test plan / regression coverage
The new test itself is the regression coverage; gate must pass.
Related docs/log entries
Detected while running the tier-1 merge gate for the harness code-review fixes (issues #785-#787).
Summary
./scripts/test-regression.shfails at the coverage gate on main (4589b90): thesubagentRunnerHandoffwrapper methods incmd/harnessd/runtime_container.go:121-161(StartRun,GetRun,Subscribe,CancelRun,RunPrompt,RunPromptWithAllowedTools,SteerRun,ParentRunID) have 0.0% coverage. Introduced by #795.Impact
The merge gate is red on main; all branches fail
verify-and-merge's pre-merge regression run.Expected vs actual
Expected: gate green on main (no zero-coverage functions). Actual: 8 zero-coverage functions.
Proposed fix direction
Unit test in
cmd/harnessd/runtime_container_test.goexercising all 8 wrappers against a real*harness.Runnerbuilt withinternal/fakeprovider, asserting delegation behavior (includingParentRunIDwith/without aParentContextHandoff).Test plan / regression coverage
The new test itself is the regression coverage; gate must pass.
Related docs/log entries
Detected while running the tier-1 merge gate for the harness code-review fixes (issues #785-#787).