chore: napi ffi update for bump deps

Author: frelodev

src/ffi/js/buffer.rs

@@ -2,7 +2,7 @@ use crate::api::controller::{AsyncReceiver, AsyncSender};
 use crate::api::{BufferUpdate, TextChange};
 use crate::buffer::controller::BufferController;
 use napi::threadsafe_function::{
-	ErrorStrategy::Fatal, ThreadSafeCallContext, ThreadsafeFunction, ThreadsafeFunctionCallMode,
+	ThreadsafeFunction, ThreadsafeFunctionCallMode,
 };
 use napi_derive::napi;
 
@@ -14,16 +14,10 @@ impl BufferController {
 		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> = fun
-			.create_threadsafe_function(
-				0,
-				|ctx: ThreadSafeCallContext<crate::buffer::controller::BufferController>| {
-					Ok(vec![ctx.value])
-				},
-			)?;
+	pub fn js_callback(&self, fun: ThreadsafeFunction<crate::buffer::controller::BufferController>) -> napi::Result<()> {
+		let tsfn: ThreadsafeFunction<crate::buffer::controller::BufferController> = fun;
 		self.callback(move |controller: BufferController| {
-			tsfn.call(controller.clone(), ThreadsafeFunctionCallMode::Blocking);
+			tsfn.call(Ok(controller.clone()), ThreadsafeFunctionCallMode::Blocking);
 			//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
 		});

src/ffi/js/cursor.rs

@@ -1,9 +1,6 @@
 use crate::api::controller::{AsyncReceiver, AsyncSender};
 use crate::cursor::controller::CursorController;
-use napi::threadsafe_function::ErrorStrategy::Fatal;
-use napi::threadsafe_function::{
-	ThreadSafeCallContext, ThreadsafeFunction, ThreadsafeFunctionCallMode,
-};
+use napi::threadsafe_function::{ ThreadsafeFunction, ThreadsafeFunctionCallMode};
 use napi_derive::napi;
 
 #[napi]
@@ -14,16 +11,10 @@ impl CursorController {
 		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> = fun
-			.create_threadsafe_function(
-				0,
-				|ctx: ThreadSafeCallContext<crate::cursor::controller::CursorController>| {
-					Ok(vec![ctx.value])
-				},
-			)?;
+	pub fn js_callback(&self, fun: ThreadsafeFunction<crate::cursor::controller::CursorController>) -> napi::Result<()> {
+		let tsfn: ThreadsafeFunction<crate::cursor::controller::CursorController> = fun;
 		self.callback(move |controller: CursorController| {
-			tsfn.call(controller.clone(), ThreadsafeFunctionCallMode::Blocking);
+			tsfn.call(Ok(controller.clone()), ThreadsafeFunctionCallMode::Blocking);
 			//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
 		});

src/ffi/js/workspace.rs

@@ -2,10 +2,7 @@ use crate::Workspace;
 use crate::api::controller::AsyncReceiver;
 use crate::buffer::controller::BufferController;
 use crate::cursor::controller::CursorController;
-use napi::threadsafe_function::ErrorStrategy::Fatal;
-use napi::threadsafe_function::{
-	ThreadSafeCallContext, ThreadsafeFunction, ThreadsafeFunctionCallMode,
-};
+use napi::threadsafe_function::{ThreadsafeFunction, ThreadsafeFunctionCallMode};
 use napi_derive::napi;
 
 use super::client::JsUser;
@@ -45,8 +42,8 @@ impl Workspace {
 
 	/// List all available buffers in this workspace
 	#[napi(js_name = "searchBuffers")]
-	pub fn js_search_buffers(&self, filter: Option<&str>) -> Vec<String> {
-		self.search_buffers(filter)
+	pub fn js_search_buffers(&self, filter: Option<String>) -> Vec<String> {
+		self.search_buffers(filter.as_deref())
 	}
 
 	/// List all user names currently in this workspace
@@ -114,13 +111,10 @@ impl Workspace {
 	}
 
 	#[napi(js_name = "callback", ts_args_type = "fun: (event: Workspace) => void")]
-	pub fn js_callback(&self, fun: napi::JsFunction) -> napi::Result<()> {
-		let tsfn: ThreadsafeFunction<crate::Workspace, Fatal> = fun
-			.create_threadsafe_function(0, |ctx: ThreadSafeCallContext<crate::Workspace>| {
-				Ok(vec![ctx.value])
-			})?;
+	pub fn js_callback(&self, fun: ThreadsafeFunction<Workspace>) -> napi::Result<()> {
+		let tsfn: ThreadsafeFunction<crate::Workspace> = fun;
 		self.callback(move |controller: Workspace| {
-			tsfn.call(controller.clone(), ThreadsafeFunctionCallMode::Blocking); //check this with tracing also we could use Ok(event) to get the error
+			tsfn.call(Ok(controller.clone()), ThreadsafeFunctionCallMode::Blocking); //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
 		});