mirror of
https://github.com/lancedb/lancedb.git
synced 2026-03-31 13:00:40 +00:00
## Summary - Upgrades `@napi-rs/cli` from v2 to v3, `napi`/`napi-derive` Rust crates to 3.x - Fixes a bug ([napi-rs#1170](https://github.com/napi-rs/napi-rs/issues/1170)) where the CLI failed to locate the built `.node` binary when a custom Cargo target directory is set (via `config.toml`) ## Changes **package.json / CLI**: - `napi.name` → `napi.binaryName`, `napi.triples` → `napi.targets` - Removed `--no-const-enum` flag and fixed output dir arg - `napi universal` → `napi universalize` **Rust API migration**: - `#[napi::module_init]` → `#[napi_derive::module_init]` - `napi::JsObject` → `Object`, `.get::<_, T>()` → `.get::<T>()` - `ErrorStrategy` removed; `ThreadsafeFunction` now takes an explicit `Return` type with `CalleeHandled = false` const generic - `JsFunction` + `create_threadsafe_function` replaced by typed `Function<Args, Return>` + `build_threadsafe_function().build()` - `RerankerCallbacks` struct removed (`Function<'env,...>` can't be stored in structs); `VectorQuery::rerank` now accepts the function directly - `ClassInstance::clone()` now returns `ClassInstance`, fixed with explicit deref - `Vec<u8>` in `#[napi(object)]` now maps to `Array<number>` in v3; changed to `Buffer` to preserve the TypeScript `Buffer` type **TypeScript**: - `inner.rerank({ rerankHybrid: async (_, args) => ... })` → `inner.rerank(async (args) => ...)` - Header provider callback wrapped in `async` to match stricter typed constructor signature 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
74 lines
2.3 KiB
Rust
74 lines
2.3 KiB
Rust
// SPDX-License-Identifier: Apache-2.0
|
|
// SPDX-FileCopyrightText: Copyright The LanceDB Authors
|
|
|
|
use napi::{bindgen_prelude::*, threadsafe_function::ThreadsafeFunction};
|
|
use napi_derive::napi;
|
|
use std::collections::HashMap;
|
|
use std::sync::Arc;
|
|
|
|
type GetHeadersFn = ThreadsafeFunction<(), Promise<HashMap<String, String>>, (), Status, false>;
|
|
|
|
/// JavaScript HeaderProvider implementation that wraps a JavaScript callback.
|
|
/// This is the only native header provider - all header provider implementations
|
|
/// should provide a JavaScript function that returns headers.
|
|
#[napi]
|
|
pub struct JsHeaderProvider {
|
|
get_headers_fn: Arc<GetHeadersFn>,
|
|
}
|
|
|
|
impl Clone for JsHeaderProvider {
|
|
fn clone(&self) -> Self {
|
|
Self {
|
|
get_headers_fn: self.get_headers_fn.clone(),
|
|
}
|
|
}
|
|
}
|
|
|
|
#[napi]
|
|
impl JsHeaderProvider {
|
|
/// Create a new JsHeaderProvider from a JavaScript callback
|
|
#[napi(constructor)]
|
|
pub fn new(
|
|
get_headers_callback: Function<(), Promise<HashMap<String, String>>>,
|
|
) -> Result<Self> {
|
|
let get_headers_fn = get_headers_callback
|
|
.build_threadsafe_function()
|
|
.build()
|
|
.map_err(|e| {
|
|
Error::new(
|
|
Status::GenericFailure,
|
|
format!("Failed to create threadsafe function: {}", e),
|
|
)
|
|
})?;
|
|
|
|
Ok(Self {
|
|
get_headers_fn: Arc::new(get_headers_fn),
|
|
})
|
|
}
|
|
}
|
|
|
|
#[cfg(feature = "remote")]
|
|
#[async_trait::async_trait]
|
|
impl lancedb::remote::HeaderProvider for JsHeaderProvider {
|
|
async fn get_headers(&self) -> lancedb::error::Result<HashMap<String, String>> {
|
|
// Call the JavaScript function asynchronously
|
|
let promise: Promise<HashMap<String, String>> =
|
|
self.get_headers_fn.call_async(()).await.map_err(|e| {
|
|
lancedb::error::Error::Runtime {
|
|
message: format!("Failed to call JavaScript get_headers: {}", e),
|
|
}
|
|
})?;
|
|
|
|
// Await the promise result
|
|
promise.await.map_err(|e| lancedb::error::Error::Runtime {
|
|
message: format!("JavaScript get_headers failed: {}", e),
|
|
})
|
|
}
|
|
}
|
|
|
|
impl std::fmt::Debug for JsHeaderProvider {
|
|
fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
|
|
write!(f, "JsHeaderProvider")
|
|
}
|
|
}
|