feat(inbox): Source product icons, PR badges, and rendering improvements by Twixes · Pull Request #1755 · PostHog/code

Twixes · 2026-04-21T10:12:35Z

Problem

The report list in Code lacked indication for source products and PR status. Users couldn't quickly identify what type of signal initiated a report or whether a PR already exists. Some of the spacing was rough too, and we were losing a lot of info due to truncated headings or description.

Changes

A few things (as summarized by Claude):

Source product icon: Replace generic FileTextIcon with colored source product icon (from SOURCE_PRODUCT_META) in report list rows, with tooltip "Report initiated by a signal from X"
PR badge in list: Show a mini PR badge pill in ReportCardContent when implementation_pr_url is set, with merged/closed/open color states via usePrDetails
PR badge in task bar: Move the PR badge from the detail header into the implementation task bar, showing "Create PR" / "Working on a PR…" / PR link badge depending on state
Clean up detail header: Remove PR badge, implementationPrUrl state, and onPrUrlChange callback from ReportDetailPane
Layout polish: Tighter list row spacing, improved summary markdown rendering (single-line, 12px font normalization)

Backend companion: PostHog/posthog#55464

How did you test this code?

Recorded a demo.

greptile-apps · 2026-04-21T10:14:53Z

Prompt To Fix All With AI

This is a comment left during a code review.
Path: apps/code/src/renderer/features/inbox/components/utils/ReportCardContent.tsx
Line: 80-82

Comment:
**Per-row `usePrDetails` calls in the list**

`ReportImplementationPrLink` calls `usePrDetails` (a tRPC + GitHub fetch) for every list row that has an `implementation_pr_url`. Previously this hook was only triggered when a single detail pane was open. With a list of N reports that all have PRs, this fires N parallel queries on mount. React Query's 60 s `staleTime` prevents re-fetching for cached URLs, but the cold-load burst could saturate the GitHub API or the main-process queue. Consider lazy-loading PR status only when a row is visible (e.g. via an `IntersectionObserver` or a separate hover-to-load trigger).

How can I resolve this? If you propose a fix, please make it concise.

---

This is a comment left during a code review.
Path: apps/code/src/renderer/features/inbox/components/list/ReportListRow.tsx
Line: 9-28

Comment:
**Multi-product tooltip implies single source**

When `sourceProducts` has more than one entry, only the first is used, and the tooltip reads "Report initiated by a signal from X" — implying a single source. If a report legitimately aggregates signals from multiple products, the icon and tooltip silently drop the rest. Either show a generic icon with an aggregate tooltip (e.g. "…from Product Analytics, Session Replay") or document that only the primary product is surfaced here.

How can I resolve this? If you propose a fix, please make it concise.

_{Reviews (1): Last reviewed commit: "feat(inbox): source product icons, PR ba..." | Re-trigger Greptile}

…nts in report list - Show source product icon (with tooltip) in report list rows - Show mini PR badge in report list when implementation_pr_url is set - Move PR badge from detail header to implementation task bar - Show "Working on a PR…" when implementation task is in progress - Extract ReportImplementationPrLink component with merged/closed states - Add source_products and implementation_pr_url to SignalReport type - Tighter list row layout, improved summary markdown rendering Depends on PostHog/posthog#55464 for backend source_products and implementation_pr_url fields.

joshsny

Nice! 🚢🚢

joshsny · 2026-04-21T18:21:10Z

+          ? "bg-red-4 text-red-11 hover:bg-red-5"
+          : "bg-green-4 text-green-11 hover:bg-green-5";
+
+  const tooltip = merged


Should we give this some kinda styling or something nice to go see the link?

We could make it a bit nicer. The button itself is already a link, so just visuals here. @cursoragent Make this tooltip show the PR in the format org/repo#pr_number, e.g. PostHog/posthog#2323

Co-authored-by: Michael Matloka <dev@twixes.com>

