fix: improve error handling and clean up resource management in runner and tui components

Author: productdevbook

internal/runner/runner.go

@@ -536,7 +536,7 @@ func (r *Runner) startHTTPService(name string, state *ServiceState) {
 		}
 		return
 	}
-	defer resp.Body.Close()
+	defer func() { _ = resp.Body.Close() }()
 
 	body, _ := io.ReadAll(resp.Body)
 	bodyStr := strings.TrimSpace(string(body))

internal/tui/model.go

@@ -457,7 +457,8 @@ func copyToClipboard(text string) error {
 		return err
 	}
 
-	if err := cmd.Start(); err != nil {
+	err = cmd.Start()
+	if err != nil {
 		return err
 	}
 
@@ -466,7 +467,8 @@ func copyToClipboard(text string) error {
 		return err
 	}
 
-	if err := pipe.Close(); err != nil {
+	err = pipe.Close()
+	if err != nil {
 		return err
 	}
 

internal/types/types.go

@@ -39,16 +39,16 @@ type LogEntry struct {
 
 // ServiceInfo provides service status for TUI
 type ServiceInfo struct {
-	Name       string
-	Color      string
-	Icon       string        // custom icon/emoji
-	Running    bool
-	Logs       []LogLine
-	Type       string        // service, oneshot, interval, http
-	Status     ServiceStatus // detailed status
-	LastRun    time.Time     // last execution time
-	NextRun    time.Time     // next scheduled run (for interval)
-	ExitCode   int           // last exit code
-	RunCount   int           // number of runs (for interval)
-	Message    string        // dynamic status message from .devir-status
+	Name     string
+	Color    string
+	Icon     string // custom icon/emoji
+	Running  bool
+	Logs     []LogLine
+	Type     string        // service, oneshot, interval, http
+	Status   ServiceStatus // detailed status
+	LastRun  time.Time     // last execution time
+	NextRun  time.Time     // next scheduled run (for interval)
+	ExitCode int           // last exit code
+	RunCount int           // number of runs (for interval)
+	Message  string        // dynamic status message from .devir-status
 }