refactor: handleHotUpdate will be removed, use hotUpdate instead

Author: ElMassimo

.claude/launch.json

@@ -0,0 +1,11 @@
+{
+  "version": "0.0.1",
+  "configurations": [
+    {
+      "name": "blog",
+      "runtimeExecutable": "pnpm",
+      "runtimeArgs": ["blog"],
+      "port": 5173
+    }
+  ]
+}

.claude/settings.local.json

@@ -0,0 +1,7 @@
+{
+  "permissions": {
+    "allow": [
+      "mcp__Claude_Preview__preview_start"
+    ]
+  }
+}

packages/iles/src/node/plugin/plugin.ts

@@ -116,9 +116,9 @@ export default function IslandsPlugins (appConfig: AppConfig): PluginOption[] {
         if (id === APP_COMPONENT_PATH && !isBuild && appConfig.debug)
           return code.replace('const DebugPanel = () => null', () => `import DebugPanel from '${DEBUG_COMPONENT_PATH}'`)
       },
-      handleHotUpdate ({ file, server }) {
-        if (file === appPath) return [server.moduleGraph.getModuleById(USER_APP_REQUEST_PATH)!]
-        if (file === sitePath) return [server.moduleGraph.getModuleById(USER_SITE_REQUEST_PATH)!]
+      hotUpdate ({ file }) {
+        if (file === appPath) return [this.environment.moduleGraph.getModuleById(USER_APP_REQUEST_PATH)!]
+        if (file === sitePath) return [this.environment.moduleGraph.getModuleById(USER_SITE_REQUEST_PATH)!]
       },
       configureServer (devServer) {
         server = devServer

packages/pages/src/hmr.ts

@@ -1,51 +1,42 @@
 import type { ViteDevServer, Plugin } from 'vite'
 import { debug, slash } from './utils'
-import { Awaitable, MODULE_ID, ResolvedOptions, PagesApi } from './types'
+import { MODULE_ID, ResolvedOptions, PagesApi } from './types'
 
-export function handleHMR (api: PagesApi, options: ResolvedOptions, clearRoutes: () => void): Plugin['handleHotUpdate'] {
+export function handleHMR (api: PagesApi, options: ResolvedOptions, clearRoutes: () => void): Plugin['hotUpdate'] {
   const server = options.server!
 
-  onPage('add', async (path) => {
-    const page = await api.addPage(path)
-    debug.hmr('add %s %O', path, page)
-    return true
-  })
-
-  onPage('unlink', (path) => {
-    api.removePage(path)
-    debug.hmr('remove', path)
-    return true
-  })
-
-  return async (ctx) => {
-    const path = slash(ctx.file)
+  return async function ({ file, type }) {
+    const path = slash(file)
     if (api.isPage(path)) {
-      const { changed, needsReload } = await api.updatePage(path)
-      if (changed) debug.hmr('change', path)
-      if (needsReload) fullReload()
+      switch (type) {
+        case 'create': {
+          const page = await api.addPage(path)
+          debug.hmr('add %s %O', path, page)
+          fullReload()
+          return []
+        }
+        case 'delete': {
+          api.removePage(path)
+          debug.hmr('remove', path)
+          fullReload()
+          return []
+        }
+        case 'update': {
+          const { changed, needsReload } = await api.updatePage(path)
+          if (changed) debug.hmr('change', path)
+          if (needsReload) fullReload()
+        }
+      }
     }
   }
 
-  function onPage (eventName: string, handler: (path: string) => Awaitable<void | boolean>) {
-    server.watcher.on(eventName, async (path) => {
-      path = slash(path)
-      if (api.isPage(path) && await handler(path))
-        fullReload()
-    })
-  }
-
   function fullReload () {
     invalidatePagesModule(server)
     clearRoutes()
     server.ws.send({ type: 'full-reload' })
   }
 }
 
-function invalidatePageFiles (path: string, { moduleGraph }: ViteDevServer) {
-  moduleGraph.getModulesByFile(path)
-    ?.forEach(mod => moduleGraph.invalidateModule(mod))
-}
-
 function invalidatePagesModule ({ moduleGraph }: ViteDevServer) {
   const mod = moduleGraph.getModuleById(MODULE_ID)
   if (mod) moduleGraph.invalidateModule(mod)

packages/pages/src/pages.ts

@@ -58,7 +58,7 @@ export default function IlesPages (): any {
       },
       async configureServer (server) {
         options.server = server
-        plugin.handleHotUpdate = handleHMR(api, options, () => { generatedRoutes = undefined })
+        plugin.hotUpdate = handleHMR(api, options, () => { generatedRoutes = undefined })
       },
       async buildStart () {
         await api.addAllPages()