fix: sanitize cached PR titles before writing state.yml

pkg/config/app_config.go

@@ -722,6 +722,14 @@ type CachedPullRequest struct {
 	HeadRepositoryOwner string `yaml:"headRepositoryOwner"`
 }
 
+// MarshalYAML trims fields that break YAML round-trips when cached to state.yml.
+func (pr CachedPullRequest) MarshalYAML() (interface{}, error) {
+	type cachedPullRequest CachedPullRequest
+	sanitized := cachedPullRequest(pr)
+	sanitized.Title = strings.TrimSpace(pr.Title)
+	return sanitized, nil
+}
+
 func getDefaultAppState() *AppState {
 	return &AppState{
 		GithubPullRequests: make(map[string][]CachedPullRequest),

pkg/config/cached_pull_request_test.go

@@ -0,0 +1,32 @@
+package config
+
+import (
+	"testing"
+
+	"gopkg.in/yaml.v3"
+)
+
+func TestCachedPullRequestTitleYAMLRoundTrip(t *testing.T) {
+	t.Parallel()
+
+	state := AppState{
+		GithubPullRequests: map[string][]CachedPullRequest{
+			"/tmp/repo": {{
+				HeadRefName: "example-branch",
+				Number:      9,
+				Title:       "\nExample pull request title",
+				State:       "MERGED",
+			}},
+		},
+	}
+
+	data, err := yaml.Marshal(&state)
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	var decoded AppState
+	if err := yaml.Unmarshal(data, &decoded); err != nil {
+		t.Fatalf("yaml unmarshal failed: %v\n%s", err, data)
+	}
+}

pkg/gui/controllers/helpers/refresh_helper.go

@@ -973,7 +973,7 @@ func (self *RefreshHelper) savePullRequestsToCache(prs []*models.GithubPullReque
 		return config.CachedPullRequest{
 			HeadRefName:         pr.HeadRefName,
 			Number:              pr.Number,
-			Title:               pr.Title,
+			Title:               strings.TrimSpace(pr.Title),
 			State:               pr.State,
 			Url:                 pr.Url,
 			HeadRepositoryOwner: pr.HeadRepositoryOwner.Login,