Rename drawer discovery to analysis

This commit is contained in:
rcourtman
2026-03-23 11:58:27 +00:00
parent ce93cbbe9a
commit 37b198299e
5 changed files with 15 additions and 15 deletions
@@ -434,7 +434,7 @@ Inside that summary, the operational and supporting context rows stay inline
instead of sitting in a collapsed `Details` disclosure or nested bordered
cards, so the first read stays like one linear sheet rather than a stack of
cards inside the summary.
Discovery support now also lives inside an overview-only `Discovery`
Discovery support now also lives inside an overview-only `Analysis`
instead of a peer drawer tab, so supplemental discovery detail stays available
without claiming the same navigation weight as runtime, identity, or
service-specific operational views.
@@ -443,8 +443,8 @@ an on-demand metadata panel, so the actionable web-interface path stays primary
while deeper discovery inspection remains available without reading like a
second peer overview surface.
For ordinary host discovery, that support row now stays even quieter: the
surface is titled `Discovery`, and the collapsed state does not repeat a
baseline `Host discovery via <hostname>` summary when the discovery target is
surface is titled `Analysis`, and the collapsed state does not repeat a
baseline `Host analysis via <hostname>` summary when the discovery target is
just the same host identity already shown elsewhere in the drawer.
The discovery metadata panel now expands directly under the outer support
disclosure instead of nesting a second bordered card, so the support surface
@@ -1064,12 +1064,12 @@ export const ResourceDetailDrawerOverviewTab: Component<ResourceDetailDrawerOver
/>
<SupportDisclosure
title="Discovery"
title="Analysis"
summary={drawer.discoveryContextSummary()}
expanded={drawer.showDiscoveryContext()}
onToggle={() => drawer.setShowDiscoveryContext((value) => !value)}
showLabel="Open discovery"
hideLabel="Hide discovery"
showLabel="Open analysis"
hideLabel="Hide analysis"
class="h-full"
dataTestId="resource-discovery-context"
>
@@ -339,11 +339,11 @@ describe('ResourceDetailDrawer change history section', () => {
expect(screen.queryByText('Container Updates')).toBeNull();
expect(screen.queryByText('Check Updates')).toBeNull();
expect(screen.queryByText('Show update controls')).toBeNull();
expect(screen.getByText('Discovery')).toBeInTheDocument();
expect(screen.getByText('Analysis')).toBeInTheDocument();
expect(
screen.queryByText('Supporting metadata only. The web interface path above stays primary.'),
).toBeNull();
expect(screen.getByRole('button', { name: 'Open discovery' })).toBeInTheDocument();
expect(screen.getByRole('button', { name: 'Open analysis' })).toBeInTheDocument();
expect(
screen
.getByTestId('resource-discovery-context')
@@ -142,9 +142,9 @@ describe('ResourceDetailDrawer runtime and identity cards', () => {
() => <ResourceDetailDrawer resource={baseResource({})} />,
);
expect(queryByRole('button', { name: 'Discovery' })).toBeNull();
expect(getByText('Discovery')).toBeInTheDocument();
expect(queryByText('Host discovery via host-1')).toBeNull();
expect(queryByRole('button', { name: 'Analysis' })).toBeNull();
expect(getByText('Analysis')).toBeInTheDocument();
expect(queryByText('Host analysis via host-1')).toBeNull();
expect(
queryByText('Supporting metadata only. The web interface path above stays primary.'),
).toBeNull();
@@ -155,9 +155,9 @@ describe('ResourceDetailDrawer runtime and identity cards', () => {
),
).toBeNull();
expect(getByRole('button', { name: 'Open discovery' })).toBeInTheDocument();
expect(getByRole('button', { name: 'Open analysis' })).toBeInTheDocument();
fireEvent.click(getByRole('button', { name: 'Open discovery' }));
fireEvent.click(getByRole('button', { name: 'Open analysis' }));
await waitFor(() => {
expect(queryByTestId('discovery-tab')).toBeInTheDocument();
@@ -240,8 +240,8 @@ export const useResourceDetailDrawerDerivedState = (
const discoveryMode =
config.resourceType === 'agent'
? 'Host discovery'
: `${formatIdentifierLabel(config.resourceType)} discovery`;
? 'Host analysis'
: `${formatIdentifierLabel(config.resourceType)} analysis`;
return config.hostname ? `${discoveryMode} via ${config.hostname}` : discoveryMode;
});