refactor(config): use strings.HasSuffix and filepath.Base for cleaner code

2 files · +3 -3

M internal/cli/styles/config.go +2 -2


@@ -2,6 +2,7 @@ package styles
2	2
3	3		import (
4	4		"fmt"
	5	+	"path/filepath"
5	6		"strings"
6	7
7	8		"github.com/charmbracelet/lipgloss"

@@ -75,8 +76,7 @@ func (r *ConfigRenderer) RenderMigrationSuccess(count int, path string) string {
75	76		pathStyle := r.theme.Subtle
76	77
77	78		// Get just the filename from the path
78		-	parts := strings.Split(path, "/")
79		-	filename := parts[len(parts)-1]
	79	+	filename := filepath.Base(path)
80	80
81	81		return fmt.Sprintf(
82	82		"\n %s Added %s new settings to %s\n",

 import (
 	"fmt"
+	"path/filepath"
 	"strings"
 	"github.com/charmbracelet/lipgloss"
 	pathStyle := r.theme.Subtle
 	// Get just the filename from the path
-	parts := strings.Split(path, "/")
+	filename := filepath.Base(path)
-	filename := parts[len(parts)-1]
 	return fmt.Sprintf(
 		"\n  %s Added %s new settings to %s\n",

M internal/infrastructure/config/migrate.go +1 -1


@@ -199,7 +199,7 @@ func (*Migrator) isUserDataSection(keyPath string) bool {
199	199		}
200	200
201	201		for _, suffix := range userDataSuffixes {
202		-	if len(keyPath) > len(suffix) && keyPath[len(keyPath)-len(suffix):] == suffix {
	202	+	if strings.HasSuffix(keyPath, suffix) {
203	203		return true
204	204		}
205	205		}


@@ -199,7 +199,7 @@ func (*Migrator) isUserDataSection(keyPath string) bool {
199		}	199		}
200			200
201		for _, suffix := range userDataSuffixes {	201		for _, suffix := range userDataSuffixes {
202	-	if len(keyPath) > len(suffix) && keyPath[len(keyPath)-len(suffix):] == suffix {	202	+	if strings.HasSuffix(keyPath, suffix) {
203		return true	203		return true
204		}	204		}
205		}	205		}