Address PR #1214 review nits

- Hoist previewUri() to a const in ResourceTemplatePanel to avoid
  duplicate computation
- Include resource.title in subscriptions search filter for consistency
  with resources and templates filters

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
cliffhall 2026-04-18 19:13:22 -04:00
commit 995c06948c
2 changed files with 5 additions and 2 deletions

View file

@ -70,6 +70,7 @@ export function ResourceControls({
const filteredSubscriptions = subscriptions.filter(
(s) =>
s.resource.name.toLowerCase().includes(query) ||
(s.resource.title?.toLowerCase().includes(query) ?? false) ||
s.resource.uri.toLowerCase().includes(query),
);

View file

@ -91,13 +91,15 @@ export function ResourceTemplatePanel({
onReadResource(resolveUri(uriTemplate, variables));
}
const preview = previewUri(uriTemplate, variables);
return (
<Stack gap="md">
<HeaderRow>
<Title order={4}>{title ?? name} Template</Title>
<UriGroup>
<UriText>{previewUri(uriTemplate, variables)}</UriText>
<CopyButton value={previewUri(uriTemplate, variables)} />
<UriText>{preview}</UriText>
<CopyButton value={preview} />
</UriGroup>
</HeaderRow>
{description && <DescriptionText>{description}</DescriptionText>}