fix: address Greptile review feedback

- Move `import json` and `import os` to module top level
- Move mcp_registry.json into litellm/proxy/ for pip distribution
- Fix `Text` component: destructure from antd Typography instead of deprecated Tremor
- Update test fixture path to match new registry location
This commit is contained in:
Ishaan Jaffer 2026-02-12 17:50:41 -08:00
parent f32e900ae2
commit a7d092f3ec
4 changed files with 7 additions and 3 deletions

View file

@ -15,6 +15,8 @@ Endpoints here:
"""
import importlib
import json
import os
from dataclasses import dataclass
from datetime import datetime, timedelta
from typing import Any, Dict, Iterable, List, Literal, Optional
@ -1181,7 +1183,7 @@ if MCP_AVAILABLE:
# --- MCP Discovery ---
_MCP_REGISTRY_PATH = os.path.join(
os.path.dirname(os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__))))),
os.path.dirname(os.path.dirname(os.path.abspath(__file__))),
"mcp_registry.json",
)

View file

@ -21,6 +21,8 @@ class TestMCPRegistryFile:
"..",
"..",
"..",
"litellm",
"proxy",
"mcp_registry.json",
)

View file

@ -1,11 +1,11 @@
import React, { useState, useMemo, useEffect } from "react";
import { Modal, Input } from "antd";
import { Typography } from "antd";
import { Modal, Input, Typography } from "antd";
import { fetchDiscoverableMCPServers } from "../networking";
import { DiscoverableMCPServer, DiscoverMCPServersResponse } from "./types";
import { mcpLogoImg } from "./create_mcp_server";
const { Search } = Input;
const { Text } = Typography;
interface MCPDiscoveryProps {
isVisible: boolean;