fix(webapp): drop unused OrgMember _count aggregate from org-list presenter

[ericallam]

What

OrganizationsPresenter.#getOrganizations selected a Prisma _count.members relation on every org-list load (hit on nearly every dashboard navigation). Prisma lowers that relation _count to a whole-OrgMember-table GROUP BY organizationId aggregate joined onto Organization. The computed membersCount field is read by nothing in the webapp, so the entire aggregate scan is wasted work.

This removes the _count select and the membersCount field. The query keeps only the indexed EXISTS membership filter and the org/project selects.

Why it's safe

• membersCount has zero consumers (whole-webapp grep finds the name only at the point of assignment). It was added in Major dashboard improvements (environment centric) #1796 (2023) and has been unused since.
• The member count shown on the org settings/team page comes from a separate presenter query, not this one. No user-visible change.

Evidence (generated SQL, before/after, seeded isolated stack)

Before (with _count.members):

SELECT ..., COALESCE(aggr._aggr_count_members, 0)
FROM "Organization"
LEFT JOIN (SELECT "organizationId", COUNT(*) AS _aggr_count_members
           FROM "OrgMember" GROUP BY "organizationId") aggr ON ...
WHERE EXISTS (... "userId" = $1 ...) AND "deletedAt" IS NULL
ORDER BY "createdAt" DESC

After:

SELECT id, slug, title, avatar, "featureFlags"
FROM "Organization"
WHERE EXISTS (... "userId" = $1 ...) AND "deletedAt" IS NULL
ORDER BY "createdAt" DESC

The whole-table GROUP BY aggregate is gone. The only remaining OrgMember access is the EXISTS on the caller's own membership (indexed by userId, a handful of rows). This is the single largest read-amplification query on the control-plane database (~1.39B rows read/day, ~719s DB CPU/day per Insights); removing it takes that portion to zero.

Webapp typecheck passes.

Rollout

Straight deploy, zero blast radius. Rollback is a plain revert, no data migration.

refs TRI-13170

[changeset-bot]

⚠️ No Changeset found

Latest commit: 43e9ff6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[coderabbitai]

Walkthrough

The organization presenter no longer retrieves _count.members from the database. It also no longer includes membersCount in organization results. A server change note documents the removal of the unused calculation during dashboard navigation.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly describes removal of the unused organization member-count aggregate from the organization list presenter.
Description check	✅ Passed	The description explains the change, rationale, safety, evidence, testing, rollout, and rollback plan, but does not use all template sections.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/tri-13170-org-list-member-count-aggregate-reads-139b-rowsday-719s-db

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.