chore(js): docs

src/ffi/js/buffer.rs

@@ -8,6 +8,8 @@ use crate::buffer::controller::BufferController;
 #[napi]
 impl BufferController {
 
+	/// Register a callback to be invoked every time a new event is available to consume
+	/// There can only be one callback registered at any given time.
 	#[napi(js_name = "callback", ts_args_type = "fun: (event: BufferController) => void")]
 	pub fn js_callback(&self, fun: napi::JsFunction) -> napi::Result<()>{
 		let tsfn : ThreadsafeFunction<crate::buffer::controller::BufferController, Fatal> = 
@@ -17,46 +19,51 @@ impl BufferController {
 			}
 		)?;
 		self.callback(move |controller : BufferController| {
-
+			tsfn.call(controller.clone(), ThreadsafeFunctionCallMode::Blocking);
-			tsfn.call(controller.clone(), ThreadsafeFunctionCallMode::Blocking); //check this with tracing also we could use Ok(event) to get the error
+			//check this with tracing also we could use Ok(event) to get the error
 			// If it blocks the main thread too many time we have to change this
-
 		});
 
 		Ok(())
 	}
 
+	/// Remove registered buffer callback
 	#[napi(js_name = "clear_callback")]
 	pub fn js_clear_callback(&self) {
 		self.clear_callback();
 	}
 	
-
+	/// Get buffer path
 	#[napi(js_name = "get_path")]
 	pub fn js_path(&self) -> &str {
 		self.path()
 	}
 
+	/// Block until next buffer event without returning it
 	#[napi(js_name = "poll")]
 	pub async fn js_poll(&self) -> napi::Result<()>{
 		Ok(self.poll().await?)
 	}
 
+	/// Return next buffer event if present
 	#[napi(js_name = "try_recv")]
 	pub async fn js_try_recv(&self) -> napi::Result<Option<TextChange>> {
 		Ok(self.try_recv().await?)
 	}
 
+	/// Wait for next buffer event and return it
 	#[napi(js_name = "recv")]
 	pub async fn js_recv(&self) -> napi::Result<TextChange> {
 		Ok(self.recv().await?)
 	}
 
+	/// Send a buffer update to workspace
 	#[napi(js_name = "send")]
 	pub async fn js_send(&self, op: TextChange) -> napi::Result<()> {
 		Ok(self.send(op).await?)
 	}
 
