refactor(config): pass project path explicitly to LoadConfig

- Changed 'LoadConfig' to accept 'projectPath' as a parameter for better flexibility.
- Updated flag variable 'projectPath' to 'ProjectPath' to align with Go naming conventions.
- Introduced 'InitializeFlags' function to parse command-line flags before execution.
- Adjusted 'main.go' to call 'InitializeFlags' before loading configuration.

Author: gifflet

cmd/review.go

@@ -22,7 +22,7 @@ func executeReview() {
 	initialCommit = formatCommitHash(initialCommit)
 
 	// Convert relative path to absolute if needed
-	absProjectPath, err := filepath.Abs(projectPath)
+	absProjectPath, err := filepath.Abs(ProjectPath)
 	if err != nil {
 		fmt.Printf("Error resolving project path: %v\n", err)
 		os.Exit(1)

cmd/root.go

@@ -13,7 +13,7 @@ var (
 	initialCommit string
 	finalCommit   string
 	mainBranch    string
-	projectPath   string
+	ProjectPath   string
 	outputDir     string
 
 	// Application version
@@ -58,6 +58,12 @@ func SetConfig(cfg *config.Config) {
 	appConfig = cfg
 }
 
+// InitializeFlags parses command line flags before executing the main command
+func InitializeFlags() error {
+	// Parse flags without executing the root command
+	return rootCmd.ParseFlags(os.Args[1:])
+}
+
 // Execute adds all child commands to the root command and sets flags
 func Execute() {
 	if err := rootCmd.Execute(); err != nil {
@@ -71,7 +77,7 @@ func init() {
 	rootCmd.Flags().StringVarP(&initialCommit, "initial", "i", "", "Starting commit hash for comparison (required)")
 	rootCmd.Flags().StringVarP(&finalCommit, "final", "f", "HEAD", "Ending commit hash (defaults to HEAD)")
 	rootCmd.Flags().StringVarP(&mainBranch, "main-branch", "m", "", "Main branch for refined comparisons")
-	rootCmd.Flags().StringVarP(&projectPath, "project-path", "p", ".", "Project directory path")
+	rootCmd.Flags().StringVarP(&ProjectPath, "project-path", "p", ".", "Project directory path")
 	rootCmd.Flags().StringVarP(&outputDir, "output-dir", "o", "git-review", "Output directory for diff files")
 
 	// Mark initialCommit as required

internal/config/config.go

@@ -20,7 +20,7 @@ type Config struct {
 }
 
 // LoadConfig reads configuration from files and environment variables
-func LoadConfig() (*Config, error) {
+func LoadConfig(projectPath string) (*Config, error) {
 	v := viper.New()
 
 	// Set config name and type
@@ -40,7 +40,7 @@ func LoadConfig() (*Config, error) {
 	v.AddConfigPath(filepath.Join(homeDir, "AppData", "Roaming", "git-review"))
 
 	// Local project config
-	v.AddConfigPath(".")
+	v.AddConfigPath(projectPath)
 	v.SetConfigName(".gitreview")
 
 	// Read config

main.go

@@ -12,8 +12,14 @@ import (
 var AppVersion = "dev"
 
 func main() {
-	// Load configuration
-	cfg, err := config.LoadConfig()
+	// Initialize command flags before loading config
+	if err := cmd.InitializeFlags(); err != nil {
+		fmt.Printf("Error initializing flags: %v\n", err)
+		os.Exit(1)
+	}
+
+	// Load configuration with project path
+	cfg, err := config.LoadConfig(cmd.ProjectPath)
 	if err != nil {
 		fmt.Printf("Error loading config: %v\n", err)
 		os.Exit(1)