Skip to content

fix(mobile): display proposed plans in thread feed - #7264

Open
tomhhealy wants to merge 2 commits into
pingdotgg:mainfrom
tomhhealy:fix/mobile-plan-display
Open

fix(mobile): display proposed plans in thread feed#7264
tomhhealy wants to merge 2 commits into
pingdotgg:mainfrom
tomhhealy:fix/mobile-plan-display

Conversation

@tomhhealy

@tomhhealy tomhhealy commented Aug 16, 2026

Copy link
Copy Markdown

What Changed

  • Add proposed plans to the mobile thread feed using the existing thread.proposedPlans data.
  • Display plans in a dedicated Markdown card with:
    • the plan title
    • copy support
    • expand/collapse controls for long plans
    • accessible labels and expansion state
  • Keep proposed plans visible when their completed turn is folded.
  • Respect the currently loaded message window when constructing the feed.
  • Add focused tests for plan formatting and feed presentation.
  • Document Plan Mode behavior for users.

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).

BeforeAfter
Plan draft after Plan draft after

Testing

  • vp test run apps/mobile/src/features/threads/proposedPlan.test.ts apps/mobile/src/lib/threadActivity.test.ts
  • 2 test files passed, 19 tests passed.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes

Note

Display proposed plans as cards in the mobile thread feed

  • Adds a proposed-plan feed entry type to buildThreadFeed in threadActivity.ts, pulling from thread.proposedPlans and ordering them by createdAt alongside other entries.
  • Renders proposed plans as ProposedPlanCard components in ThreadFeed.tsx, with assistant markdown styles, link press handling, and skill props.
  • Adds utility functions in proposedPlan.ts to extract the plan title, strip the title/summary heading from the displayed body, and build a collapsed preview with an ellipsis for long plans.
  • Fixes stripDisplayedPlanMarkdown in apps/web/src/proposedPlan.ts to correctly skip leading blank lines before dropping the title and summary headings.

Macroscope summarized a982546.

@coderabbitai

coderabbitai Bot commented Aug 16, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3e9a9040-5957-4748-b3ed-cb701c8126bf

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

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.

❤️ Share

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

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 16, 2026
Comment thread apps/mobile/src/features/threads/proposedPlan.ts
@macroscopeapp

macroscopeapp Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100-499 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant