From 5e54d492be59fb4254427704c68cb39d2fbe1616 Mon Sep 17 00:00:00 2001 From: shaw Date: Wed, 29 Apr 2026 21:35:18 +0800 Subject: [PATCH] fix(lint): check type assertion error in codex transform test The errcheck linter flagged an unchecked type assertion on item["type"].(string). Use the two-value form with require.True to satisfy the linter and fail clearly on unexpected types. --- backend/internal/service/openai_codex_transform_test.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/backend/internal/service/openai_codex_transform_test.go b/backend/internal/service/openai_codex_transform_test.go index c6f147d8..7ab6bfc0 100644 --- a/backend/internal/service/openai_codex_transform_test.go +++ b/backend/internal/service/openai_codex_transform_test.go @@ -1176,7 +1176,9 @@ func TestFilterCodexInput_DropsReasoningItemsRegardlessOfPreserveReferences(t *t for _, raw := range filtered { item, ok := raw.(map[string]any) require.True(t, ok) - gotTypes[item["type"].(string)]++ + typ, ok := item["type"].(string) + require.True(t, ok) + gotTypes[typ]++ } require.Equal(t, 1, gotTypes["message"]) require.Equal(t, 1, gotTypes["function_call"])