Merge pull request #402 from dgageot/simpler-fetch-tests

Simpler fetch tests

Author: rumpl

.golangci.yml

@@ -29,6 +29,9 @@ linters:
     forbidigo:
       forbid:
         - pattern: context\.Background()
+          msg: "do not use context.Background() in tests, use t.Context()"
+        - pattern: context\.TODO()
+          msg: "do not use context.TODO() in tests, use t.Context()"
     depguard:
       rules:
         internal:

pkg/tools/builtin/fetch.go

@@ -47,8 +47,7 @@ func (h *fetchHandler) CallTool(ctx context.Context, toolCall tools.ToolCall) (*
 		Timeout: h.timeout,
 	}
 	if params.Timeout > 0 {
-		timeout := time.Duration(params.Timeout) * time.Second
-		client = &http.Client{Timeout: timeout}
+		client.Timeout = time.Duration(params.Timeout) * time.Second
 	}
 
 	var results []FetchResult