fix: correct struct field alignment and spelling

Fix remaining golangci-lint issues:

Field Alignment:
- Reorder Cache struct: pointers first (store, logger), then map, then RWMutex
- This ordering reduces struct size from 48 to 24 pointer bytes
- Aligns with Go's memory layout optimization requirements

Spelling:
- Fix 'cancelled' → 'canceled' in test comment
- Maintains consistent American English spelling throughout codebase

Both issues were caught by golangci-lint v1.64 fieldalignment and misspell linters.

Author: Rihoj

cache/cache.go

@@ -10,10 +10,10 @@ import (
 
 // Cache provides a high-performance in-memory cache
 type Cache struct {
-	mu     sync.RWMutex
-	ttls   map[string]time.Time
 	store  *ristretto.Cache[string, any]
 	logger *zap.Logger
+	ttls   map[string]time.Time
+	mu     sync.RWMutex
 }
 
 // Config holds cache configuration

server_test.go

@@ -338,26 +338,26 @@ func TestServer_Shutdown(t *testing.T) {
 				}
 			}
 
-			srv, err := New(cfg, logger)
-			if err != nil {
-				t.Fatalf("failed to create server: %v", err)
-			}
-
-			var ctx context.Context
-			var cancel context.CancelFunc
-
-			if tt.withTimeout {
-				// Create a context that's already cancelled to simulate timeout
-				ctx, cancel = context.WithCancel(context.Background())
-				cancel()
-			} else {
-				ctx = context.Background()
-			}
-
-			err = srv.Shutdown(ctx)
-			if (err != nil) != tt.wantErr {
-				t.Errorf("Shutdown() error = %v, wantErr %v", err, tt.wantErr)
-			}
+		srv, err := New(cfg, logger)
+		if err != nil {
+			t.Fatalf("failed to create server: %v", err)
+		}
+
+		var ctx context.Context
+		var cancel context.CancelFunc
+
+		if tt.withTimeout {
+			// Create a context that's already canceled to simulate timeout
+			ctx, cancel = context.WithCancel(context.Background())
+			cancel()
+		} else {
+			ctx = context.Background()
+		}
+
+		err = srv.Shutdown(ctx)
+		if (err != nil) != tt.wantErr {
+			t.Errorf("Shutdown() error = %v, wantErr %v", err, tt.wantErr)
+		}
 		})
 	}
 }