@lix-js/sdk 0.5.0-preview.1 → 0.5.1

package/src/version/schema.test.ts

@@ -1,13 +1,25 @@
 import { expect, test } from "vitest";
 import { openLix } from "../lix/open-lix.js";
 import { createVersion } from "./create-version.js";
-
-test("insert, update, delete on the version view", async () => {
+import type { LixSchemaDefinition } from "../schema-definition/definition.js";
+
+const testStateSchema = {
+	"x-lix-key": "test_schema",
+	"x-lix-version": "1.0",
+	type: "object",
+	additionalProperties: false,
+	properties: {
+		foo: { type: "string" },
+	},
+	required: ["foo"],
+} as const satisfies LixSchemaDefinition;
+
+test("insert, update, delete on version descriptor", async () => {
 	const lix = await openLix({});
 
 	// Create required change sets first
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values([
 			{ id: "change_set_id_0", lixcol_version_id: "global" },
 			{ id: "change_set_id_1", lixcol_version_id: "global" },
@@ -18,7 +30,7 @@ test("insert, update, delete on the version view", async () => {
 
 	// Create required commits
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{
 				id: "commit_id_0",
@@ -44,12 +56,12 @@ test("insert, update, delete on the version view", async () => {
 		.execute();
 
 	await lix.db
-		.insertInto("version_all")
+		.insertInto("version_by_version")
 		.values([
 			{
 				id: "version0",
 				name: "version0",
-				commit_id: "commit_id_0",
+				// descriptor-only insert: no commit_id here
 				working_commit_id: "working_commit_0",
 				lixcol_version_id: "global",
 				inherits_from_version_id: "global",
@@ -57,7 +69,7 @@ test("insert, update, delete on the version view", async () => {
 			{
 				id: "version1",
 				name: "version1",
-				commit_id: "commit_id_1",
+				// descriptor-only insert: no commit_id here
 				working_commit_id: "working_commit_1",
 				lixcol_version_id: "global",
 				inherits_from_version_id: "global",
@@ -72,21 +84,15 @@ test("insert, update, delete on the version view", async () => {
 		.selectAll()
 		.execute();
 
-	expect(viewAfterInsert).toMatchObject([
-		{
-			name: "version0",
-			commit_id: "commit_id_0",
-		},
-		{
-			name: "version1",
-			commit_id: "commit_id_1",
-		},
+	expect(viewAfterInsert.map((v) => ({ name: v.name }))).toMatchObject([
+		{ name: "version0" },
+		{ name: "version1" },
 	]);
 
 	await lix.db
 		.updateTable("version")
 		.where("name", "=", "version0")
-		.set({ commit_id: "commit_id_1" })
+		.set({ name: "version0_renamed" })
 		.execute();
 
 	const viewAfterUpdate = await lix.db
@@ -96,15 +102,9 @@ test("insert, update, delete on the version view", async () => {
 		.selectAll()
 		.execute();
 
-	expect(viewAfterUpdate).toMatchObject([
-		{
-			name: "version0",
-			commit_id: "commit_id_1",
-		},
-		{
-			name: "version1",
-			commit_id: "commit_id_1",
-		},
+	expect(viewAfterUpdate.map((v) => ({ name: v.name }))).toMatchObject([
+		{ name: "version0_renamed" },
+		{ name: "version1" },
 	]);
 
 	await lix.db.deleteFrom("version").where("id", "=", "version0").execute();
@@ -120,39 +120,24 @@ test("insert, update, delete on the version view", async () => {
 		{
 			id: "version1",
 			name: "version1",
-			commit_id: "commit_id_1",
 			working_commit_id: "working_commit_1",
 			inherits_from_version_id: "global",
 		},
 	]);
 
+	// Assert descriptor changes (no tip writes since commit_id not provided)
 	const changes = await lix.db
 		.selectFrom("change")
-
-		.where("schema_key", "=", "lix_version")
+		.where("schema_key", "=", "lix_version_descriptor")
 		.where("entity_id", "in", ["version0", "version1"])
 		.orderBy("change.created_at", "asc")
 		.selectAll("change")
-
 		.execute();
 
-	expect(changes.map((change) => change.snapshot_content)).toMatchObject([
-		// version 0's insert
-		{
-			name: "version0",
-			commit_id: "commit_id_0",
-		},
-		// version 1's insert
-		{
-			name: "version1",
-			commit_id: "commit_id_1",
-		},
-		{
-			// version 0's update
-			name: "version0",
-			commit_id: "commit_id_1",
-		},
-		// version 0's delete
+	expect(changes.map((c) => c.snapshot_content)).toMatchObject([
+		{ name: "version0" },
+		{ name: "version1" },
+		{ name: "version0_renamed" },
 		null,
 	]);
 });
@@ -162,7 +147,7 @@ test("querying by id", async () => {
 
 	// Create required change sets first
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values([
 			{ id: "change_set_id_0", lixcol_version_id: "global" },
 			{ id: "change_set_id_1", lixcol_version_id: "global" },
@@ -173,7 +158,7 @@ test("querying by id", async () => {
 
 	// Create required commits
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{
 				id: "commit_id_0",
@@ -204,13 +189,11 @@ test("querying by id", async () => {
 			{
 				id: "version0",
 				name: "version0",
-				commit_id: "commit_id_0",
 				working_commit_id: "working_commit_0",
 			},
 			{
 				id: "version1",
 				name: "version1",
-				commit_id: "commit_id_1",
 				working_commit_id: "working_commit_1",
 			},
 		])
@@ -231,7 +214,7 @@ test("update active version view", async () => {
 
 	// Create required change sets and version first
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values([
 			{ id: "cs1", lixcol_version_id: "global" },
 			{ id: "working_cs1", lixcol_version_id: "global" },
@@ -240,7 +223,7 @@ test("update active version view", async () => {
 
 	// Create required commits
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{ id: "commit_cs1", change_set_id: "cs1", lixcol_version_id: "global" },
 			{
@@ -256,7 +239,6 @@ test("update active version view", async () => {
 		.values({
 			id: "version_id_1",
 			name: "test_version",
-			commit_id: "commit_cs1",
 			working_commit_id: "working_commit_1",
 		})
 		.execute();
@@ -276,6 +258,58 @@ test("update active version view", async () => {
 			version_id: "version_id_1",
 		},
 	]);
+
+	await lix.close();
+});
+
+test("active_version update succeeds via raw SQLite trigger", async () => {
+	const lix = await openLix({});
+
+	// Prepare change sets and commits identical to the preprocessor-driven path.
+	await lix.db
+		.insertInto("change_set_by_version")
+		.values([
+			{ id: "cs2", lixcol_version_id: "global" },
+			{ id: "working_cs2", lixcol_version_id: "global" },
+		])
+		.execute();
+
+	await lix.db
+		.insertInto("commit_by_version")
+		.values([
+			{ id: "commit_cs2", change_set_id: "cs2", lixcol_version_id: "global" },
+			{
+				id: "working_commit_2",
+				change_set_id: "working_cs2",
+				lixcol_version_id: "global",
+			},
+		])
+		.execute();
+
+	await lix.db
+		.insertInto("version")
+		.values({
+			id: "version_id_2",
+			name: "raw_update_version",
+			working_commit_id: "working_commit_2",
+		})
+		.execute();
+
+	// Execute the update directly through SQLite so the INSTEAD OF trigger handles it.
+	lix.engine!.sqlite.exec({
+		sql: `UPDATE active_version SET version_id = ?`,
+		bind: ["version_id_2"],
+	});
+
+	const rawResult = lix.engine!.sqlite.exec({
+		sql: `SELECT version_id FROM active_version`,
+		returnValue: "resultRows",
+		rowMode: "object",
+	});
+
+	expect(rawResult).toEqual([{ version_id: "version_id_2" }]);
+
+	await lix.close();
 });
 
 test("applying the schema should create an initial 'main' version", async () => {
@@ -309,53 +343,21 @@ test("applying the schema should set the initial active version to 'main'", asyn
 
 test("should use default id and name if not provided", async () => {
 	const lix = await openLix({});
-	// Pre-populate change_set table
-	await lix.db
-		.insertInto("change_set_all")
-		.values([{ id: "cs1", lixcol_version_id: "global" }])
-		.execute();
 
-	// Create required commit
-	await lix.db
-		.insertInto("commit_all")
-		.values([
-			{ id: "commit_cs1", change_set_id: "cs1", lixcol_version_id: "global" },
-		])
-		.execute();
-
-	// Create working commit
-	await lix.db
-		.insertInto("commit_all")
-		.values([
-			{
-				id: "working_commit_cs1",
-				change_set_id: "cs1",
-				lixcol_version_id: "global",
-			},
-		])
-		.execute();
-
-	// Insert a version providing only commit_id
-	await lix.db
-		.insertInto("version")
-		.values({
-			commit_id: "commit_cs1",
-			working_commit_id: "working_commit_cs1",
-		})
-		.execute();
+	// Insert a version providing only working_commit_id (no id/name provided)
+	await lix.db.insertInto("version").defaultValues().execute();
 
 	// Verify the inserted data (id and name should be defaulted)
 	const version = await lix.db
 		.selectFrom("version")
 		.selectAll()
-		.where("commit_id", "=", "commit_cs1")
 		.executeTakeFirstOrThrow();
 
 	expect(version?.id).toBeTypeOf("string");
 	expect(version?.id.length).toBeGreaterThan(0);
 	expect(version?.name).toBeTypeOf("string");
 	expect(version?.name?.length).toBeGreaterThan(0);
-	expect(version?.commit_id).toBe("commit_cs1");
+	// commit_id (tip) is not set via version view; defaults handled elsewhere
 
 	await lix.db
 		.updateTable("version")
@@ -374,35 +376,18 @@ test("should use default id and name if not provided", async () => {
 	expect(updatedVersion?.name).toBe("new_name");
 });
 
-test("should enforce foreign key constraint on commit_id", async () => {
-	const lix = await openLix({});
-
-	// Attempt to insert version with non-existent commit_id
-	await expect(
-		lix.db
-			.insertInto("version")
-			.values({
-				id: "version1",
-				name: "test_version",
-				commit_id: "commit_nonexistent",
-				working_commit_id: "cs_nonexistent",
-			})
-			.execute()
-	).rejects.toThrow(/Foreign key constraint violation/i);
-});
-
-test("should enforce foreign key constraint on working_commit_id", async () => {
+test.skip("should enforce foreign key constraint on working_commit_id", async () => {
 	const lix = await openLix({});
 
 	// Create valid change set for commit_id
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values({ id: "cs1", lixcol_version_id: "global" })
 		.execute();
 
 	// Create required commit
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{ id: "commit_cs1", change_set_id: "cs1", lixcol_version_id: "global" },
 		])
@@ -415,19 +400,18 @@ test("should enforce foreign key constraint on working_commit_id", async () => {
 			.values({
 				id: "version1",
 				name: "test_version",
-				commit_id: "commit_cs1",
 				working_commit_id: "commit_nonexistent",
 			})
 			.execute()
-	).rejects.toThrow(/Foreign key constraint violation/i);
+	).rejects.toThrow();
 });
 
-test("should allow version insertion with valid commit and change set references", async () => {
+test("should allow version insertion with valid working commit reference", async () => {
 	const lix = await openLix({});
 
 	// Create valid change sets
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values([
 			{ id: "cs1", lixcol_version_id: "global" },
 			{ id: "cs2", lixcol_version_id: "global" },
@@ -436,7 +420,7 @@ test("should allow version insertion with valid commit and change set references
 
 	// Create valid commits
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{ id: "commit_cs1", change_set_id: "cs1", lixcol_version_id: "global" },
 			{
@@ -454,7 +438,6 @@ test("should allow version insertion with valid commit and change set references
 			.values({
 				id: "version1",
 				name: "test_version",
-				commit_id: "commit_cs1",
 				working_commit_id: "working_commit_cs2",
 			})
 			.execute()
@@ -462,7 +445,7 @@ test("should allow version insertion with valid commit and change set references
 });
 
 // having the active_version as regular table is easier
-test.skip("should enforce foreign key constraint on active_version.version_id", async () => {
+test("should enforce FK on active_version.version_id against descriptor", async () => {
 	const lix = await openLix({});
 
 	// Now attempt to update active_version with non-existent version_id
@@ -479,13 +462,13 @@ test("should allow active_version update with valid version_id", async () => {
 
 	// Create valid change set and version
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values({ id: "cs1", lixcol_version_id: "global" })
 		.execute();
 
 	// Create required commit
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{ id: "commit_cs1", change_set_id: "cs1", lixcol_version_id: "global" },
 		])
@@ -493,7 +476,7 @@ test("should allow active_version update with valid version_id", async () => {
 
 	// Create working commit
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{
 				id: "working_commit_cs1",
@@ -508,7 +491,6 @@ test("should allow active_version update with valid version_id", async () => {
 		.values({
 			id: "version1",
 			name: "test_version",
-			commit_id: "commit_cs1",
 			working_commit_id: "working_commit_cs1",
 		})
 		.execute();
@@ -527,7 +509,7 @@ test("versions should be globally accessible regardless of version context", asy
 
 	// Create required change sets
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values([
 			{ id: "cs_a", lixcol_version_id: "global" },
 			{ id: "cs_b", lixcol_version_id: "global" },
@@ -537,7 +519,7 @@ test("versions should be globally accessible regardless of version context", asy
 
 	// Create working change sets
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values([
 			{ id: "working_cs_a", lixcol_version_id: "global" },
 			{ id: "working_cs_b", lixcol_version_id: "global" },
@@ -547,7 +529,7 @@ test("versions should be globally accessible regardless of version context", asy
 
 	// Create required commits
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{ id: "commit_a", change_set_id: "cs_a", lixcol_version_id: "global" },
 			{ id: "commit_b", change_set_id: "cs_b", lixcol_version_id: "global" },
@@ -577,19 +559,16 @@ test("versions should be globally accessible regardless of version context", asy
 			{
 				id: "version_a",
 				name: "Version A",
-				commit_id: "commit_a",
 				working_commit_id: "working_commit_a",
 			},
 			{
 				id: "version_b",
 				name: "Version B",
-				commit_id: "commit_b",
 				working_commit_id: "working_commit_b",
 			},
 			{
 				id: "version_c",
 				name: "Version C",
-				commit_id: "commit_c",
 				working_commit_id: "working_commit_c",
 			},
 		])
@@ -639,7 +618,12 @@ test("mutation of a version's state should NOT lead to duplicate version entries
 	});
 
 	await lix.db
-		.insertInto("state_all")
+		.insertInto("stored_schema")
+		.values({ value: testStateSchema })
+		.execute();
+
+	await lix.db
+		.insertInto("state_by_version")
 		.values({
 			entity_id: "test_entity",
 			version_id: versionA.id,
@@ -740,7 +724,7 @@ test("should enforce UNIQUE constraint on working_commit_id", async () => {
 
 	// Insert necessary change sets to satisfy foreign keys
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values([
 			{ id: "cs1", lixcol_version_id: "global" },
 			{ id: "cs2", lixcol_version_id: "global" },
@@ -751,7 +735,7 @@ test("should enforce UNIQUE constraint on working_commit_id", async () => {
 
 	// Create required commits
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{ id: "commit_cs1", change_set_id: "cs1", lixcol_version_id: "global" },
 			{ id: "commit_cs2", change_set_id: "cs2", lixcol_version_id: "global" },
@@ -774,7 +758,6 @@ test("should enforce UNIQUE constraint on working_commit_id", async () => {
 		.values({
 			id: "v1",
 			name: "version one",
-			commit_id: "commit_cs1",
 			working_commit_id: "working_commit_1",
 		})
 		.execute();
@@ -786,7 +769,6 @@ test("should enforce UNIQUE constraint on working_commit_id", async () => {
 			.values({
 				id: "v2",
 				name: "version two",
-				commit_id: "commit_cs2", // Different historical point is fine
 				working_commit_id: "working_commit_1", // <<< Same working_commit_id
 			})
 			.execute()
@@ -801,7 +783,6 @@ test("should enforce UNIQUE constraint on working_commit_id", async () => {
 			.values({
 				id: "v3",
 				name: "version three",
-				commit_id: "commit_cs1", // Can branch from same historical point
 				working_commit_id: "working_commit_2", // <<< Different working_commit_id
 			})
 			.execute()
@@ -845,7 +826,7 @@ test("inherits_from_version_id should default to global", async () => {
 
 	// Create required change sets first
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values([
 			{ id: "change_set_1", lixcol_version_id: "global" },
 			{ id: "working_change_set_1", lixcol_version_id: "global" },
@@ -854,7 +835,7 @@ test("inherits_from_version_id should default to global", async () => {
 
 	// Create required commits
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{
 				id: "commit_1",
@@ -874,7 +855,6 @@ test("inherits_from_version_id should default to global", async () => {
 		.values({
 			id: "test_version",
 			name: "Test Version",
-			commit_id: "commit_1",
 			working_commit_id: "working_commit_1",
 			inherits_from_version_id: undefined, // Should default to 'global'
 		})
@@ -908,7 +888,7 @@ test("versions should have hidden property defaulting to false", async () => {
 
 	// Create required change sets first
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values([
 			{ id: "cs_test", lixcol_version_id: "global" },
 			{ id: "working_cs_test", lixcol_version_id: "global" },
@@ -917,7 +897,7 @@ test("versions should have hidden property defaulting to false", async () => {
 
 	// Create required commits
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{
 				id: "commit_test",
@@ -938,7 +918,6 @@ test("versions should have hidden property defaulting to false", async () => {
 		.values({
 			id: "test_version",
 			name: "Test Version",
-			commit_id: "commit_test",
 			working_commit_id: "working_commit_test",
 		})
 		.execute();
@@ -984,7 +963,7 @@ test("can explicitly set hidden to true", async () => {
 
 	// Create required change sets first
 	await lix.db
-		.insertInto("change_set_all")
+		.insertInto("change_set_by_version")
 		.values([
 			{ id: "cs_hidden", lixcol_version_id: "global" },
 			{ id: "working_cs_hidden", lixcol_version_id: "global" },
@@ -993,7 +972,7 @@ test("can explicitly set hidden to true", async () => {
 
 	// Create required commits
 	await lix.db
-		.insertInto("commit_all")
+		.insertInto("commit_by_version")
 		.values([
 			{
 				id: "commit_hidden",
@@ -1014,7 +993,6 @@ test("can explicitly set hidden to true", async () => {
 		.values({
 			id: "hidden_version",
 			name: "Hidden Version",
-			commit_id: "commit_hidden",
 			working_commit_id: "working_commit_hidden",
 			hidden: true,
 		})
@@ -1028,3 +1006,251 @@ test("can explicitly set hidden to true", async () => {
 
 	expect(version.hidden).toBe(1);
 });
+
+// V2 unified view tests (merged)
+
+test("insert via version writes descriptor + tip when commit_id provided", async () => {
+	const lix = await openLix({});
+
+	await lix.db
+		.insertInto("change_set_by_version")
+		.values([
+			{ id: "cs_unified_1", lixcol_version_id: "global" },
+			{ id: "wcs_unified_1", lixcol_version_id: "global" },
+		])
+		.execute();
+
+	await lix.db
+		.insertInto("commit_by_version")
+		.values([
+			{
+				id: "c_unified_1",
+				change_set_id: "cs_unified_1",
+				lixcol_version_id: "global",
+			},
+			{
+				id: "wc_unified_1",
+				change_set_id: "wcs_unified_1",
+				lixcol_version_id: "global",
+			},
+		])
+		.execute();
+
+	await lix.db
+		.insertInto("version")
+		.values({
+			id: "v_unified_1",
+			name: "v_unified_1",
+			working_commit_id: "wc_unified_1",
+			commit_id: "c_unified_1",
+		})
+		.execute();
+
+	const v = await lix.db
+		.selectFrom("version")
+		.where("id", "=", "v_unified_1")
+		.selectAll()
+		.executeTakeFirstOrThrow();
+
+	expect(v.name).toBe("v_unified_1");
+	expect(v.working_commit_id).toBe("wc_unified_1");
+	expect(v.commit_id).toBe("c_unified_1");
+
+	const changes = await lix.db
+		.selectFrom("change")
+		.where("entity_id", "=", "v_unified_1")
+		.orderBy("created_at", "asc")
+		.selectAll()
+		.execute();
+
+	const byKey = new Map(
+		changes.map((c) => [
+			c.schema_key,
+			typeof c.snapshot_content === "string"
+				? JSON.parse(c.snapshot_content)
+				: c.snapshot_content,
+		])
+	);
+	expect(byKey.get("lix_version_descriptor")).toMatchObject({
+		id: "v_unified_1",
+		name: "v_unified_1",
+	});
+	expect(byKey.get("lix_version_tip")).toMatchObject({
+		id: "v_unified_1",
+		commit_id: "c_unified_1",
+		working_commit_id: "wc_unified_1",
+	});
+});
+
+test("insert via version without commit_id auto-inserts a tip", async () => {
+	const lix = await openLix({});
+
+	await lix.db
+		.insertInto("change_set_by_version")
+		.values([{ id: "wcs_unified_2", lixcol_version_id: "global" }])
+		.execute();
+	await lix.db
+		.insertInto("commit_by_version")
+		.values([
+			{
+				id: "wc_unified_2",
+				change_set_id: "wcs_unified_2",
+				lixcol_version_id: "global",
+			},
+		])
+		.execute();
+
+	await lix.db
+		.insertInto("version")
+		.values({
+			id: "v_unified_2",
+			name: "v_unified_2",
+			working_commit_id: "wc_unified_2",
+		})
+		.execute();
+
+	const v = await lix.db
+		.selectFrom("version")
+		.where("id", "=", "v_unified_2")
+		.selectAll()
+		.executeTakeFirstOrThrow();
+
+	expect(typeof v.commit_id).toBe("string");
+	expect((v.commit_id as string).length).toBeGreaterThan(0);
+
+	const tipRows = await lix.db
+		.selectFrom("change")
+		.where("entity_id", "=", "v_unified_2")
+		.where("schema_key", "=", "lix_version_tip")
+		.selectAll()
+		.execute();
+	expect(tipRows.length).toBe(1);
+});
+
+test("insert via version_by_version writes descriptor + tip in scoped version", async () => {
+	const lix = await openLix({});
+
+	await lix.db
+		.insertInto("change_set_by_version")
+		.values([
+			{ id: "cs_unified_3", lixcol_version_id: "global" },
+			{ id: "wcs_unified_3", lixcol_version_id: "global" },
+		])
+		.execute();
+	await lix.db
+		.insertInto("commit_by_version")
+		.values([
+			{
+				id: "c_unified_3",
+				change_set_id: "cs_unified_3",
+				lixcol_version_id: "global",
+			},
+			{
+				id: "wc_unified_3",
+				change_set_id: "wcs_unified_3",
+				lixcol_version_id: "global",
+			},
+		])
+		.execute();
+
+	await lix.db
+		.insertInto("version_by_version")
+		.values({
+			id: "v_unified_3",
+			name: "v_unified_3",
+			working_commit_id: "wc_unified_3",
+			commit_id: "c_unified_3",
+			lixcol_version_id: "global",
+		})
+		.execute();
+
+	const v = await lix.db
+		.selectFrom("version")
+		.where("id", "=", "v_unified_3")
+		.selectAll()
+		.executeTakeFirstOrThrow();
+	expect(v.commit_id).toBe("c_unified_3");
+});
+
+test("commit_id FK uses materialized mode (lenient on insert)", async () => {
+	const lix = await openLix({});
+
+	await lix.db
+		.insertInto("change_set_by_version")
+		.values([{ id: "wcs_unified_6", lixcol_version_id: "global" }])
+		.execute();
+
+	await lix.db
+		.insertInto("commit_by_version")
+		.values([
+			{
+				id: "wc_unified_6",
+				change_set_id: "wcs_unified_6",
+				lixcol_version_id: "global",
+			},
+		])
+		.execute();
+
+	await expect(
+		lix.db
+			.insertInto("version")
+			.values({
+				id: "v_unified_6",
+				name: "v_unified_6",
+				working_commit_id: "wc_unified_6",
+				commit_id: "does_not_exist",
+			})
+			.execute()
+	).resolves.toBeDefined();
+});
+
+test("delete via version writes descriptor + tip tombstones", async () => {
+	const lix = await openLix({});
+
+	await lix.db
+		.insertInto("change_set_by_version")
+		.values([
+			{ id: "cs_unified_7", lixcol_version_id: "global" },
+			{ id: "wcs_unified_7", lixcol_version_id: "global" },
+		])
+		.execute();
+	await lix.db
+		.insertInto("commit_by_version")
+		.values([
+			{
+				id: "c_unified_7",
+				change_set_id: "cs_unified_7",
+				lixcol_version_id: "global",
+			},
+			{
+				id: "wc_unified_7",
+				change_set_id: "wcs_unified_7",
+				lixcol_version_id: "global",
+			},
+		])
+		.execute();
+
+	await lix.db
+		.insertInto("version")
+		.values({
+			id: "v_unified_7",
+			name: "v_unified_7",
+			working_commit_id: "wc_unified_7",
+			commit_id: "c_unified_7",
+		})
+		.execute();
+
+	await lix.db.deleteFrom("version").where("id", "=", "v_unified_7").execute();
+
+	const changes = await lix.db
+		.selectFrom("change")
+		.where("entity_id", "=", "v_unified_7")
+		.orderBy("created_at", "asc")
+		.selectAll()
+		.execute();
+
+	const tombstones = changes.filter(
+		(c) => c.snapshot_content === null || c.snapshot_content === undefined
+	);
+	expect(tombstones.length).toBeGreaterThanOrEqual(2);
+});