fix: config filepath.Abs errors, file-exist check, git error handling, iteragent v1.5.0

Author: Patel230

cmd/iterate/config.go

@@ -120,10 +120,10 @@ func saveConfig(cfg iterConfig) {
 	tomlPath := configPathTOML()
 	jsonPath := configPath()
 
-	_, tomlExists := os.Stat(tomlPath)
-	_, jsonExists := os.Stat(jsonPath)
+	_, tomlErr := os.Stat(tomlPath)
+	_, jsonErr := os.Stat(jsonPath)
 
-	if tomlExists == nil || jsonExists != nil {
+	if tomlErr == nil || jsonErr != nil {
 		// Write TOML.
 		if err := os.MkdirAll(filepath.Dir(tomlPath), 0o755); err != nil {
 			slog.Warn("failed to create TOML config dir", "err", err)
@@ -226,7 +226,10 @@ func checkDirPermission(cfg iterConfig, filePath string) (denied bool) {
 	}
 	// DenyDirs: block if path is under any denied directory.
 	for _, d := range cfg.DenyDirs {
-		dAbs, _ := filepath.Abs(d)
+		dAbs, err := filepath.Abs(d)
+		if err != nil {
+			continue
+		}
 		rel, err := filepath.Rel(dAbs, abs)
 		if err == nil && !strings.HasPrefix(rel, "..") {
 			return true
@@ -235,7 +238,10 @@ func checkDirPermission(cfg iterConfig, filePath string) (denied bool) {
 	// AllowDirs: if set, path must be under at least one allowed dir.
 	if len(cfg.AllowDirs) > 0 {
 		for _, d := range cfg.AllowDirs {
-			dAbs, _ := filepath.Abs(d)
+			dAbs, err := filepath.Abs(d)
+			if err != nil {
+				continue
+			}
 			rel, err := filepath.Rel(dAbs, abs)
 			if err == nil && !strings.HasPrefix(rel, "..") {
 				return false

cmd/iterate/features_prompts.go

@@ -99,9 +99,9 @@ func buildMockPrompt(filePath string) string {
 // ---------------------------------------------------------------------------
 
 func buildReviewPrompt(repoPath string) string {
-	out, _ := exec.Command("git", "-C", repoPath, "diff", "HEAD").Output()
+	out, err := exec.Command("git", "-C", repoPath, "diff", "HEAD").Output()
 	diff := strings.TrimSpace(string(out))
-	if diff == "" {
+	if diff == "" || err != nil {
 		out, _ = exec.Command("git", "-C", repoPath, "diff").Output()
 		diff = strings.TrimSpace(string(out))
 	}

go.mod

@@ -4,7 +4,7 @@ go 1.25.0
 
 require (
 	github.com/BurntSushi/toml v1.6.0
-	github.com/GrayCodeAI/iteragent v1.4.0
+	github.com/GrayCodeAI/iteragent v1.5.0
 	github.com/google/go-github/v61 v61.0.0
 	golang.org/x/oauth2 v0.36.0
 	golang.org/x/term v0.41.0

go.sum

@@ -2,6 +2,8 @@ github.com/BurntSushi/toml v1.6.0 h1:dRaEfpa2VI55EwlIW72hMRHdWouJeRF7TPYhI+AUQjk
 github.com/BurntSushi/toml v1.6.0/go.mod h1:ukJfTF/6rtPPRCnwkur4qwRxa8vTRFBF0uk2lLoLwho=
 github.com/GrayCodeAI/iteragent v1.4.0 h1:BSW/Zj0142sXO6vZrC47LUqLt5O3M6eepgpG+VEDMxk=
 github.com/GrayCodeAI/iteragent v1.4.0/go.mod h1:wsIv7o17FthxfJ1zzbqgPdnVSSlUOxmEC2ch96S+4hY=
+github.com/GrayCodeAI/iteragent v1.5.0 h1:Z7weM5U6SOgFBf6AH2clSxPGeXB0URxGXF1kfhqhpG4=
+github.com/GrayCodeAI/iteragent v1.5.0/go.mod h1:wsIv7o17FthxfJ1zzbqgPdnVSSlUOxmEC2ch96S+4hY=
 github.com/google/go-cmp v0.6.0 h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI=
 github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
 github.com/google/go-github/v61 v61.0.0 h1:VwQCBwhyE9JclCI+22/7mLB1PuU9eowCXKY5pNlu1go=