mirror of
https://github.com/shankar0123/certctl.git
synced 2026-06-12 00:58:53 +00:00
2057e76706
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.
188 lines
7.3 KiB
TypeScript
188 lines
7.3 KiB
TypeScript
import { useParams, useNavigate } from 'react-router-dom';
|
|
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';
|
|
import DataTable from '../components/DataTable';
|
|
import type { Column } from '../components/DataTable';
|
|
import ErrorState from '../components/ErrorState';
|
|
import { formatDateTime } from '../api/utils';
|
|
import type { Certificate, Issuer } from '../api/types';
|
|
import { typeLabels, redactConfig } from '../config/issuerTypes';
|
|
|
|
function InfoRow({ label, value }: { label: string; value: React.ReactNode }) {
|
|
return (
|
|
<div className="flex justify-between py-2 border-b border-surface-border/50">
|
|
<span className="text-sm text-ink-muted">{label}</span>
|
|
<span className="text-sm text-ink">{value}</span>
|
|
</div>
|
|
);
|
|
}
|
|
|
|
// Derive display status from backend `enabled` boolean.
|
|
//
|
|
// D-2 (diff-05x06-97fab8783a5c, master): pre-D-2 the fall-through here
|
|
// was `issuer.status || 'Unknown'`, but `Issuer.status` was a TS phantom
|
|
// the Go-side struct never emitted (see types.ts::Issuer docblock for the
|
|
// full closure rationale). Post-D-2 the phantom is gone; this function
|
|
// derives the displayed status from `enabled` exclusively.
|
|
function issuerStatus(issuer: Issuer): string {
|
|
return issuer.enabled ? 'Enabled' : 'Disabled';
|
|
}
|
|
|
|
export default function IssuerDetailPage() {
|
|
const { id } = useParams<{ id: string }>();
|
|
const navigate = useNavigate();
|
|
|
|
const { data: issuer, isLoading, error, refetch } = useQuery({
|
|
queryKey: ['issuer', id],
|
|
queryFn: () => getIssuer(id!),
|
|
enabled: !!id,
|
|
});
|
|
|
|
const { data: certsData } = useQuery({
|
|
queryKey: ['certificates', { issuer_id: id }],
|
|
queryFn: () => getCertificates({ issuer_id: id! }),
|
|
enabled: !!id,
|
|
});
|
|
|
|
const testMutation = useTrackedMutation({
|
|
mutationFn: () => testIssuerConnection(id!),
|
|
invalidates: [['issuer', id]],
|
|
});
|
|
|
|
if (error) {
|
|
return (
|
|
<>
|
|
<PageHeader title="Issuer Details" />
|
|
<ErrorState error={error as Error} onRetry={() => refetch()} />
|
|
</>
|
|
);
|
|
}
|
|
|
|
if (isLoading || !issuer) {
|
|
return (
|
|
<>
|
|
<PageHeader title="Issuer Details" />
|
|
<div className="flex items-center justify-center py-20">
|
|
<div className="text-sm text-ink-muted">Loading issuer...</div>
|
|
</div>
|
|
</>
|
|
);
|
|
}
|
|
|
|
const safeConfig = issuer.config ? redactConfig(issuer.config) : {};
|
|
|
|
const certColumns: Column<Certificate>[] = [
|
|
{
|
|
key: 'name',
|
|
label: 'Certificate',
|
|
render: (c) => (
|
|
<div>
|
|
<div className="font-medium text-ink text-sm">{c.common_name}</div>
|
|
<div className="text-xs text-ink-faint font-mono">{c.id}</div>
|
|
</div>
|
|
),
|
|
},
|
|
{ key: 'status', label: 'Status', render: (c) => <StatusBadge status={c.status} /> },
|
|
{ key: 'expires', label: 'Expires', render: (c) => <span className="text-xs text-ink-muted">{formatDateTime(c.expires_at)}</span> },
|
|
];
|
|
|
|
return (
|
|
<>
|
|
<PageHeader
|
|
title={issuer.name}
|
|
subtitle={typeLabels[issuer.type] || issuer.type}
|
|
action={
|
|
<div className="flex gap-2">
|
|
<button
|
|
onClick={() => navigate(`/issuers?edit=${issuer.id}`)}
|
|
className="px-3 py-1.5 border border-surface-border rounded text-ink text-xs hover:bg-surface-hover transition-colors font-medium"
|
|
>
|
|
Edit
|
|
</button>
|
|
<button
|
|
onClick={() => testMutation.mutate()}
|
|
disabled={testMutation.isPending}
|
|
className="btn btn-primary text-xs disabled:opacity-50"
|
|
>
|
|
{testMutation.isPending ? 'Testing...' : 'Test Connection'}
|
|
</button>
|
|
</div>
|
|
}
|
|
/>
|
|
|
|
<div className="flex-1 overflow-y-auto px-6 py-4 space-y-6">
|
|
{testMutation.isSuccess && (
|
|
<div className="px-4 py-2.5 bg-emerald-50 border border-emerald-200 rounded-lg text-sm text-emerald-700">
|
|
Connection test passed.
|
|
</div>
|
|
)}
|
|
{testMutation.isError && (
|
|
<div className="px-4 py-2.5 bg-red-50 border border-red-200 rounded-lg text-sm text-red-700">
|
|
Connection test failed: {(testMutation.error as Error).message}
|
|
</div>
|
|
)}
|
|
|
|
<div className="grid grid-cols-1 lg:grid-cols-2 gap-6">
|
|
{/* Issuer info */}
|
|
<div className="bg-surface border border-surface-border rounded p-5 shadow-sm">
|
|
<h3 className="text-sm font-semibold text-ink-muted mb-4">Issuer Information</h3>
|
|
<InfoRow label="ID" value={<span className="font-mono text-xs">{issuer.id}</span>} />
|
|
<InfoRow label="Name" value={issuer.name} />
|
|
<InfoRow label="Type" value={typeLabels[issuer.type] || issuer.type} />
|
|
<InfoRow label="Status" value={<StatusBadge status={issuerStatus(issuer)} />} />
|
|
<InfoRow label="Source" value={
|
|
<span className={`text-xs px-2 py-0.5 rounded-full ${
|
|
issuer.source === 'env' ? 'bg-amber-100 text-amber-700' : 'bg-blue-100 text-blue-700'
|
|
}`}>
|
|
{issuer.source === 'env' ? 'Environment Variable' : 'GUI Configured'}
|
|
</span>
|
|
} />
|
|
<InfoRow label="Connection Test" value={
|
|
issuer.test_status === 'success' ? (
|
|
<span className="text-xs text-emerald-600 font-medium">Passed {issuer.last_tested_at ? formatDateTime(issuer.last_tested_at) : ''}</span>
|
|
) : issuer.test_status === 'failed' ? (
|
|
<span className="text-xs text-red-600 font-medium">Failed {issuer.last_tested_at ? formatDateTime(issuer.last_tested_at) : ''}</span>
|
|
) : (
|
|
<span className="text-xs text-ink-faint">Not tested</span>
|
|
)
|
|
} />
|
|
<InfoRow label="Created" value={formatDateTime(issuer.created_at)} />
|
|
</div>
|
|
|
|
{/* Config (redacted) */}
|
|
<div className="bg-surface border border-surface-border rounded p-5 shadow-sm">
|
|
<h3 className="text-sm font-semibold text-ink-muted mb-4">Configuration</h3>
|
|
{Object.keys(safeConfig).length > 0 ? (
|
|
<div className="space-y-0">
|
|
{Object.entries(safeConfig).map(([key, val]) => (
|
|
<InfoRow key={key} label={key} value={
|
|
<span className="font-mono text-xs truncate max-w-xs inline-block">{String(val)}</span>
|
|
} />
|
|
))}
|
|
</div>
|
|
) : (
|
|
<div className="text-sm text-ink-faint py-4 text-center">No configuration data</div>
|
|
)}
|
|
</div>
|
|
</div>
|
|
|
|
{/* Issued certificates */}
|
|
<div className="bg-surface border border-surface-border rounded p-5 shadow-sm">
|
|
<h3 className="text-sm font-semibold text-ink-muted mb-4">
|
|
Issued Certificates {certsData ? `(${certsData.total})` : ''}
|
|
</h3>
|
|
<DataTable
|
|
columns={certColumns}
|
|
data={certsData?.data || []}
|
|
isLoading={!certsData}
|
|
emptyMessage="No certificates issued by this issuer"
|
|
/>
|
|
</div>
|
|
</div>
|
|
</>
|
|
);
|
|
}
|