From abe10a1b885cdeb970f268c4d7053841c0071719 Mon Sep 17 00:00:00 2001 From: seal Date: Wed, 12 Aug 2026 16:24:56 -0400 Subject: [PATCH] fix(runner): repair errorResult callsite skew breaking main build MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The DeliverControl dispatch arm at dispatch.go:470 calls a bare `errorResult(id, err)`, but the only definition is the method `d.errorResult(ctx, id, err)` (:488) — so `go build ./...` fails module-wide with `undefined: errorResult`, and main does not compile. This is a semantic merge-skew between two individually-green PRs: #288 added the DeliverControl arm calling `d.errorResult(ctx, id, err)`, and #286 (landing on top, touching the same file) predated that arm, so neither PR's CI compiled the combined tree. The textual auto-merge succeeded but left the callsite in the pre-refactor bare form. Correcting :470 to `d.errorResult(ctx, id, err)` (ctx is in scope in the arm) restores the module build; it is the sole bare callsite (the other seven already use the method). Spec-impact: none. Co-authored-by: Matt Wilkinson --- go/internal/runner/dispatch.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/go/internal/runner/dispatch.go b/go/internal/runner/dispatch.go index 2bf6088a..be3d816d 100644 --- a/go/internal/runner/dispatch.go +++ b/go/internal/runner/dispatch.go @@ -467,7 +467,7 @@ func (d *dispatcher) execute(ctx context.Context, id string, cmd *compassv1inter // unadvanced for the D2 reconnect sweep to redeliver (errSessionUnknown // maps to NOT_FOUND via errorResult). if err := d.host.Deliver(ctx, c.DeliverControl.GetSessionId(), c.DeliverControl.GetOp()); err != nil { - return errorResult(id, err) + return d.errorResult(ctx, id, err) } return nil default: