refactor(tui): simplify stale skill result handling tests

Generated with [codeagent](https://github.com/qbox/codeagent)
Co-authored-by: Cai-Tang-www <106404101+Cai-Tang-www@users.noreply.github.com>

Author: xgopilot

internal/tui/core/app/update.go

@@ -225,6 +225,7 @@ func (a App) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		requestSessionID := strings.TrimSpace(typed.RequestSessionID)
 		activeSessionID := strings.TrimSpace(a.state.ActiveSessionID)
 		if requestSessionID != "" && !strings.EqualFold(requestSessionID, activeSessionID) {
+			a.recordStaleSkillCommandResult(requestSessionID, activeSessionID, typed.Err)
 			return a, tea.Batch(cmds...)
 		}
 		if typed.Err != nil {
@@ -1701,6 +1702,15 @@ func (a *App) applyInlineCommandError(message string) {
 	a.rebuildTranscript()
 }
 
+// recordStaleSkillCommandResult 记录来自旧会话的技能命令结果，避免在会话切换后错误被静默丢弃。
+func (a *App) recordStaleSkillCommandResult(requestSessionID, activeSessionID string, runErr error) {
+	detail := fmt.Sprintf("result from session %q ignored after switching to %q", requestSessionID, activeSessionID)
+	if runErr != nil {
+		detail = fmt.Sprintf("%s; original error: %s", detail, runErr.Error())
+	}
+	a.appendActivity("skills", "Ignored stale skill command result", detail, runErr != nil)
+}
+
 func (a *App) appendActivity(kind string, title string, detail string, isError bool) {
 	previousCount := len(a.activities)
 	title = strings.TrimSpace(title)

internal/tui/core/app/update_test.go

@@ -4189,22 +4189,46 @@ func TestRebuildActivityWithHeightAndPersistPathGuard(t *testing.T) {
 	app.persistLogEntriesForActiveSession()
 }
 
+func updateWithSkillCommandResult(t *testing.T, app App, result skillCommandResultMsg) App {
+	t.Helper()
+
+	model, _ := app.Update(result)
+	return model.(App)
+}
+
+func assertIgnoredStaleSkillResultActivity(t *testing.T, app App, beforeActivities int, wantError bool) tuistate.ActivityEntry {
+	t.Helper()
+
+	if len(app.activities) != beforeActivities+1 {
+		t.Fatalf("expected stale skill result to be logged, got %d activities", len(app.activities))
+	}
+	last := app.activities[len(app.activities)-1]
+	if last.Title != "Ignored stale skill command result" {
+		t.Fatalf("expected stale result activity title, got %q", last.Title)
+	}
+	if last.IsError != wantError {
+		t.Fatalf("expected stale result error flag=%v, got %v", wantError, last.IsError)
+	}
+	return last
+}
+
 func TestUpdateIgnoresStaleSkillCommandResultBySession(t *testing.T) {
 	t.Parallel()
 
 	app, _ := newTestApp(t)
 	app.state.ActiveSessionID = "session-current"
 	app.state.StatusText = "before"
+	beforeActivities := len(app.activities)
 
-	model, _ := app.Update(skillCommandResultMsg{
+	app = updateWithSkillCommandResult(t, app, skillCommandResultMsg{
 		Notice:           "should be ignored",
 		RequestSessionID: "session-old",
 	})
-	app = model.(App)
 
 	if app.state.StatusText != "before" {
 		t.Fatalf("expected stale skill result to be ignored, got status %q", app.state.StatusText)
 	}
+	assertIgnoredStaleSkillResultActivity(t, app, beforeActivities, false)
 }
 
 func TestUpdateAcceptsSkillCommandResultForCurrentSession(t *testing.T) {
@@ -4213,13 +4237,34 @@ func TestUpdateAcceptsSkillCommandResultForCurrentSession(t *testing.T) {
 	app, _ := newTestApp(t)
 	app.state.ActiveSessionID = "session-current"
 
-	model, _ := app.Update(skillCommandResultMsg{
+	app = updateWithSkillCommandResult(t, app, skillCommandResultMsg{
 		Notice:           "Skill command completed.",
 		RequestSessionID: "session-current",
 	})
-	app = model.(App)
 
 	if app.state.StatusText != "Skill command completed." {
 		t.Fatalf("expected status to be updated, got %q", app.state.StatusText)
 	}
 }
+
+func TestUpdateLogsStaleSkillCommandErrorBySession(t *testing.T) {
+	t.Parallel()
+
+	app, _ := newTestApp(t)
+	app.state.ActiveSessionID = "session-current"
+	app.state.StatusText = "before"
+	beforeActivities := len(app.activities)
+
+	app = updateWithSkillCommandResult(t, app, skillCommandResultMsg{
+		Err:              errors.New("activate failed"),
+		RequestSessionID: "session-old",
+	})
+
+	if app.state.StatusText != "before" {
+		t.Fatalf("expected stale skill error to keep current status, got %q", app.state.StatusText)
+	}
+	last := assertIgnoredStaleSkillResultActivity(t, app, beforeActivities, true)
+	if !strings.Contains(last.Detail, "activate failed") {
+		t.Fatalf("expected stale error detail to include original error, got %q", last.Detail)
+	}
+}