refactor(ui): rename the create MCP server component to PascalCase (#35686)

Pure rename, no behavior change. create_mcp_server.tsx and its test move
to CreateMCPServer, the two importers and one stale e2e comment follow,
and the local/filename-pascal-case suppression drops now that the file
passes the rule on its own.

The rename is scoped to this one component rather than the whole
directory because three PRs are currently open against its snake_case
siblings; the rest can follow once those land.
This commit is contained in:
yuneng-jiang 2026-08-03 13:39:07 -07:00 • committed by GitHub
parent db98413e3b
commit 9d5984b358
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
6 changed files with 18 additions and 21 deletions

View file

@ -36,7 +36,7 @@ test.describe("MCP Servers", () => {
await formModal.locator('input[id="url"]').fill("https://e2e-fake-mcp.test.local/mcp");
// Authentication: None
// The auth_type Form.Item has no label prop (create_mcp_server.tsx:795), so
// The auth_type Form.Item has no label prop (CreateMCPServer.tsx), so
// it can't be anchored by label text. Scope via the enclosing Collapse
// panel ("Authentication") instead — that anchor is stable even if the
// placeholder copy changes.

View file

@ -779,6 +779,20 @@
"count": 1
}
},
"src/app/(dashboard)/mcp-servers/_components/CreateMCPServer.tsx": {
"max-lines": {
"count": 1
},
"no-nested-ternary": {
"count": 1
},
"no-restricted-imports": {
"count": 2
},
"react-hooks/set-state-in-effect": {
"count": 4
}
},
"src/app/(dashboard)/mcp-servers/_components/DcrBridgeToggle.tsx": {
"no-restricted-imports": {
"count": 1
@ -900,23 +914,6 @@
"count": 1
}
},
"src/app/(dashboard)/mcp-servers/_components/create_mcp_server.tsx": {
"local/filename-pascal-case": {
"count": 1
},
"max-lines": {
"count": 1
},
"no-nested-ternary": {
"count": 1
},
"no-restricted-imports": {
"count": 2
},
"react-hooks/set-state-in-effect": {
"count": 4
}
},
"src/app/(dashboard)/mcp-servers/_components/index.tsx": {
"local/filename-pascal-case": {
"count": 1

View file

@ -3,7 +3,7 @@ import userEvent from "@testing-library/user-event";
import { beforeEach, describe, expect, it, vi } from "vitest";
import * as networking from "@/components/networking";
import { setToken } from "@/utils/mcpTokenStore";
import CreateMCPServer from "./create_mcp_server";
import CreateMCPServer from "./CreateMCPServer";
import { selectAntOption } from "./testUtils";
vi.mock("@/components/networking", () => ({

View file

@ -7,7 +7,7 @@ import { Skeleton } from "@/components/ui/skeleton";
import { cn } from "@/lib/cva.config";
import { fetchDiscoverableMCPServers } from "@/components/networking";
import { DiscoverableMCPServer, DiscoverMCPServersResponse } from "@/components/mcp_tools/types";
import { mcpLogoImg } from "./create_mcp_server";
import { mcpLogoImg } from "./CreateMCPServer";
import { resolveLogoSrc } from "@/lib/assetPaths";
interface MCPDiscoveryProps {

View file

@ -24,7 +24,7 @@ import NotificationsManager from "@/components/molecules/notifications_manager";
import { deleteMCPServer } from "@/components/networking";
import { MCPSubmissionsTab } from "./MCPSubmissionsTab";
import { MCPToolsetsTab } from "./MCPToolsetsTab";
import CreateMCPServer from "./create_mcp_server";
import CreateMCPServer from "./CreateMCPServer";
import MCPConnect from "./mcp_connect";
import MCPServerCard from "./MCPServerCard";
import { MCPServerView } from "./mcp_server_view";