fixing setup

Author: algospider

internal/config/config.go

@@ -11,6 +11,16 @@ const (
 	File = Dir + "/config.json"
 )
 
+const (
+	stateDirName  = ".git-genius"
+	stateFileName = "state.json"
+)
+
+type appState struct {
+	ActiveWorkDir  string   `json:"active_work_dir"`
+	RecentWorkDirs []string `json:"recent_work_dirs"`
+}
+
 // Config holds Git Genius configuration
 type Config struct {
 	/* ---------------- Git basics ---------------- */
@@ -40,18 +50,27 @@ type Config struct {
 
 // Load reads config from disk and applies safe defaults
 func Load() Config {
-	data, err := os.ReadFile(File)
+	dir := preferredWorkDir()
+	data, err := os.ReadFile(configPath(dir))
 	if err != nil {
-		return defaultConfig()
+		c := defaultConfig()
+		c.WorkDir = dir
+		return c
 	}
 
 	var c Config
 	if err := json.Unmarshal(data, &c); err != nil {
-		return defaultConfig()
+		c = defaultConfig()
+		c.WorkDir = dir
+		return c
 	}
 
 	applyDefaults(&c)
 	normalizePaths(&c)
+	if c.WorkDir == "" {
+		c.WorkDir = dir
+	}
+	updateState(c.WorkDir)
 
 	return c
 }
@@ -61,14 +80,20 @@ func Save(c Config) {
 	applyDefaults(&c)
 	normalizePaths(&c)
 
-	_ = os.MkdirAll(Dir, 0700)
+	if c.WorkDir == "" {
+		c.WorkDir = preferredWorkDir()
+	}
+
+	p := configPath(c.WorkDir)
+	_ = os.MkdirAll(filepath.Dir(p), 0700)
 
 	data, err := json.MarshalIndent(c, "", "  ")
 	if err != nil {
 		return
 	}
 
-	_ = os.WriteFile(File, data, 0600)
+	_ = os.WriteFile(p, data, 0600)
+	updateState(c.WorkDir)
 }
 
 /* ============================================================
@@ -121,6 +146,107 @@ func (c Config) GetWorkDir() string {
 	if c.WorkDir != "" {
 		return c.WorkDir
 	}
+	if d := preferredWorkDir(); d != "" {
+		return d
+	}
+	wd, _ := os.Getwd()
+	return wd
+}
+
+func RecentWorkDirs() []string {
+	s := loadState()
+	out := make([]string, 0, len(s.RecentWorkDirs))
+	for _, d := range s.RecentWorkDirs {
+		if d != "" {
+			if info, err := os.Stat(d); err != nil || !info.IsDir() {
+				continue
+			}
+			out = append(out, d)
+		}
+	}
+	return out
+}
+
+func preferredWorkDir() string {
+	s := loadState()
+	if s.ActiveWorkDir != "" {
+		if info, err := os.Stat(s.ActiveWorkDir); err == nil && info.IsDir() {
+			return s.ActiveWorkDir
+		}
+	}
 	wd, _ := os.Getwd()
 	return wd
 }
+
+func configPath(workDir string) string {
+	if workDir == "" {
+		return File
+	}
+	return filepath.Join(workDir, ".git", ".genius", "config.json")
+}
+
+func statePath() string {
+	home, err := os.UserHomeDir()
+	if err != nil || home == "" {
+		return ""
+	}
+	return filepath.Join(home, stateDirName, stateFileName)
+}
+
+func loadState() appState {
+	p := statePath()
+	if p == "" {
+		return appState{}
+	}
+
+	data, err := os.ReadFile(p)
+	if err != nil {
+		return appState{}
+	}
+
+	var s appState
+	if err := json.Unmarshal(data, &s); err != nil {
+		return appState{}
+	}
+	return s
+}
+
+func saveState(s appState) {
+	p := statePath()
+	if p == "" {
+		return
+	}
+
+	_ = os.MkdirAll(filepath.Dir(p), 0700)
+	b, err := json.MarshalIndent(s, "", "  ")
+	if err != nil {
+		return
+	}
+	_ = os.WriteFile(p, b, 0600)
+}
+
+func updateState(workDir string) {
+	if workDir == "" {
+		return
+	}
+
+	abs, err := filepath.Abs(workDir)
+	if err != nil {
+		return
+	}
+
+	s := loadState()
+	s.ActiveWorkDir = abs
+
+	next := []string{abs}
+	for _, d := range s.RecentWorkDirs {
+		if d != abs && d != "" {
+			next = append(next, d)
+		}
+		if len(next) >= 10 {
+			break
+		}
+	}
+	s.RecentWorkDirs = next
+	saveState(s)
+}

internal/config/history.go

@@ -0,0 +1,140 @@
+package config
+
+import (
+	"bufio"
+	"encoding/json"
+	"os"
+	"path/filepath"
+	"strings"
+	"time"
+)
+
+type HistoryEntry struct {
+	Timestamp int64  `json:"timestamp"`
+	WorkDir   string `json:"work_dir"`
+	Section   string `json:"section"`
+	Action    string `json:"action"`
+	Success   bool   `json:"success"`
+	Note      string `json:"note,omitempty"`
+}
+
+func RecordHistory(workDir, section, action string, success bool, note string) {
+	p := historyPath()
+	if p == "" {
+		return
+	}
+	_ = os.MkdirAll(filepath.Dir(p), 0700)
+
+	e := HistoryEntry{
+		Timestamp: time.Now().UnixMilli(),
+		WorkDir:   workDir,
+		Section:   section,
+		Action:    action,
+		Success:   success,
+		Note:      strings.TrimSpace(note),
+	}
+
+	b, err := json.Marshal(e)
+	if err != nil {
+		return
+	}
+
+	f, err := os.OpenFile(p, os.O_APPEND|os.O_CREATE|os.O_WRONLY, 0600)
+	if err != nil {
+		return
+	}
+	defer f.Close()
+	_, _ = f.Write(append(b, '\n'))
+}
+
+func RecentHistory(limit int) []HistoryEntry {
+	if limit <= 0 {
+		limit = 20
+	}
+	p := historyPath()
+	if p == "" {
+		return nil
+	}
+	f, err := os.Open(p)
+	if err != nil {
+		return nil
+	}
+	defer f.Close()
+
+	var all []HistoryEntry
+	s := bufio.NewScanner(f)
+	for s.Scan() {
+		line := strings.TrimSpace(s.Text())
+		if line == "" {
+			continue
+		}
+		var e HistoryEntry
+		if json.Unmarshal([]byte(line), &e) == nil {
+			all = append(all, e)
+		}
+	}
+	if len(all) <= limit {
+		return all
+	}
+	return all[len(all)-limit:]
+}
+
+func HistorySuggestions(workDir string) []string {
+	recent := RecentHistory(60)
+	if len(recent) == 0 {
+		return []string{"Run Tools -> Setup / Reconfigure to initialize this project."}
+	}
+
+	var inDir []HistoryEntry
+	for _, e := range recent {
+		if e.WorkDir == workDir {
+			inDir = append(inDir, e)
+		}
+	}
+	if len(inDir) == 0 {
+		return []string{"No history for this repo yet. Start with Setup / Reconfigure."}
+	}
+
+	failures := 0
+	pushCount := 0
+	pullCount := 0
+	changeDirCount := 0
+	for _, e := range inDir {
+		if !e.Success {
+			failures++
+		}
+		if e.Action == "push" {
+			pushCount++
+		}
+		if e.Action == "pull" || e.Action == "smart_pull" {
+			pullCount++
+		}
+		if e.Action == "change_project_dir" {
+			changeDirCount++
+		}
+	}
+
+	out := make([]string, 0, 3)
+	if failures > 0 {
+		out = append(out, "Recent failures detected. Run Tools -> Doctor first.")
+	}
+	if pushCount >= 3 && pullCount == 0 {
+		out = append(out, "You push often without pulling. Use Smart Pull before push to avoid conflicts.")
+	}
+	if changeDirCount >= 2 {
+		out = append(out, "You switch repos often. Use Setup recent-directory shortcuts (1/2/3).")
+	}
+	if len(out) == 0 {
+		out = append(out, "Workflow looks healthy. Continue with Daily Git Operations.")
+	}
+	return out
+}
+
+func historyPath() string {
+	home, err := os.UserHomeDir()
+	if err != nil || home == "" {
+		return ""
+	}
+	return filepath.Join(home, ".git-genius", "history.jsonl")
+}
+

internal/doctor/doctor.go

@@ -2,7 +2,6 @@ package doctor
 
 import (
 	"os"
-	"path/filepath"
 
 	"git-genius/internal/config"
 	"git-genius/internal/github"
@@ -29,6 +28,7 @@ func Run() {
 	checkRemote()
 	checkGitHubToken()
 	checkGitHubRepo()
+	checkGitCredentialHelper()
 	checkErrorLog()
 
 	ui.Success("Doctor check completed")
@@ -174,14 +174,7 @@ func checkGitHubRepo() {
 }
 
 func checkErrorLog() {
-	cfg := config.Load()
-
-	logPath := filepath.Join(
-		cfg.GetWorkDir(),
-		".git",
-		".genius",
-		"error.log",
-	)
+	logPath := system.ErrorLogPath()
 
 	if _, err := os.Stat(logPath); err == nil {
 		ui.Warn("Error log exists")
@@ -190,3 +183,15 @@ func checkErrorLog() {
 		ui.Success("No error log found")
 	}
 }
+
+func checkGitCredentialHelper() {
+	out, err := system.GitOutput("config", "--global", "--get", "credential.helper")
+	if err != nil || out == "" {
+		return
+	}
+
+	if out == "!/usr/bin/gh auth git-credential" && !system.CommandExists("gh") {
+		ui.Warn("Git credential helper uses gh, but gh is not installed")
+		ui.Info("Push may still work, but auth helper warnings will appear")
+	}
+}