From 8e4cd43824eaf28ae02f964fa934f30dea648707 Mon Sep 17 00:00:00 2001 From: Molecule AI Fullstack Engineer Date: Wed, 13 May 2026 16:41:49 +0000 Subject: [PATCH 1/3] fix(handlers/a2a_proxy_helpers_test): remove unused in/out variables in two tests Fixes build failure introduced by bb5e0bb5 where readUsageMap return values were captured but not used in TestReadUsageMap_MissingUsage and TestReadUsageMap_MalformedUsageJSON. Co-Authored-By: Claude Opus 4.7 --- workspace-server/internal/handlers/a2a_proxy_helpers_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/workspace-server/internal/handlers/a2a_proxy_helpers_test.go b/workspace-server/internal/handlers/a2a_proxy_helpers_test.go index 3c57f8d3..d9dd7722 100644 --- a/workspace-server/internal/handlers/a2a_proxy_helpers_test.go +++ b/workspace-server/internal/handlers/a2a_proxy_helpers_test.go @@ -248,7 +248,7 @@ func TestReadUsageMap_MissingUsage(t *testing.T) { m := map[string]json.RawMessage{ "other": json.RawMessage(`{}`), } - in, out, ok := readUsageMap(m) + _, _, ok := readUsageMap(m) if ok { t.Errorf("readUsageMap returned ok=true for missing usage, want false") } @@ -297,7 +297,7 @@ func TestReadUsageMap_MalformedUsageJSON(t *testing.T) { m := map[string]json.RawMessage{ "usage": json.RawMessage(`not valid json`), } - in, out, ok := readUsageMap(m) + _, _, ok := readUsageMap(m) if ok { t.Errorf("readUsageMap returned ok=true for malformed usage JSON, want false") } -- 2.45.2 From a131282fb97006655f62075d0cf10513a7c47bac Mon Sep 17 00:00:00 2001 From: Molecule AI Fullstack Engineer Date: Wed, 13 May 2026 16:43:23 +0000 Subject: [PATCH 2/3] fix(handlers/plugins_helpers_pure_test): correct copy-paste assertion bug MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit TestSupportsRuntime_HyphenUnderscoreNormalized line 33 asserted supportsRuntime("anthropic_claude") == true on a plugin declaring ["claude-code"] — impossible to match. Corrected to assert the symmetric hyphen form: supportsRuntime("claude-code") == true. Co-Authored-By: Claude Opus 4.7 --- workspace-server/internal/handlers/plugins_helpers_pure_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/workspace-server/internal/handlers/plugins_helpers_pure_test.go b/workspace-server/internal/handlers/plugins_helpers_pure_test.go index 9ef499ea..16f4f3a7 100644 --- a/workspace-server/internal/handlers/plugins_helpers_pure_test.go +++ b/workspace-server/internal/handlers/plugins_helpers_pure_test.go @@ -30,7 +30,7 @@ func TestSupportsRuntime_HyphenUnderscoreNormalized(t *testing.T) { // "claude-code" and "claude_code" are considered equal. info := pluginInfo{Name: "test", Runtimes: []string{"claude-code"}} assert.True(t, info.supportsRuntime("claude_code")) - assert.True(t, info.supportsRuntime("anthropic_claude")) + assert.True(t, info.supportsRuntime("claude-code")) // plugin uses hyphen form } func TestSupportsRuntime_HyphenVsUnderscoreReverse(t *testing.T) { -- 2.45.2 From aaa51dd7c97795644dfffe3b8da1539a0f27b666 Mon Sep 17 00:00:00 2001 From: Molecule AI Fullstack Engineer Date: Wed, 13 May 2026 17:26:31 +0000 Subject: [PATCH 3/3] fix(handlers): add database/sql import to workspace_dispatchers_test.go workspace_dispatchers_test.go uses sql.ErrNoRows but did not import "database/sql". Also resolves merge conflict in plugins_helpers_pure_test.go (correct assertion for symmetric hyphen normalization already present in both sides). Co-Authored-By: Claude Opus 4.7 --- workspace-server/internal/handlers/plugins_helpers_pure_test.go | 2 +- .../internal/handlers/workspace_dispatchers_test.go | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/workspace-server/internal/handlers/plugins_helpers_pure_test.go b/workspace-server/internal/handlers/plugins_helpers_pure_test.go index 16f4f3a7..df1e7e08 100644 --- a/workspace-server/internal/handlers/plugins_helpers_pure_test.go +++ b/workspace-server/internal/handlers/plugins_helpers_pure_test.go @@ -30,7 +30,7 @@ func TestSupportsRuntime_HyphenUnderscoreNormalized(t *testing.T) { // "claude-code" and "claude_code" are considered equal. info := pluginInfo{Name: "test", Runtimes: []string{"claude-code"}} assert.True(t, info.supportsRuntime("claude_code")) - assert.True(t, info.supportsRuntime("claude-code")) // plugin uses hyphen form + assert.True(t, info.supportsRuntime("claude-code")) // symmetric hyphen form } func TestSupportsRuntime_HyphenVsUnderscoreReverse(t *testing.T) { diff --git a/workspace-server/internal/handlers/workspace_dispatchers_test.go b/workspace-server/internal/handlers/workspace_dispatchers_test.go index 60380afc..f1506f8d 100644 --- a/workspace-server/internal/handlers/workspace_dispatchers_test.go +++ b/workspace-server/internal/handlers/workspace_dispatchers_test.go @@ -2,6 +2,7 @@ package handlers import ( "context" + "database/sql" "testing" "github.com/DATA-DOG/go-sqlmock" -- 2.45.2