Merge fix/M-029-pass1-batch1: 4 single-mutation pages migrated

This commit is contained in:
shankar0123
2026-04-27 02:37:30 +00:00
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' });
},
});
+4 -2
View File
@@ -1,5 +1,6 @@
import { useState } from 'react';
import { useQuery, useMutation, useQueryClient } from '@tanstack/react-query';
import { useQuery, useQueryClient } from '@tanstack/react-query';
import { useTrackedMutation } from '../hooks/useTrackedMutation';
import { useNavigate } from 'react-router-dom';
import { getCertificates, createCertificate, revokeCertificate, getOwners, getTeams, getRenewalPolicies, getProfiles, getIssuers, bulkRevokeCertificates, bulkRenewCertificates, bulkReassignCertificates } from '../api/client';
import { useAuth } from '../components/AuthProvider';
@@ -73,7 +74,7 @@ function CreateCertificateModal({ onClose, onSuccess }: { onClose: () => void; o
: `${Math.round(selectedProfile.max_ttl_seconds / 86400)}d`
: null;
const mutation = useMutation({
const mutation = useTrackedMutation({
mutationFn: () => {
const payload: Record<string, unknown> = { ...form };
// Convert comma-separated SANs to array
@@ -95,6 +96,7 @@ function CreateCertificateModal({ onClose, onSuccess }: { onClose: () => void; o
}
return createCertificate(payload);
},
invalidates: [['certificates']],
onSuccess: () => onSuccess(),
onError: (err: Error) => setError(err.message),
});
+5 -2
View File
@@ -1,5 +1,6 @@
import { useState } from 'react';
import { useQuery, useMutation } from '@tanstack/react-query';
import { useQuery } from '@tanstack/react-query';
import { useTrackedMutation } from '../hooks/useTrackedMutation';
import { previewDigest, sendDigest } from '../api/client';
import PageHeader from '../components/PageHeader';
import ErrorState from '../components/ErrorState';
@@ -13,8 +14,10 @@ export default function DigestPage() {
retry: false,
});
const sendMutation = useMutation({
const sendMutation = useTrackedMutation({
mutationFn: sendDigest,
invalidates: 'noop',
noopReason: 'sendDigest dispatches an email server-side; no cached client query reflects digest-send state',
onSuccess: () => setShowConfirm(false),
});
+4 -3
View File
@@ -1,5 +1,6 @@
import { useParams, useNavigate } from 'react-router-dom';
import { useQuery, useMutation } from '@tanstack/react-query';
import { useQuery } from '@tanstack/react-query';
import { useTrackedMutation } from '../hooks/useTrackedMutation';
import { getIssuer, testIssuerConnection, getCertificates } from '../api/client';
import PageHeader from '../components/PageHeader';
import StatusBadge from '../components/StatusBadge';
@@ -46,9 +47,9 @@ export default function IssuerDetailPage() {
enabled: !!id,
});
const testMutation = useMutation({
const testMutation = useTrackedMutation({
mutationFn: () => testIssuerConnection(id!),
onSuccess: () => refetch(),
invalidates: [['issuer', id]],
});
if (error) {