M-029 Pass 1 batch 1: migrate 4 single-mutation pages to useTrackedMutation

Drains the Bundle 8 useMutation backlog (56 -> 52). Each migration declares

explicit invalidates per the M-009 contract; the wrapper invalidates BEFORE

calling the caller's onSuccess so user code drops the redundant qc.invalidateQueries.

Pages migrated:

  - AgentsPage.tsx        invalidates: [['agents'], ['agents', 'retired']]

  - CertificatesPage.tsx  invalidates: [['certificates']]

  - DigestPage.tsx        invalidates: 'noop' (sendDigest is a server-side email

                            dispatch — no client query reflects digest-send state)

  - IssuerDetailPage.tsx  invalidates: [['issuer', id]] (testIssuerConnection

                            updates last_tested_at server-side)

Verification:

  legacy useMutation count   56 -> 52 (-4 sites)

  useTrackedMutation count    0 ->  4 (+4 sites)

  invalidation surface      82 -> 84 (+2; DigestPage is noop, AgentsPage

                                  collapses 2 invalidates into 1 array, others +1)

Closes 4 of 56 sites toward M-029 Pass 1 completion.
This commit is contained in:
shankar0123
2026-04-27 02:37:25 +00:00
parent 0b58662e9a
commit 2057e76706
4 changed files with 17 additions and 12 deletions
+4 -5
View File
@@ -1,6 +1,7 @@
import { useState } from 'react';
import { useNavigate } from 'react-router-dom';
import { useQuery, useMutation, useQueryClient } from '@tanstack/react-query';
import { useQuery } from '@tanstack/react-query';
import { useTrackedMutation } from '../hooks/useTrackedMutation';
import {
getAgents,
listRetiredAgents,
@@ -45,7 +46,6 @@ type ModalMode =
export default function AgentsPage() {
const navigate = useNavigate();
const qc = useQueryClient();
const [tab, setTab] = useState<TabKey>('active');
const [modal, setModal] = useState<ModalMode>({ kind: 'closed' });
@@ -67,12 +67,11 @@ export default function AgentsPage() {
// and we invalidate both queries on success so the retired tab refreshes and
// the active tab drops the row. 409s are converted into modal.mode=blocked so
// the operator can escalate to force; everything else becomes modal.mode=error.
const mutation = useMutation({
const mutation = useTrackedMutation({
mutationFn: (input: { agent: Agent; force?: boolean; reason?: string }) =>
retireAgent(input.agent.id, { force: input.force, reason: input.reason }),
invalidates: [['agents'], ['agents', 'retired']],
onSuccess: () => {
qc.invalidateQueries({ queryKey: ['agents'] });
qc.invalidateQueries({ queryKey: ['agents', 'retired'] });
setModal({ kind: 'closed' });
},
});