@lobomfz/db 0.3.0 → 0.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@lobomfz/db",
-	"version": "0.3.0",
+	"version": "0.3.1",
 	"description": "Bun SQLite database with Arktype schemas and typed Kysely client",
 	"keywords": [
 		"arktype",

package/src/database.ts

@@ -1,5 +1,5 @@
 import { Database as BunDatabase } from "bun:sqlite";
-import { Kysely } from "kysely";
+import { Kysely, ParseJSONResultsPlugin } from "kysely";
 import { BunSqliteDialect } from "./dialect/dialect";
 import type { Type } from "arktype";
 import type { GeneratedPreset } from "./generated";
@@ -82,7 +82,7 @@ export class Database<T extends SchemaRecord> {
 
 		this.kysely = new Kysely<TablesFromSchemas<T>>({
 			dialect: new BunSqliteDialect({ database: this.sqlite }),
-			plugins: [new DeserializePlugin(this.columns, validation)],
+			plugins: [new DeserializePlugin(this.columns, validation), new ParseJSONResultsPlugin()],
 		});
 	}
 
@@ -335,6 +335,7 @@ export class Database<T extends SchemaRecord> {
 
 			const indexes = (schemaIndexes?.[name] ?? []).map((indexDef) => ({
 				name: this.generateIndexName(name, indexDef.columns, indexDef.unique ?? false),
+				columns: indexDef.columns,
 				sql: this.generateCreateIndexSQL(name, indexDef),
 			}));
 

package/src/migration/diff.ts

@@ -7,6 +7,7 @@ export interface DesiredColumn extends ColumnSchema {
 
 export type DesiredIndex = {
 	name: string;
+	columns: string[];
 	sql: string;
 };
 
@@ -162,7 +163,7 @@ export class Differ {
 
 			if (this.rebuiltTables.has(table.name)) {
 				for (const idx of tableIndexes) {
-					this.ops.push({ type: "CreateIndex", sql: idx.sql });
+					this.ops.push({ type: "CreateIndex", table: table.name, columns: idx.columns, sql: idx.sql });
 				}
 
 				continue;
@@ -179,7 +180,7 @@ export class Differ {
 
 			for (const idx of tableIndexes) {
 				if (!existingNames.has(idx.name)) {
-					this.ops.push({ type: "CreateIndex", sql: idx.sql });
+					this.ops.push({ type: "CreateIndex", table: table.name, columns: idx.columns, sql: idx.sql });
 				}
 			}
 

package/src/migration/execute.ts

@@ -30,7 +30,11 @@ export class Executor {
 		try {
 			this.db.transaction(() => {
 				for (const op of this.ops) {
-					this.executeOp(op);
+					try {
+						this.executeOp(op);
+					} catch (e) {
+						throw this.parseError(e, op);
+					}
 				}
 
 				if (restoreFk) {
@@ -71,6 +75,22 @@ export class Executor {
 		}
 	}
 
+	private parseError(e: unknown, op: MigrationOp) {
+		if (
+			op.type === "CreateIndex" &&
+			e instanceof Error &&
+			e.message.includes("UNIQUE constraint failed")
+		) {
+			const cols = op.columns.map((c) => `"${c}"`).join(", ");
+
+			return new Error(
+				`Cannot create unique index on table "${op.table}" (${cols}): duplicate values exist`,
+			);
+		}
+
+		return e;
+	}
+
 	private rebuildTable(op: RebuildTableOp) {
 		const tempName = `__new_${op.table}`;
 		const tempSql = op.createSql.replace(

package/src/migration/types.ts

@@ -29,6 +29,8 @@ export type RebuildTableOp = {
 
 export type CreateIndexOp = {
 	type: "CreateIndex";
+	table: string;
+	columns: string[];
 	sql: string;
 };