fix(mobile): back from a finished subagent in the feed returns to its parent

[AKolenda]

Follow-up to #15068.

#15068 fixed the live subagent group card (thread-subagent-group.tsx) to push the agent's thread so Android back returns to the parent. But that card only shows while agents are running. Once an agent finishes, the entry point is the "Subagent … finished" row in the thread work log (thread-work-log.tsx), which still called navigate("Thread") — that reuses the current Thread route, drops the parent from the stack, and sends back straight to Home.

This flips that one call to StackActions.push, matching #15068, so opening a finished subagent from the feed and pressing back returns to the parent thread.

Before / after (Pixel 9, Android 17)

Opening the finished "Scan Supabase backend" subagent, then pressing back — left is current behaviour, right is this PR:

Note: the "Open agents" sheet (ThreadAgentsSheet.tsx) intentionally uses StackActions.replace and is left as-is.

Built with Claude Opus 5.5.

[macroscopeapp]

Approvability

Verdict: Approved at 9fe527e

Macroscope's review found this PR approvable — This is a small, self-contained mobile navigation bug fix that changes only the subagent-thread transition from route reuse to stack push, preserving the parent thread for back navigation. No product defaults, schemas, infrastructure, or static-analysis overrides are involved.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration

• Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: ad1862df-d0da-416c-877b-4862c4449515

📥 Commits

Reviewing files that changed from the base of the PR and between ed5b29b and 9f6f8be.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: ef52399d-5852-4b93-a038-c52cabd4723e

📥 Commits

Reviewing files that changed from the base of the PR and between 01f894e and 9fe527e.

📒 Files selected for processing (1)

• apps/mobile/src/features/threads/thread-work-log.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

Pressing a work-log row for a subagent notification now pushes the notified thread onto the navigation stack. The route receives the same environment and thread IDs.

Changes

Subagent thread navigation

Layer / File(s)	Summary
Push the notified thread apps/mobile/src/features/threads/thread-work-log.tsx	The notification row dispatches StackActions.push to the Thread route with the environment and notified thread IDs.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to 9fe52

The notified subagent opens above its parent thread, so returning from it preserves the intended parent-thread navigation. No merge-blocking risk is evident.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly describes the navigation fix for finished subagents and is specific to the main change.
Description check	✅ Passed	The description explains the problem and change, identifies the focused scope, and reports a device check with before-and-after evidence. It mostly covers the required template.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

_{Comment @coderabbitai help to get the list of available commands.}

[AKolenda]

Hey @Yash-Singh1 — small follow-up to #15068. Forgot to make back work from the finished "Subagent … finished" row in the feed (thread-work-log.tsx), so it still popped to Home there. Pushes it now like the group card does.