feat: require harness install hook #968

Closed
coilyco-ops wants to merge 1 commit from issue-956 into main
Member

closes #956

closes #956
feat: require harness install hook
All checks were successful
test / test (pull_request) Successful in 26s
ecbc312dd3
Author
Member

Blocking this PR from the director lane.

This made the agents API interface require an Install hook, but the missing harness implementations are still effectively optional no-ops. That is not the intended behavior. The feature request was to make install behavior part of the agents API contract, so a harness that cannot install itself should fail explicitly or report unsupported behavior through a real contract, not satisfy the interface with an empty method.

Please do not merge this shape. A corrective issue will carry the behavior requirement explicitly.

Blocking this PR from the director lane. This made the agents API interface require an `Install` hook, but the missing harness implementations are still effectively optional no-ops. That is not the intended behavior. The feature request was to make install behavior part of the agents API contract, so a harness that cannot install itself should fail explicitly or report unsupported behavior through a real contract, not satisfy the interface with an empty method. Please do not merge this shape. A corrective issue will carry the behavior requirement explicitly.
coilyco-ops closed this pull request 2026-07-10 08:31:01 +00:00
All checks were successful
test / test (pull_request) Successful in 26s

Pull request closed

Sign in to join this conversation.
No description provided.