+	/// Return buffer whole content
 	#[napi(js_name = "content")]
 	pub async fn js_content(&self) -> napi::Result<String> {
 		Ok(self.content().await?)

src/ffi/js/cursor.rs

@@ -4,7 +4,6 @@ use napi::threadsafe_function::{ThreadsafeFunction, ThreadSafeCallContext, Threa
 use crate::api::Controller;
 use crate::cursor::controller::CursorController;
 
-
 #[napi(object, js_name = "Cursor")]
 pub struct JsCursor {
 	/// range of text change, as char indexes in buffer previous state
@@ -26,6 +25,7 @@ impl From<JsCursor> for crate::api::Cursor {
 		}
 	}
 }
+
 impl From<crate::api::Cursor> for JsCursor {
 	fn from(value: crate::api::Cursor) -> Self {
 		JsCursor {
@@ -44,6 +44,8 @@ impl From<crate::api::Cursor> for JsCursor {
 #[napi]
 impl CursorController {
 
+	/// Register a callback to be called on receive.
+	/// There can only be one callback registered at any given time.
 	#[napi(js_name = "callback", ts_args_type = "fun: (event: CursorController) => void")]
 	pub fn js_callback(&self, fun: napi::JsFunction) -> napi::Result<()>{
 		let tsfn : ThreadsafeFunction<crate::cursor::controller::CursorController, Fatal> = 
@@ -53,32 +55,35 @@ impl CursorController {
 			}
 		)?;
 		self.callback(move |controller : CursorController| {
-
+			tsfn.call(controller.clone(), ThreadsafeFunctionCallMode::Blocking);
-			tsfn.call(controller.clone(), ThreadsafeFunctionCallMode::Blocking); //check this with tracing also we could use Ok(event) to get the error
+			//check this with tracing also we could use Ok(event) to get the error
 			// If it blocks the main thread too many time we have to change this
-
 		});
 
 		Ok(())
 	}
 
+	/// Clear the registered callback
 	#[napi(js_name = "clear_callback")]
 	pub fn js_clear_callback(&self) {
 		self.clear_callback();
 	}
 
+	/// Send a new cursor event to remote
 	#[napi(js_name = "send")]
 	pub async fn js_send(&self, pos: JsCursor) -> napi::Result<()> {
 		Ok(self.send(crate::api::Cursor::from(pos)).await?)
 	}
 
 
+	/// Get next cursor event if available without blocking
 	#[napi(js_name= "try_recv")]
 	pub async fn js_try_recv(&self) -> napi::Result<Option<JsCursor>> {
 		Ok(self.try_recv().await?
 		.map(JsCursor::from))
 	}
 
+  /// Block until next 
 	#[napi(js_name= "recv")]
 	pub async fn js_recv(&self) -> napi::Result<JsCursor> {
 		Ok(self.recv().await?.into())

src/ffi/js/ext.rs

@@ -1,11 +1,14 @@
 use napi_derive::napi;
 
 
+/// Hash function
 #[napi(js_name = "hash")]
 pub fn js_hash(data: String) -> i64 {
     crate::ext::hash(data)
 }
 
+
+/// Get the current version of the client
 #[napi(js_name = "version")]
 pub fn js_version() -> String {
     crate::version()

src/ffi/js/workspace.rs

@@ -22,56 +22,70 @@ impl From<crate::api::Event> for JsEvent {
 
 #[napi]
 impl Workspace {
+	/// Get the unique workspace id
 	#[napi(js_name = "id")]
 	pub fn js_id(&self) -> String {
 		self.id()
 	}
 	
+	/// List all available buffers in this workspace
 	#[napi(js_name = "filetree")]
 	pub fn js_filetree(&self, filter: Option<&str>, strict: bool) -> Vec<String> {
 		self.filetree(filter, strict)
 	}
 
+	/// List all user names currently in this workspace
 	#[napi(js_name = "user_list")]
 	pub fn js_user_list(&self) -> Vec<String> {
 		self.user_list()
 	}
 
+	/// List all currently active buffers
 	#[napi(js_name = "buffer_list")]
 	pub fn js_buffer_list(&self) -> Vec<String> {
 		self.buffer_list()
 	}
 
+	/// Get workspace's Cursor Controller
 	#[napi(js_name = "cursor")]
 	pub fn js_cursor(&self) -> CursorController {
 		self.cursor()
 	}
 
+	/// Get a buffer controller by its name (path)
 	#[napi(js_name = "buffer_by_name")]
 	pub fn js_buffer_by_name(&self, path: String) -> Option<BufferController> {
 		self.buffer_by_name(&path)
 	}
 
+	/// Create a new buffer in the current workspace
 	#[napi(js_name = "create")]
 	pub async fn js_create(&self, path: String) -> napi::Result<()> {
 		Ok(self.create(&path).await?)
 	}
 
+	
+	/// Attach to a workspace buffer, starting a BufferController
 	#[napi(js_name = "attach")]
 	pub async fn js_attach(&self, path: String) -> napi::Result<BufferController> {
 		Ok(self.attach(&path).await?)
 	}
 	
+	/// Delete a buffer from workspace
 	#[napi(js_name = "delete")]
 	pub async fn js_delete(&self, path: String) -> napi::Result<()> {
 		Ok(self.delete(&path).await?)
 	}
 
+	/// Detach from an active buffer, stopping its underlying worker
+	/// this method returns true if no reference or last reference was held, false if there are still
+	/// dangling references to clear
 	#[napi(js_name = "detach")]
 	pub async fn js_detach(&self, path: String) -> bool {
 		self.detach(&path)
 	}
 
+	/// Wait for next workspace event and return it
 	#[napi(js_name = "event")]
 	pub async fn js_event(&self) -> napi::Result<JsEvent> {
 		Ok(JsEvent::from(self.event().await?))