cleanups

Author: penberg

index.d.ts

@@ -12,6 +12,7 @@ export interface RunResult {
   duration: number
   lastInsertRowid: number
 }
+/** Retrieve next row from an iterator synchronously. Needed for better-sqlite3 API compatibility. */
 export declare function iteratorNextSync(iter: RowsIterator): Record
 export declare class SqliteError {
   message: string
@@ -48,6 +49,7 @@ export declare class Statement {
   safeIntegers(toggle?: boolean | undefined | null): this
   interrupt(): void
 }
+/** A raw iterator over rows. The JavaScript layer wraps this in a iterable. */
 export declare class RowsIterator {
   next(): Promise<Record>
 }

src/lib.rs

@@ -7,7 +7,6 @@ mod auth;
 #[macro_use]
 extern crate napi_derive;
 
-use libsql::Row;
 use napi::bindgen_prelude::{Array, FromNapiValue, ToNapiValue};
 use napi::{Env, JsUnknown, Result, ValueType};
 use once_cell::sync::OnceCell;
@@ -642,14 +641,13 @@ impl Statement {
             Ok::<_, napi::Error>((rows, column_names))
         };
         let (rows, column_names) = rt.block_on(future)?;
-        let iter = RowsIterator::new(
+        Ok(RowsIterator::new(
             Arc::new(tokio::sync::Mutex::new(rows)),
             column_names,
             safe_ints,
             raw,
             pluck,
-        )?;
-        Ok(iter)
+        ))
     }
 
     #[napi]