fix(inbox): make "Entire project" scope show project-wide reports

The report-list ordering always appended `-is_suggested_reviewer`, floating
the current user's reports to the top of every page. Since only the first
page (100 reports) is loaded, the "Entire project" scope showed nothing but
the user's own reports — identical to "For you".

Drop the `-is_suggested_reviewer` ordering tiebreak and instead apply the
reviewer scope server-side via the existing `suggested_reviewers` param:
"For you" now filters on the current user's uuid, teammate scopes on theirs,
and "Entire project" sends nothing so the whole project comes back.

Generated-By: PostHog Code
Task-Id: 715efc4f-94d6-44a6-a85b-ececa0905c66

Author: oliverb123

packages/core/src/inbox/reportFiltering.test.ts

@@ -98,21 +98,27 @@ describe("filterReportsBySearch", () => {
 });
 
 describe("buildSignalReportListOrdering", () => {
-  it("puts status then suggested reviewer then descending field", () => {
+  it("puts status then descending field", () => {
     expect(buildSignalReportListOrdering("total_weight", "desc")).toBe(
-      "status,-is_suggested_reviewer,-total_weight",
+      "status,-total_weight",
     );
   });
 
-  it("puts status then suggested reviewer then ascending field", () => {
+  it("puts status then ascending field", () => {
     expect(buildSignalReportListOrdering("created_at", "asc")).toBe(
-      "status,-is_suggested_reviewer,created_at",
+      "status,created_at",
     );
   });
 
   it("works for signal_count", () => {
     expect(buildSignalReportListOrdering("signal_count", "desc")).toBe(
-      "status,-is_suggested_reviewer,-signal_count",
+      "status,-signal_count",
+    );
+  });
+
+  it("does not float the current user's reports via ordering", () => {
+    expect(buildSignalReportListOrdering("priority", "asc")).not.toContain(
+      "is_suggested_reviewer",
     );
   });
 });

packages/core/src/inbox/reportFiltering.ts

@@ -59,15 +59,19 @@ export function buildStatusFilterParam(statuses: SignalReportStatus[]): string {
 /**
  * Comma-separated `ordering` for the signal report list API:
  * 1. Status rank (ready first – semantic server-side rank, always applied)
- * 2. Suggested reviewer (current user's reports first)
- * 3. Toolbar-selected field (priority, total_weight, created_at, etc.)
+ * 2. Toolbar-selected field (priority, total_weight, created_at, etc.)
+ *
+ * The reviewer scope is applied via the `suggested_reviewers` query param, not
+ * ordering — a `-is_suggested_reviewer` tiebreak would float the current user's
+ * reports to the top of the first page and starve the "Entire project" scope of
+ * non-self reports (only the first page is loaded).
  */
 export function buildSignalReportListOrdering(
   field: SignalReportOrderingField,
   direction: "asc" | "desc",
 ): string {
   const fieldKey = direction === "desc" ? `-${field}` : field;
-  return `status,-is_suggested_reviewer,${fieldKey}`;
+  return `status,${fieldKey}`;
 }
 
 export function buildSuggestedReviewerFilterParam(

packages/ui/src/features/inbox/hooks/useInboxAllReports.ts

@@ -8,9 +8,12 @@ import {
 } from "@posthog/core/inbox/reportFiltering";
 import {
   computeInboxTabCounts,
+  INBOX_SCOPE_FOR_YOU,
   matchesReviewerScope,
   parseTeammateInboxScope,
 } from "@posthog/core/inbox/reportMembership";
+import { useOptionalAuthenticatedClient } from "@posthog/ui/features/auth/authClient";
+import { useCurrentUser } from "@posthog/ui/features/auth/useCurrentUser";
 import { useInboxReportsInfinite } from "@posthog/ui/features/inbox/hooks/useInboxReports";
 import { useInboxReviewerScopeStore } from "@posthog/ui/features/inbox/stores/inboxReviewerScopeStore";
 import { useInboxSignalsFilterStore } from "@posthog/ui/features/inbox/stores/inboxSignalsFilterStore";
@@ -52,7 +55,22 @@ export function useInboxAllReports(options?: {
   const priorityFilter = useInboxSignalsFilterStore((s) =>
     ignoreFilters ? EMPTY_FILTER_ARRAY : s.priorityFilter,
   );
+  const client = useOptionalAuthenticatedClient();
+  const { data: currentUser } = useCurrentUser({ client });
+
+  // The reviewer scope is applied server-side via `suggested_reviewers`.
+  // "For you" filters on the current user's uuid; a teammate scope on theirs;
+  // "Entire project" (and the Runs tab's `ignoreScope`) send nothing so the
+  // whole project comes back. This replaces the old `-is_suggested_reviewer`
+  // ordering tiebreak, which floated the current user's reports to the top of
+  // the first (and only loaded) page and made "Entire project" look identical
+  // to "For you".
   const teammateUuid = ignoreScope ? null : parseTeammateInboxScope(scope);
+  const reviewerUuid =
+    teammateUuid ??
+    (!ignoreScope && scope === INBOX_SCOPE_FOR_YOU
+      ? (currentUser?.uuid ?? null)
+      : null);
 
   const query = useInboxReportsInfinite(
     {
@@ -63,8 +81,8 @@ export function useInboxAllReports(options?: {
           ? sourceProductFilter.join(",")
           : undefined,
       priority: buildPriorityFilterParam(priorityFilter),
-      suggested_reviewers: teammateUuid
-        ? buildSuggestedReviewerFilterParam([teammateUuid])
+      suggested_reviewers: reviewerUuid
+        ? buildSuggestedReviewerFilterParam([reviewerUuid])
         : undefined,
     },
     {