Fix useSql on Windows when the db is locked (#66)

Author: mathieudutour

docs/utils-reference/getting-started.md

@@ -16,6 +16,10 @@ npm install --save @raycast/utils
 
 ## Changelog
 
+### v2.2.3
+
+- Fixed an issue with `useSQL` on Windows where the query would refuse to be executed because the database is locked
+
 ### v2.2.2
 
 - Fix `useCachedState` to preserve Date objects more precisely.

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@raycast/utils",
-  "version": "2.2.2",
+  "version": "2.2.3",
   "description": "Set of utilities to streamline building Raycast extensions",
   "author": "Raycast Technologies Ltd.",
   "homepage": "https://developers.raycast.com/utilities/getting-started",

package.json

@@ -44,9 +44,17 @@ export async function baseExecuteSQL<T = unknown>(
 
   try {
     db.open();
+
+    const statement = db.prepare(query);
+    checkAborted(abortSignal);
+
+    const result = statement.all();
+
+    db.close();
+
+    return result as T[];
   } catch (error: any) {
-    console.log(error);
-    if (error.message.match("(5)") || error.message.match("(14)")) {
+    if (error.errcode === 5 || error.errcode === 14 || error.message.match("(5)") || error.message.match("(14)")) {
       // That means that the DB is busy because of another app is locking it
       // This happens when Chrome or Arc is opened: they lock the History db.
       // As an ugly workaround, we duplicate the file and read that instead
@@ -69,17 +77,19 @@ export async function baseExecuteSQL<T = unknown>(
       db = new sqlite3.DatabaseSync(workaroundCopiedDb, { open: false, readOnly: true });
       db.open();
       checkAborted(abortSignal);
-    }
-  }
 
-  const statement = db.prepare(query);
-  checkAborted(abortSignal);
+      const statement = db.prepare(query);
+      checkAborted(abortSignal);
+
+      const result = statement.all();
 
-  const result = statement.all();
+      db.close();
 
-  db.close();
+      return result as T[];
+    }
 
-  return result as T[];
+    throw error;
+  }
 }
 
 async function sqliteFallback<T = unknown>(