fix(mcp): fix options parsing

Author: stainless-app[bot]

packages/mcp-server/src/options.ts

@@ -55,7 +55,7 @@ export function parseCLIOptions(): CLIOptions {
   const transport = argv.transport as 'stdio' | 'http';
 
   return {
-    includeDocsTools,
+    ...(includeDocsTools !== undefined && { includeDocsTools }),
     transport,
     port: argv.port,
     socket: argv.socket,
@@ -87,6 +87,6 @@ export function parseQueryOptions(defaultOptions: McpOptions, query: unknown): M
     : defaultOptions.includeDocsTools;
 
   return {
-    includeDocsTools: docsTools,
+    ...(docsTools !== undefined && { includeDocsTools: docsTools }),
   };
 }

packages/mcp-server/tests/options.test.ts

@@ -26,7 +26,7 @@ describe('parseCLIOptions', () => {
     const result = parseCLIOptions();
 
     expect(result.transport).toBe('http');
-    expect(result.port).toBe('2222');
+    expect(result.port).toBe(2222);
     cleanup();
   });
 });
@@ -38,13 +38,13 @@ describe('parseQueryOptions', () => {
     const query = '';
     const result = parseQueryOptions(defaultOptions, query);
 
-    expect(result).toBe({});
+    expect(result).toEqual({});
   });
 
   it('should handle invalid query string gracefully', () => {
-    const query = 'invalid=value&operation=invalid-operation';
+    const query = 'invalid=value&tools=invalid-operation';
 
-    // Should throw due to Zod validation for invalid operation
+    // Should throw due to Zod validation for invalid tools
     expect(() => parseQueryOptions(defaultOptions, query)).toThrow();
   });
 });