node-sqlite-kv 0.3.0 → 0.3.1

package/dist/index.d.mts

@@ -8,7 +8,7 @@ declare const journalModes: JournalMode[];
 declare class KVSync<T = any> {
     #private;
     constructor(options?: KVSyncOptions);
-    set<K = T>(key: string, value: K | undefined): K | null;
+    set<K = T>(key: string, value: K | undefined): K;
     get<K = T>(key: string): K | null;
     delete(key: string): KVSync;
     all<K = T>(filter?: (key: string, value: K) => boolean): {

package/dist/index.d.ts

@@ -8,7 +8,7 @@ declare const journalModes: JournalMode[];
 declare class KVSync<T = any> {
     #private;
     constructor(options?: KVSyncOptions);
-    set<K = T>(key: string, value: K | undefined): K | null;
+    set<K = T>(key: string, value: K | undefined): K;
     get<K = T>(key: string): K | null;
     delete(key: string): KVSync;
     all<K = T>(filter?: (key: string, value: K) => boolean): {

package/dist/index.js

@@ -63,12 +63,9 @@ var KVSync = class {
     }
     this.#db = new import_node_sqlite.DatabaseSync(dbPath);
     this.setJournalMode(options?.journalMode ?? "DELETE");
-    this.#db.exec(`
-            CREATE TABLE IF NOT EXISTS kv (
-                key TEXT PRIMARY KEY NOT NULL,
-                value BLOB NOT NULL
-            ) STRICT;
-        `);
+    this.#db.exec(
+      "CREATE TABLE IF NOT EXISTS kv (key TEXT PRIMARY KEY NOT NULL, value BLOB NOT NULL) STRICT;"
+    );
   }
   /**
    * Set a key in the database
@@ -83,8 +80,9 @@ var KVSync = class {
       );
     }
     if (value === void 0) {
-      this.delete(key);
-      return null;
+      throw new Error(
+        "[KVSync]: Provided value is undefined. Did you mean to use delete() instead?"
+      );
     }
     this.#db.prepare("INSERT OR REPLACE INTO kv (key, value) VALUES (?, ?)").run(key, (0, import_node_v8.serialize)(value));
     return value;
@@ -169,7 +167,6 @@ var KVSync = class {
         `[KVSync]: Transaction callback must be of type function. Received: ${typeof callback}`
       );
     }
-    this.#db.exec("BEGIN TRANSACTION;");
     const oldMap = /* @__PURE__ */ new Map();
     const newMap = /* @__PURE__ */ new Map();
     const tx = Object.create(this);
@@ -178,9 +175,8 @@ var KVSync = class {
         const oldValue = this.get(key);
         oldMap.set(key, oldValue === null ? void 0 : oldValue);
       }
-      const result = this.set(key, value);
-      newMap.set(key, result);
-      return result;
+      newMap.set(key, value);
+      return value ?? null;
     };
     tx.delete = (key) => {
       if (!oldMap.has(key)) {
@@ -188,11 +184,18 @@ var KVSync = class {
         oldMap.set(key, oldValue === null ? void 0 : oldValue);
       }
       newMap.set(key, null);
-      this.delete(key);
       return tx;
     };
     try {
       callback(tx);
+      this.#db.exec("BEGIN TRANSACTION;");
+      for (const [key, value] of newMap.entries()) {
+        if (value === null) {
+          this.delete(key);
+        } else {
+          this.set(key, value);
+        }
+      }
       this.#db.exec("COMMIT;");
     } catch (error) {
       this.#db.exec("ROLLBACK;");

package/dist/index.mjs

@@ -30,12 +30,9 @@ var KVSync = class {
     }
     this.#db = new DatabaseSync(dbPath);
     this.setJournalMode(options?.journalMode ?? "DELETE");
-    this.#db.exec(`
-            CREATE TABLE IF NOT EXISTS kv (
-                key TEXT PRIMARY KEY NOT NULL,
-                value BLOB NOT NULL
-            ) STRICT;
-        `);
+    this.#db.exec(
+      "CREATE TABLE IF NOT EXISTS kv (key TEXT PRIMARY KEY NOT NULL, value BLOB NOT NULL) STRICT;"
+    );
   }
   /**
    * Set a key in the database
@@ -50,8 +47,9 @@ var KVSync = class {
       );
     }
     if (value === void 0) {
-      this.delete(key);
-      return null;
+      throw new Error(
+        "[KVSync]: Provided value is undefined. Did you mean to use delete() instead?"
+      );
     }
     this.#db.prepare("INSERT OR REPLACE INTO kv (key, value) VALUES (?, ?)").run(key, serialize(value));
     return value;
@@ -136,7 +134,6 @@ var KVSync = class {
         `[KVSync]: Transaction callback must be of type function. Received: ${typeof callback}`
       );
     }
-    this.#db.exec("BEGIN TRANSACTION;");
     const oldMap = /* @__PURE__ */ new Map();
     const newMap = /* @__PURE__ */ new Map();
     const tx = Object.create(this);
@@ -145,9 +142,8 @@ var KVSync = class {
         const oldValue = this.get(key);
         oldMap.set(key, oldValue === null ? void 0 : oldValue);
       }
-      const result = this.set(key, value);
-      newMap.set(key, result);
-      return result;
+      newMap.set(key, value);
+      return value ?? null;
     };
     tx.delete = (key) => {
       if (!oldMap.has(key)) {
@@ -155,11 +151,18 @@ var KVSync = class {
         oldMap.set(key, oldValue === null ? void 0 : oldValue);
       }
       newMap.set(key, null);
-      this.delete(key);
       return tx;
     };
     try {
       callback(tx);
+      this.#db.exec("BEGIN TRANSACTION;");
+      for (const [key, value] of newMap.entries()) {
+        if (value === null) {
+          this.delete(key);
+        } else {
+          this.set(key, value);
+        }
+      }
       this.#db.exec("COMMIT;");
     } catch (error) {
       this.#db.exec("ROLLBACK;");

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "node-sqlite-kv",
   "description": "Key-value store with node:sqlite",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "repository": {
     "url": "https://github.com/e60m5ss/node-sqlite-kv"
   },
@@ -26,7 +26,7 @@
     "useTabs": false
   },
   "devDependencies": {
-    "@types/node": "^25.1.0",
+    "@types/node": "^25.2.0",
     "prettier": "^3.8.1",
     "tsup": "^8.5.1",
     "typescript": "^5.9.3"