0x1d
784f0f601f
- Add package comments to all packages (pkg/config, pkg/logger, internal/*, cmd/platform)
- Fix context key warnings by using custom ContextKey type
- Export ContextKey type to avoid unexported-return warnings
- Update all context value operations to use ContextKey instead of string
- Update RequestIDKey() and UserIDKey() to return ContextKey
- Fix error checking issues (errcheck)
- Properly handle os.Chdir errors in defer statements
- Properly handle os.Setenv/os.Unsetenv errors in tests
- Fix security warnings (gosec)
- Change directory permissions from 0755 to 0750 in tests
- Change file permissions from 0644 to 0600 in tests
- Fix unused parameter warnings (revive)
- Replace unused parameters with _ in:
* RegisterLifecycleHooks lifecycle functions
* Mock logger implementations
* noOpLogger methods
- Fix type assertion issues (staticcheck)
- Remove unnecessary type assertions in tests
- Use simpler compile-time checks
- Fix exported type stuttering warning
- Add nolint directive for ConfigProvider (standard interface pattern)
- Update golangci-lint configuration
- Add version: 2 field (required for newer versions)
- Remove unsupported linters (typecheck, gosimple)
- Move formatters (gofmt, goimports) to separate formatters section
- Simplify linter list to only well-supported linters
All linting issues resolved (0 issues reported by golangci-lint).
All tests pass and code compiles successfully.
44 lines
1.5 KiB
Go
44 lines
1.5 KiB
Go
// Package config provides the configuration management interface.
|
|
// It defines the ConfigProvider interface that implementations must satisfy.
|
|
package config
|
|
|
|
import "time"
|
|
|
|
// ConfigProvider defines the interface for configuration management.
|
|
// It provides type-safe access to configuration values from various sources
|
|
// (YAML files, environment variables, etc.).
|
|
//
|
|
//nolint:revive // ConfigProvider is a standard interface name pattern; stuttering is acceptable
|
|
type ConfigProvider interface {
|
|
// Get retrieves a configuration value by key.
|
|
// Returns nil if the key is not found.
|
|
Get(key string) any
|
|
|
|
// Unmarshal unmarshals the entire configuration into the provided struct.
|
|
// The struct should use appropriate tags for mapping configuration keys.
|
|
Unmarshal(v any) error
|
|
|
|
// GetString retrieves a string value by key.
|
|
// Returns empty string if the key is not found.
|
|
GetString(key string) string
|
|
|
|
// GetInt retrieves an integer value by key.
|
|
// Returns 0 if the key is not found or cannot be converted.
|
|
GetInt(key string) int
|
|
|
|
// GetBool retrieves a boolean value by key.
|
|
// Returns false if the key is not found or cannot be converted.
|
|
GetBool(key string) bool
|
|
|
|
// GetStringSlice retrieves a string slice value by key.
|
|
// Returns nil if the key is not found.
|
|
GetStringSlice(key string) []string
|
|
|
|
// GetDuration retrieves a duration value by key.
|
|
// Returns 0 if the key is not found or cannot be parsed.
|
|
GetDuration(key string) time.Duration
|
|
|
|
// IsSet checks if a configuration key is set.
|
|
IsSet(key string) bool
|
|
}
|