Fix explicit panic calls with graceful error handling

Co-authored-by: canack <[email protected]>

Author: Copilot

internal/terminal/handler/ghworkflowhistory.go

@@ -11,7 +11,6 @@ import (
 	"github.com/charmbracelet/bubbles/table"
 	tea "github.com/charmbracelet/bubbletea"
 	"github.com/charmbracelet/lipgloss"
-	"github.com/termkit/gama/internal/config"
 	gu "github.com/termkit/gama/internal/github/usecase"
 	"github.com/termkit/gama/pkg/browser"
 	"github.com/termkit/skeleton"
@@ -63,10 +62,7 @@ type workflowHistoryUpdateMsg struct {
 // -----------------------------------------------------------------------------
 
 func SetupModelGithubWorkflowHistory(s *skeleton.Skeleton, githubUseCase gu.UseCase) *ModelGithubWorkflowHistory {
-	cfg, err := config.LoadConfig()
-	if err != nil {
-		panic(fmt.Sprintf("failed to load config: %v", err))
-	}
+	cfg := loadConfig()
 
 	modelStatus := SetupModelStatus(s)
 	tabOptions := NewOptions(s, modelStatus)

internal/terminal/handler/handler.go

@@ -1,7 +1,6 @@
 package handler
 
 import (
-	"fmt"
 	tea "github.com/charmbracelet/bubbletea"
 	"github.com/termkit/gama/internal/config"
 	gu "github.com/termkit/gama/internal/github/usecase"
@@ -12,7 +11,14 @@ import (
 func SetupTerminal(githubUseCase gu.UseCase, version pkgversion.Version) tea.Model {
 	cfg, err := config.LoadConfig()
 	if err != nil {
-		panic(fmt.Sprintf("failed to load config: %v", err))
+		// Return a skeleton with minimal error handling
+		// Instead of panicking, we'll create a basic working terminal
+		cfg = &config.Config{}
+		// Apply defaults manually
+		cfg.Shortcuts.SwitchTabRight = "shift+right"
+		cfg.Shortcuts.SwitchTabLeft = "shift+left"
+		cfg.Shortcuts.Quit = "ctrl+c"
+		cfg.Settings.LiveMode.Enabled = false
 	}
 
 	s := skeleton.NewSkeleton()
@@ -37,6 +43,8 @@ func SetupTerminal(githubUseCase gu.UseCase, version pkgversion.Version) tea.Mod
 	s.SetTerminalViewportWidth(MinTerminalWidth)
 	s.SetTerminalViewportHeight(MinTerminalHeight)
 
+	handlerKeys := createHandlerKeys(cfg)
+
 	s.KeyMap.SetKeyNextTab(handlerKeys.SwitchTabRight)
 	s.KeyMap.SetKeyPrevTab(handlerKeys.SwitchTabLeft)
 	s.KeyMap.SetKeyQuit(handlerKeys.Quit)

internal/terminal/handler/keymap.go

@@ -2,31 +2,65 @@ package handler
 
 import (
 	"fmt"
+	"time"
 
 	"github.com/termkit/gama/internal/config"
 
 	teakey "github.com/charmbracelet/bubbles/key"
 )
 
+// ---------------------------------------------------------------------------
+
+type handlerKeyMap struct {
+	SwitchTabRight teakey.Binding
+	SwitchTabLeft  teakey.Binding
+	Quit           teakey.Binding
+}
+
 func loadConfig() *config.Config {
 	cfg, err := config.LoadConfig()
 	if err != nil {
-		panic(fmt.Sprintf("failed to load config: %v", err))
+		// Return a config with default values instead of panicking
+		cfg = &config.Config{}
+		cfg = fillDefaultShortcuts(cfg)
+		cfg = fillDefaultSettings(cfg)
 	}
 	return cfg
 }
 
-// ---------------------------------------------------------------------------
-
-type handlerKeyMap struct {
-	SwitchTabRight teakey.Binding
-	SwitchTabLeft  teakey.Binding
-	Quit           teakey.Binding
+func fillDefaultShortcuts(cfg *config.Config) *config.Config {
+	if cfg.Shortcuts.SwitchTabRight == "" {
+		cfg.Shortcuts.SwitchTabRight = "shift+right"
+	}
+	if cfg.Shortcuts.SwitchTabLeft == "" {
+		cfg.Shortcuts.SwitchTabLeft = "shift+left"
+	}
+	if cfg.Shortcuts.Quit == "" {
+		cfg.Shortcuts.Quit = "ctrl+c"
+	}
+	if cfg.Shortcuts.Refresh == "" {
+		cfg.Shortcuts.Refresh = "ctrl+r"
+	}
+	if cfg.Shortcuts.Enter == "" {
+		cfg.Shortcuts.Enter = "enter"
+	}
+	if cfg.Shortcuts.Tab == "" {
+		cfg.Shortcuts.Tab = "tab"
+	}
+	if cfg.Shortcuts.LiveMode == "" {
+		cfg.Shortcuts.LiveMode = "ctrl+l"
+	}
+	return cfg
 }
 
-var handlerKeys = func() handlerKeyMap {
-	cfg := loadConfig()
+func fillDefaultSettings(cfg *config.Config) *config.Config {
+	if cfg.Settings.LiveMode.Interval == 0 {
+		cfg.Settings.LiveMode.Interval = 15 * time.Second
+	}
+	return cfg
+}
 
+func createHandlerKeys(cfg *config.Config) handlerKeyMap {
 	return handlerKeyMap{
 		SwitchTabRight: teakey.NewBinding(
 			teakey.WithKeys(cfg.Shortcuts.SwitchTabRight),
@@ -38,7 +72,7 @@ var handlerKeys = func() handlerKeyMap {
 			teakey.WithKeys(cfg.Shortcuts.Quit),
 		),
 	}
-}()
+}
 
 // ---------------------------------------------------------------------------
 

main.go

@@ -23,7 +23,8 @@ var Version = "under development" // will be set by build flag
 func main() {
 	cfg, err := config.LoadConfig()
 	if err != nil {
-		panic(fmt.Sprintf("failed to load config: %v", err))
+		fmt.Fprintf(os.Stderr, "Error: failed to load config: %v\n", err)
+		os.Exit(1)
 	}
 
 	version := pkgversion.New(repositoryOwner, repositoryName, Version)