chore: fix linting issues

Author: bartventer

internal/responsestorerer.go

@@ -60,7 +60,8 @@ func (r *responseStorer) StoreResponse(
 	// Remove hop-by-hop headers as per RFC 9111 §3.1
 	removeHopByHopHeaders(resp)
 
-	vary := strings.Join(resp.Header.Values("Vary"), ",") // Join multiple Vary headers into a single string
+	// Join multiple Vary headers into a single string
+	vary := strings.Join(resp.Header.Values("Vary"), ",")
 	varyResolved := maps.Collect(
 		r.vhn.NormalizeVaryHeader(vary, req.Header),
 	)

roundtripper_test.go

@@ -998,15 +998,23 @@ func Test_transport_Vary_MultipleHeaders(t *testing.T) {
 	// Request 1: MISS — first request for en-us + 2025-01-01
 	resp := makeReq("en-us", "2025-01-01")
 	testutil.AssertEqual(t, http.StatusOK, resp.StatusCode)
-	testutil.AssertEqual(t, internal.CacheStatusMiss.Value, resp.Header.Get(internal.CacheStatusHeader))
+	testutil.AssertEqual(
+		t,
+		internal.CacheStatusMiss.Value,
+		resp.Header.Get(internal.CacheStatusHeader),
+	)
 	body, _ := io.ReadAll(resp.Body)
 	_ = resp.Body.Close()
 	testutil.AssertEqual(t, "lang=en-us date=2025-01-01", string(body))
 
 	// Request 2: HIT — same headers
 	resp = makeReq("en-us", "2025-01-01")
 	testutil.AssertEqual(t, http.StatusOK, resp.StatusCode)
-	testutil.AssertEqual(t, internal.CacheStatusHit.Value, resp.Header.Get(internal.CacheStatusHeader))
+	testutil.AssertEqual(
+		t,
+		internal.CacheStatusHit.Value,
+		resp.Header.Get(internal.CacheStatusHeader),
+	)
 	body, _ = io.ReadAll(resp.Body)
 	_ = resp.Body.Close()
 	testutil.AssertEqual(t, "lang=en-us date=2025-01-01", string(body))
@@ -1015,16 +1023,24 @@ func Test_transport_Vary_MultipleHeaders(t *testing.T) {
 	// This is the bug: without the fix, this incorrectly returns a HIT
 	resp = makeReq("en-us", "2026-01-01")
 	testutil.AssertEqual(t, http.StatusOK, resp.StatusCode)
-	testutil.AssertEqual(t, internal.CacheStatusMiss.Value, resp.Header.Get(internal.CacheStatusHeader),
-		"X-Compatibility-Date is a Vary header; different value must produce a cache MISS")
+	testutil.AssertEqual(
+		t,
+		internal.CacheStatusMiss.Value,
+		resp.Header.Get(internal.CacheStatusHeader),
+		"X-Compatibility-Date is a Vary header; different value must produce a cache MISS",
+	)
 	body, _ = io.ReadAll(resp.Body)
 	_ = resp.Body.Close()
 	testutil.AssertEqual(t, "lang=en-us date=2026-01-01", string(body))
 
 	// Request 4: HIT — same as request 3
 	resp = makeReq("en-us", "2026-01-01")
 	testutil.AssertEqual(t, http.StatusOK, resp.StatusCode)
-	testutil.AssertEqual(t, internal.CacheStatusHit.Value, resp.Header.Get(internal.CacheStatusHeader))
+	testutil.AssertEqual(
+		t,
+		internal.CacheStatusHit.Value,
+		resp.Header.Get(internal.CacheStatusHeader),
+	)
 	body, _ = io.ReadAll(resp.Body)
 	_ = resp.Body.Close()
 	testutil.AssertEqual(t, "lang=en-us date=2026-01-01", string(body))

store/expapi/expapi.go

@@ -98,7 +98,11 @@ func retrieve(conn driver.Conn) http.Handler {
 			if errors.Is(err, driver.ErrNotExist) {
 				http.Error(w, fmt.Sprintf("key %q not found", key), http.StatusNotFound)
 			} else {
-				http.Error(w, fmt.Sprintf("failed to get value for key %q: %v", key, err), http.StatusInternalServerError)
+				http.Error(
+					w,
+					fmt.Sprintf("failed to get value for key %q: %v", key, err),
+					http.StatusInternalServerError,
+				)
 			}
 			return
 		}
@@ -121,7 +125,11 @@ func destroy(conn driver.Conn) http.Handler {
 			if errors.Is(err, driver.ErrNotExist) {
 				http.Error(w, fmt.Sprintf("key %q not found", key), http.StatusNotFound)
 			} else {
-				http.Error(w, fmt.Sprintf("failed to delete value for key %q: %v", key, err), http.StatusInternalServerError)
+				http.Error(
+					w,
+					fmt.Sprintf("failed to delete value for key %q: %v", key, err),
+					http.StatusInternalServerError,
+				)
 			}
 			return
 		}