fix: on awareness craah prevention and memory leak onStoreDocument (#1032)

* fix: on awareness craah prevention and memory leak onStoreDocument

* fix: prevent awareness crash and memory leak in onStoreDocument

- Fix BUG #1: Lock memory leak - store acquired lock in locks map
- Fix BUG #3: Awareness crash when no connections - check connections before publish

Fixes #1027

* chore: removed prettier locally

* reformat

---------

Co-authored-by: Jan Thurau <jan@janthurau.de>

Author: matteotarantino-algor

.gitignore

@@ -13,3 +13,4 @@ npm-debug.log*
 yarn-debug.log*
 yarn-error.log*
 /database
+.prettierignore

packages/extension-redis/src/Redis.ts

@@ -246,11 +246,12 @@ export class Redis implements Extension {
 		const resource = this.lockKey(documentName);
 		const ttl = this.configuration.lockTimeout;
 		try {
-			await this.redlock.acquire([resource], ttl);
+			const lock = await this.redlock.acquire([resource], ttl);
 			const oldLock = this.locks.get(resource);
-			if (oldLock) {
+			if (oldLock?.release) {
 				await oldLock.release;
 			}
+			this.locks.set(resource, { lock });
 		} catch (error) {
 			//based on: https://github.com/sesamecare/redlock/blob/508e00dcd1e4d2bc6373ce455f4fe847e98a9aab/src/index.ts#L347-L349
 			if (
@@ -328,7 +329,13 @@ export class Redis implements Extension {
 		added,
 		updated,
 		removed,
+		document,
 	}: onAwarenessUpdatePayload) {
+		// Do not publish if there is no connection: it fixes this issue: "https://github.com/ueberdosis/hocuspocus/issues/1027"
+		const connections = document?.connections.size || 0;
+		if (connections === 0) {
+			return; // avoids exception
+		}
 		const changedClients = added.concat(updated, removed);
 		const message = new OutgoingMessage(
 			documentName,