feat(cli): add max history length

cmd/history_cmd.go

@@ -11,7 +11,8 @@ import (
 
 var historyCmd = &cobra.Command{
 	Use:   "history",
-	Short: "Zeigt die Befehlshistorie an",
+	Short: "show command history",
+	Long:  "Shows history of the last 2000 commands",
 	Run: func(cmd *cobra.Command, args []string) {
 		dirname, err := os.UserConfigDir()
 		if err != nil {

utils/utils.go

@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"os"
 	"path/filepath"
+	"strings"
 	"time"
 
 	"github.com/charmbracelet/huh"
@@ -34,23 +35,68 @@ func max(a, b int) int {
 	return b
 }
 
+const (
+	maxHistoryLines = 2000
+	historyFileName = "sst_history"
+)
+
+// func LogToHistory(action, pkg, manager string) error {
+// 	dirname, err := os.UserConfigDir()
+// 	if err != nil {
+// 		return fmt.Errorf("cant obtain config dir: %v", err)
+// 	}
+// 	historyFile := filepath.Join(dirname, "sst", "sst_history")
+// 	f, err := os.OpenFile(historyFile, os.O_APPEND|os.O_CREATE|os.O_WRONLY, 0644)
+// 	if err != nil {
+// 		return err
+// 	}
+// 	defer f.Close()
+
+// 	timestamp := time.Now().Format("2006-01-02 15:04:05")
+// 	logEntry := fmt.Sprintf("%s: action: [%s] - package: [%s] - packagemanager: [%s]\n", timestamp, action, pkg, manager)
+
+// 	if _, err := f.WriteString(logEntry); err != nil {
+// 		return err
+// 	}
+// 	return nil
+// }
+//
+
 func LogToHistory(action, pkg, manager string) error {
 	dirname, err := os.UserConfigDir()
 	if err != nil {
 		return fmt.Errorf("cant obtain config dir: %v", err)
 	}
-	historyFile := filepath.Join(dirname, "sst", "sst_history")
+	historyFile := filepath.Join(dirname, "sst", historyFileName)
-	f, err := os.OpenFile(historyFile, os.O_APPEND|os.O_CREATE|os.O_WRONLY, 0644)
+
+	lines, err := readHistoryLines(historyFile)
 	if err != nil {
 		return err
 	}
-	defer f.Close()
 
 	timestamp := time.Now().Format("2006-01-02 15:04:05")
-	logEntry := fmt.Sprintf("%s: action: [%s] - package: [%s] - packagemanager: [%s]\n", timestamp, action, pkg, manager)
+	newLine := fmt.Sprintf("%s: action: [%s] - package: [%s] - packagemanager: [%s]", timestamp, action, pkg, manager)
+	lines = append(lines, newLine)
 
-	if _, err := f.WriteString(logEntry); err != nil {
+	if len(lines) > maxHistoryLines {
-		return err
+		lines = lines[len(lines)-maxHistoryLines:]
 	}
-	return nil
+
+	return writeHistoryLines(historyFile, lines)
+}
+
+func readHistoryLines(filePath string) ([]string, error) {
+	content, err := os.ReadFile(filePath)
+	if err != nil {
+		if os.IsNotExist(err) {
+			return []string{}, nil
+		}
+		return nil, err
+	}
+	return strings.Split(string(content), "\n"), nil
+}
+
+func writeHistoryLines(filePath string, lines []string) error {
+	content := strings.Join(lines, "\n")
+	return os.WriteFile(filePath, []byte(content), 0644)
 }