borogove

created pr with borogove/sqlite-injectable-driver.1 on 2026-11-09T19:46:34Z · by 3c31bc8c

commented on 2026-11-09T19:46:34Z · by 3c31bc8c

Changes are available at https://tangled.org/mrry.band/borogove on branch sqlite-injectable-driver.

added borogove/sqlite-injectable-driver.2 on 2026-11-09T20:05:07Z · by 3c31bc8c


                      1:  981fe7d = 1:  981fe7d Use arrays


                      2:  ef9e639 = 2:  ef9e639 Make prepare public, remove formatParam and first


                      3:  f949361 ! 3:  a97c836 Allow injectable drivers for Sqlite

commented on 2026-11-09T20:49:02Z · by 9e834e0d

browser tests dont pass

added borogove/sqlite-injectable-driver.3 on 2026-11-09T21:28:45Z · by 3c31bc8c


                      1:  981fe7d = 1:  981fe7d Use arrays


                      2:  ef9e639 ! 2:  11eb826 Make prepare public, remove formatParam and first


                      3:  a97c836 ! 3:  34cee7d Allow injectable drivers for Sqlite

cmds

checkout latest patchset:

ssh pr.pico.sh pull borogove:sqlite-injectable-driver | git am -3

checkout specific patchset revision:

ssh pr.pico.sh pull borogove:sqlite-injectable-driver [rev] | git am -3

add changes to patch request:

git format-patch main --stdout | ssh pr.pico.sh borogove:sqlite-injectable-driver

add comment to patch request:

ssh pr.pico.sh comment borogove:sqlite-injectable-driver "lgtm!"

Patchset `borogove/sqlite-injectable-driver.1` on 2026-11-09T19:46:34Z · commit f949361

Use arrays

Eric Roberts 2026-11-09T19:27:25Z

Make prepare public, remove formatParam and first

Eric Roberts 2026-11-09T19:29:48Z

→ Allow injectable drivers for Sqlite

Eric Roberts 2026-11-09T19:41:16Z

Allow injectable drivers for Sqlite

Eric Roberts 2026-11-09T19:41:16Z

In Tauri I want to use https://v2.tauri.app/plugin/sql/ so we can put
the db on the filesystem. To do that I need to be able to pass in a
custom driver in borogove. This does that.

Semantic diff summary

0 added, 0 modified, 0 signature changed, 0 removed across 0 analyzed files (2 files skipped: unsupported file type)

+18 -4 borogove/persistence/Sqlite.hx #

......@@ -34,13 +34,21 @@ typedef InsertColumn = {
	value:(ChatMessage)->Dynamic
}

@:expose
typedef SqliteConnection = {
	function exec(sql: String, ?params: Array<Dynamic>): Promise<Array<Dynamic>>;
	function execMany(qs: Array<{ sql: String, ?params: Array<Dynamic> }>): Promise<Array<Dynamic>>;
}

typedef SqliteDriverFactory = (String, (Array<String>->Promise<Array<Dynamic>>)->Promise<Any>)->SqliteConnection;

@:expose
#if cpp
@:build(HaxeCBridge.expose())
@:build(HaxeSwiftBridge.expose())
#end
class Sqlite implements Persistence implements KeyValueStore {
	final db: SqliteDriver;
	final db: SqliteConnection;
	final media: MediaStore;
	final storeMessagesSerialized = new AsyncLock();

......@@ -83,12 +91,13 @@ class Sqlite implements Persistence implements KeyValueStore {

		@param dbfile path to sqlite database
		@param media a MediaStore to use for media
		@param driver optional JavaScript driver factory; defaults to the built-in SQLite driver
		@returns new persistence layer
	**/
	public function new(dbfile: String, media: MediaStore) {
	public function new(dbfile: String, media: MediaStore #if js , ?driver: SqliteDriverFactory #end) {
		this.media = media;
		media.setKV(this);
		db = new SqliteDriver(dbfile, (exec) -> {
		final migrate = (exec: Array<String>->Promise<Array<Dynamic>>) -> {
			exec(["PRAGMA user_version"]).then(rows -> {
				final version = Std.parseInt(rows[0]?.user_version) ?? 0;
				return Promise.resolve(null).then(_ -> {

......@@ -387,7 +396,12 @@ class Sqlite implements Persistence implements KeyValueStore {
					return Promise.resolve(null);
				});
			});
		});
		};
		#if js
		db = driver == null ? new SqliteDriver(dbfile, migrate) : driver(dbfile, migrate);
		#else
		db = new SqliteDriver(dbfile, migrate);
		#end
	}

	@HaxeCBridge.noemit

+29 -0 test/TestSqlite.hx #

......@@ -104,6 +104,35 @@ class TestSqlite extends utest.Test {
		mediaStore = new MockMediaStore();
		persistence = new Sqlite("file:" + ID.unique() + "?mode=memory&cache=shared", mediaStore);
	}
	#if js
	public function testInjectedDriver(async: Async) {
		var migrated = Promise.resolve(null);
		var migrationQueries: Array<String> = [];
		final injected = new Sqlite("injected.sqlite3", mediaStore, (file, migrate) -> {
			Assert.equals("injected.sqlite3", file);
			migrated = migrate(sql -> {
				migrationQueries = migrationQueries.concat(sql);
				return Promise.resolve([{ user_version: 24 }]);
			});
			return {
				exec: (sql: String, ?params: Array<Dynamic>) -> {
					Assert.equals("SELECT v FROM keyvaluepairs WHERE k=? LIMIT 1", sql);
					Assert.equals("key", params[0]);
					return Promise.resolve([{ v: "injected value" }]);
				},
				execMany: qs -> Promise.resolve([])
			};
		});
		migrated.then(_ -> injected.get("key")).then(value -> {
			Assert.same(["PRAGMA user_version"], migrationQueries);
			Assert.equals("injected value", value);
			async.done();
		}).catchError(e -> {
			Assert.fail(Std.string(e));
			async.done();
		});
	}
	#end

	public function testOrder(async: Async) {
		final account = "alice@example.com";

...	...		@@ -387,7 +396,12 @@ class Sqlite implements Persistence implements KeyValueStore {
387	396		return Promise.resolve(null);
388	397		});
389	398		});
390		-	});
	399	+	};
	400	+	#if js
	401	+	db = driver == null ? new SqliteDriver(dbfile, migrate) : driver(dbfile, migrate);
	402	+	#else
	403	+	db = new SqliteDriver(dbfile, migrate);
	404	+	#end
