Comments: Make `getThreads` async

[AntoLC]

Problem

getThreads is necessary to load the "Threads" but the interface does not accept Promise response, so it does not seem easy to include this part with REST:

BlockNote/packages/core/src/comments/threadstore/ThreadStore.ts

Line 119 in ac322a1

abstract getThread(threadId: string): ThreadData;

---

A REST example implementation is provided but it does not include the getThreads part:
https://github.com/TypeCellOS/BlockNote/blob/ac322a1aa38fd48e64afdf7cfaebb10a6b77c364/packages/core/src/comments/threadstore/yjs/RESTYjsThreadStore.ts

Based on that:

BlockNote/packages/core/src/comments/threadstore/yjs/YjsThreadStoreBase.ts

Lines 29 to 35 in ac322a1

	public getThreads(): Map<string, ThreadData> {
	const threadMap = new Map<string, ThreadData>();
	this.threadsYMap.forEach((yThread, id) => {
	threadMap.set(id, yMapToThread(yThread));
	});
	return threadMap;
	}

Request

Could we change the interface to accept promise response with getThreads ?

Thx!

Bonus

• I'm a sponsor and would appreciate if you could look into this sooner than later 💖

[nperez0111]

@AntoLC, this is because the thread is stored within the Y.js document so it can be fetched from the document synchronously. If you were to implement reading of threads for the Y.js document with REST, it would not have sufficient information to describe the location of the thread within the document (because that is what we can derive from the Y.js implementation).

So, the RESTYjsThreadStore is necessarily coupled to the Y.js storage of comments at this time.