use normal try catch

Author: tim-smart

packages/sql-sqlite-bun/src/SqliteClient.ts

@@ -105,13 +105,13 @@ export const make = (
         Effect.withFiberRuntime<Array<any>, SqlError>((fiber) => {
           const useSafeIntegers = Context.get(fiber.currentContext, Client.SafeIntegers)
           const statement = db.query(sql)
-
           // @ts-ignore bun-types missing safeIntegers method, fixed in https://github.com/oven-sh/bun/pull/26627
           statement.safeIntegers(useSafeIntegers)
-          return Effect.try({
-            try: () => (statement.all(...(params as any)) ?? []) as Array<any>,
-            catch: (cause) => new SqlError({ cause, message: "Failed to execute statement" })
-          })
+          try {
+            return Effect.succeed((statement.all(...(params as any)) ?? []) as Array<any>)
+          } catch (cause) {
+            return Effect.fail(new SqlError({ cause, message: "Failed to execute statement" }))
+          }
         })
 
       const runValues = (
@@ -121,13 +121,13 @@ export const make = (
         Effect.withFiberRuntime<Array<any>, SqlError>((fiber) => {
           const useSafeIntegers = Context.get(fiber.currentContext, Client.SafeIntegers)
           const statement = db.query(sql)
-
           // @ts-ignore bun-types missing safeIntegers method, fixed in https://github.com/oven-sh/bun/pull/26627
           statement.safeIntegers(useSafeIntegers)
-          return Effect.try({
-            try: () => (statement.values(...(params as any)) ?? []) as Array<any>,
-            catch: (cause) => new SqlError({ cause, message: "Failed to execute statement" })
-          })
+          try {
+            return Effect.succeed((statement.values(...(params as any)) ?? []) as Array<any>)
+          } catch (cause) {
+            return Effect.fail(new SqlError({ cause, message: "Failed to execute statement" }))
+          }
         })
 
       return identity<SqliteConnection>({