fix(vite,nuxt): skip on explicit file extensions

Author: harlan-zw

packages/nuxt/src/runtime/server/middleware/mdream.ts

@@ -90,8 +90,19 @@ export default defineEventHandler(async (event) => {
   let path = event.path
   const config = useRuntimeConfig(event).mdream as ModuleRuntimeConfig
 
-  // never run on API routes, internal routes or explicit html requests
-  if (path.startsWith('/api') || path.startsWith('/_') || path.endsWith('.html')) {
+  // never run on API routes or internal routes
+  if (path.startsWith('/api') || path.startsWith('/_') || path.startsWith('/@')) {
+    return
+  }
+
+  // Extract file extension from path (e.g., /file.js -> .js, /path/to/file.css -> .css)
+  const lastSegment = path.split('/').pop() || ''
+  const hasExtension = lastSegment.includes('.')
+  const extension = hasExtension ? lastSegment.substring(lastSegment.lastIndexOf('.')) : ''
+
+  // Only run on .md extension or no extension at all
+  // Skip all other file extensions (.js, .css, .html, .json, etc.)
+  if (hasExtension && extension !== '.md') {
     return
   }
 

packages/vite/src/plugin.ts

@@ -192,8 +192,19 @@ export function viteHtmlToMarkdownPlugin(userOptions: ViteHtmlToMarkdownOptions
         req.headers['sec-fetch-dest'] as string | undefined,
       )
 
-      // never run on API routes, internal routes or explicit html requests
-      if (path.startsWith('/api') || path.startsWith('/_') || path.endsWith('.html')) {
+      // never run on API routes or internal routes
+      if (path.startsWith('/api') || path.startsWith('/_') || path.startsWith('/@')) {
+        return next()
+      }
+
+      // Extract file extension from path (e.g., /file.js -> .js, /path/to/file.css -> .css)
+      const lastSegment = path.split('/').pop() || ''
+      const hasExtension = lastSegment.includes('.')
+      const extension = hasExtension ? lastSegment.substring(lastSegment.lastIndexOf('.')) : ''
+
+      // Only run on .md extension or no extension at all
+      // Skip all other file extensions (.js, .css, .html, .json, etc.)
+      if (hasExtension && extension !== '.md') {
         return next()
       }
 

packages/vite/test/unit/plugin-route-filtering.test.ts

@@ -75,6 +75,30 @@ describe('viteHtmlToMarkdownPlugin - Route Filtering', () => {
     expect(mockRes.setHeader).not.toHaveBeenCalled()
   })
 
+  it('should skip /@ Vite internal routes', async () => {
+    const plugin = viteHtmlToMarkdownPlugin({ verbose: false }) as Plugin & { configureServer: Function }
+
+    const mockServer = {
+      transformRequest: vi.fn(),
+      middlewares: { use: vi.fn() },
+    } as unknown as ViteDevServer
+
+    plugin.configureServer(mockServer)
+    const middleware = (mockServer.middlewares.use as ReturnType<typeof vi.fn>).mock.calls[0][0]
+
+    const mockReq = {
+      url: '/@vite/client',
+      headers: { accept: '*/*' },
+    }
+    const mockRes = { setHeader: vi.fn(), end: vi.fn() }
+    const mockNext = vi.fn()
+
+    await middleware(mockReq, mockRes, mockNext)
+
+    expect(mockNext).toHaveBeenCalled()
+    expect(mockRes.setHeader).not.toHaveBeenCalled()
+  })
+
   it('should skip explicit .html requests', async () => {
     const plugin = viteHtmlToMarkdownPlugin({ verbose: false }) as Plugin & { configureServer: Function }
 
@@ -131,4 +155,109 @@ describe('viteHtmlToMarkdownPlugin - Route Filtering', () => {
     expect(mockRes.setHeader).toHaveBeenCalledWith('Content-Type', 'text/markdown; charset=utf-8')
     expect(mockNext).not.toHaveBeenCalled()
   })
+
+  it('should skip .js files', async () => {
+    const plugin = viteHtmlToMarkdownPlugin({ verbose: false }) as Plugin & { configureServer: Function }
+
+    const mockServer = {
+      transformRequest: vi.fn(),
+      middlewares: { use: vi.fn() },
+    } as unknown as ViteDevServer
+
+    plugin.configureServer(mockServer)
+    const middleware = (mockServer.middlewares.use as ReturnType<typeof vi.fn>).mock.calls[0][0]
+
+    const mockReq = {
+      url: '/app.js',
+      headers: { accept: '*/*' },
+    }
+    const mockRes = { setHeader: vi.fn(), end: vi.fn() }
+    const mockNext = vi.fn()
+
+    await middleware(mockReq, mockRes, mockNext)
+
+    expect(mockNext).toHaveBeenCalled()
+    expect(mockRes.setHeader).not.toHaveBeenCalled()
+  })
+
+  it('should skip .css files', async () => {
+    const plugin = viteHtmlToMarkdownPlugin({ verbose: false }) as Plugin & { configureServer: Function }
+
+    const mockServer = {
+      transformRequest: vi.fn(),
+      middlewares: { use: vi.fn() },
+    } as unknown as ViteDevServer
+
+    plugin.configureServer(mockServer)
+    const middleware = (mockServer.middlewares.use as ReturnType<typeof vi.fn>).mock.calls[0][0]
+
+    const mockReq = {
+      url: '/styles.css',
+      headers: { accept: '*/*' },
+    }
+    const mockRes = { setHeader: vi.fn(), end: vi.fn() }
+    const mockNext = vi.fn()
+
+    await middleware(mockReq, mockRes, mockNext)
+
+    expect(mockNext).toHaveBeenCalled()
+    expect(mockRes.setHeader).not.toHaveBeenCalled()
+  })
+
+  it('should skip .json files', async () => {
+    const plugin = viteHtmlToMarkdownPlugin({ verbose: false }) as Plugin & { configureServer: Function }
+
+    const mockServer = {
+      transformRequest: vi.fn(),
+      middlewares: { use: vi.fn() },
+    } as unknown as ViteDevServer
+
+    plugin.configureServer(mockServer)
+    const middleware = (mockServer.middlewares.use as ReturnType<typeof vi.fn>).mock.calls[0][0]
+
+    const mockReq = {
+      url: '/data.json',
+      headers: { accept: '*/*' },
+    }
+    const mockRes = { setHeader: vi.fn(), end: vi.fn() }
+    const mockNext = vi.fn()
+
+    await middleware(mockReq, mockRes, mockNext)
+
+    expect(mockNext).toHaveBeenCalled()
+    expect(mockRes.setHeader).not.toHaveBeenCalled()
+  })
+
+  it('should allow routes with no extension when client prefers markdown', async () => {
+    const plugin = viteHtmlToMarkdownPlugin({ verbose: false }) as Plugin & { configureServer: Function }
+
+    const mockTransformRequest = vi.fn().mockResolvedValue({
+      code: '<html><body>Content</body></html>',
+    })
+
+    const mockServer = {
+      transformRequest: mockTransformRequest,
+      middlewares: { use: vi.fn() },
+    } as unknown as ViteDevServer
+
+    plugin.configureServer(mockServer)
+    const middleware = (mockServer.middlewares.use as ReturnType<typeof vi.fn>).mock.calls[0][0]
+
+    const mockReq = {
+      url: '/about',
+      headers: { accept: '*/*' },
+    }
+    const mockRes = {
+      setHeader: vi.fn(),
+      end: vi.fn(),
+      statusCode: 200,
+    }
+    const mockNext = vi.fn()
+
+    await middleware(mockReq, mockRes, mockNext)
+
+    expect(mockTransformRequest).toHaveBeenCalled()
+    expect(mockRes.setHeader).toHaveBeenCalledWith('Content-Type', 'text/markdown; charset=utf-8')
+    expect(mockNext).not.toHaveBeenCalled()
+  })
 })