Combination of index route and dynamic route doesn't seem to get resolved properly

[bux]

With the above setup of pages, when calling localhost:3000/menuplan/ it seems that the dynamic route is being used instead of the index route.

Am I doing something wrong here?

Version: 0.7.0

[hannoeru]

It works, can you provide more context?
or try to upgrade vue-router to latest version.

[bux]

A simple repro would be:

• Clone vitesse by antfu (or create a new github project by using this template)
• Create a new vue file below ~/src/pages/hi and call it index.vue
• Add a <template/> with some demo content
• Run npm run dev
• Try to open: http://localhost:3333/hi/

Observe how the dynamic route is being used instead of the newly created index route

[hannoeru]

That is vite-plugin-vue-layouts's bug, the work around for now is:

// add to main.ts
routes[0].children.forEach((i: any) => {
  i.path = i.path.substring(1)
  if (i.path.endsWith('?'))
    i.path = i.path.slice(0, -1)
})

P.S. That didn't handle deep routes
Cc @JohnCampionJr

[bux]

Thank you. This has worked. Do you want me to open an issue on vite-plugin-vue-layouts's repo?

[hannoeru]

Ok, then I will close this one.

[shroudedcode]

I'm also running into this issue and I'm not using vite-plugin-vue-layouts.

My pages directory looks like this:

├── [...notfound].vue
├── things
│   ├── [thing].vue
│   └── index.vue
└── index.vue

And these are my generated routes:

[
  { "name": "notfound", "path": "/:notfound(.*)" },
  { "name": "index", "path": "/" },
  // This route is always being matched:
  { "name": "things-thing", "path": "/things/:thing?" },
  { "name": "things", "path": "/things" }
]

Edit: Manually removing the ? at the end of the path (which is sort of what you suggested here) fixes the issue, but I don't see why it should have been added there in the first place. Under what circumstances would someone want their route's parameter to be optional and are those circumstances common enough to be the default behavior? 🤔

[hannoeru]

Optional parameters is removed from v0.9.0

[shroudedcode]

The new version fixed it. Thanks a lot for this great package, @hannoeru! 💚