Improvements and optimization

Author: web-flow

eslint.config.js

@@ -94,7 +94,8 @@ export default [
         'warn',
         { allowConstantExport: true },
       ],
-      '@typescript-eslint/no-explicit-any': 'warn',
+      // Allow 'any' for dynamic API parsing, error handling, and template helpers
+      '@typescript-eslint/no-explicit-any': 'off',
       '@typescript-eslint/no-unused-vars': [
         'warn',
         { argsIgnorePattern: '^_' },

src/client/components/steps/StepApiConfig.tsx

@@ -47,6 +47,7 @@ function StepApiConfig({
     if (onParseRequest) {
       onParseRequest(handleParse, canParse, parsing);
     }
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [canParse, parsing, formData, inputMethod]);
 
   const handleParse = async () => {
@@ -73,7 +74,7 @@ function StepApiConfig({
           if (!parsed.info || !parsed.item) {
             throw new Error("Invalid Postman collection format");
           }
-        } catch (e) {
+        } catch {
           setParseError("Invalid Postman collection JSON format");
           setParsing(false);
           return;

src/client/components/steps/StepSelectEndpoints.tsx

@@ -30,22 +30,24 @@ function StepSelectEndpoints({
   selectedEndpoints,
   setSelectedEndpoints,
 }: StepSelectEndpointsProps) {
-  const endpoints = apiConfig?.endpoints || [];
   const [searchTerm, setSearchTerm] = useState("");
   const [page, setPage] = useState(0);
   const [rowsPerPage, setRowsPerPage] = useState(25);
 
+  const endpoints = apiConfig?.endpoints || [];
+
   // Filter endpoints based on search
   const filteredEndpoints = useMemo(() => {
-    if (!searchTerm) return endpoints;
+    const endpointList = apiConfig?.endpoints || [];
+    if (!searchTerm) return endpointList;
     const searchLower = searchTerm.toLowerCase();
-    return endpoints.filter(
+    return endpointList.filter(
       (endpoint: any) =>
         endpoint.path?.toLowerCase().includes(searchLower) ||
         endpoint.method?.toLowerCase().includes(searchLower) ||
         endpoint.description?.toLowerCase().includes(searchLower),
     );
-  }, [endpoints, searchTerm]);
+  }, [apiConfig?.endpoints, searchTerm]);
 
   // Paginate filtered endpoints
   const paginatedEndpoints = useMemo(() => {

src/client/pages/CreateServer.tsx

@@ -170,15 +170,6 @@ function CreateServer() {
     setActiveStep((prev) => prev - 1);
   };
 
-  const handleReset = () => {
-    setActiveStep(0);
-    setApiConfig(null);
-    setSelectedEndpoints([]);
-    setServerConfig({ transport: "stdio" });
-    setGeneratedServerId(null);
-    setError(null);
-  };
-
   const getStepContent = (step: number) => {
     switch (step) {
       case 0:

src/client/pages/McpServers.tsx

@@ -118,7 +118,7 @@ function McpServers() {
       // Then deploy (install + build)
       return apiService.deployServer(serverId);
     },
-    onSuccess: (response) => {
+    onSuccess: () => {
       queryClient.invalidateQueries({ queryKey: ["mcp-servers"] });
       enqueueSnackbar("STDIO server regenerated and rebuilt successfully", {
         variant: "success",

src/generator/ApiParser.ts

@@ -45,10 +45,10 @@ export class ApiParser {
       if (typeof response.data === "string") {
         try {
           spec = JSON.parse(response.data);
-        } catch (jsonError) {
+        } catch {
           try {
             spec = YAML.parse(response.data);
-          } catch (yamlError) {
+          } catch {
             throw new Error(
               "Invalid OpenAPI specification from URL: must be valid JSON or YAML format",
             );
@@ -62,10 +62,10 @@ export class ApiParser {
       // Try to parse as JSON first, then fall back to YAML
       try {
         spec = JSON.parse(specUrlOrJson);
-      } catch (jsonError) {
+      } catch {
         try {
           spec = YAML.parse(specUrlOrJson);
-        } catch (yamlError) {
+        } catch {
           throw new Error(
             "Invalid OpenAPI specification: must be valid JSON or YAML format",
           );
@@ -154,10 +154,14 @@ export class ApiParser {
           endpoints.push(currentEndpoint);
         }
 
+        const method = methodMatch[1].toUpperCase();
+        const path = methodMatch[2].trim();
+
         currentEndpoint = {
-          method: methodMatch[1].toUpperCase(),
-          path: methodMatch[2].trim(),
+          method,
+          path,
           description: methodMatch[3].trim(),
+          operationId: this.generateOperationId(method, path),
           parameters: [],
         };
         continue;
@@ -602,11 +606,13 @@ export class ApiParser {
             : item.request.url.raw;
 
         const path = this.extractPath(url);
+        const method = item.request.method;
 
         endpoints.push({
           path,
-          method: item.request.method,
+          method,
           description: item.name || "",
+          operationId: this.generateOperationId(method, path),
           parameters: this.extractPostmanParameters(item.request),
         });
       }
@@ -617,6 +623,41 @@ export class ApiParser {
     }
   }
 
+  /**
+   * Generate operationId from method and path
+   * e.g., GET /submissions -> getSubmissions
+   * e.g., POST /submissions/{id}/documents -> postSubmissionsIdDocuments
+   */
+  private generateOperationId(method: string, path: string): string {
+    // Convert method to lowercase
+    const methodLower = method.toLowerCase();
+
+    // Clean and convert path to camelCase
+    // Remove leading slash and split by /
+    const pathParts = path
+      .replace(/^\//, "") // Remove leading slash
+      .split("/")
+      .filter((part) => part.length > 0) // Remove empty parts
+      .map((part) => {
+        // Replace path parameters {id} with just "id"
+        if (part.startsWith("{") && part.endsWith("}")) {
+          return part.slice(1, -1);
+        }
+        // Replace hyphens and underscores with spaces for proper casing
+        return part.replace(/[-_]/g, " ");
+      })
+      .map((part) => {
+        // Convert to title case for all parts
+        return part
+          .split(" ")
+          .map((word) => word.charAt(0).toUpperCase() + word.slice(1))
+          .join("");
+      })
+      .join("");
+
+    return methodLower + (pathParts || "Root");
+  }
+
   /**
    * Extract path from URL
    */

src/server/routes/deployments.ts

@@ -11,7 +11,6 @@ import {
   appendLog,
   readLogs,
   rotateLogIfNeeded,
-  deleteServerLogs,
 } from "../utils/logger.js";
 
 const router = express.Router();