From e63b662161ecb0c549d83cee1a7eeb2e573b58cd Mon Sep 17 00:00:00 2001
From: Ishaan Jaffer <ishaanjaffer0324@gmail.com>
Date: Tue, 10 Mar 2026 11:30:41 -0700
Subject: [PATCH] fix: use Shopify 2023-10 REST spec, co-locate rest section
 header with tool rows

---
 litellm/proxy/openapi_registry.json           |  4 +-
 .../mcp_tools/mcp_tool_configuration.tsx      | 70 ++++++++++---------
 2 files changed, 38 insertions(+), 36 deletions(-)

diff --git a/litellm/proxy/openapi_registry.json b/litellm/proxy/openapi_registry.json
index ff59f13ac14..fe83e71de5f 100644
--- a/litellm/proxy/openapi_registry.json
+++ b/litellm/proxy/openapi_registry.json
@@ -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",
diff --git a/ui/litellm-dashboard/src/components/mcp_tools/mcp_tool_configuration.tsx b/ui/litellm-dashboard/src/components/mcp_tools/mcp_tool_configuration.tsx
index 1a7c58fe10e..b9f4aae6b02 100644
--- a/ui/litellm-dashboard/src/components/mcp_tools/mcp_tool_configuration.tsx
+++ b/ui/litellm-dashboard/src/components/mcp_tools/mcp_tool_configuration.tsx
@@ -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>