@tldraw/sync-core 4.2.0-canary.fda2467026d5 → 4.2.0-next.47462e908ff5

package/src/lib/TLSyncClient.ts

@@ -31,7 +31,7 @@ import {
  * @param cb - Callback function that receives the event value
  * @returns Function to call when you want to unsubscribe from the events
  *
- * @public
+ * @internal
  */
 export type SubscribingFn<T> = (cb: (val: T) => void) => () => void
 
@@ -149,9 +149,9 @@ export type TLCustomMessageHandler = (this: null, data: any) => void
  * Event object describing changes in socket connection status.
  * Contains either a basic status change or an error with details.
  *
- * @public
+ * @internal
  */
-export type TLSocketStatusChangeEvent =
+export type TlSocketStatusChangeEvent =
 	| {
 			/** Connection came online or went offline */
 			status: 'online' | 'offline'
@@ -169,7 +169,7 @@ export type TLSocketStatusChangeEvent =
  *
  * @internal
  */
-export type TLSocketStatusListener = (params: TLSocketStatusChangeEvent) => void
+export type TLSocketStatusListener = (params: TlSocketStatusChangeEvent) => void
 
 /**
  * Possible connection states for a persistent client socket.
@@ -183,7 +183,7 @@ export type TLPersistentClientSocketStatus = 'online' | 'offline' | 'error'
  * Mode for handling presence information in sync sessions.
  * Controls whether presence data (cursors, selections) is shared with other clients.
  *
- * @public
+ * @internal
  */
 export type TLPresenceMode =
 	/** No presence sharing - client operates independently */
@@ -217,12 +217,9 @@ export type TLPresenceMode =
  * }
  * ```
  *
- * @public
+ * @internal
  */
-export interface TLPersistentClientSocket<
-	ClientSentMessage extends object = object,
-	ServerSentMessage extends object = object,
-> {
+export interface TLPersistentClientSocket<R extends UnknownRecord = UnknownRecord> {
 	/** Current connection state - online means actively connected and ready */
 	connectionStatus: 'online' | 'offline' | 'error'
 
@@ -230,32 +227,27 @@ export interface TLPersistentClientSocket<
 	 * Send a protocol message to the sync server
 	 * @param msg - Message to send (connect, push, ping, etc.)
 	 */
-	sendMessage(msg: ClientSentMessage): void
+	sendMessage(msg: TLSocketClientSentEvent<R>): void
 
 	/**
 	 * Subscribe to messages received from the server
 	 * @param callback - Function called for each received message
 	 * @returns Cleanup function to remove the listener
 	 */
-	onReceiveMessage: SubscribingFn<ServerSentMessage>
+	onReceiveMessage: SubscribingFn<TLSocketServerSentEvent<R>>
 
 	/**
 	 * Subscribe to connection status changes
 	 * @param callback - Function called when connection status changes
 	 * @returns Cleanup function to remove the listener
 	 */
-	onStatusChange: SubscribingFn<TLSocketStatusChangeEvent>
+	onStatusChange: SubscribingFn<TlSocketStatusChangeEvent>
 
 	/**
 	 * Force a connection restart (disconnect then reconnect)
 	 * Used for error recovery or when connection health checks fail
 	 */
 	restart(): void
-
-	/**
-	 * Close the connection
-	 */
-	close(): void
 }
 
 const PING_INTERVAL = 5000
@@ -320,7 +312,7 @@ const MAX_TIME_TO_WAIT_FOR_SERVER_INTERACTION_BEFORE_RESETTING_CONNECTION = PING
  * })
  * ```
  *
- * @public
+ * @internal
  */
 export class TLSyncClient<R extends UnknownRecord, S extends Store<R> = Store<R>> {
 	/** The last clock time from the most recent server update */
@@ -343,19 +335,14 @@ export class TLSyncClient<R extends UnknownRecord, S extends Store<R> = Store<R>
 
 	private disposables: Array<() => void> = []
 
-	/** @internal */
 	readonly store: S
-	/** @internal */
-	readonly socket: TLPersistentClientSocket<TLSocketClientSentEvent<R>, TLSocketServerSentEvent<R>>
+	readonly socket: TLPersistentClientSocket<R>
 
-	/** @internal */
 	readonly presenceState: Signal<R | null> | undefined
-	/** @internal */
 	readonly presenceMode: Signal<TLPresenceMode> | undefined
 
 	// isOnline is true when we have an open socket connection and we have
 	// established a connection with the server room (i.e. we have received a 'connect' message)
-	/** @internal */
 	isConnectedToRoom = false
 
 	/**
@@ -379,7 +366,7 @@ export class TLSyncClient<R extends UnknownRecord, S extends Store<R> = Store<R>
 	 * @param details - Connection details
 	 *   - isReadonly - Whether the connection is in read-only mode
 	 */
-	private readonly onAfterConnect?: (self: this, details: { isReadonly: boolean }) => void
+	public readonly onAfterConnect?: (self: this, details: { isReadonly: boolean }) => void
 
 	private readonly onCustomMessageReceived?: TLCustomMessageHandler
 
@@ -393,7 +380,7 @@ export class TLSyncClient<R extends UnknownRecord, S extends Store<R> = Store<R>
 
 	private readonly presenceType: R['typeName'] | null
 
-	private didCancel?: () => boolean
+	didCancel?: () => boolean
 
 	/**
 	 * Creates a new TLSyncClient instance to manage synchronization with a remote server.
@@ -413,7 +400,7 @@ export class TLSyncClient<R extends UnknownRecord, S extends Store<R> = Store<R>
 	 */
 	constructor(config: {
 		store: S
-		socket: TLPersistentClientSocket<any, any>
+		socket: TLPersistentClientSocket<R>
 		presence: Signal<R | null>
 		presenceMode?: Signal<TLPresenceMode>
 		onLoad(self: TLSyncClient<R, S>): void
@@ -529,7 +516,6 @@ export class TLSyncClient<R extends UnknownRecord, S extends Store<R> = Store<R>
 		}
 	}
 
-	/** @internal */
 	latestConnectRequestId: string | null = null
 
 	/**
@@ -655,7 +641,7 @@ export class TLSyncClient<R extends UnknownRecord, S extends Store<R> = Store<R>
 		this.lastServerClock = event.serverClock
 	}
 
-	private incomingDiffBuffer: TLSocketServerSentDataEvent<R>[] = []
+	incomingDiffBuffer: TLSocketServerSentDataEvent<R>[] = []
 
 	/** Handle events received from the server */
 	private handleServerEvent(event: TLSocketServerSentEvent<R>) {
@@ -715,7 +701,7 @@ export class TLSyncClient<R extends UnknownRecord, S extends Store<R> = Store<R>
 		this.scheduleRebase.cancel?.()
 	}
 
-	private lastPushedPresenceState: R | null = null
+	lastPushedPresenceState: R | null = null
 
 	private pushPresence(nextPresence: R | null) {
 		// make sure we push any document changes first

package/src/test/TestSocketPair.ts

@@ -62,7 +62,7 @@ export class TestSocketPair<R extends UnknownRecord> {
 	}
 	didReceiveFromClient?: (msg: TLSocketClientSentEvent<R>) => void = undefined
 	clientDisconnected?: () => void = undefined
-	clientSocket: TLPersistentClientSocket<TLSocketClientSentEvent<R>, TLSocketServerSentEvent<R>> = {
+	clientSocket: TLPersistentClientSocket<R> = {
 		connectionStatus: 'offline',
 		onStatusChange: (cb) => {
 			this.callbacks.onStatusChange = cb
@@ -76,7 +76,7 @@ export class TestSocketPair<R extends UnknownRecord> {
 				this.callbacks.onReceiveMessage = null
 			}
 		},
-		sendMessage: (msg) => {
+		sendMessage: (msg: TLSocketClientSentEvent<R>) => {
 			if (this.clientSocket.connectionStatus !== 'online') {
 				throw new Error('trying to send before open')
 			}
@@ -87,9 +87,6 @@ export class TestSocketPair<R extends UnknownRecord> {
 			this.disconnect()
 			this.connect()
 		},
-		close: () => {
-			this.disconnect()
-		},
 	}
 
 	callbacks = {