fix: consider basePath when generating presenter URLs

Author: coaxial

packages/client/logic/slides.ts

@@ -20,7 +20,9 @@ export function getSlidePath(
   if (typeof route === 'number' || typeof route === 'string')
     route = getSlide(route)!
   const no = route.meta.slide?.frontmatter.routeAlias ?? route.no
-  return exporting ? `/export/${no}` : presenter ? `/presenter/${no}` : `/${no}`
+  const basePath = import.meta.env.BASE_URL.replace(/\/$/, '') ?? ''
+  const path = exporting ? `/export/${no}` : presenter ? `/presenter/${no}` : `/${no}`
+  return `${basePath}${path}`
 }
 
 export function useIsSlideActive() {

test/utils.test.ts

@@ -1,6 +1,7 @@
-import type { ResolvedFontOptions, SlideInfo } from '@slidev/types'
+import type { ResolvedFontOptions, SlideInfo, SlideRoute } from '@slidev/types'
 import { relative, resolve } from 'node:path'
 import { slash } from '@antfu/utils'
+import { getSlidePath } from '@slidev/client/logic/slides'
 import MarkdownIt from 'markdown-it'
 import { describe, expect, it } from 'vitest'
 import YAML from 'yaml'
@@ -145,4 +146,38 @@ describe('utils', () => {
       "
     `)
   })
+
+  it('getSlidePath with base path', () => {
+    // Mock import.meta.env.BASE_URL
+    const originalBaseUrl = import.meta.env.BASE_URL
+
+    // Test with base path
+    import.meta.env.BASE_URL = '/my_monorepo/my_prez/'
+
+    const mockRoute: SlideRoute = {
+      no: 2,
+      meta: {
+        slide: {
+          frontmatter: {},
+        },
+      },
+    }
+
+    expect(getSlidePath(mockRoute, false, false)).toBe(
+      '/my_monorepo/my_prez/2',
+    )
+    expect(getSlidePath(mockRoute, true, false)).toBe(
+      '/my_monorepo/my_prez/presenter/2',
+    )
+    expect(getSlidePath(mockRoute, false, true)).toBe(
+      '/my_monorepo/my_prez/export/2',
+    )
+
+    // Test with root path
+    import.meta.env.BASE_URL = '/'
+    expect(getSlidePath(mockRoute, false, false)).toBe('/2')
+
+    // Restore original
+    import.meta.env.BASE_URL = originalBaseUrl
+  })
 })