Apply suggestions from code review

Mostly nil reference checks, most (if not all) would be impossible to happen, but better safe than sorry.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: nullun

cmd/root.go

@@ -165,7 +165,7 @@ func runTUI(cmd *cobra.Command, algodData string, incentivesFlag bool, version s
 
 		// Display Hybrid Notice on launch
 		// Only shown if EnableP2PHybridMode is unset/false and hasn't already been set to "do not show again"
-		hybridEnabled := m.Data.Config.EnableP2PHybridMode != nil && *m.Data.Config.EnableP2PHybridMode
+		hybridEnabled := m.Data.Config != nil && m.Data.Config.EnableP2PHybridMode != nil && *m.Data.Config.EnableP2PHybridMode
 		if !hybridEnabled && algodutils.ShowHybridPopUp() {
 			p.Send(app.HybridModal)
 		}

internal/algod/config/config.go

@@ -7,7 +7,13 @@ type Config struct {
 
 // IsEqual compares two Config objects and returns true if all their fields have the same values, otherwise false.
 func (c Config) IsEqual(conf Config) bool {
-	return c.EnableP2PHybridMode == conf.EnableP2PHybridMode
+	if c.EnableP2PHybridMode == nil && conf.EnableP2PHybridMode == nil {
+		return true
+	}
+	if c.EnableP2PHybridMode == nil || conf.EnableP2PHybridMode == nil {
+		return false
+	}
+	return *c.EnableP2PHybridMode == *conf.EnableP2PHybridMode
 }
 
 // MergeAlgodConfigs merges two Config objects, with non-zero and non-default fields in 'b' overriding those in 'a'.

ui/modals/hybrid/model.go

@@ -70,7 +70,7 @@ func (m ViewModel) BorderColor() string {
 
 // Controls returns a formatted string displaying the available control options (yes or no) with styled color representations.
 func (m ViewModel) Controls() string {
-	hybridEnabled := m.State.Config.EnableP2PHybridMode != nil && *m.State.Config.EnableP2PHybridMode
+	hybridEnabled := m.State.Config != nil && m.State.Config.EnableP2PHybridMode != nil && *m.State.Config.EnableP2PHybridMode
 	controls := "| "
 	if !hybridEnabled && utils.ShowHybridPopUp() {
 		controls += style.Red.Render("(d)on't show again") + " | "

ui/status.go

@@ -94,7 +94,7 @@ func (m StatusViewModel) View() string {
 	// Last Round
 	row1 := lipgloss.JoinHorizontal(lipgloss.Left, beginning, middle, end)
 
-	if m.Data.Config.EnableP2PHybridMode != nil && *m.Data.Config.EnableP2PHybridMode {
+	if m.Data.Config != nil && m.Data.Config.EnableP2PHybridMode != nil && *m.Data.Config.EnableP2PHybridMode {
 		end = "P2P: " + style.Green.Render("YES") + " "
 	} else {
 		end = "P2P: " + style.Red.Render("NO") + " "