Skip to content

Commit bf705b1

Browse files
Zie619claude
andcommitted
fix(lint): handle all errcheck violations for golangci-lint
Wrap all unhandled error returns with blank identifier assignments to satisfy the errcheck linter: defer Close(), rand.Read(), json Encode/Decode, and w.Write() calls across source, tests, and examples. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent f5e35e7 commit bf705b1

File tree

6 files changed

+22
-22
lines changed

6 files changed

+22
-22
lines changed

events.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ type Event struct {
3131
// generateID creates a random hex ID
3232
func generateID() string {
3333
b := make([]byte, 16)
34-
rand.Read(b)
34+
_, _ = rand.Read(b)
3535
return hex.EncodeToString(b)
3636
}
3737

examples/basic/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ func main() {
1515
trusera.WithAgentID("my-agent-123"),
1616
trusera.WithFlushInterval(10*time.Second),
1717
)
18-
defer client.Close()
18+
defer func() { _ = client.Close() }()
1919

2020
// Track a tool call event
2121
toolEvent := trusera.NewEvent(trusera.EventToolCall, "calculator").

examples/block-mode/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ func main() {
3131
if err != nil {
3232
log.Fatalf("Failed to initialize: %v", err)
3333
}
34-
defer client.Close()
34+
defer func() { _ = client.Close() }()
3535

3636
fmt.Println("Agent registered and HTTP client wrapped with block mode")
3737

examples/http-interceptor/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ func main() {
1717
trusera.WithAgentID("web-agent"),
1818
trusera.WithFlushInterval(30*time.Second),
1919
)
20-
defer client.Close()
20+
defer func() { _ = client.Close() }()
2121

2222
// Wrap HTTP client with Trusera interception
2323
httpClient := trusera.WrapHTTPClient(&http.Client{}, client, trusera.InterceptorOptions{

interceptor_test.go

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ func TestWrapHTTPClient(t *testing.T) {
1717
defer truseraServer.Close()
1818

1919
truseraClient := NewClient("test-key", WithBaseURL(truseraServer.URL))
20-
defer truseraClient.Close()
20+
defer func() { _ = truseraClient.Close() }()
2121

2222
httpClient := WrapHTTPClient(&http.Client{}, truseraClient, InterceptorOptions{
2323
Enforcement: ModeLog,
@@ -50,7 +50,7 @@ func TestInterceptorRecordsRequests(t *testing.T) {
5050
defer truseraServer.Close()
5151

5252
truseraClient := NewClient("test-key", WithBaseURL(truseraServer.URL))
53-
defer truseraClient.Close()
53+
defer func() { _ = truseraClient.Close() }()
5454

5555
httpClient := WrapHTTPClient(&http.Client{}, truseraClient, InterceptorOptions{
5656
Enforcement: ModeLog,
@@ -85,7 +85,7 @@ func TestExcludePatterns(t *testing.T) {
8585
defer truseraServer.Close()
8686

8787
truseraClient := NewClient("test-key", WithBaseURL(truseraServer.URL))
88-
defer truseraClient.Close()
88+
defer func() { _ = truseraClient.Close() }()
8989

9090
httpClient := WrapHTTPClient(&http.Client{}, truseraClient, InterceptorOptions{
9191
Enforcement: ModeLog,
@@ -122,7 +122,7 @@ func TestBlockModeRejectsRequests(t *testing.T) {
122122
defer truseraServer.Close()
123123

124124
truseraClient := NewClient("test-key", WithBaseURL(truseraServer.URL))
125-
defer truseraClient.Close()
125+
defer func() { _ = truseraClient.Close() }()
126126

127127
httpClient := WrapHTTPClient(&http.Client{}, truseraClient, InterceptorOptions{
128128
Enforcement: ModeBlock,
@@ -157,7 +157,7 @@ func TestWarnModeAllowsBlockedRequests(t *testing.T) {
157157
defer truseraServer.Close()
158158

159159
truseraClient := NewClient("test-key", WithBaseURL(truseraServer.URL))
160-
defer truseraClient.Close()
160+
defer func() { _ = truseraClient.Close() }()
161161

162162
httpClient := WrapHTTPClient(&http.Client{}, truseraClient, InterceptorOptions{
163163
Enforcement: ModeWarn,
@@ -193,7 +193,7 @@ func TestLogModeAllowsAllRequests(t *testing.T) {
193193
defer truseraServer.Close()
194194

195195
truseraClient := NewClient("test-key", WithBaseURL(truseraServer.URL))
196-
defer truseraClient.Close()
196+
defer func() { _ = truseraClient.Close() }()
197197

198198
httpClient := WrapHTTPClient(&http.Client{}, truseraClient, InterceptorOptions{
199199
Enforcement: ModeLog,
@@ -259,7 +259,7 @@ func TestRequestBodyCapture(t *testing.T) {
259259
defer truseraServer.Close()
260260

261261
truseraClient := NewClient("test-key", WithBaseURL(truseraServer.URL))
262-
defer truseraClient.Close()
262+
defer func() { _ = truseraClient.Close() }()
263263

264264
httpClient := WrapHTTPClient(&http.Client{}, truseraClient, InterceptorOptions{
265265
Enforcement: ModeLog,
@@ -282,7 +282,7 @@ func TestCreateInterceptedClient(t *testing.T) {
282282
defer truseraServer.Close()
283283

284284
truseraClient := NewClient("test-key", WithBaseURL(truseraServer.URL))
285-
defer truseraClient.Close()
285+
defer func() { _ = truseraClient.Close() }()
286286

287287
httpClient := CreateInterceptedClient(truseraClient, InterceptorOptions{
288288
Enforcement: ModeLog,
@@ -311,7 +311,7 @@ func TestConcurrentRequests(t *testing.T) {
311311
defer truseraServer.Close()
312312

313313
truseraClient := NewClient("test-key", WithBaseURL(truseraServer.URL))
314-
defer truseraClient.Close()
314+
defer func() { _ = truseraClient.Close() }()
315315

316316
httpClient := WrapHTTPClient(&http.Client{}, truseraClient, InterceptorOptions{
317317
Enforcement: ModeLog,

trusera_test.go

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import (
1111

1212
func TestNewClient(t *testing.T) {
1313
client := NewClient("test-api-key")
14-
defer client.Close()
14+
defer func() { _ = client.Close() }()
1515

1616
if client.apiKey != "test-api-key" {
1717
t.Errorf("expected apiKey 'test-api-key', got %s", client.apiKey)
@@ -37,7 +37,7 @@ func TestClientWithOptions(t *testing.T) {
3737
WithAgentID(customAgent),
3838
WithBatchSize(customBatch),
3939
)
40-
defer client.Close()
40+
defer func() { _ = client.Close() }()
4141

4242
if client.baseURL != customURL {
4343
t.Errorf("expected baseURL %s, got %s", customURL, client.baseURL)
@@ -54,7 +54,7 @@ func TestClientWithOptions(t *testing.T) {
5454

5555
func TestTrackEvent(t *testing.T) {
5656
client := NewClient("test-key")
57-
defer client.Close()
57+
defer func() { _ = client.Close() }()
5858

5959
event := NewEvent(EventToolCall, "test-tool")
6060

@@ -104,7 +104,7 @@ func TestFlush(t *testing.T) {
104104
defer server.Close()
105105

106106
client := NewClient("test-key", WithBaseURL(server.URL))
107-
defer client.Close()
107+
defer func() { _ = client.Close() }()
108108

109109
event1 := NewEvent(EventToolCall, "tool1")
110110
event2 := NewEvent(EventAPICall, "api1")
@@ -165,12 +165,12 @@ func TestRegisterAgent(t *testing.T) {
165165
}
166166

167167
w.Header().Set("Content-Type", "application/json")
168-
json.NewEncoder(w).Encode(response)
168+
_ = json.NewEncoder(w).Encode(response)
169169
}))
170170
defer server.Close()
171171

172172
client := NewClient("test-key", WithBaseURL(server.URL))
173-
defer client.Close()
173+
defer func() { _ = client.Close() }()
174174

175175
agentID, err := client.RegisterAgent("test-agent", "langchain")
176176
if err != nil {
@@ -188,7 +188,7 @@ func TestRegisterAgent(t *testing.T) {
188188

189189
func TestRegisterAgentEmptyName(t *testing.T) {
190190
client := NewClient("test-key")
191-
defer client.Close()
191+
defer func() { _ = client.Close() }()
192192

193193
_, err := client.RegisterAgent("", "framework")
194194
if err == nil {
@@ -236,7 +236,7 @@ func TestBatchAutoFlush(t *testing.T) {
236236
var payload struct {
237237
Events []Event `json:"events"`
238238
}
239-
json.NewDecoder(r.Body).Decode(&payload)
239+
_ = json.NewDecoder(r.Body).Decode(&payload)
240240

241241
mu.Lock()
242242
receivedCount += len(payload.Events)
@@ -251,7 +251,7 @@ func TestBatchAutoFlush(t *testing.T) {
251251
WithBaseURL(server.URL),
252252
WithBatchSize(5),
253253
)
254-
defer client.Close()
254+
defer func() { _ = client.Close() }()
255255

256256
for i := 0; i < 10; i++ {
257257
event := NewEvent(EventToolCall, "tool")

0 commit comments

Comments
 (0)