fix(mobile): display proposed plans in thread feed - #7264
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Needs human review This PR introduces new user-facing functionality by adding proposed plan display to the mobile thread feed. While the implementation follows existing web patterns and includes good test coverage, new features with new UI components and data flow warrant human review. You can customize Macroscope's approvability policy. Learn more. |
What Changed
thread.proposedPlansdata.Why
The mobile app was receiving the proposed-plan data but not adding it to the thread feed - completed plans were not visible in the conversation hindering usability of the plan mode in the mobile app.
REndering the data as a dedicated feed entry brings mobile in line with the web experience without changing contracts, provider adapters, or orchestration behavior. Keeping the plan outside the folded work entries ensures users can still find and copy the result after a turn completes.
UI Changes
Non-visible plan on the left, plan mode visible on the right (this PR).
Testing
vp test run apps/mobile/src/features/threads/proposedPlan.test.ts apps/mobile/src/lib/threadActivity.test.tsChecklist
Note
Display proposed plans as cards in the mobile thread feed
proposed-planfeed entry type tobuildThreadFeedin threadActivity.ts, pulling fromthread.proposedPlansand ordering them bycreatedAtalongside other entries.ProposedPlanCardcomponents in ThreadFeed.tsx, with assistant markdown styles, link press handling, and skill props.stripDisplayedPlanMarkdownin apps/web/src/proposedPlan.ts to correctly skip leading blank lines before dropping the title and summary headings.Macroscope summarized a982546.