Remove commented-out code and unnecessary comments from server, route collector, and HTML parser files for improved readability and maintainability.

Author: scalvert

src/mcp/server.ts

@@ -83,7 +83,6 @@ export class McpDocsServer {
   private registerTools(server: McpServer): void {
     const toolOverrides = this.config.tools;
 
-    // Register docs_search tool
     server.registerTool(
       docsSearchTool.name,
       {
@@ -118,7 +117,6 @@ export class McpDocsServer {
       }
     );
 
-    // Register docs_fetch tool
     server.registerTool(
       docsFetchTool.name,
       {
@@ -199,13 +197,11 @@ export class McpDocsServer {
   }
 
   private async _doInitialize(): Promise<void> {
-    // Load the search provider (specifier string or pre-instantiated provider)
     const searchSpecifier = this.config.search ?? 'flexsearch';
     this.searchProvider = await loadSearchProvider(searchSpecifier, {
       flexsearch: this.config.flexsearch,
     });
 
-    // Build provider context
     const providerContext: ProviderContext = {
       baseUrl: this.config.baseUrl ?? '',
       serverName: this.config.name,
@@ -228,7 +224,6 @@ export class McpDocsServer {
       throw new Error('Invalid server config: must provide either file paths or pre-loaded data');
     }
 
-    // Initialize the search provider
     await this.searchProvider.initialize(providerContext, initData);
 
     this.initialized = true;
@@ -260,14 +255,11 @@ export class McpDocsServer {
       enableJsonResponse: true, // Return JSON instead of SSE streams
     });
 
-    // Connect the server to this transport
     await server.connect(transport);
 
     try {
-      // Let the transport handle the request
       await transport.handleRequest(req, res, parsedBody);
     } finally {
-      // Clean up the transport after request
       await transport.close();
     }
   }
@@ -287,20 +279,16 @@ export class McpDocsServer {
 
     const server = this.createMcpServer();
 
-    // Create a stateless transport for Web Standards
     const transport = new WebStandardStreamableHTTPServerTransport({
       sessionIdGenerator: undefined, // Stateless mode
       enableJsonResponse: true,
     });
 
-    // Connect the server to this transport
     await server.connect(transport);
 
     try {
-      // Let the transport handle the request and return the response
       return await transport.handleRequest(request);
     } finally {
-      // Clean up the transport after request
       await transport.close();
     }
   }

src/plugin/route-collector.ts

@@ -10,15 +10,13 @@ export function flattenRoutes(routes: RouteConfig[]): FlattenedRoute[] {
   const flattened: FlattenedRoute[] = [];
 
   function traverse(route: RouteConfig): void {
-    // Add the route if it has a path
     if (route.path) {
       flattened.push({
         path: route.path,
         htmlPath: '', // Will be resolved later
       });
     }
 
-    // Recursively process child routes
     if (route.routes && Array.isArray(route.routes)) {
       for (const childRoute of route.routes) {
         traverse(childRoute);
@@ -42,15 +40,12 @@ export function flattenRoutes(routes: RouteConfig[]): FlattenedRoute[] {
  * - /api -> build/api/index.html
  */
 export function routeToHtmlPath(routePath: string, outDir: string): string {
-  // Normalize the route path
   let normalizedPath = routePath;
 
-  // Remove trailing slash if present (except for root)
   if (normalizedPath.length > 1 && normalizedPath.endsWith('/')) {
     normalizedPath = normalizedPath.slice(0, -1);
   }
 
-  // Handle root path
   if (normalizedPath === '/') {
     return path.join(outDir, 'index.html');
   }
@@ -98,7 +93,6 @@ export async function discoverHtmlFiles(outDir: string): Promise<FlattenedRoute[
       const fullPath = path.join(dir, entry.name);
 
       if (entry.isDirectory()) {
-        // Skip common non-content directories
         if (['assets', 'img', 'static'].includes(entry.name)) {
           continue;
         }
@@ -159,10 +153,7 @@ export async function collectRoutes(
   outDir: string,
   excludePatterns: string[]
 ): Promise<FlattenedRoute[]> {
-  // Discover all HTML files in the build directory
   const allRoutes = await discoverHtmlFiles(outDir);
-
-  // Filter out excluded routes
   const filteredRoutes = filterRoutes(allRoutes, excludePatterns);
 
   // Deduplicate routes by path. When the same route is emitted as both

src/processing/html-parser.ts

@@ -135,7 +135,6 @@ export function cleanContentElement(element: Element, excludeSelectors: string[]
         }
       }
 
-      // Recursively clean children
       removeUnwanted(childElement);
       return true;
     });