@remit/drizzle-service 0.0.75 → 0.0.76

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/drizzle-service",
-  "version": "0.0.75",
+  "version": "0.0.76",
   "description": "Drizzle ORM service over SQLite",
   "type": "module",
   "main": "src/index.ts",
@@ -9,6 +9,10 @@
     ".": {
       "types": "./src/index.ts",
       "default": "./src/index.ts"
+    },
+    "./test-sqlite": {
+      "types": "./src/test-sqlite.ts",
+      "default": "./src/test-sqlite.ts"
     }
   },
   "scripts": {

package/src/repos/i4-account.ts

@@ -19,6 +19,17 @@ import { rowToMailbox } from "./i4-mailbox.js";
 
 type DB = Db<Record<string, unknown>>;
 
+/**
+ * A stored failure is a sentence the account card shows, not a transcript.
+ * Long enough for what a mail server says when it refuses, short enough that a
+ * chatty one — or a stack-carrying `Error.message` from a sync — cannot fill the
+ * column. Clamped here so every writer is bounded, whatever it hands over.
+ */
+const LAST_ERROR_MAX_LENGTH = 500;
+
+const clampLastError = (value: string | undefined): string | undefined =>
+	value === undefined ? undefined : value.slice(0, LAST_ERROR_MAX_LENGTH);
+
 export function rowToAccount(
 	row: typeof accountTable.$inferSelect,
 ): AccountItem {
@@ -88,7 +99,7 @@ export class AccountRepo implements IAccountRepository {
 				connectionState: input.connectionState,
 				lastConnectedAt: input.lastConnectedAt,
 				lastSyncAt: input.lastSyncAt,
-				lastError: input.lastError,
+				lastError: clampLastError(input.lastError),
 				syncPhase: input.syncPhase,
 				mailboxCountTotal: input.mailboxCountTotal,
 				mailboxCountSynced: input.mailboxCountSynced,
@@ -159,7 +170,8 @@ export class AccountRepo implements IAccountRepository {
 		if (input.lastConnectedAt !== undefined)
 			updates.lastConnectedAt = input.lastConnectedAt;
 		if (input.lastSyncAt !== undefined) updates.lastSyncAt = input.lastSyncAt;
-		if (input.lastError !== undefined) updates.lastError = input.lastError;
+		if (input.lastError !== undefined)
+			updates.lastError = clampLastError(input.lastError);
 		if (input.syncPhase !== undefined) updates.syncPhase = input.syncPhase;
 		if (input.mailboxCountTotal !== undefined)
 			updates.mailboxCountTotal = input.mailboxCountTotal;

package/src/test-sqlite.ts

@@ -0,0 +1,39 @@
+import Database from "better-sqlite3";
+import { drizzle } from "drizzle-orm/better-sqlite3";
+import type { Db } from "./db.js";
+import {
+	applyMigration,
+	migrationJournal,
+} from "./test-shipped-sqlite-schema.js";
+
+/**
+ * The store a test runs against: the committed SQLite entity migrations, in the
+ * order the migrator runs them, over the same engine a self-host deployment
+ * boots. Applying the shipped DDL rather than pushing the drizzle table objects
+ * is the point — a test then fails when the two drift (reader#73).
+ */
+export const applyShippedMigrations = (sqlite: Database.Database): void => {
+	for (const entry of [...migrationJournal()].sort(
+		(left, right) => left.idx - right.idx,
+	)) {
+		applyMigration(sqlite, entry.tag);
+	}
+};
+
+/**
+ * A migrated in-memory database and a drizzle handle over it, for a test that
+ * wants one repo and no file to clean up.
+ */
+export const createShippedSqliteDb = (): {
+	db: Db<Record<string, unknown>>;
+	sqlite: Database.Database;
+	close: () => void;
+} => {
+	const sqlite = new Database(":memory:");
+	applyShippedMigrations(sqlite);
+	return {
+		db: drizzle(sqlite) as unknown as Db<Record<string, unknown>>,
+		sqlite,
+		close: () => sqlite.close(),
+	};
+};