fix: remove buildJsQueryResult and simplify types

Author: chrispader

package/src/operations/execute.ts

@@ -1,5 +1,4 @@
 import { HybridNitroSQLite } from '../nitro'
-import type { NitroSQLiteQueryResult } from '../specs/NitroSQLiteQueryResult.nitro'
 import type { QueryResult, QueryResultRow, SQLiteQueryParams } from '../types'
 import NitroSQLiteError from '../NitroSQLiteError'
 
@@ -10,7 +9,7 @@ export function execute<Row extends QueryResultRow = never>(
 ): QueryResult<Row> {
   try {
     const result = HybridNitroSQLite.execute(dbName, query, params)
-    return buildJsQueryResult<Row>(result)
+    return result as QueryResult<Row>
   } catch (error) {
     throw NitroSQLiteError.fromError(error)
   }
@@ -23,25 +22,8 @@ export async function executeAsync<Row extends QueryResultRow = never>(
 ): Promise<QueryResult<Row>> {
   try {
     const result = await HybridNitroSQLite.executeAsync(dbName, query, params)
-    return buildJsQueryResult<Row>(result)
+    return result as QueryResult<Row>
   } catch (error) {
     throw NitroSQLiteError.fromError(error)
   }
 }
-
-function buildJsQueryResult<Row extends QueryResultRow = never>(
-  result: NitroSQLiteQueryResult,
-): QueryResult<Row> {
-  const data = result.results as Row[]
-
-  return {
-    ...result,
-    insertId: result.insertId,
-    rowsAffected: result.rowsAffected,
-    rows: {
-      _array: data,
-      length: data.length,
-      item: (idx: number) => data[idx],
-    },
-  } as QueryResult<Row>
-}

package/src/types.ts

@@ -1,5 +1,4 @@
 import type {
-  NitroSQLiteQueryColumnMetadata,
   NitroSQLiteQueryResult,
   NitroSQLiteQueryResultRows,
 } from './specs/NitroSQLiteQueryResult.nitro'
@@ -42,17 +41,8 @@ export type QueryResultRow = Record<string, SQLiteValue>
 
 export type QueryResult<Row extends QueryResultRow = QueryResultRow> =
   NitroSQLiteQueryResult & {
-    readonly rowsAffected: number
-    readonly insertId?: number
-
-    /** Query results */
-    readonly results: Row[]
-
     /** Query results in a row format for TypeORM compatibility */
-    readonly rows?: NitroSQLiteQueryResultRows<Row>
-
-    /** Table metadata */
-    readonly metadata?: Record<string, NitroSQLiteQueryColumnMetadata>
+    rows?: NitroSQLiteQueryResultRows<Row>
   }
 
 export type ExecuteQuery = <Row extends QueryResultRow = QueryResultRow>(