@abraca/dabra 0.1.5 → 0.1.7

package/dist/abracadabra-provider.cjs

@@ -2116,6 +2116,10 @@ var SubdocMessage = class extends OutgoingMessage {
 
 //#endregion
 //#region packages/provider/src/AbracadabraProvider.ts
+/** Validate that a string is a UUID acceptable by the server's DocId parser. */
+function isValidDocId(id) {
+	return /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(id);
+}
 /**
 * AbracadabraProvider extends HocuspocusProvider with:
 *
@@ -2142,6 +2146,7 @@ var AbracadabraProvider = class AbracadabraProvider extends HocuspocusProvider {
 		super(resolved);
 		this.effectiveRole = null;
 		this.childProviders = /* @__PURE__ */ new Map();
+		this.pendingLoads = /* @__PURE__ */ new Map();
 		this.boundHandleYSubdocsChange = this.handleYSubdocsChange.bind(this);
 		this._client = client;
 		this.abracadabraConfig = configuration;
@@ -2251,7 +2256,10 @@ var AbracadabraProvider = class AbracadabraProvider extends HocuspocusProvider {
 	* We intercept additions to register them with the Abracadabra server.
 	*/
 	handleYSubdocsChange({ added, removed }) {
-		for (const subdoc of added) this.registerSubdoc(subdoc);
+		for (const subdoc of added) {
+			if (!isValidDocId(subdoc.guid)) continue;
+			this.registerSubdoc(subdoc);
+		}
 		for (const subdoc of removed) this.unloadChild(subdoc.guid);
 	}
 	/**
@@ -2272,17 +2280,35 @@ var AbracadabraProvider = class AbracadabraProvider extends HocuspocusProvider {
 	}
 	/**
 	* Create (or return cached) a child AbracadabraProvider for a given
-	* child document id.  The child shares the parent's WebSocket connection.
+	* child document id.  Each child opens its own WebSocket connection because
+	* the server is document-scoped (one WebSocket ↔ one document).
 	*/
 	async loadChild(childId) {
+		if (!isValidDocId(childId)) throw new Error(`loadChild: "${childId}" is not a valid document ID (must be a UUID). If this node was created with an older version of the app, delete it and recreate it.`);
 		if (this.childProviders.has(childId)) return this.childProviders.get(childId);
+		if (this.pendingLoads.has(childId)) return this.pendingLoads.get(childId);
+		const load = this._doLoadChild(childId);
+		this.pendingLoads.set(childId, load);
+		load.finally(() => this.pendingLoads.delete(childId));
+		return load;
+	}
+	async _doLoadChild(childId) {
 		const childDoc = new yjs.Doc({ guid: childId });
-		const parentWsp = this.configuration.websocketProvider;
+		await new Promise((resolve) => {
+			const onRegistered = ({ childId: cid }) => {
+				if (cid === childId) {
+					this.off("subdocRegistered", onRegistered);
+					resolve();
+				}
+			};
+			this.on("subdocRegistered", onRegistered);
+			this.registerSubdoc(childDoc);
+			setTimeout(resolve, 3e3);
+		});
 		const childProvider = new AbracadabraProvider({
 			name: childId,
 			document: childDoc,
-			url: parentWsp.configuration.url,
-			WebSocketPolyfill: parentWsp.configuration.WebSocketPolyfill,
+			url: this.abracadabraConfig.url ?? this.configuration.websocketProvider?.url,
 			token: this.configuration.token,
 			subdocLoading: this.subdocLoading,
 			disableOfflineStore: this.abracadabraConfig.disableOfflineStore,