chore: use OS-appropriate cache directory

Author: maniac-en

internal/collections/manager_test.go

@@ -9,7 +9,6 @@ import (
 	"github.com/maniac-en/req/internal/testutils"
 )
 
-
 func TestCollectionsManagerCRUD(t *testing.T) {
 	db := testutils.SetupTestDB(t, "collections")
 	manager := NewCollectionsManager(db)

internal/endpoints/manager_test.go

@@ -8,7 +8,6 @@ import (
 	"github.com/maniac-en/req/internal/testutils"
 )
 
-
 func TestEndpointsManagerCRUD(t *testing.T) {
 	db := testutils.SetupTestDB(t, "collections", "endpoints")
 	manager := NewEndpointsManager(db)
@@ -402,4 +401,4 @@ func TestEndpointsManagerValidation(t *testing.T) {
 			t.Errorf("Expected ErrInvalidInput, got %v", err)
 		}
 	})
-}
+}

internal/history/manager_test.go

@@ -8,7 +8,6 @@ import (
 	"github.com/maniac-en/req/internal/testutils"
 )
 
-
 func TestHistoryManagerCRUD(t *testing.T) {
 	ctx := context.Background()
 	db := testutils.SetupTestDB(t, "history")

internal/log/handler_test.go

@@ -51,4 +51,4 @@ func TestMultiHandler(t *testing.T) {
 	if !strings.Contains(buf2.String(), "multi test") {
 		t.Error("second handler should receive message")
 	}
-}
+}

internal/log/logger_test.go

@@ -12,21 +12,21 @@ import (
 
 func TestLoggerFactory(t *testing.T) {
 	factory := NewLoggerFactory()
-	
+
 	t.Run("creates working logger", func(t *testing.T) {
 		config := Config{Level: slog.LevelInfo, Verbose: true}
 		logger := factory.CreateLogger(config)
-		
+
 		if logger == nil {
 			t.Fatal("factory should create logger")
 		}
 	})
-	
+
 	t.Run("creates independent instances", func(t *testing.T) {
 		config := Config{Level: slog.LevelInfo}
 		logger1 := factory.CreateLogger(config)
 		logger2 := factory.CreateLogger(config)
-		
+
 		if logger1 == logger2 {
 			t.Error("factory should create independent instances")
 		}
@@ -36,15 +36,15 @@ func TestLoggerFactory(t *testing.T) {
 func TestDualHandler(t *testing.T) {
 	var buf bytes.Buffer
 	handler := NewDualHandler(&buf, false, slog.LevelInfo)
-	
+
 	record := slog.NewRecord(time.Now(), slog.LevelInfo, "test message", 0)
 	record.Add("key", "value")
-	
+
 	err := handler.Handle(context.Background(), record)
 	if err != nil {
 		t.Fatalf("handler failed: %v", err)
 	}
-	
+
 	output := buf.String()
 	if !strings.Contains(output, "test message") {
 		t.Error("output should contain message")
@@ -57,17 +57,17 @@ func TestDualHandler(t *testing.T) {
 func TestRequestIDFunctions(t *testing.T) {
 	id1 := GenerateRequestID()
 	id2 := GenerateRequestID()
-	
+
 	if id1 == id2 {
 		t.Error("IDs should be unique")
 	}
 	if !strings.HasPrefix(id1, "req_") {
 		t.Error("ID should have req_ prefix")
 	}
-	
+
 	ctx := ContextWithRequestID(context.Background(), "test123")
 	retrieved := RequestIDFromContext(ctx)
-	
+
 	if retrieved != "test123" {
 		t.Errorf("expected test123, got %s", retrieved)
 	}
@@ -76,10 +76,10 @@ func TestRequestIDFunctions(t *testing.T) {
 func TestGlobalLogger(t *testing.T) {
 	// Test that global functions work
 	Info("test info message")
-	Debug("test debug message") 
+	Debug("test debug message")
 	Warn("test warn message")
 	Error("test error message")
-	
+
 	logger := Global()
 	if logger == nil {
 		t.Error("Global() should return logger")
@@ -93,17 +93,17 @@ func TestLoggerWithFile(t *testing.T) {
 	}
 	defer os.Remove(tempFile.Name())
 	tempFile.Close()
-	
+
 	factory := NewLoggerFactory()
 	logger := factory.CreateLogger(Config{
 		Level:       slog.LevelInfo,
 		LogFilePath: tempFile.Name(),
 	})
-	
+
 	logger.Info("test file logging")
-	
+
 	err = logger.Close()
 	if err != nil {
 		t.Errorf("Close should not error: %v", err)
 	}
-}
+}

internal/testutils/database.go

@@ -23,7 +23,7 @@ func SetupTestDB(t *testing.T, tables ...string) *database.Queries {
 		if schema == "" {
 			t.Fatalf("Unknown table: %s", table)
 		}
-		
+
 		if _, err := db.Exec(schema); err != nil {
 			t.Fatalf("Failed to create %s table: %v", table, err)
 		}
@@ -75,7 +75,7 @@ func getTableSchema(table string) string {
 				executed_at TEXT NOT NULL DEFAULT CURRENT_TIMESTAMP
 			);`,
 	}
-	
+
 	return schemas[table]
 }
 
@@ -86,4 +86,4 @@ func CreateTestCollection(t *testing.T, db *database.Queries, name string) int64
 		t.Fatalf("Failed to create test collection: %v", err)
 	}
 	return collection.ID
-}
+}

main.go

@@ -45,13 +45,19 @@ type Config struct {
 }
 
 func initPaths() error {
-	// setup paths based on user's home directory
+	// setup paths using OS-appropriate cache directory
 	userHomeDir, err := os.UserHomeDir()
 	if err != nil {
 		return fmt.Errorf("error reading user's home path: %w", err)
 	}
 	USERHOMEDIR = userHomeDir
-	APPDIR = filepath.Join(USERHOMEDIR, ".cache", "req")
+
+	// use OS-appropriate cache directory
+	userCacheDir, err := os.UserCacheDir()
+	if err != nil {
+		return fmt.Errorf("error reading user's cache path: %w", err)
+	}
+	APPDIR = filepath.Join(userCacheDir, "req")
 	if err := os.MkdirAll(APPDIR, 0o755); err != nil {
 		return fmt.Errorf("error creating app directory: %w", err)
 	}