fix: do not allow duplicate registries

Fixes: #2

Author: FredrikMWold

src/bubbles/new_registry_input/commands.go

@@ -0,0 +1,10 @@
+package newregistryinput
+
+import (
+	tea "github.com/charmbracelet/bubbletea"
+	"github.com/fredrikmwold/jsrepo-tui/src/commands/manifest"
+)
+
+func duplicateRegistryErrorMessage() tea.Msg {
+	return manifest.ManifestErrorMessage("You have already added this registry")
+}

src/bubbles/new_registry_input/new_registry_input.go

@@ -41,12 +41,14 @@ func (m Model) Update(msg tea.Msg) (Model, tea.Cmd) {
 		case tea.KeyEnter:
 			registries := viper.GetStringSlice("registries")
 			newRegistry := m.newRegistryInput.Value()
-			result := manifest.GetManifest(newRegistry)()
-			if _, ok := result.(manifest.ManifestErrorMessage); ok {
-				return m, func() tea.Msg {
-					return result
-				}
+			if m.checkIfDuplicateRegistry(newRegistry) {
+				return m, duplicateRegistryErrorMessage
 			}
+
+			if valid, cmd := m.checkIfValidRegistry(newRegistry); !valid {
+				return m, cmd
+			}
+
 			registries = append(registries, newRegistry)
 			viper.Set("registries", registries)
 			viper.WriteConfig()
@@ -68,3 +70,23 @@ func (m Model) View() string {
 		Padding(0, 1).
 		Render(m.newRegistryInput.View())
 }
+
+func (m Model) checkIfDuplicateRegistry(newRegistry string) bool {
+	registries := viper.GetStringSlice("registries")
+	for _, registry := range registries {
+		if registry == newRegistry {
+			return true
+		}
+	}
+	return false
+}
+
+func (m Model) checkIfValidRegistry(newRegistry string) (bool, tea.Cmd) {
+	result := manifest.GetManifest(newRegistry)()
+	if _, ok := result.(manifest.ManifestErrorMessage); ok {
+		return false, func() tea.Msg {
+			return result
+		}
+	}
+	return true, nil
+}