feat(ssg): Ignore Dynamic Route

deno_dist/helper/ssg/index.ts

@@ -78,7 +78,7 @@ export const fetchRoutesContent = async <
   const baseURL = 'http://localhost'
 
   for (const route of inspectRoutes(app)) {
-    if (route.isMiddleware) continue
+    if (route.isMiddleware || isDynamicRoute(route.path)) continue
 
     const url = new URL(route.path, baseURL).toString()
     const response = await app.fetch(new Request(url))
@@ -91,6 +91,10 @@ export const fetchRoutesContent = async <
   return htmlMap
 }
 
+const isDynamicRoute = (path: string): boolean => {
+  return path.split('/').some((segment) => segment.startsWith(':') || segment.includes('*'))
+}
+
 /**
  * @experimental
  * `saveContentToFiles` is an experimental feature.

src/helper/ssg/index.test.tsx

@@ -155,3 +155,34 @@ describe('saveContentToFiles function', () => {
     )
   })
 })
+
+describe('Dynamic route handling', () => {
+  let app: Hono
+  beforeEach(() => {
+    app = new Hono()
+    app.get('/shops/:id', (c) => c.html('Shop Page'))
+    app.get('/shops/:id/:comments([0-9]+)', (c) => c.html('Comments Page'))
+    app.get('/foo/*', (c) => c.html('Foo Page'))
+    app.get('/foo:bar', (c) => c.html('Foo Bar Page'))
+  })
+
+  it('should skip /shops/:id dynamic route', async () => {
+    const htmlMap = await fetchRoutesContent(app)
+    expect(htmlMap.has('/shops/:id')).toBeFalsy()
+  })
+
+  it('should skip /shops/:id/:comments([0-9]+) dynamic route', async () => {
+    const htmlMap = await fetchRoutesContent(app)
+    expect(htmlMap.has('/shops/:id/:comments([0-9]+)')).toBeFalsy()
+  })
+
+  it('should skip /foo/* dynamic route', async () => {
+    const htmlMap = await fetchRoutesContent(app)
+    expect(htmlMap.has('/foo/*')).toBeFalsy()
+  })
+
+  it('should not skip /foo:bar dynamic route', async () => {
+    const htmlMap = await fetchRoutesContent(app)
+    expect(htmlMap.has('/foo:bar')).toBeTruthy()
+  })
+})

src/helper/ssg/index.ts

@@ -77,7 +77,7 @@ export const fetchRoutesContent = async <
   const baseURL = 'http://localhost'
 
   for (const route of inspectRoutes(app)) {
-    if (route.isMiddleware) continue
+    if (route.isMiddleware || isDynamicRoute(route.path)) continue
 
     const url = new URL(route.path, baseURL).toString()
     const response = await app.fetch(new Request(url))
@@ -90,6 +90,10 @@ export const fetchRoutesContent = async <
   return htmlMap
 }
 
+const isDynamicRoute = (path: string): boolean => {
+  return path.split('/').some((segment) => segment.startsWith(':') || segment.includes('*'))
+}
+
 /**
  * @experimental
  * `saveContentToFiles` is an experimental feature.