Implementation in HMR event, 'vite:afterUpdate'

[eddie0329]

Clear and concise description of the problem

The problem is that I cannot precisely handle HMR events, specifically 'after update'.

According to rollup.js, there is feature like below:

import.meta.hot.afterUpdate(() => {
   // after update called
}) 

and other bundlers have similar features, such as wepack.addStatus('idle', () => { //... }).

Suggested solution

Abstract implementation is somehow like below:

Add notifyListeners('vite:afterUpdate') after payload.updates.forEach loop ends.

async function handleMessage(payload: HMRPayload) {
  switch (payload.type) {
    case 'update':  
      notifyListeners('vite:beforeUpdate', payload)
      payload.updates.forEach((update) => {
         // ... js and cs updates
      })
     notifyListeners('vite:afterUpdate') // ⭐️  fire events 'vite:afterUpdate'
      break;

Alternative

No response

Additional context

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

[bluwy]

I think this make sense, but it would be helpful if you can show a usecase for it too to further justify this feature request.

[eddie0329]

@bluwy Thanks for reply. Should I refer ordinary usecase, using other bundlers, such as webpack, rollup? (usecase for afterUpdate.
Or Should I get mine using vite?

[potty]

For me would be useful to update styles after update, because my app has multiple iframes and I have to keep styles in sync between them manually.

[alvarosabu]

I was looking for something like this, this will be specially useful for the work I'm doing with ThreeJS, since is tricky to have a good DX while updating files, with this hooks I could basically dispose renderers and initialize them

[patak-dev]

@alvarosabu see #9810, it will be available in Vite 4 (the first alpha is going to be out next week, so you could start using it if you need it before the stable release)