fix: use Shopify 2023-10 REST spec, co-locate rest section header with tool rows

This commit is contained in:
Ishaan Jaffer 2026-03-10 11:30:41 -07:00
parent ffbd7ef525
commit e63b662161
2 changed files with 38 additions and 36 deletions

View file

@ -187,9 +187,9 @@
{
"name": "shopify",
"title": "Shopify",
"description": "Products, orders, customers, and store management (requires your store subdomain)",
"description": "Products, orders, customers, and store management via Shopify Admin REST API (requires your store subdomain)",
"icon_url": "https://cdn.simpleicons.org/shopify",
"spec_url": "https://raw.githubusercontent.com/Shopify/shopify-api-specs/main/admin/rest/2024-01/openapi.json",
"spec_url": "https://raw.githubusercontent.com/Shopify/shopify-api-specs/main/admin/rest/2023-10/openapi.json",
"oauth": {
"authorization_url": "https://your-store.myshopify.com/admin/oauth/authorize",
"token_url": "https://your-store.myshopify.com/admin/oauth/access_token",

View file

@ -468,42 +468,44 @@ const MCPToolConfiguration: React.FC<MCPToolConfigurationProps> = ({
</>
)}
{restFiltered.length > 0 && (
<div className="flex items-center justify-between px-1 pt-2">
<p className="text-xs font-semibold text-gray-500 uppercase tracking-wide">
{pinnedFiltered.length > 0 ? "All tools" : "Tools"}
</p>
<div className="flex gap-2">
<button
type="button"
onClick={handleEnableRest}
className="text-xs text-blue-600 hover:text-blue-700"
>
Enable all
</button>
<button
type="button"
onClick={handleDisableRest}
className="text-xs text-gray-500 hover:text-gray-700"
>
Disable all
</button>
<>
<div className="flex items-center justify-between px-1 pt-2">
<p className="text-xs font-semibold text-gray-500 uppercase tracking-wide">
{pinnedFiltered.length > 0 ? "All tools" : "Tools"}
</p>
<div className="flex gap-2">
<button
type="button"
onClick={handleEnableRest}
className="text-xs text-blue-600 hover:text-blue-700"
>
Enable all
</button>
<button
type="button"
onClick={handleDisableRest}
className="text-xs text-gray-500 hover:text-gray-700"
>
Disable all
</button>
</div>
</div>
</div>
{restFiltered.map((tool) => (
<ToolRow
key={tool.name}
tool={tool}
isEnabled={allowedTools.includes(tool.name)}
isEditExpanded={expandedTools.has(tool.name)}
toolNameToDisplayName={toolNameToDisplayName}
toolNameToDescription={toolNameToDescription}
onToggle={handleToolToggle}
onToggleExpand={handleToggleEditExpanded}
onDisplayNameChange={handleDisplayNameChange}
onDescriptionChange={handleDescriptionChange}
/>
))}
</>
)}
{restFiltered.map((tool) => (
<ToolRow
key={tool.name}
tool={tool}
isEnabled={allowedTools.includes(tool.name)}
isEditExpanded={expandedTools.has(tool.name)}
toolNameToDisplayName={toolNameToDisplayName}
toolNameToDescription={toolNameToDescription}
onToggle={handleToolToggle}
onToggleExpand={handleToggleEditExpanded}
onDisplayNameChange={handleDisplayNameChange}
onDescriptionChange={handleDescriptionChange}
/>
))}
</div>
)}
</div>