What is the correct way to apply document migrations?

[Gin-Quin]

I’ve got more informations about this danger. From another post:

@dmonad

You should only reuse a client-id if you are absolutely sure that no one else with the same client-id is producing conflicting changes. If you can assure that, you can set ydoc.clientID = myclientid

And from the FAQ:

If you still want to retain a ClientID, you can do so by simply overwriting the ydoc.clientID property. But you must ensure that no other Y.Doc instance is currently holding that ClientID. This is not always possible: A user might open several browser windows with the same user account. When two Y.Doc instances with the same ClientID exist, the document might get permanently corrupted without a way to recover. So do this with caution.

So, it seems safe if the migration function is the only place where the client ID is manually set. I think an even safer version would be to have a client ID equals to the document version. This ensures that two conflicting updates due to a migration will never have the same client ID:

function applyDocumentMigration(doc: Y.Doc) {
  const clientID = doc.clientID

  const version = doc.getMap().get("version")

  switch (version) {
    case 0:
      doc.clientID = 0
      // ... apply migrations from v0 to v1
    case 1:
      doc.clientID = 1
      // ... apply migrations from v1 to v2
  }

  doc.clientID = clientID
}

What do you think? Am I about to do something horribly wrong? Even though I think it’s safe, the repeated warnings make me a bit anxious.