Don't create a query match for rule-from-exchanges without queries

In most cases, this is just noise, and maybe mildly confusing. Clearer
to skip it (matching all requests and ignoring query) except in
endpoints that are actually using queries. Users can still modify as
they like, so we're just aiming for good simple defaults really.

Author: pimterry

src/model/rules/rule-creation.ts

@@ -128,14 +128,19 @@ function buildRequestMatchers(request: HtkRequest) {
 
     const urlParts = request.parsedUrl.toString().split('?');
     const path = urlParts[0];
+
+    const hasQuery = request.url.includes('?'); // Not just with parameters, but also trailing '?'
     const query = urlParts.slice(1).join('?');
+    const queryMatcher = hasQuery
+        ? [new matchers.QueryMatcher(
+            querystring.parse(query) as ({ [key: string]: string | string[] })
+        )]
+        : [];
 
     return [
         new (HttpRule.MethodMatchers[request.method as MethodName] || HttpRule.WildcardMatcher)(),
         new matchers.SimplePathMatcher(path),
-        new matchers.QueryMatcher(
-            querystring.parse(query) as ({ [key: string]: string | string[] })
-        ),
+        ...queryMatcher,
         ...bodyMatcher
     ];
 }