Nov 6, 2025 · Nov 7, 2025 · Nov 7, 2025 · Nov 6, 2025
diff --git a/packages/nuxt/src/imports/module.ts b/packages/nuxt/src/imports/module.ts
    }

    let ctx: Unimport
    const viteServers = new Set<any>()

    // Track all Vite servers (client + SSR)
    nuxt.hook('vite:serverCreated', (server) => {
      viteServers.add(server)
    })

    // Vite plugin to detect and handle export type changes BEFORE HMR
    nuxt.hook('vite:configResolved', (viteConfig) => {
      if (!viteConfig.plugins) { return }
      viteConfig.plugins.push({
        name: 'nuxt:composables-hmr',
        handleHotUpdate: async (hmrContext) => {
          // Only handle composable files
          if (!composablesDirs.some(dir => hmrContext.file.startsWith(dir + '/'))) {
            return
          }

          // Get current imports to compare
          const currentImports = await ctx.getImports()
          const oldImports = currentImports.filter(i => i.from === hmrContext.file)

          // Scan the updated file
          const newImports = await scanDirExports([hmrContext.file], {
            fileFilter: file => !isIgnored(file),
          })

          // Check if export type changed (default ↔ named)
          let exportTypeChanged = false
          for (const newImport of newImports) {
            const oldImport = oldImports.find(i => (i.as || i.name) === (newImport.as || newImport.name))
            if (oldImport && oldImport.name !== newImport.name) {
              exportTypeChanged = true
              break
            }
          }

          if (exportTypeChanged) {
            // Invalidate module cache recursively in all Vite servers (client + SSR)
            for (const server of viteServers) {
              const visited = new Set()
              const invalidateRecursively = (mod: any) => {
                if (!mod || visited.has(mod)) { return }
                visited.add(mod)
                mod.importers.forEach((importer: any) => invalidateRecursively(importer))
                server.moduleGraph.invalidateModule(mod)
              }

              const modules = server.moduleGraph.getModulesByFile(hmrContext.file)
              if (modules) {
                modules.forEach((mod: any) => invalidateRecursively(mod))
              }
            }

            // Trigger full reload - the changed export type cannot be handled with HMR
            hmrContext.server.ws.send({ type: 'full-reload', path: '*' })
            return []
          }

          return
        },
      })
    })

    // initialise unimport only after all modules
    // have had a chance to register their hooks
Original file line number	Diff line number	Diff line change
Expand Up		@@ -89,6 +89,69 @@ export default defineNuxtModule<Partial<ImportsOptions>>({
		}

		let ctx: Unimport
		const viteServers = new Set<any>()

		// Track all Vite servers (client + SSR)
		nuxt.hook('vite:serverCreated', (server) => {
		viteServers.add(server)
		})

		// Vite plugin to detect and handle export type changes BEFORE HMR
		nuxt.hook('vite:configResolved', (viteConfig) => {
		if (!viteConfig.plugins) { return }
Copy link Contributor HigherOrderLogicNov 7, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Shouldnt we initialise the list here and push the plugin in anyway? Copy link CollaboratorAuthor Flo0806Nov 7, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. `viteConfig.plugins`? I guess no. Because it is`readonly` and the`vite:configResolved` hook is called after Vite's config resolution, which always includes a plugins array. The early return is defensive programming - if`plugins` doesn't exist, something is fundamentally wrong with the Vite setup and we shouldn't proceed.
		viteConfig.plugins.push({
		name: 'nuxt:composables-hmr',
		handleHotUpdate: async (hmrContext) => {
		// Only handle composable files
		if (!composablesDirs.some(dir => hmrContext.file.startsWith(dir + '/'))) {
		return
		}

		// Get current imports to compare
		const currentImports = await ctx.getImports()
		const oldImports = currentImports.filter(i => i.from === hmrContext.file)

		// Scan the updated file
		const newImports = await scanDirExports([hmrContext.file], {
		fileFilter: file => !isIgnored(file),
		})

		// Check if export type changed (default ↔ named)
		let exportTypeChanged = false
		for (const newImport of newImports) {
		const oldImport = oldImports.find(i => (i.as \|\| i.name) === (newImport.as \|\| newImport.name))
		if (oldImport && oldImport.name !== newImport.name) {
		exportTypeChanged = true
		break
		}
		}

Comment on lines +119 to +128 Copy link coderabbitaibotNov 6, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. ⚠️ Potential issue \|🟠 Major Detection misses default↔named switches when the alias changes Line 120: if the export flips between`default` and named while you also rename the symbol (e.g.`export default function useKey()` →`export function useKeyRenamed()`),`oldImport` is never found so`exportTypeChanged` stays`false`. The module cache is left untouched and the original`(…).default is not a function` crash still reproduces. Please derive`exportTypeChanged` from the presence of a`name === 'default'` entry before versus after (and only fall back to the alias comparison when that check is inconclusive) so we catch these alias-renaming cases as well. Otherwise the fix remains incomplete.
		if (exportTypeChanged) {
		// Invalidate module cache recursively in all Vite servers (client + SSR)
		for (const server of viteServers) {
		const visited = new Set()
		const invalidateRecursively = (mod: any) => {
		if (!mod \|\| visited.has(mod)) { return }
		visited.add(mod)
		mod.importers.forEach((importer: any) => invalidateRecursively(importer))
		server.moduleGraph.invalidateModule(mod)
		}

		const modules = server.moduleGraph.getModulesByFile(hmrContext.file)
		if (modules) {
		modules.forEach((mod: any) => invalidateRecursively(mod))
		}
		}

		// Trigger full reload - the changed export type cannot be handled with HMR
		hmrContext.server.ws.send({ type: 'full-reload', path: '*' })
		return []
		}

		return
		},
		})
		})

		// initialise unimport only after all modules
		// have had a chance to register their hooks
Expand Down