391	405		}
392	406
393	407		@HaxeCBridge.noemit

...	...		@@ -104,6 +104,35 @@ class TestSqlite extends utest.Test {
104	104		mediaStore = new MockMediaStore();
105	105		persistence = new Sqlite("file:" + ID.unique() + "?mode=memory&cache=shared", mediaStore);
106	106		}
	107	+	#if js
	108	+	public function testInjectedDriver(async: Async) {
	109	+	var migrated = Promise.resolve(null);
	110	+	var migrationQueries: Array<String> = [];
	111	+	final injected = new Sqlite("injected.sqlite3", mediaStore, (file, migrate) -> {
	112	+	Assert.equals("injected.sqlite3", file);
	113	+	migrated = migrate(sql -> {
	114	+	migrationQueries = migrationQueries.concat(sql);
	115	+	return Promise.resolve([{ user_version: 24 }]);
	116	+	});
	117	+	return {
	118	+	exec: (sql: String, ?params: Array<Dynamic>) -> {
	119	+	Assert.equals("SELECT v FROM keyvaluepairs WHERE k=? LIMIT 1", sql);
	120	+	Assert.equals("key", params[0]);
	121	+	return Promise.resolve([{ v: "injected value" }]);
	122	+	},
	123	+	execMany: qs -> Promise.resolve([])
	124	+	};
	125	+	});
	126	+	migrated.then(_ -> injected.get("key")).then(value -> {
	127	+	Assert.same(["PRAGMA user_version"], migrationQueries);
	128	+	Assert.equals("injected value", value);
	129	+	async.done();
	130	+	}).catchError(e -> {
	131	+	Assert.fail(Std.string(e));
	132	+	async.done();
	133	+	});
	134	+	}
	135	+	#end
107	136
108	137		public function testOrder(async: Async) {
109	138		final account = "alice@example.com";

...	...		@@ -34,13 +34,21 @@ typedef InsertColumn = {
34	34		value:(ChatMessage)->Dynamic
35	35		}
36	36
	37	+	@:expose
	38	+	typedef SqliteConnection = {
	39	+	function exec(sql: String, ?params: Array<Dynamic>): Promise<Array<Dynamic>>;
	40	+	function execMany(qs: Array<{ sql: String, ?params: Array<Dynamic> }>): Promise<Array<Dynamic>>;
	41	+	}
	42	+
	43	+	typedef SqliteDriverFactory = (String, (Array<String>->Promise<Array<Dynamic>>)->Promise<Any>)->SqliteConnection;
	44	+
37	45		@:expose
38	46		#if cpp
39	47		@:build(HaxeCBridge.expose())
40	48		@:build(HaxeSwiftBridge.expose())
41	49		#end
42	50		class Sqlite implements Persistence implements KeyValueStore {
43		-	final db: SqliteDriver;
	51	+	final db: SqliteConnection;
44	52		final media: MediaStore;
45	53		final storeMessagesSerialized = new AsyncLock();
46	54

...	...		@@ -83,12 +91,13 @@ class Sqlite implements Persistence implements KeyValueStore {
83	91
84	92		@param dbfile path to sqlite database
85	93		@param media a MediaStore to use for media
	94	+	@param driver optional JavaScript driver factory; defaults to the built-in SQLite driver
86	95		@returns new persistence layer
87	96		**/
88		-	public function new(dbfile: String, media: MediaStore) {
	97	+	public function new(dbfile: String, media: MediaStore #if js , ?driver: SqliteDriverFactory #end) {
89	98		this.media = media;
90	99		media.setKV(this);
91		-	db = new SqliteDriver(dbfile, (exec) -> {
	100	+	final migrate = (exec: Array<String>->Promise<Array<Dynamic>>) -> {
92	101		exec(["PRAGMA user_version"]).then(rows -> {
93	102		final version = Std.parseInt(rows[0]?.user_version) ?? 0;
94	103		return Promise.resolve(null).then(_ -> {

borogove

Patchset borogove/sqlite-injectable-driver.1 on 2026-11-09T19:46:34Z · commit f949361

Patchset `borogove/sqlite-injectable-driver.1` on 2026-11-09T19:46:34Z · commit f949361