fix: make UriTemplate query params optional and order-independent

[Christian-Sidak]

Per RFC 6570, {?param1,param2} query parameter templates should match URIs with no query params, a subset of params, or params in a different order than declared in the template.

Changes

• partToRegExp(): removed the strict per-param pattern generation for ?/& operators; query params are now handled in match() directly
• partToRegExp(): changed the default ('') operator pattern from [^/,]+ to [^/?#,]+ so path variables don't greedily consume ? and # delimiters
• match(): added optional (?:\?[^#]*)? regex group for the query string, then parses query params from the URI independently (order-agnostic, all optional)

Before

const template = new UriTemplate("dom://{pageId}{?selector,includeAttributes}");
template.match("dom://test-id");                        // null (wrong)
template.match("dom://test-id?selector=body");          // null (wrong)
template.match("dom://test-id?includeAttributes=true"); // null (wrong)

After

template.match("dom://test-id");                        // { pageId: "test-id" }
template.match("dom://test-id?selector=body");          // { pageId: "test-id", selector: "body" }
template.match("dom://test-id?includeAttributes=true"); // { pageId: "test-id", includeAttributes: "true" }

Fixes #1079

[changeset-bot]

🦋 Changeset detected

Latest commit: 6e01958

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@modelcontextprotocol/core	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

Open in StackBlitz

@modelcontextprotocol/client

npm i https://pkg.pr.new/@modelcontextprotocol/client@1929

@modelcontextprotocol/server

npm i https://pkg.pr.new/@modelcontextprotocol/server@1929

@modelcontextprotocol/express

npm i https://pkg.pr.new/@modelcontextprotocol/express@1929

@modelcontextprotocol/fastify

npm i https://pkg.pr.new/@modelcontextprotocol/fastify@1929

@modelcontextprotocol/hono

npm i https://pkg.pr.new/@modelcontextprotocol/hono@1929

@modelcontextprotocol/node

npm i https://pkg.pr.new/@modelcontextprotocol/node@1929

commit: 6e01958