fix: register server handler only for server provider (#524)

Author: onmax

src/module.ts

@@ -92,10 +92,12 @@ export default defineNuxtModule<ModuleOptions>({
       global: true,
       filePath: await resolver.resolvePath('./runtime/components/index'),
     })
-    addServerHandler({
-      route: `${options.localApiEndpoint || '/api/_nuxt_icon'}/:collection`,
-      handler: resolver.resolve('./runtime/server/api'),
-    })
+    if (options.provider === 'server') {
+      addServerHandler({
+        route: `${options.localApiEndpoint || '/api/_nuxt_icon'}/:collection`,
+        handler: resolver.resolve('./runtime/server/api'),
+      })
+    }
 
     await setupCustomCollectionsWatcher(options, nuxt, ctx)
 

test/fixtures/iconify-provider/nuxt.config.ts

@@ -0,0 +1,10 @@
+import NuxtIcon from '../../../src/module'
+
+export default defineNuxtConfig({
+  modules: [
+    [NuxtIcon, {
+      provider: 'iconify',
+      serverBundle: false,
+    }],
+  ],
+})

test/module.test.ts

@@ -0,0 +1,19 @@
+import { fileURLToPath } from 'node:url'
+import { describe, expect, it } from 'vitest'
+import { setup, useTestContext } from '@nuxt/test-utils/e2e'
+
+describe('module', async () => {
+  await setup({
+    rootDir: fileURLToPath(new URL('./fixtures/iconify-provider', import.meta.url)),
+    build: true,
+    server: false,
+  })
+
+  it('does not register the local API handler for the iconify provider', () => {
+    const handlers = useTestContext().nuxt?.options.serverHandlers
+
+    expect(handlers).not.toContainEqual(expect.objectContaining({
+      route: '/api/_nuxt_icon/:collection',
+    }))
+  })
+})