style: run precommit

Signed-off-by: John Panos <[email protected]>

Author: johnpanos

api/v1alpha1/zz_generated.deepcopy.go

@@ -54,8 +54,8 @@ metadata:
 spec:
   responseCache:
     enabled: true
-    ttl: 1h                      # Cache TTL (default: 1h)
-    respectCacheControl: true    # Honor HTTP Cache-Control headers (default: true)
+    ttl: 1h # Cache TTL (default: 1h)
+    respectCacheControl: true # Honor HTTP Cache-Control headers (default: true)
   # ... rest of config
 ```
 
@@ -78,18 +78,18 @@ When `respectCacheControl: true` (default), the gateway honors standard HTTP Cac
 
 ### Request Headers (Client Control)
 
-| Header | Behavior |
-|--------|----------|
+| Header                    | Behavior                                          |
+| ------------------------- | ------------------------------------------------- |
 | `Cache-Control: no-cache` | Bypass cache lookup, but still cache the response |
-| `Cache-Control: no-store` | Bypass cache entirely (no lookup, no store) |
+| `Cache-Control: no-store` | Bypass cache entirely (no lookup, no store)       |
 
 ### Response Headers (Backend Control)
 
-| Header | Behavior |
-|--------|----------|
-| `Cache-Control: no-store` | Do not cache this response |
-| `Cache-Control: no-cache` | Do not cache this response |
-| `Cache-Control: private` | Do not cache (shared cache should not store) |
+| Header                     | Behavior                                           |
+| -------------------------- | -------------------------------------------------- |
+| `Cache-Control: no-store`  | Do not cache this response                         |
+| `Cache-Control: no-cache`  | Do not cache this response                         |
+| `Cache-Control: private`   | Do not cache (shared cache should not store)       |
 | `Cache-Control: max-age=N` | Use N seconds as TTL instead of configured default |
 
 ### Examples

examples/response-cache/README.md

@@ -24,8 +24,8 @@ metadata:
     app: redis
 spec:
   ports:
-  - name: redis
-    port: 6379
+    - name: redis
+      port: 6379
   selector:
     app: redis
 ---
@@ -45,17 +45,17 @@ spec:
         app: redis
     spec:
       containers:
-      - image: redis:alpine
-        imagePullPolicy: IfNotPresent
-        name: redis
-        ports:
-        - name: redis
-          containerPort: 6379
-        resources:
-          requests:
-            memory: "64Mi"
-            cpu: "50m"
-          limits:
-            memory: "256Mi"
-            cpu: "200m"
+        - image: redis:alpine
+          imagePullPolicy: IfNotPresent
+          name: redis
+          ports:
+            - name: redis
+              containerPort: 6379
+          resources:
+            requests:
+              memory: "64Mi"
+              cpu: "50m"
+            limits:
+              memory: "256Mi"
+              cpu: "200m"
       restartPolicy: Always

examples/response-cache/redis.yaml

@@ -23,9 +23,9 @@ metadata:
 spec:
   gatewayClassName: eg
   listeners:
-  - name: http
-    protocol: HTTP
-    port: 8080
+    - name: http
+      protocol: HTTP
+      port: 8080
 ---
 apiVersion: aigateway.envoyproxy.io/v1alpha1
 kind: AIGatewayRoute
@@ -34,9 +34,9 @@ metadata:
   namespace: default
 spec:
   parentRefs:
-  - name: ai-gateway-with-cache
-    kind: Gateway
-    group: gateway.networking.k8s.io
+    - name: ai-gateway-with-cache
+      kind: Gateway
+      group: gateway.networking.k8s.io
   # Response caching configuration
   responseCache:
     # Enable response caching for this route
@@ -52,13 +52,13 @@ spec:
     #   - Response "Cache-Control: max-age=N" overrides TTL
     respectCacheControl: true
   rules:
-  - matches:
-    - headers:
-      - type: Exact
-        name: x-ai-eg-model
-        value: gpt-4o-mini
-    backendRefs:
-    - name: openai-backend
+    - matches:
+        - headers:
+            - type: Exact
+              name: x-ai-eg-model
+              value: gpt-4o-mini
+      backendRefs:
+        - name: openai-backend
 ---
 apiVersion: aigateway.envoyproxy.io/v1alpha1
 kind: AIServiceBackend
@@ -80,9 +80,9 @@ metadata:
   namespace: default
 spec:
   targetRefs:
-  - group: aigateway.envoyproxy.io
-    kind: AIServiceBackend
-    name: openai-backend
+    - group: aigateway.envoyproxy.io
+      kind: AIServiceBackend
+      name: openai-backend
   type: APIKey
   apiKey:
     secretRef:
@@ -96,9 +96,9 @@ metadata:
   namespace: default
 spec:
   endpoints:
-  - fqdn:
-      hostname: api.openai.com
-      port: 443
+    - fqdn:
+        hostname: api.openai.com
+        port: 443
 ---
 apiVersion: gateway.networking.k8s.io/v1alpha3
 kind: BackendTLSPolicy
@@ -107,9 +107,9 @@ metadata:
   namespace: default
 spec:
   targetRefs:
-  - group: gateway.envoyproxy.io
-    kind: Backend
-    name: openai-backend
+    - group: gateway.envoyproxy.io
+      kind: Backend
+      name: openai-backend
   validation:
     wellKnownCACertificates: System
     hostname: api.openai.com

examples/response-cache/response-cache.yaml

@@ -34,7 +34,7 @@ func TestHashBody(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			result := HashBody(tt.body)
-			require.True(t, len(result) > len(KeyPrefix), "hash should be longer than prefix")
+			require.Greater(t, len(result), len(KeyPrefix), "hash should be longer than prefix")
 			require.Contains(t, result, KeyPrefix, "hash should contain prefix")
 
 			// Verify determinism

internal/cache/cache_test.go

@@ -11,8 +11,8 @@ import (
 	"time"
 )
 
-// CacheDirectives holds parsed Cache-Control header directives.
-type CacheDirectives struct {
+// Directives holds parsed Cache-Control header directives.
+type Directives struct {
 	// NoCache indicates the response should not be served from cache without revalidation.
 	// For requests: bypass cache lookup but still cache the response.
 	// For responses: treat as no-store for simplicity (no revalidation support).
@@ -38,9 +38,9 @@ type CacheDirectives struct {
 //   - private: do not store in shared cache
 //   - max-age=N: use N seconds as TTL
 //
-// Example: "no-cache, max-age=3600" -> CacheDirectives{NoCache: true, MaxAge: 1h}
-func ParseCacheControl(value string) CacheDirectives {
-	var d CacheDirectives
+// Example: "no-cache, max-age=3600" -> Directives{NoCache: true, MaxAge: 1h}
+func ParseCacheControl(value string) Directives {
+	var d Directives
 	if value == "" {
 		return d
 	}
@@ -74,12 +74,12 @@ func ParseCacheControl(value string) CacheDirectives {
 
 // ShouldSkipCacheLookup returns true if the request Cache-Control directives
 // indicate that cache lookup should be skipped.
-func (d CacheDirectives) ShouldSkipCacheLookup() bool {
+func (d Directives) ShouldSkipCacheLookup() bool {
 	return d.NoStore || d.NoCache
 }
 
 // ShouldSkipCacheStore returns true if the directives indicate that
 // the response should not be stored in cache.
-func (d CacheDirectives) ShouldSkipCacheStore() bool {
+func (d Directives) ShouldSkipCacheStore() bool {
 	return d.NoStore || d.NoCache || d.Private
 }