fix(hmr): cannot reload after missing import on server startup (#9534)

packages/vite/src/node/plugins/importAnalysis.ts

@@ -294,6 +294,8 @@ export function importAnalysisPlugin(config: ResolvedConfig): Plugin {
           if (ssr) {
             return [url, url]
           }
+          // fix#9534, prevent the importerModuleNode being stopped from propagating updates
+          importerModule.isSelfAccepting = false
           return this.error(
             `Failed to resolve import "${url}" from "${path.relative(
               process.cwd(),

playground/hmr/__tests__/hmr.spec.ts

@@ -670,4 +670,28 @@ if (!isBuild) {
       return await el.textContent()
     }, '[wow]1')
   })
+
+  test('keep hmr reload after missing import on server startup', async () => {
+    const file = 'missing-import/a.js'
+    const importCode = "import 'missing-modules'"
+    const unImportCode = `// ${importCode}`
+    const timeout = 2000
+
+    await page.goto(viteTestUrl + '/missing-import/index.html')
+
+    browserLogs.length = 0
+    expect(browserLogs).toMatchObject([])
+
+    editFile(file, (code) => code.replace(importCode, unImportCode))
+
+    await page.waitForNavigation({ timeout })
+    expect(browserLogs.some((msg) => msg.match('missing test'))).toBe(true)
+    browserLogs.length = 0
+
+    editFile(file, (code) => code.replace(unImportCode, importCode))
+
+    await page.waitForNavigation({ timeout })
+    expect(browserLogs.some((msg) => msg.includes('500'))).toBe(true)
+    browserLogs.length = 0
+  })
 }

playground/hmr/missing-import/a.js

@@ -0,0 +1,3 @@
+import 'missing-modules'
+
+console.log('missing test')

playground/hmr/missing-import/index.html

@@ -0,0 +1,2 @@
+<div>Page</div>
+<script type="module" src="main.js"></script>

playground/hmr/missing-import/main.js

@@ -0,0 +1 @@
+import './a.js'