fix: optional params was wrapped inside quotes (#138)

close #137

Author: yesmeck

src/build.ts

@@ -116,7 +116,18 @@ function parse(routes: RouteManifestEntry[]) {
           .split('/')
           .filter((seg) => seg.startsWith(':') || seg == '*')
           .map((param) => param.split('.')[0])
-          .map((param) => param.replace(':', '')),
+          .map((param) => {
+            let keyable = param.replace(':', '');
+            const isOptional = keyable.match(/\?$/);
+            if (isOptional) {
+              keyable = keyable.replace(/\?$/, '');
+            }
+            keyable = `'${keyable}'`
+            if (isOptional) {
+              keyable = `${keyable}?`
+            }
+            return keyable;
+          })
       )
     );
   });

src/template.ts

@@ -23,7 +23,7 @@ function exportedQuery(ctx: Context) {
 function routes(ctx: Context) {
   const routes = ctx.routes.map(({ route, params, fileName }) =>
     `"${route}": {
-      params: ${params.length > 0 ? `{${params.map(param => `'${param}': string | number`).join('; ')}}` : 'never'},
+      params: ${params.length > 0 ? `{${params.map(param => `${param}: string | number`).join('; ')}}` : 'never'},
       query: ExportedQuery<import('${ctx.relativeAppDirPath}/${fileName}').SearchParams>,
     }`
   );

tests/__snapshots__/build.test.ts.snap

@@ -14,9 +14,13 @@ exports[`gen route types 1`] = `
       query: ExportedQuery<import('../app/root').SearchParams>,
     },
     "/:lang?/about": {
-      params: {'lang?': string | number},
+      params: {'lang'?: string | number},
       query: ExportedQuery<import('../app/routes/($lang).about').SearchParams>,
     },
+    "/:provider-key?/about": {
+      params: {'provider-key'?: string | number},
+      query: ExportedQuery<import('../app/routes/($provider-key).about').SearchParams>,
+    },
     "/admin": {
       params: never,
       query: ExportedQuery<import('../app/routes/admin._index').SearchParams>,
@@ -166,6 +170,7 @@ exports[`gen route types 1`] = `
             | 'routes/blog'
             | 'routes/blog._index'
             | 'routes/auth.$provider-key'
+            | 'routes/($provider-key).about'
             | 'catchall';
 
   export function $path<

tests/fixture.ts

@@ -190,6 +190,12 @@ export const testRoutes =
     path: "auth/:provider-key",
     parentId: "root",
   },
+  "routes/($provider-key).about": {
+    file: "routes/($provider-key).about.tsx",
+    id: "routes/($provider-key).about",
+    path: ":provider-key?/about",
+    parentId: "root",
+  },
   catchall: {
     path: "/somewhere/cool/*",
     index: undefined,