…ability ordering to reports (#55464) ## Problem The signal reports API lacks metadata needed by PostHog Code to show source product icons and PR badges in the inbox list, and the inbox ordering doesn't distinguish between actionable and not-actionable ready reports. ## Changes As summarized by Claude: - Added `source_products` field to `SignalReportSerializer` — batch-fetched from ClickHouse per page via `fetch_source_products_for_reports()` - Added `implementation_pr_url` field — annotated via correlated subquery through `SignalReportTask` → `Task` → `TaskRun` → `output->'pr_url'` - Split ready status ordering by actionability: rank 0 = actionable (or no judgment), rank 1 = not_actionable - Suppressed `is_suggested_reviewer` for not_actionable reports - Extracted `_Q_READY_NOT_ACTIONABLE` shared constant to avoid duplicate Q expressions - Dropped legacy `choice` fallback in serializer (only `actionability` key is read now) - Passed serializer context in the `signals` detail action See what this looks like in the Code app in the companion PR: PostHog/code#1755 ## How did you test this code? Bunch of tests. ## Publish to changelog? No

greptile-apps Bot reviewed Apr 21, 2026

View reviewed changes

Comment thread apps/code/src/renderer/features/inbox/components/utils/ReportCardContent.tsx

Comment thread apps/code/src/renderer/features/inbox/components/list/ReportListRow.tsx

Twixes force-pushed the twixes/report-pr-rendering branch from 5505938 to 0765488 Compare April 21, 2026 15:45

Twixes changed the title ~~feat(inbox): source product icons, PR badges, and rendering improvements~~ feat(inbox): Source product icons, PR badges, and rendering improvements Apr 21, 2026

Twixes force-pushed the twixes/report-pr-rendering branch 2 times, most recently from fcb4bdf to 20c2118 Compare April 21, 2026 16:36

Twixes force-pushed the twixes/report-pr-rendering branch from 20c2118 to dea86d7 Compare April 21, 2026 16:37

Twixes mentioned this pull request Apr 21, 2026

feat(signals): Add source_products, implementation_pr_url, and actionability ordering to reports PostHog/posthog#55464

Merged

Twixes requested a review from a team April 21, 2026 17:28

joshsny approved these changes Apr 21, 2026

View reviewed changes

sortafreel approved these changes Apr 22, 2026

View reviewed changes

cursoragent and others added 4 commits April 22, 2026 09:40

Merge origin/main into twixes/report-pr-rendering

12aec0a

Co-authored-by: Michael Matloka <dev@twixes.com>

fix(inbox): format PR link tooltip as org/repo#number

5f862ee

Co-authored-by: Michael Matloka <dev@twixes.com>

style(inbox): satisfy biome formatting for PR tooltip parser

1702017

Co-authored-by: Michael Matloka <dev@twixes.com>

Merge origin/main into twixes/report-pr-rendering

5996296

Co-authored-by: Michael Matloka <dev@twixes.com>

Twixes merged commit 723c58c into main Apr 22, 2026
15 checks passed

Twixes deleted the twixes/report-pr-rendering branch April 22, 2026 13:23

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(inbox): Source product icons, PR badges, and rendering improvements#1755

feat(inbox): Source product icons, PR badges, and rendering improvements#1755
Twixes merged 5 commits intomainfrom
twixes/report-pr-rendering

Twixes commented Apr 21, 2026 •

edited

Loading

Uh oh!

greptile-apps Bot commented Apr 21, 2026

Uh oh!

Uh oh!

Uh oh!

joshsny left a comment

Uh oh!

joshsny Apr 21, 2026

Uh oh!

Twixes Apr 22, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

Twixes commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Problem

Changes

How did you test this code?

Uh oh!

greptile-apps Bot commented Apr 21, 2026

Uh oh!

Uh oh!

Uh oh!

joshsny left a comment

Choose a reason for hiding this comment

Uh oh!

joshsny Apr 21, 2026

Choose a reason for hiding this comment

Uh oh!

Twixes Apr 22, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Twixes commented Apr 21, 2026 •

edited

Loading