[Bug?]: Route with (...) resolved incorrectly

[yinonburgansky]

Duplicates

• I have searched the existing issues

Latest version

• I have tested the latest version

Current behavior 😯

the following file route:

|-- routes/
    |-- product/
        |-- (product)about.tsx

is converted to route:

/productabout

Expected behavior 🤔

expecting:

/product/about

Steps to reproduce 🕹

https://stackblitz.com/edit/github-4r8246qf?file=src%2Fapp.tsx

Context 🔦

having multiple pages with the same name can be confusing
prefixing them with (some name) can help identify them

solid-start/packages/start/src/router/routes.ts

Line 39 in 13f68c1

path: id.replace(/\/\([^)/]+\)/g, "").replace(/\([^)/]+\)/g, "")

maybe something like this instead?

// remove all `(...)` and then remove multiple slashes to account for (nested) route groups
 id.replace(/\([^)/]+\)/g, "").replace(/\/+/g, "/")

Your environment 🌎