API suggestion: catch-all routes use the chosen parameter name

[osdiab]

This is irrelevant if we go with #21 , but right now the getRoute function refers to the catch-all part of a route to be path, ignoring the name in the actual route. For instance, /foo/[...bar].ts would be populated with getRoute({route: '/foo/[...bar].ts', path: 'baz'}) instead of getRoute({route: '/foo/[...bar].ts', params: {bar: 'baz'}}).

I found it counterintuitive that while NextJS refers to those catch-all params as params, this library doesn't.

[ckastbjerg]

It's been awhile since I worked on the library, so forgive me if I'm not remembering correctly here...

But I believe I chose the path as you can have both params and path. E.g.:

getRoute({
  route: "/nested-catch-all/[dynamic]/slugs",
  params: { dynamic: 1 },
  path: "/a/b/c",
})

More details here: #13

Or are you arguing that they should be treated the same?

But yes, if the API surfaces was limited/the implementation changed, it might not matter I guess

[osdiab]

Yes, just saying it would be more consistent if they were treated the same.