fix(vite-node): top-level throw in module is not reported properly (#…

…6840)

Co-authored-by: Hiroshi Ogawa <hi.ogawa.zz@gmail.com>

packages/vite-node/src/hmr/hmr.ts

@@ -160,8 +160,8 @@ async function fetchUpdate(
   }
 }
 
-function warnFailedFetch(err: Error, path: string | string[]) {
-  if (!err.message.match('fetch')) {
+function warnFailedFetch(err: unknown, path: string | string[]) {
+  if (!(err instanceof Error) || !err.message.match('fetch')) {
     console.error(err)
   }
 

test/vite-node/src/hmr-throw.js

@@ -0,0 +1,11 @@
+if (import.meta.hot) {
+  import.meta.hot.accept(() => {})
+  if (import.meta.hot.data.value == null) {
+    import.meta.hot.data.value = 0
+  }
+  else {
+    // eslint-disable-next-line no-throw-literal
+    throw 'some error'
+  }
+}
+console.error('ready')

test/vite-node/test/hmr.test.ts

@@ -15,3 +15,16 @@ test('hmr.accept works correctly', async () => {
   await viteNode.waitForStderr('Accept')
   await viteNode.waitForStdout(`[vite-node] hot updated: ${scriptFile}`)
 })
+
+test('can handle top-level throw in self-accepting module', async () => {
+  const scriptFile = resolve(__dirname, '../src/hmr-throw.js')
+
+  const { viteNode } = await runViteNodeCli('--watch', scriptFile)
+
+  await viteNode.waitForStderr('ready')
+
+  editFile(scriptFile, content => `${content}\nconsole.error("done")`)
+
+  await viteNode.waitForStderr('some error')
+  await viteNode.waitForStderr(`[hmr] Failed to reload ${scriptFile}. This could be due to syntax errors or importing non-existent modules. (see errors above)`)
+})