ref: Ensure All Queries Use Object Syntax

[nicholas-codecov]

Description

This PR updates a handful of queries to use the object syntax in preparation for the migration to TanStack v5.

Setup work for codecov/engineering-team#678

Notable Changes

• Update a handful of hooks to object syntax
• Update MemberTable and how it invalidates queries

[netlify]

✅ Deploy Preview for gazebo-staging ready!

Name	Link
🔨 Latest commit	37c9976
🔍 Latest deploy log	https://app.netlify.com/sites/gazebo-staging/deploys/654be3404211ce000804c6aa
😎 Deploy Preview	https://deploy-preview-2389--gazebo-staging.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[codecov-staging]

Codecov Report

Merging #2389 (37c9976) into main (38a6095) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2389      +/-   ##
==========================================
- Coverage   96.15%   96.15%   -0.01%     
==========================================
  Files         748      748              
  Lines        9327     9325       -2     
  Branches     2332     2288      -44     
==========================================
- Hits         8968     8966       -2     
- Misses        341      343       +2     
+ Partials       18       16       -2     

Files	Coverage Δ
...AdminAccess/AdminAccessTable/useAdminAccessList.js	100.00% <100.00%> (ø)
...inSettings/AdminMembers/MemberList/MemberTable.jsx	100.00% <100.00%> (ø)
src/services/commits/useCommits.js	100.00% <100.00%> (ø)
src/services/commits/useCommitsTeam.tsx	100.00% <ø> (ø)
src/services/pull/usePullTeam.tsx	100.00% <ø> (ø)
src/services/repo/useRepoFlags.js	100.00% <100.00%> (ø)
src/services/repo/useRepoFlagsSelect.js	100.00% <100.00%> (ø)
src/services/repos/useReposTeam.tsx	95.23% <100.00%> (ø)
src/services/selfHosted/useSelfHostedUserList.js	100.00% <100.00%> (ø)
src/services/users/useInfiniteUser.js	100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 38a6095...37c9976. Read the comment docs.

[codecov-qa]

Codecov Report

Merging #2389 (37c9976) into main (38a6095) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2389      +/-   ##
==========================================
- Coverage   96.15%   96.15%   -0.01%     
==========================================
  Files         748      748              
  Lines        9327     9325       -2     
  Branches     2332     2332              
==========================================
- Hits         8968     8966       -2     
  Misses        341      341              
  Partials       18       18              

Files	Coverage Δ
...AdminAccess/AdminAccessTable/useAdminAccessList.js	100.00% <100.00%> (ø)
...inSettings/AdminMembers/MemberList/MemberTable.jsx	100.00% <100.00%> (ø)
src/services/commits/useCommits.js	100.00% <100.00%> (ø)
src/services/commits/useCommitsTeam.tsx	100.00% <ø> (ø)
src/services/pull/usePullTeam.tsx	100.00% <ø> (ø)
src/services/repo/useRepoFlags.js	100.00% <100.00%> (ø)
src/services/repo/useRepoFlagsSelect.js	100.00% <100.00%> (ø)
src/services/repos/useReposTeam.tsx	95.23% <100.00%> (ø)
src/services/selfHosted/useSelfHostedUserList.js	100.00% <100.00%> (ø)
src/services/users/useInfiniteUser.js	100.00% <100.00%> (ø)

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 38a6095...37c9976. Read the comment docs.

[codecov]

Codecov Report

Merging #2389 (37c9976) into main (38a6095) will not change coverage.
The diff coverage is 100.00%.

@@          Coverage Diff          @@
##            main   #2389   +/-   ##
=====================================
  Coverage   96.15   96.15           
=====================================
  Files        748     748           
  Lines       9327    9325    -2     
  Branches    2319    2332   +13     
=====================================
- Hits        8968    8966    -2     
- Misses       341     343    +2     
+ Partials      18      16    -2     

Files	Coverage Δ
...AdminAccess/AdminAccessTable/useAdminAccessList.js	100.00% <100.00%> (ø)
...inSettings/AdminMembers/MemberList/MemberTable.jsx	100.00% <100.00%> (ø)
src/services/commits/useCommits.js	100.00% <100.00%> (ø)
src/services/commits/useCommitsTeam.tsx	100.00% <ø> (ø)
src/services/pull/usePullTeam.tsx	100.00% <ø> (ø)
src/services/repo/useRepoFlags.js	100.00% <100.00%> (ø)
src/services/repo/useRepoFlagsSelect.js	100.00% <100.00%> (ø)
src/services/repos/useReposTeam.tsx	95.23% <100.00%> (ø)
src/services/selfHosted/useSelfHostedUserList.js	100.00% <100.00%> (ø)
src/services/users/useInfiniteUser.js	100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 38a6095...37c9976. Read the comment docs.

[adrian-codecov]

You don't need the whole sets of params to invalidate the query right? Like if you had a query key with

['SelfHostedUserList', 'gh']
['SelfHostedUserList', 'gh', 'codecov']
['SelfHostedUserList']

That prompt will invalidate any of them right?

[nicholas-codecov]

I believe so, they're not 100% explicit, in the docs they say:

The invalidateQueries method can be used to invalidate and refetch single or multiple queries in the cache based on their query keys or any other functionally accessible property/state of the query. By default, all matching queries are immediately marked as invalid and active queries are refetched in the background.