feat: onServerPrefetch

[Akryum]

Add a Composition API lifecycle hook onServerPrefetch that merge the passed handler function to the serverPrefetch component option.
Related to vuejs/apollo#1102

[github-actions]

Size report

Path	Size
vue.global.prod.js	41.81 KB (+0.13% 🔺)
runtime-dom.global.prod.js	27.66 KB (+0.16% 🔺)
size-check.global.prod.js	16.84 KB (+0.24% 🔺)

[LinusBorg]

Would we technically consider this a new feature, requiring a minor release?

We're putting together 3.0.6, if we include this, we'd have to go to 3.1 instead.

Just mentioning as we had other features, not coming right now, labelled for 3.1. so we would have to relabel them for 3.2

[Akryum]

The GH action can't download the actions/checkout action for some reason 🤷‍♂️

[HcySunYang]

@Akryum How about this #2902, it supports hooks from mixin/extend.

[Akryum]

@Akryum How about this #2902, it supports hooks from mixin/extend.

I think both can co exist

[HcySunYang]

Oh, sorry, I took a closer look at that PR(#2902), the conclusion is that these two cannot coexist because they basically do the same thing. The #2902 also provides an implementation of onServerPrefetch, and the implementation mechanism is the same as other hooks. That mechanism provides the ability to pause tracking, see injectHook pauseTracking and it's necessary. Of course, that PR(#2902) also has some flaws, but I think we can review and try to use it?

[HcySunYang]

@LinusBorg @posva , Can you help me take a look at these two PRs(#2902 and #3070)? thanks.

[Akryum]

I've taken some parts of #2902 to make it work like a lifecycle hook. Do we merge this PR into @tabjy's PR to give him credits?

[Akryum]

Note: error display is already done by the wrapped lifecycle hook function.

[tabjy]

Sorry I've been a bit busy recently.

I've taken some parts of #2902 to make it work like a lifecycle hook. Do we merge this PR into @tabjy's PR to give him credits?

That sounds like a lot of work. Alternatively, maybe you can add me as a co-author when merging this request? In which case please use Co-authored-by: tabjy <8033899+tabjy@users.noreply.github.com>.

I'll close #2902 once this is merged.

[oliverzy]

How is this feature related to "Suspense + Async Setup"?

[araratmartirossyan]

I still have this issue with vite. A patch that I found in the issue fixed it but only for dev. After build it's coming back. Any updates on how it can be fixed on build time?