fix: match multi-variable URI template paths

Author: he-yufeng

.changeset/fix-uri-template-multi-var.md

@@ -0,0 +1,5 @@
+---
+'@modelcontextprotocol/core': patch
+---
+
+Fix `UriTemplate.match()` for comma-separated simple expressions such as `/users/{userId,format}`.

packages/core/src/shared/uriTemplate.ts

@@ -227,6 +227,15 @@ export class UriTemplate {
 
         switch (part.operator) {
             case '': {
+                if (!part.exploded && part.names.length > 1) {
+                    for (let i = 0; i < part.names.length; i++) {
+                        patterns.push({
+                            pattern: (i === 0 ? '' : ',') + '([^/,]+)',
+                            name: part.names[i]!
+                        });
+                    }
+                    return patterns;
+                }
                 pattern = part.exploded ? '([^/,]+(?:,[^/,]+)*)' : '([^/,]+)';
                 break;
             }

packages/core/test/shared/uriTemplate.test.ts

@@ -98,6 +98,12 @@ describe('UriTemplate', () => {
             expect(match).toEqual({ username: 'fred', postId: '123' });
         });
 
+        it('should match comma-separated variables in one expression', () => {
+            const template = new UriTemplate('/users/{userId,format}');
+            const match = template.match('/users/42,json');
+            expect(match).toEqual({ userId: '42', format: 'json' });
+        });
+
         it('should return null for non-matching URIs', () => {
             const template = new UriTemplate('/users/{username}');
             const match = template.match('/posts/123');