Merge pull request #2 from andreagrandi/add-config-support

Add configuration file support

Author: andreagrandi

README.md

@@ -49,12 +49,33 @@ go build -o sentire ./cmd/sentire
 
 ## Configuration
 
-Before using sentire, you must set your Sentry API token as an environment variable:
+Sentire supports two methods for providing your Sentry API token: environment variables and configuration files.
+
+### Environment Variable (Recommended)
+
+Set your Sentry API token as an environment variable:
 
 ```bash
 export SENTRY_API_TOKEN=your_sentry_api_token_here
 ```
 
+### Configuration File
+
+Alternatively, you can create a configuration file at `~/.config/sentire/config.json`:
+
+```json
+{
+  "sentry_api_token": "your_sentry_api_token_here"
+}
+```
+
+### Configuration Precedence
+
+If both are provided, the environment variable takes precedence over the configuration file. This allows you to:
+
+- Use a config file for your default token
+- Override it temporarily with an environment variable when needed
+
 You can obtain an API token from your Sentry organization settings under "Auth Tokens".
 
 ## Usage
@@ -317,6 +338,6 @@ Licensed under the MIT License. See [LICENSE](LICENSE) for details.
 Future enhancements may include:
 
 - ✅ **Multiple output formats** (JSON, table, text, markdown) - **COMPLETED**
-- Configuration file support
+- ✅ **Configuration file support** - **COMPLETED**
 - Additional Sentry API endpoints
 - Export functionality (CSV, JSON files)

internal/client/client.go

@@ -6,16 +6,15 @@ import (
 	"io"
 	"net/http"
 	"net/url"
-	"os"
+	"sentire/internal/config"
 	"strconv"
 	"strings"
 	"time"
 )
 
 const (
-	BaseURL     = "https://sentry.io/api/0"
-	UserAgent   = "sentire/1.0.0"
-	TokenEnvVar = "SENTRY_API_TOKEN"
+	BaseURL   = "https://sentry.io/api/0"
+	UserAgent = "sentire/1.0.0"
 )
 
 // Client represents the Sentry API client
@@ -51,17 +50,17 @@ type Response struct {
 
 // NewClient creates a new Sentry API client
 func NewClient() (*Client, error) {
-	token := os.Getenv(TokenEnvVar)
-	if token == "" {
-		return nil, fmt.Errorf("environment variable %s is required", TokenEnvVar)
+	cfg, err := config.LoadConfig()
+	if err != nil {
+		return nil, err
 	}
 
 	return &Client{
 		BaseURL: BaseURL,
 		HTTPClient: &http.Client{
 			Timeout: 30 * time.Second,
 		},
-		Token:     token,
+		Token:     cfg.SentryAPIToken,
 		RateLimit: &RateLimiter{},
 	}, nil
 }

internal/config/config.go

@@ -0,0 +1,98 @@
+package config
+
+import (
+	"encoding/json"
+	"fmt"
+	"os"
+	"path/filepath"
+)
+
+// Config holds the application configuration
+type Config struct {
+	SentryAPIToken string `json:"sentry_api_token"`
+}
+
+// LoadConfig loads configuration from environment variables or config file
+// Environment variables take precedence over config file values
+func LoadConfig() (*Config, error) {
+	config := &Config{}
+
+	// First, try to load from environment variable
+	if token := os.Getenv("SENTRY_API_TOKEN"); token != "" {
+		config.SentryAPIToken = token
+		return config, nil
+	}
+
+	// If env var is not set, try to load from config file
+	configPath, err := getConfigPath()
+	if err != nil {
+		return nil, fmt.Errorf("failed to determine config path: %w", err)
+	}
+
+	if err := loadFromFile(configPath, config); err != nil {
+		// If config file doesn't exist or has issues, return error about missing token
+		return nil, fmt.Errorf("SENTRY_API_TOKEN environment variable is required (or configure ~/.config/sentire/config.json)")
+	}
+
+	// Validate that we have a token
+	if config.SentryAPIToken == "" {
+		return nil, fmt.Errorf("SENTRY_API_TOKEN is required in config file")
+	}
+
+	return config, nil
+}
+
+// getConfigPath returns the path to the config file
+func getConfigPath() (string, error) {
+	homeDir, err := os.UserHomeDir()
+	if err != nil {
+		return "", fmt.Errorf("failed to get user home directory: %w", err)
+	}
+
+	return filepath.Join(homeDir, ".config", "sentire", "config.json"), nil
+}
+
+// loadFromFile loads configuration from a JSON file
+func loadFromFile(path string, config *Config) error {
+	file, err := os.Open(path)
+	if err != nil {
+		return fmt.Errorf("failed to open config file: %w", err)
+	}
+	defer file.Close()
+
+	decoder := json.NewDecoder(file)
+	if err := decoder.Decode(config); err != nil {
+		return fmt.Errorf("failed to parse config file: %w", err)
+	}
+
+	return nil
+}
+
+// SaveConfig saves the current configuration to the config file
+// This is mainly for future use when we add more configuration options
+func SaveConfig(config *Config) error {
+	configPath, err := getConfigPath()
+	if err != nil {
+		return fmt.Errorf("failed to determine config path: %w", err)
+	}
+
+	// Create directory if it doesn't exist
+	configDir := filepath.Dir(configPath)
+	if err := os.MkdirAll(configDir, 0755); err != nil {
+		return fmt.Errorf("failed to create config directory: %w", err)
+	}
+
+	file, err := os.Create(configPath)
+	if err != nil {
+		return fmt.Errorf("failed to create config file: %w", err)
+	}
+	defer file.Close()
+
+	encoder := json.NewEncoder(file)
+	encoder.SetIndent("", "  ")
+	if err := encoder.Encode(config); err != nil {
+		return fmt.Errorf("failed to write config file: %w", err)
+	}
+
+	return nil
+}