feat: unit test cli list parsing

Author: gregnr

packages/mcp-server-supabase/src/transports/stdio.ts

@@ -5,6 +5,7 @@ import { parseArgs } from 'node:util';
 import packageJson from '../../package.json' with { type: 'json' };
 import { createSupabaseApiPlatform } from '../platform/api-platform.js';
 import { createSupabaseMcpServer } from '../server.js';
+import { parseList } from './util.js';
 
 const { version } = packageJson;
 
@@ -16,7 +17,7 @@ async function main() {
       ['read-only']: readOnly,
       ['api-url']: apiUrl,
       ['version']: showVersion,
-      ['features']: features,
+      ['features']: cliFeatures,
     },
   } = parseArgs({
     options: {
@@ -47,18 +48,17 @@ async function main() {
     process.exit(0);
   }
 
-  // Use access token from CLI argument or environment variable
   const accessToken = cliAccessToken ?? process.env.SUPABASE_ACCESS_TOKEN;
 
-  const featureList = features?.split(',') ?? [];
-
   if (!accessToken) {
     console.error(
       'Please provide a personal access token (PAT) with the --access-token flag or set the SUPABASE_ACCESS_TOKEN environment variable'
     );
     process.exit(1);
   }
 
+  const features = cliFeatures ? parseList(cliFeatures) : undefined;
+
   const platform = createSupabaseApiPlatform({
     accessToken,
     apiUrl,
@@ -68,7 +68,7 @@ async function main() {
     platform,
     projectId,
     readOnly,
-    features: featureList,
+    features,
   });
 
   const transport = new StdioServerTransport();

packages/mcp-server-supabase/src/transports/util.test.ts

@@ -0,0 +1,44 @@
+import { describe, expect, test } from 'vitest';
+import { parseList } from './util.js';
+
+describe('parseList', () => {
+  test('should parse comma-delimited list', () => {
+    const result = parseList('item1,item2,item3');
+    expect(result).toEqual(['item1', 'item2', 'item3']);
+  });
+
+  test('should handle spaces around items', () => {
+    const result = parseList('item1, item2 , item3');
+    expect(result).toEqual(['item1', 'item2', 'item3']);
+  });
+
+  test('should filter out empty items', () => {
+    const result = parseList('item1,,item2,');
+    expect(result).toEqual(['item1', 'item2']);
+  });
+
+  test('should handle custom delimiter', () => {
+    const result = parseList('item1|item2|item3', '|');
+    expect(result).toEqual(['item1', 'item2', 'item3']);
+  });
+
+  test('should handle single item', () => {
+    const result = parseList('item1');
+    expect(result).toEqual(['item1']);
+  });
+
+  test('should handle empty string', () => {
+    const result = parseList('');
+    expect(result).toEqual([]);
+  });
+
+  test('should handle string with only delimiters', () => {
+    const result = parseList(',,,');
+    expect(result).toEqual([]);
+  });
+
+  test('should handle semicolon delimiter', () => {
+    const result = parseList('item1; item2; item3', ';');
+    expect(result).toEqual(['item1', 'item2', 'item3']);
+  });
+});

packages/mcp-server-supabase/src/transports/util.ts

@@ -0,0 +1,10 @@
+/**
+ * Parses a delimited list of items into an array,
+ * trimming whitespace and filtering out empty items.
+ *
+ * Default delimiter is a comma (`,`).
+ */
+export function parseList(list: string, delimiter = ','): string[] {
+  const items = list.split(delimiter).map((feature) => feature.trim());
+  return items.filter((feature) => feature !== '');
+}