Do you need the WebSocket instance? It concerns me a bit that WebSocket instance has a broad interface. (e.g. socket.send())

It isn't required for the original request. @antfu maybe we should add this param once we have a real use case? And once we go there, it could be a Payload object like the others so we could extend it in the future if needed.

I don't have a strong opinion either on having it or not. I was just thinking why not, since we have that information.

A few notes added to the consideration:

• WebSocket it's a web standard so I don't need to worry about the API been changed
• If we say web socket is an implementation detail, vite:ws already indicates it's web socket. Later if we really find an alternative way of making the connection, those hooks won't be called so there will be no conflicts like the general vite:connect
• On the node side, plugins already have logics with WebSocket. It's not really an implementation detail we can't reveal.

I don't expect this will be changed and won't add much maintenance interface to us.

I do see one downside is that ppl might start using it and mess it around to break Vite - and then send requests to support very niche usages. But that could happen everywhere and we need to prevent that for sure.

Good points, I think we can talk about this in the next team meeting 👍🏼
I assume you also don't think it is a good idea to have a Payload object here.