fix: take last value in query params

Author: posva

packages/router/src/experimental/route-resolver/matchers/matcher-pattern-query.spec.ts

@@ -19,15 +19,15 @@ describe('MatcherPatternQueryParam', () => {
       expect(matcher.match({ user_id: 'abc123' })).toEqual({ userId: 'abc123' })
     })
 
-    it('takes first value from array', () => {
+    it('takes last value from array', () => {
       const matcher = new MatcherPatternQueryParam(
         'userId',
         'user_id',
         'value',
         PARAM_PARSER_DEFAULTS
       )
       expect(matcher.match({ user_id: ['first', 'second'] })).toEqual({
-        userId: 'first',
+        userId: 'second',
       })
     })
 

packages/router/src/experimental/route-resolver/matchers/matcher-pattern-query.ts

@@ -32,12 +32,13 @@ export class MatcherPatternQueryParam<T, ParamName extends string>
   match(query: MatcherQueryParams): Record<ParamName, T> {
     const queryValue: MatcherQueryParamsValue | undefined = query[this.queryKey]
 
-    // Check if query param is missing for default value handling
-
+    // normalize the value coming from the query based on the expected format
+    // value => keep the last value if multiple
+    // array => null becomes [], single value becomes [value]
     let valueBeforeParse =
       this.format === 'value'
         ? Array.isArray(queryValue)
-          ? queryValue[0]
+          ? queryValue.at(-1)
           : queryValue
         : // format === 'array'
           Array.isArray(queryValue)