fix: improve error handling when user enters invalid new registry

Author: FredrikMWold

src/bubbles/app.go

@@ -71,6 +71,7 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 	var cmd tea.Cmd
 	switch msg := msg.(type) {
 	case tea.KeyMsg:
+		m.hasBannerMessage = false
 		switch {
 		case key.Matches(msg, m.keys.DownloadBlocks):
 			m.isDownloading = true

src/bubbles/new_registry_input/new_registry_input.go

@@ -4,6 +4,7 @@ import (
 	"github.com/charmbracelet/bubbles/textinput"
 	tea "github.com/charmbracelet/bubbletea"
 	"github.com/charmbracelet/lipgloss"
+	"github.com/fredrikmwold/jsrepo-tui/src/commands/manifest"
 	"github.com/fredrikmwold/jsrepo-tui/src/config"
 	"github.com/spf13/viper"
 )
@@ -39,7 +40,14 @@ func (m Model) Update(msg tea.Msg) (Model, tea.Cmd) {
 		switch msg.Type {
 		case tea.KeyEnter:
 			registries := viper.GetStringSlice("registries")
-			registries = append(registries, m.newRegistryInput.Value())
+			newRegistry := m.newRegistryInput.Value()
+			result := manifest.GetManifest(newRegistry)()
+			if _, ok := result.(manifest.ManifestErrorMessage); ok {
+				return m, func() tea.Msg {
+					return result
+				}
+			}
+			registries = append(registries, newRegistry)
 			viper.Set("registries", registries)
 			viper.WriteConfig()
 			return m, config.LoadConfig

src/commands/manifest/fetcher.go

@@ -24,7 +24,7 @@ func GetManifest(provider string) tea.Cmd {
 		url = fmt.Sprintf("%s%s", url, "/refs/heads/"+branch+"/jsrepo-manifest.json")
 		resp, err := http.Get(url)
 		if err != nil {
-			return ManifestErrorMessage(err.Error())
+			return ManifestErrorMessage("Error getting manifest from: " + provider)
 		}
 		defer resp.Body.Close()
 
@@ -35,7 +35,7 @@ func GetManifest(provider string) tea.Cmd {
 		}
 
 		if resp.StatusCode == 404 {
-			return ManifestErrorMessage("Manifest not found")
+			return ManifestErrorMessage("Error getting manifest from: " + provider)
 		}
 
 		var response ManifestResponse