Skip to content

fix(mobile): break signal report summary sections onto separate lines (port #2317)#2543

Open
Gilbert09 wants to merge 1 commit into
mainfrom
posthog-code/mobile-signal-summary-line-breaks
Open

fix(mobile): break signal report summary sections onto separate lines (port #2317)#2543
Gilbert09 wants to merge 1 commit into
mainfrom
posthog-code/mobile-signal-summary-line-breaks

Conversation

@Gilbert09

Copy link
Copy Markdown
Member

Ports #2317 to the mobile app.

Problem

Agent-generated signal report summaries often pack the labelled sections onto a single line, e.g.:

What's happening: text Root cause: more text How to resolve: ...

which renders as one run-on paragraph in MarkdownText.

Change

  • Adds formatSignalReportSummaryMarkdown to apps/mobile/src/features/inbox/utils.ts. It inserts blank lines before each bold section header (**What's happening:**, **Root cause:**, **How to resolve:**) and after the header label, so each section and its body render on their own line. Matching is case-insensitive.
  • Applies it to the summary passed into MarkdownText in both render sites:
    • the report detail screen (apps/mobile/src/app/inbox/[...id].tsx)
    • the expanded report card in TinderView.tsx
  • Adds unit tests covering packed headers, already-separated headers, header-free content, and case-insensitive matching.

Pure presentation formatting in the renderer layer — no main-process or business-logic changes.

Testing

  • pnpm --filter mobile test — 202 passed
  • pnpm --filter mobile lint — clean

… (port #2317)

Agent-generated signal report summaries often pack the labelled sections
(**What's happening:**, **Root cause:**, **How to resolve:**) onto a single
line. Run the summary markdown through a new formatSignalReportSummaryMarkdown
util before rendering it via MarkdownText, so each section header and its body
land on their own line.

Applied in the report detail screen and the expanded Tinder card. Ports the
desktop change in #2317.

Generated-By: PostHog Code
Task-Id: 73013ebe-bbe1-49b3-8739-1cf570a37c9d
@Gilbert09 Gilbert09 requested a review from a team June 9, 2026 10:38
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit a719284.

@greptile-apps

greptile-apps Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(mobile): break signal report summary..." | Re-trigger Greptile

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant