@remit/drizzle-service 0.0.59 → 0.0.61

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/drizzle-service",
-  "version": "0.0.59",
+  "version": "0.0.61",
   "description": "Drizzle ORM service over SQLite",
   "type": "module",
   "main": "src/index.ts",

package/src/id.test.ts

@@ -0,0 +1,69 @@
+import assert from "node:assert/strict";
+import { describe, test } from "node:test";
+import {
+	bodyPartContentId,
+	bodyPartId,
+	bodyPartParameterId,
+	deterministicBase36Id,
+	envelopeAddressId,
+	envelopeId,
+	generateDeterministicId,
+	rootBodyPartId,
+} from "./id.js";
+
+const MESSAGE = "message-golden";
+
+describe("derived ids are stored primary keys: a changed value orphans every row already written and needs a migration, never a new expectation here", () => {
+	test("generateDeterministicId pins the namespace and the UUIDv5 encoding", () => {
+		assert.equal(
+			generateDeterministicId("golden-seed"),
+			"68dd8eea-ad03-5221-b37c-52bbf1772c30",
+		);
+	});
+
+	test("deterministicBase36Id pins the base36 encoding shared with the electrodb adapter", () => {
+		assert.equal(
+			deterministicBase36Id("golden-seed"),
+			"67hxxs95ofdk5frbprqyrgwmo",
+		);
+	});
+
+	test("bodyPartId pins the body_part primary key", () => {
+		assert.equal(
+			bodyPartId(MESSAGE, "1.2"),
+			"022d58cc-585c-5391-8106-f2d2c1c0b228",
+		);
+	});
+
+	test("rootBodyPartId pins the root MIME node's key, part path 0", () => {
+		assert.equal(
+			rootBodyPartId(MESSAGE),
+			"f15cb8cc-8e03-5d68-812a-70bac87e2261",
+		);
+	});
+
+	test("bodyPartParameterId pins the body_part_parameter primary key", () => {
+		assert.equal(
+			bodyPartParameterId(MESSAGE, "1.2", "charset"),
+			"fdac41cc-7cbf-5bf7-bfb3-48f66e2e7f0c",
+		);
+	});
+
+	test("envelopeId pins the envelope primary key", () => {
+		assert.equal(envelopeId(MESSAGE), "fdfd34a1-84fb-535f-a586-965338211e99");
+	});
+
+	test("envelopeAddressId pins the envelope_address primary key", () => {
+		assert.equal(
+			envelopeAddressId(MESSAGE, "to", 1),
+			"5f4e930e-e418-51a2-9402-657822f8688b",
+		);
+	});
+
+	test("bodyPartContentId pins the body_part_content primary key", () => {
+		assert.equal(
+			bodyPartContentId(MESSAGE, "bodypart-golden"),
+			"c0ee5f7d-1894-5957-9f65-d9aafdf306ab",
+		);
+	});
+});

package/src/repos/i4-address.test.ts

@@ -1,7 +1,6 @@
 import assert from "node:assert";
 import { randomUUID } from "node:crypto";
 import { after, before, describe, test } from "node:test";
-import { envelopeAddressId } from "../id.js";
 import { createTestDb, randomId, type TestDb } from "../test-db.js";
 import { AddressRepo } from "./i4-address.js";
 
@@ -140,7 +139,6 @@ describe("AddressRepo", () => {
 	test("createEnvelopeAddress and getEnvelopeAddress", async () => {
 		const messageId = randomUUID();
 		const addressId = randomUUID();
-		const envId = envelopeAddressId(messageId, "from", 0);
 
 		const addr = await repo.createAddress({
 			...makeAddressInput(randomId()),
@@ -155,18 +153,18 @@ describe("AddressRepo", () => {
 			addressOrder: 0,
 		});
 
-		assert.equal(ea.envelopeAddressId, envId);
 		assert.equal(ea.messageId, messageId);
 		assert.equal(ea.addressRole, "from");
 
-		const fetched = await repo.getEnvelopeAddress(envId);
-		assert.equal(fetched.envelopeAddressId, envId);
+		const fetched = await repo.getEnvelopeAddress(ea.envelopeAddressId);
+		assert.equal(fetched.envelopeAddressId, ea.envelopeAddressId);
+		assert.equal(fetched.addressId, addressId);
 
-		await repo.deleteEnvelopeAddress(envId);
+		await repo.deleteEnvelopeAddress(ea.envelopeAddressId);
 		await repo.deleteAddress(addr.accountConfigId, addr.addressId);
 	});
 
-	test("upsertEnvelopeAddress is idempotent", async () => {
+	test("upsertEnvelopeAddress keys the row on messageId, role and order", async () => {
 		const messageId = randomUUID();
 		const input = {
 			messageId,
@@ -176,14 +174,22 @@ describe("AddressRepo", () => {
 			addressOrder: 1,
 		};
 		const first = await repo.upsertEnvelopeAddress(input);
-		const second = await repo.upsertEnvelopeAddress(input);
-		assert.equal(
-			first.envelopeAddressId,
-			envelopeAddressId(messageId, "to", 1),
-		);
+		const second = await repo.upsertEnvelopeAddress({
+			...input,
+			addressId: randomUUID(),
+			normalizedEmail: "y@example.com",
+		});
 		assert.equal(second.envelopeAddressId, first.envelopeAddressId);
+		assert.equal(second.normalizedEmail, "x@example.com");
+
+		const reordered = await repo.upsertEnvelopeAddress({
+			...input,
+			addressOrder: 2,
+		});
+		assert.notEqual(reordered.envelopeAddressId, first.envelopeAddressId);
 
 		await repo.deleteEnvelopeAddress(first.envelopeAddressId);
+		await repo.deleteEnvelopeAddress(reordered.envelopeAddressId);
 	});
 
 	test("deleteManyEnvelopeAddresses removes in batch", async () => {

package/src/repos/i4-mailbox.test.ts

@@ -135,6 +135,14 @@ describe("MailboxRepo", () => {
 		);
 	});
 
+	test("findByPathPrefix finds nothing in a flat namespace", async () => {
+		const accountId = randomId();
+		await repo.create(makeMailboxInput(accountId, "Work"));
+		await repo.create(makeMailboxInput(accountId, "Workshop"));
+
+		assert.deepEqual(await repo.findByPathPrefix(accountId, "Work", ""), []);
+	});
+
 	test("renameChildPaths updates all children", async () => {
 		const accountId = randomId();
 		await repo.create(makeMailboxInput(accountId, "OldName/Sub1"));

package/src/repos/i4-mailbox.ts

@@ -306,6 +306,9 @@ export class MailboxRepo implements IMailboxRepository {
 		pathPrefix: string,
 		delimiter = "/",
 	): Promise<MailboxItem[]> {
+		// A flat namespace nests nothing: with no delimiter the prefix is the
+		// folder’s own path, which would match every sibling starting with it.
+		if (delimiter.length === 0) return [];
 		const rows = await this.db
 			.select()
 			.from(mailboxTable)

package/src/repos/thread-message.test.ts

@@ -7,6 +7,7 @@ import { createSqliteTestDb } from "../test-db-sqlite.js";
 import {
 	clampThreadSearchLimit,
 	DrizzleThreadMessageRepository,
+	deriveThreadMessageId,
 	THREAD_SEARCH_MAX_LIMIT,
 } from "./thread-message.js";
 
@@ -936,3 +937,12 @@ describe("DrizzleThreadMessageRepository — core CRUD", () => {
 		assert.equal(await repo.countByThread(other, threadId), 0);
 	});
 });
+
+describe("derived ids are stored primary keys: a changed value orphans every row already written and needs a migration, never a new expectation here", () => {
+	test("deriveThreadMessageId pins the thread_message primary key", () => {
+		assert.equal(
+			deriveThreadMessageId("thread-golden", "message-golden"),
+			"agmlamuvd9feee9fmfvqxrb35",
+		);
+	});
+});