PR Summary

Low Risk
Low risk: changes are limited to React Query cache invalidation timing/coverage, but could cause extra refetches since invalidations now run even on failed mutations.

Overview
Fixes stale credential UI by invalidating oauthCredentials queries whenever workspace credentials are created/updated/deleted, ensuring OAuth-backed credential selectors refresh immediately.

Updates all credential-set mutations to use onSettled (instead of onSuccess) so related membership/invitation/list caches are invalidated reliably even when a mutation errors.

^{Reviewed by Cursor Bugbot for commit ca32a83. Configure here.}

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 9, 2026 6:23pm

Greptile Summary

This PR adds cross-cache invalidation so that workspace credential mutations (create, update, delete) also invalidate oauthCredentials queries, ensuring OAuth credential-selector dropdowns refresh immediately. It also migrates all credential-sets.ts mutation handlers from onSuccess to onSettled for consistent cache reconciliation on both success and error paths.

The OAUTH_CREDENTIALS_KEY constant (['oauthCredentials']) correctly matches oauthCredentialKeys.all from oauth/oauth-credentials.ts, and the duplication is clearly documented to explain the circular-import constraint.

Confidence Score: 5/5

Safe to merge — changes are additive cache invalidations and non-breaking lifecycle handler migrations.

All findings are P2 or below. The OAUTH_CREDENTIALS_KEY constant correctly mirrors oauthCredentialKeys.all, onSettled signatures are correct for all 8 mutations in credential-sets.ts, and the three new cross-cache invalidations are logically correct for credential CRUD operations. No logic errors, data loss, or security concerns found.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant UI as Credential UI
    participant Mutation as useCreate/Update/DeleteWorkspaceCredential
    participant QC as QueryClient
    participant WC as workspaceCredential queries
    participant OC as oauthCredentials queries

    UI->>Mutation: mutate(payload)
    Mutation->>API: POST/PUT/DELETE /api/credentials
    API-->>Mutation: response
    Note over Mutation: onSettled (success or error)
    Mutation->>QC: invalidateQueries(workspaceCredentialKeys.lists())
    QC->>WC: refetch
    Mutation->>QC: invalidateQueries(['oauthCredentials'])
    QC->>OC: refetch
    OC-->>UI: credential-selector dropdowns update

_{Reviews (1): Last reviewed commit: "fix(credentials): add cross-cache invali..." | Re-trigger Greptile}