Provide access to the client connection instance in all lifecycle hooks about hocuspocus HOT 7 OPEN

Ah, I should have explained!

We've implemented a schema versioning system. Anytime new node types are added or existing ones modified, we bump a schema version in the code. Any client that connects to Hocuspocus passes its schema version via a query param.

If a user on schema version 1 is connected and editing the doc, then a user with schema version 2 connects, the user on schema version 1 must be put in read only mode and asked to reload, as their client may not be able to understand the content of the newer schema.

Another simpler use case is that a client's access level changes. If the user keeps a long lived connection to a document, their permissions could change and either require that their connection is set to read only or closed.

from hocuspocus.

What if you’d have access to other connections in the onConnect hook? So you could write something like that:

async onConnect({ requestParameters, connection }) {
  // All connections to this document …
  connections
    // … with an older version …
    .filter(connection => requestParameters.version > connection.requestParameters.version)
    // … should be closed …
    .forEach(connection => {
      // … with a proper code and message.
      connection.close(Outdated.code, Outdated.reason)
    })
}

from hocuspocus.

@jamesopti What do you think? Would that help? 🙃

from hocuspocus.

Out of interest: How does the client get notified that the schema is outdated? The schema version is stored in the Y.js document?

Yup, exactly. We set a property on the doc like this:

async onCreateDocument({ document, documentName }) {
  document.getMap(this.configuration.metaDataKey).set('READY', false)
  await this.load(documentName)
  // Apply updates
  ...
  document.getMap(this.configuration.metaDataKey).set('READY', true)
}

@jamesopti What do you think? Would that help? 🙃

So this would partially solve our problem (the piece where the document is already loaded into memory).

But we also need to handle the case where the document has not been loaded, for example:

• Document in cold storage (postgres) is at version 4
• New client connects with version 3
• Document is fetched from cold storage
• Document version 4 is newer than connecting client (version 3)
• Connecting client must now be put in readonly mode

Perhaps onCreateDocument also needs to know the connection which triggered it to fire?

from hocuspocus.

I think it's worth considering how this versioning usecase could be a first class citizen of Hocuspocus. The other implementation of yjs that I've taken to production had to have exactly the same considerations, because the way that y-prosemirror is written if a node isn't understood it's just filtered out (deleted).

https://github.com/yjs/y-prosemirror/blob/master/src/plugins/sync-plugin.js#L448-L455

This means you absolutely cannot have old client versions connected or you'll lose content from newer clients as soon as they load it 🙈

from hocuspocus.

In which hook you'd like to set the connection to readOnly? I’d assume in the onChange hook?

Would you mind to elaborate on the use case? I think once someone is able to write to the document, it seems frustrating to ignore changes later (after the users was idle or something).

And would you make it writeable again at some point? That would mean we’d need to trigger another hook for blocked changes.

from hocuspocus.

Ah, got it! That’s a great use case.

Out of interest: How does the client get notified that the schema is outdated? The schema version is stored in the Y.js document?

from hocuspocus.