fix: exclude /^\/app\/.+\.tsx?/ in vite dev-server (#251)

* fix: exclude `/^\/app\/.+\.tsx?/` in vite dev-server * correct the test
honojs · Jan 10, 2025 · 5ac6155 · 5ac6155
1 parent a3956ea
commit 5ac6155
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 2 deletions.
diff --git a/mocks/app/routes/app/nested/index.tsx b/mocks/app/routes/app/nested/index.tsx
@@ -0,0 +1,3 @@
+export default function () {
+  return <h1>Nested</h1>
+}
diff --git a/src/vite/index.ts b/src/vite/index.ts
@@ -33,7 +33,7 @@ function honox(options?: Options): PluginOption[] {
       entry,
       exclude: [
         ...devServerDefaultOptions.exclude,
-        /^\/app\/.+/,
+        /^\/app\/.+\.tsx?/,
         /^\/favicon.ico/,
         /^\/static\/.+/,
       ],

diff --git a/test-e2e/e2e.test.ts b/test-e2e/e2e.test.ts
@@ -131,3 +131,9 @@ test('server-side suspense contains island components', async ({ page }) => {
   await container.getByText('Count: 7').click()
   await container.getByText('Suspense Islands').click()
 })
+
+test('/app/nested', async ({ page }) => {
+  await page.goto('/app/nested')
+  const contentH1 = await page.textContent('h1')
+  expect(contentH1).toBe('Nested')
+})
diff --git a/test-integration/apps.test.ts b/test-integration/apps.test.ts
@@ -96,8 +96,12 @@ describe('Basic', () => {
         path: '/throw_error',
         method: 'GET',
       },
+      {
+        path: '/app/nested',
+        method: 'GET',
+      },
     ]
-    expect(app.routes).toHaveLength(46)
+    expect(app.routes).toHaveLength(48)
     expect(app.routes).toEqual(
       expect.arrayContaining(
         routes.map(({ path, method }) => {