Skip to content

Commit c7e7ae6

Browse files
authored
Merge pull request #265 from dgageot/fixes-ui
Fixes to the UI
2 parents 52676bf + 5e779a6 commit c7e7ae6

File tree

7 files changed

+15
-16
lines changed

7 files changed

+15
-16
lines changed

src/extension/ui/src/components/tabs/ToolCatalog.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,16 +25,16 @@ const ToolCatalog: React.FC<ToolCatalogProps> = ({
2525
// Memoize the filtered catalog items to prevent unnecessary recalculations
2626
const all = useMemo(() => {
2727
const filteredItems = catalogItems.filter((item) => {
28-
return item.name.toLowerCase().includes(search.toLowerCase());
28+
return item.title.toLowerCase().includes(search.toLowerCase());
2929
});
3030

3131
return sort === 'name-asc'
3232
? filteredItems.sort((a, b) => {
33-
return a.name.localeCompare(b.name);
33+
return a.title.localeCompare(b.title);
3434
})
3535
: sort === 'name-desc'
3636
? filteredItems.sort((a, b) => {
37-
return b.name.localeCompare(a.name);
37+
return b.title.localeCompare(a.title);
3838
})
3939
: filteredItems;
4040
}, [catalogItems, search, sort]);

src/extension/ui/src/components/tile/Modal.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -195,12 +195,12 @@ const ConfigurationModal = ({
195195
}}
196196
/>
197197
}
198-
{catalogItem.title ?? catalogItem.name}
198+
{catalogItem.title}
199199
<Tooltip
200200
placement="right"
201201
title={
202202
!catalogItem.canRegister
203-
? 'You must assign all secrets and configure the item before it can be used.'
203+
? 'You must configure the item before it can be used.'
204204
: ''
205205
}
206206
>
@@ -217,12 +217,12 @@ const ConfigurationModal = ({
217217
/>
218218
</span>
219219
</Tooltip>
220+
{catalogItem.missingConfig && (
221+
<Chip label="Requires parameters" color="warning" />
222+
)}
220223
{catalogItem.missingSecrets && (
221224
<Chip label="Requires secrets" color="warning" />
222225
)}
223-
{catalogItem.missingConfig && (
224-
<Chip label="Requires configuration" color="warning" />
225-
)}
226226
</Stack>
227227
</DialogTitle>
228228
<IconButton

src/extension/ui/src/components/tile/Top.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ export default function Top({ item, onToggleRegister }: TopProps) {
5252
color: 'text.primary',
5353
}}
5454
>
55-
{item.title ?? item.name}
55+
{item.title}
5656
</Typography>
5757
}
5858
action={
@@ -74,7 +74,7 @@ export default function Top({ item, onToggleRegister }: TopProps) {
7474
/>
7575
</Tooltip>
7676
) : (
77-
<Tooltip title={`Enabling ${item.name} requires configuration`}>
77+
<Tooltip title={`Enabling ${item.title} requires configuration`}>
7878
<span>
7979
<Switch checked={false} disabled />
8080
</span>

src/extension/ui/src/mcp-clients/ClaudeDesktop.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ class ClaudeDesktopClient implements MCPClient {
55
name = "Claude Desktop";
66
url = "https://claude.ai/download";
77
manualConfigSteps = [
8-
"Open Claude Desktop",
98
"Select <strong>Claude Settings</strong>",
109
"Click on the <strong>Developer</strong> tab",
1110
"Click on the <strong>Edit Config</strong> button",

src/extension/ui/src/mcp-clients/ContinueDotDev.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@ class ContinueDotDev implements MCPClient {
66
name = "Continue.dev";
77
url = "https://continue.dev/";
88
manualConfigSteps = [
9-
"Open <strong>Continue.dev Settings </strong>",
10-
"in your global .continue folder (~/.continue on Mac, %USERPROFILE%.continue) within .continue/assistants. The name of the file will be used as the display name of the assistant, e.g. My Assistant.yaml",
11-
"Add block mcpServers:",
9+
"Open <strong>Continue.dev Settings</strong>",
10+
"In your global .continue folder (~/.continue on Mac/Linux, %USERPROFILE%.continue on Windows), within .continue/assistants. The name of the file will be used as the display name of the assistant, e.g. My Assistant.yaml",
11+
"Add block <code>mcpServers</code>:",
1212
stringify(SAMPLE_MCP_CONFIG),
1313
];
1414
expectedConfigPath = {

src/extension/ui/src/mcp-clients/Gordon.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ class GordonMCPClient implements MCPClient {
1717
url = 'https://docs.docker.com/desktop/features/gordon/mcp/yaml/';
1818
manualConfigSteps = [
1919
'Enable Gordon in Docker Desktop',
20-
'Write gordon-mcp.yml to the directory you want to connect to',
20+
'Write <code>gordon-mcp.yml</code> to the directory you want to connect to',
2121
'Add MCP_DOCKER to the <code>services</code> section:' +
2222
'<pre style="font-family: monospace; overflow: auto; width: 80%; background-color: grey.200; padding: 1; border-radius: 1; font-size: 12px;">' +
2323
DOCKER_MCP_CONFIG_YAML +

src/extension/ui/src/types/catalog/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { JsonSchema } from "json-schema-library";
55
*/
66
export interface CatalogItem {
77
description?: string;
8-
title?: string;
8+
title: string;
99
source?: string;
1010
icon?: string;
1111
secrets?: { name: string }[];

0 commit comments

Comments
 (0)