@voltro/sql-mssql 0.43.2 → 0.44.1

package/CHANGELOG.md

@@ -39,6 +39,56 @@ _Changes staged for the next release accumulate here (rolled up from
 
 ---
 
+## [0.44.1] — 2026-08-19
+
+### Fixed
+
+- **@voltro/database, @voltro/data-transfer, @voltro/cli** — A refused import row reported `(FiberFailure) SqlError: Failed to execute statement` — our runtime's rendering of a rejection, marker and stack frames and all — instead of the constraint that fired. It names neither a rule, nor a code, nor even which layer refused, and every row refused for the same cause carries it identically.
+
+  Two defects, and fixing either alone still leaves a reader stuck.
+
+  **`Cause.squash` elects a branch, and first is a position, not a ranking.** A `Cause` is a tree, and a transaction routinely produces one with more than one leaf: the statement that failed, and whatever the rollback or a finalizer did on the way out. When the first leaf was the bare wrapper, the driver error sitting in the sibling branch was never looked at. Measured: `sequential(bareSqlError, sqlErrorWithDriver)` classified as nothing while the same two branches in the opposite order classified as `unique constraint PRIMARY … [ER_DUP_ENTRY/1062]`. `extractDbCause` flattens failures AND defects now, in Cause order, expanding a nested `FiberFailure` leaf, and elects the branch that names a driver — the others' chains are appended rather than dropped.
+
+  **A row's reason may never read like a stack trace.** The runtime rendering is stripped before any tier looks at the text, so the failure mode cannot return invisibly. And when no driver detail is reachable at all, the reason now names the CHAIN of wrappers the failure passed through — the difference between "the database refused this row" and "the connection died mid-import" — while the run logs the full rendering of the first few such failures. Never returned over the wire: it carries frames, and on some engines a driver's sentence carries row data.
+
+  `RowsRefusedError` also gained `byTable`: **complete** per-table counts. `rows` is capped at 20, so per-table counts tallied off the printed list sum to the cap rather than to the failure — and nothing else in the payload offered any.
+
+---
+
+## [0.44.0] — 2026-08-19
+
+### ⚠ BREAKING
+
+- **@voltro/data-transfer, @voltro/cli** — `runImport` returns an `ImportOutcome` instead of the bundle's `Manifest`, and `--mode replace` runs as ONE transaction by default.
+
+  **Why the return type changed.** The summary line counted the rows the BUNDLE carries, not the rows the run wrote. Those differ most exactly where it matters: a bundle directory carries its own resume ledger, so a copied directory imports nothing — correctly, with a warning naming the file to delete — and the run then printed `import complete … rows: 242950` over a target it had not touched. The warning was one line above, which is one line too far for anyone piping the output through `tail -1`. The outcome carries `rowsWritten`, `rowsSkipped` and `fullyResumed`, the CLI reports written-vs-carried, and a fully-skipped run says so on its LAST line.
+
+  Migration: `runImport(...)` now resolves to `{ manifest, tablesWritten, rowsWritten, tablesSkipped, rowsSkipped, fullyResumed }`. Read `.manifest` where you read the manifest before.
+
+  **Why replace is atomic now.** The all-or-nothing guarantee was written for the emptying step, and read — reasonably — as covering the run. A replace that died partway through the LOAD left the target emptied of its old rows and holding part of the new ones, measured on a live instance over sixteen minutes. There is no useful state for a replace to stop in, so it is a default rather than a flag you have to know about. It also closes the window that produced that failure: with the tables emptied and the load uncommitted, a concurrent writer in the application waits instead of inserting a row the bundle is about to insert too.
+
+  `--no-atomic` (CLI) / `atomic: false` (API) opts out. The trade is stated where it bites: every write to those tables waits for the load, and on postgres the bulk `COPY` loader cannot join a transaction it does not own — an atomic run now says that once rather than being quietly slower.
+
+  Two smaller things from the same report: `--mode replace` against a running instance warns about the empty-target window when it is NOT atomic, and the `--target api` upload reports progress per chunk plus a line explaining that the final request stays open for the whole import — sixteen minutes of silence is indistinguishable from a hang, and one operator killed a run that had finished.
+
+  **`voltro update` carries you across this** — codemod `0.44.0/01_import-outcome`.
+
+### Fixed
+
+- **@voltro/sql-mysql, @voltro/sql-postgres, @voltro/sql-sqlite, @voltro/sql-mssql, @voltro/data-transfer** — A write and its write-recorders now succeed or fail together on every SQL dialect, and the data importer's retry is idempotent.
+
+  A recorder (a versioning trail, an audit log) runs on the caller's connection and is ALLOWED to fail — a recorder that throws must take the write down with it, that is its contract. Outside a caller transaction the two were not one unit: the row's statement committed on its own, and the recorder's INSERT ran afterwards as a second autocommit statement. So a recorder that threw left a COMMITTED row behind a write that reported failure. Measured directly: `insert` throws, the row is in the table, and a second attempt at the same row is `ER_DUP_ENTRY` on PRIMARY.
+
+  Anything that retries a failed write then meets its own row. The data importer retries by design — it holds a row whose write failed and tries again once the remaining tables have streamed — so a `--mode replace` that had just emptied a table failed on a duplicate key IN that table. From outside, that is the impossible-looking thing: an import that emptied a table and then failed because something was already in it. Reproduced verbatim, including the table name and `[ER_DUP_ENTRY/1062]`, by putting the old retry back.
+
+  Both ends are closed, and they are independent on purpose:
+
+  - **The cause.** A table that HAS recorders writes inside a transaction now, so the row and the trail commit together or not at all — on mysql, mariadb, postgres, sqlite and mssql, verified by one suite that asks all five the same question. A table with no recorders — the default — takes the direct path unchanged; `recordsTable` is a Map-size check first, so it costs one comparison. - **The defence.** The importer's retry upserts instead of inserting in `replace` mode. That covers every OTHER way a write can land while reporting failure: a driver timeout on a write the server applied, a connection lost after the commit, a concurrent writer inserting the same key. It is sound precisely because the table was emptied by this same run — there is nothing in it that is not ours.
+
+  The test that pinned the divergence went red when the fix landed, exactly as its own note said it would, and is inverted with that note kept.
+
+---
+
 ## [0.43.2] — 2026-08-18
 
 ### Fixed

package/dist/index.d.ts

@@ -222,6 +222,13 @@ export declare class MssqlStore implements DataStore {
      * stores is what lets the parity test ask the question at all.
      */
     private localWrite;
+    /**
+     * A write and its write-recorders, as ONE unit — see `MysqlStore.
+     * writeWithRecorders` for the failure this closes: a recorder that throws
+     * after the row's statement committed leaves a committed row behind a write
+     * that reported failure, and a caller that retries meets its own row.
+     */
+    private writeWithRecorders;
     insert(t: string, r: Row): Promise<Readonly<Record<string, unknown>>>;
     insertMany(t: string, rows: ReadonlyArray<Row>): Promise<readonly Readonly<Record<string, unknown>>[]>;
     patchJson(t: string, pk: string, path: string, value: unknown): Promise<Readonly<Record<string, unknown>> | null>;

package/dist/index.js

@@ -1,31 +1,31 @@
 import { MssqlClient as e } from "@effect/sql-mssql";
-import { Config as t, Duration as n, Effect as r, Layer as i, ManagedRuntime as a, Redacted as o } from "effect";
-import { CDC_OFFSETS_TABLE as s, DEFAULT_ACQUIRE_TIMEOUT_MS as c, EagerCardinalityError as l, _voltroMssqlCdcOffsetsTable as u, attachEagerLoads as d, attributionFields as f, bulkInsertLimitsFor as p, chunkRowsForInsert as m, compileEagerJson as h, compilePredicate as g, compileRawFragment as _, compileSelect as v, decodeRowsFromSchema as y, externalChangeEvent as b, hasEagerLoads as ee, isTableReactive as x, makeEagerFallbackReporter as te, observeDbOp as S, qualifyTable as C, quoteIdent as w, raiseChangeListenerCeiling as T, recordsTable as E, requireTable as ne, runStoreTransaction as D, runWriteRecorders as O, stampGeneratedId as k, stampGeneratedIds as A, withCapturedAttribution as j } from "@voltro/database";
-import { EventEmitter as M } from "node:events";
-import { createLogger as N } from "@voltro/logger";
-import { SqlClient as P, TransactionConnection as F } from "@effect/sql/SqlClient";
+import { Config as t, Duration as n, Effect as r, Layer as i, ManagedRuntime as a, Option as o, Redacted as s } from "effect";
+import { CDC_OFFSETS_TABLE as c, DEFAULT_ACQUIRE_TIMEOUT_MS as l, EagerCardinalityError as u, _voltroMssqlCdcOffsetsTable as d, attachEagerLoads as f, attributionFields as p, bulkInsertLimitsFor as m, chunkRowsForInsert as h, compileEagerJson as g, compilePredicate as _, compileRawFragment as v, compileSelect as y, decodeRowsFromSchema as b, externalChangeEvent as ee, hasEagerLoads as x, isTableReactive as S, makeEagerFallbackReporter as te, observeDbOp as C, qualifyTable as ne, quoteIdent as w, raiseChangeListenerCeiling as T, recordsTable as E, requireTable as D, runStoreTransaction as O, runWriteRecorders as k, stampGeneratedId as A, stampGeneratedIds as j, withCapturedAttribution as re } from "@voltro/database";
+import { EventEmitter as ie } from "node:events";
+import { createLogger as M } from "@voltro/logger";
+import { SqlClient as N, TransactionConnection as P } from "@effect/sql/SqlClient";
 //#region src/sqlLayer.ts
-var re = {
+var ae = {
 	...e.defaultParameterTypes,
 	null: e.defaultParameterTypes.object
-}, ie = (e) => {
-	let t = e.acquireTimeoutMs ?? c;
+}, oe = (e) => {
+	let t = e.acquireTimeoutMs ?? l;
 	return t > 0 ? t : void 0;
-}, I = (r) => {
-	let i = ie(r);
+}, F = (r) => {
+	let i = oe(r);
 	return e.layerConfig({
 		server: t.succeed(r.server),
-		parameterTypes: t.succeed(re),
+		parameterTypes: t.succeed(ae),
 		...i === void 0 ? {} : { connectTimeout: t.succeed(n.millis(i)) },
 		...r.port === void 0 ? {} : { port: t.succeed(r.port) },
 		...r.database === void 0 ? {} : { database: t.succeed(r.database) },
 		...r.username === void 0 ? {} : { username: t.succeed(r.username) },
-		...r.password === void 0 ? {} : { password: t.succeed(o.make(r.password)) },
+		...r.password === void 0 ? {} : { password: t.succeed(s.make(r.password)) },
 		...r.encrypt === void 0 ? {} : { encrypt: t.succeed(r.encrypt) },
 		...r.trustServer === void 0 ? {} : { trustServer: t.succeed(r.trustServer) },
 		...r.maxConnections === void 0 ? {} : { maxConnections: t.succeed(r.maxConnections) }
 	});
-}, L = (e) => {
+}, I = (e) => {
 	let t = e.get("sslmode");
 	if (t !== null) {
 		if (t === "require") return !0;
@@ -40,7 +40,7 @@ var re = {
 			throw Error(`DB_URL '?${t}=${n}' is not supported by the mssql dialect — use 'true'/'1' or 'false'/'0'.`);
 		}
 	}
-}, R = (e) => {
+}, L = (e) => {
 	let t = e.acquireTimeoutMs === void 0 ? {} : { acquireTimeoutMs: e.acquireTimeoutMs }, n = (e) => e === void 0 ? {} : {
 		encrypt: e,
 		...e ? { trustServer: !0 } : {}
@@ -53,7 +53,7 @@ var re = {
 			username: decodeURIComponent(r.username || "sa"),
 			password: decodeURIComponent(r.password || ""),
 			database: r.pathname.replace(/^\//, "") || "master",
-			...n(e.ssl ?? L(r.searchParams)),
+			...n(e.ssl ?? I(r.searchParams)),
 			...e.maxConnections === void 0 ? {} : { maxConnections: e.maxConnections },
 			...t
 		};
@@ -68,7 +68,7 @@ var re = {
 		...e.maxConnections === void 0 ? {} : { maxConnections: e.maxConnections },
 		...t
 	};
-}, z = (e) => I(R(e)), B = /* @__PURE__ */ new Set(["1205"]), V = (e) => {
+}, R = (e) => F(L(e)), z = /* @__PURE__ */ new Set(["1205"]), B = (e) => {
 	let t = e;
 	for (let e = 0; e < 5 && typeof t == "object" && t; e++) {
 		let e = t.number;
@@ -77,36 +77,36 @@ var re = {
 		if (typeof n == "string") return n;
 		t = t.cause;
 	}
-}, H = (e) => {
-	let t = V(e);
-	return t !== void 0 && B.has(t);
-}, U = /* @__PURE__ */ new Set(["2601", "2627"]), W = (e) => {
-	let t = V(e);
-	return t !== void 0 && U.has(t);
-}, G = (e) => H(e) ? "retry" : "noRetry", K = {
+}, V = (e) => {
+	let t = B(e);
+	return t !== void 0 && z.has(t);
+}, H = /* @__PURE__ */ new Set(["2601", "2627"]), U = (e) => {
+	let t = B(e);
+	return t !== void 0 && H.has(t);
+}, W = (e) => V(e) ? "retry" : "noRetry", G = {
 	I: "insert",
 	U: "update",
 	D: "delete"
-}, q = (e, t) => e(r.gen(function* () {
-	let e = yield* P;
+}, K = (e, t) => e(r.gen(function* () {
+	let e = yield* N;
 	if (((yield* e`
         SELECT COUNT(*) AS ${e("on")}
         FROM sys.change_tracking_databases
         WHERE database_id = DB_ID()`)[0]?.on ?? 0) === 0) return yield* r.fail(/* @__PURE__ */ Error("mssql Change Tracking CDC requires CT enabled on the database. Run `ALTER DATABASE <db> SET CHANGE_TRACKING = ON (CHANGE_RETENTION = 2 DAYS, AUTO_CLEANUP = ON)`, or set CDC=0 for inline emit."));
 	for (let n of t) yield* e.unsafe(`IF NOT EXISTS (
              SELECT 1 FROM sys.change_tracking_tables
-             WHERE object_id = OBJECT_ID(${J(n)})
+             WHERE object_id = OBJECT_ID(${q(n)})
            )
-           ALTER TABLE ${Y(n)} ENABLE CHANGE_TRACKING WITH (TRACK_COLUMNS_UPDATED = OFF)`);
-})).then(() => void 0), J = (e) => `'${e.replace(/'/g, "''")}'`, Y = (e) => `[${e.replace(/\]/g, "]]")}]`, X = async (e) => {
-	let t = N({ scope: "voltro:mssql:cdc" }), n = e.pollIntervalMs ?? 500;
-	await q(e.run, e.tables);
+           ALTER TABLE ${J(n)} ENABLE CHANGE_TRACKING WITH (TRACK_COLUMNS_UPDATED = OFF)`);
+})).then(() => void 0), q = (e) => `'${e.replace(/'/g, "''")}'`, J = (e) => `[${e.replace(/\]/g, "]]")}]`, Y = async (e) => {
+	let t = M({ scope: "voltro:mssql:cdc" }), n = e.pollIntervalMs ?? 500;
+	await K(e.run, e.tables);
 	let i = async () => (await e.run(r.gen(function* () {
-		return yield* (yield* P)`SELECT CONVERT(VARCHAR(40), CHANGE_TRACKING_CURRENT_VERSION()) AS v`;
+		return yield* (yield* N)`SELECT CONVERT(VARCHAR(40), CHANGE_TRACKING_CURRENT_VERSION()) AS v`;
 	})))[0]?.v ?? "0", a = e.startVersion ?? await i(), o = !1, s = null, c = !1, l = (e, t) => BigInt(e) > BigInt(t), u = async (n, o) => {
 		let s = await e.run(r.gen(function* () {
-			let e = yield* P, t = e.unsafe(Y(n)), r = (yield* e`
-          SELECT CONVERT(VARCHAR(40), CHANGE_TRACKING_MIN_VALID_VERSION(OBJECT_ID(${e.unsafe(J(n))}))) AS floor`)[0]?.floor;
+			let e = yield* N, t = e.unsafe(J(n)), r = (yield* e`
+          SELECT CONVERT(VARCHAR(40), CHANGE_TRACKING_MIN_VALID_VERSION(OBJECT_ID(${e.unsafe(q(n))}))) AS floor`)[0]?.floor;
 			return r != null && BigInt(o) < BigInt(r) ? {
 				resync: !0,
 				rows: []
@@ -124,7 +124,7 @@ var re = {
 			return;
 		}
 		for (let t of s.rows) {
-			let r = K[t.__ct_op];
+			let r = G[t.__ct_op];
 			if (!r) continue;
 			let { __ct_op: i, __ct_id: a, ...o } = t;
 			if (r === "delete") e.onChange({
@@ -134,7 +134,7 @@ var re = {
 				new: null
 			});
 			else {
-				let t = y([o], n, "mssql")[0];
+				let t = b([o], n, "mssql")[0];
 				e.onChange({
 					table: n,
 					op: r,
@@ -175,26 +175,26 @@ var re = {
 		},
 		currentVersion: () => a
 	};
-}, Z = N({ scope: "voltro:mssql" }), Q = async (e) => {
+}, X = M({ scope: "voltro:mssql" }), Z = async (e) => {
 	let t = e.changeStrategy ?? "inline", n = t;
-	t === "cdc" && !e.cdcConfig && (Z.warn("changeStrategy='cdc' requires cdcConfig (replicaId + includeTables); falling back to 'inline'."), n = "inline");
-	let r = e.tracerLayer ? i.mergeAll(e.sqlLayer, e.tracerLayer) : e.sqlLayer, o = a.make(r), s = new ae(await o.runPromise(P), o, n);
+	t === "cdc" && !e.cdcConfig && (X.warn("changeStrategy='cdc' requires cdcConfig (replicaId + includeTables); falling back to 'inline'."), n = "inline");
+	let r = e.tracerLayer ? i.mergeAll(e.sqlLayer, e.tracerLayer) : e.sqlLayer, o = a.make(r), s = new se(await o.runPromise(N), o, n);
 	return n === "cdc" && e.cdcConfig && await s.startCdcConsumer(e.cdcConfig), s;
-}, ae = class e {
+}, se = class e {
 	sql;
 	runtime;
 	changeStrategy;
 	namespace;
 	emitter;
 	inflightTxns = 0;
-	reportEagerFallback = te(Z);
+	reportEagerFallback = te(X);
 	cdcHandle = null;
 	cdcCheckpointTimer = null;
 	cdcPendingVersion = null;
 	cdcReplicaId = "";
 	cdcStreamName = "default";
 	constructor(e, t, n = "inline", r = null, i) {
-		this.sql = e, this.runtime = t, this.changeStrategy = n, this.namespace = r, this.emitter = i ?? new M(), T(this.emitter);
+		this.sql = e, this.runtime = t, this.changeStrategy = n, this.namespace = r, this.emitter = i ?? new ie(), T(this.emitter);
 	}
 	get inlineEmit() {
 		return this.changeStrategy === "inline";
@@ -203,16 +203,16 @@ var re = {
 		return t === this.namespace ? this : new e(this.sql, this.runtime, this.changeStrategy, t, this.emitter);
 	}
 	nsT(e) {
-		return C(this.namespace, e);
+		return ne(this.namespace, e);
 	}
 	__mssqlReplicationFriend = { runEffect: (e) => this.runtime.runPromise(e) };
 	async executeQuery(e, t) {
-		let n = v(e, this.sql, this.namespace), i = t ? r.provideService(n, F, t) : n, a = await S("mssql", "select", () => this.runtime.runPromise(i));
-		return y(a, e.table, "mssql");
+		let n = y(e, this.sql, this.namespace), i = t ? r.provideService(n, P, t) : n, a = await C("mssql", "select", () => this.runtime.runPromise(i));
+		return b(a, e.table, "mssql");
 	}
 	async executeInsert(e, t, n, i, a) {
-		t = k(e, t);
-		let o = this.sql, s = o`INSERT INTO ${o(this.nsT(e))} ${o.insert(t).returning("*")}`, c = n ? r.provideService(s, F, n) : s, l = (await this.runtime.runPromise(c))[0];
+		t = A(e, t);
+		let o = this.sql, s = o`INSERT INTO ${o(this.nsT(e))} ${o.insert(t).returning("*")}`, c = n ? r.provideService(s, P, n) : s, l = (await this.runtime.runPromise(c))[0];
 		if (!l) throw Error(`MssqlStore.insert: row not returned post-insert in '${e}'`);
 		return await this.routeEvent({
 			table: e,
@@ -222,7 +222,7 @@ var re = {
 		}, i, n, a), l;
 	}
 	async executeUpdate(e, t, n, i, a, o) {
-		let s = this.sql, c = s`UPDATE ${s(this.nsT(e))} SET ${s.update(n).returning("*")} WHERE ${s("id")} = ${t}`, l = i ? r.provideService(c, F, i) : c, u = (await this.runtime.runPromise(l))[0];
+		let s = this.sql, c = s`UPDATE ${s(this.nsT(e))} SET ${s.update(n).returning("*")} WHERE ${s("id")} = ${t}`, l = i ? r.provideService(c, P, i) : c, u = (await this.runtime.runPromise(l))[0];
 		return u ? (await this.routeEvent({
 			table: e,
 			op: "update",
@@ -231,27 +231,27 @@ var re = {
 		}, a, i, o), u) : null;
 	}
 	async executeInsertMany(e, t, n, i, a) {
-		if (t = A(e, t), t.length === 0) return [];
-		let o = this.sql, s = m(t, p("mssql")), c = (t) => o`INSERT INTO ${o(this.nsT(e))} ${o.insert(t).returning("*")}`, l;
+		if (t = j(e, t), t.length === 0) return [];
+		let o = this.sql, s = h(t, m("mssql")), c = (t) => o`INSERT INTO ${o(this.nsT(e))} ${o.insert(t).returning("*")}`, l;
 		if (s.length === 1) {
-			let e = c(s[0]), t = n ? r.provideService(e, F, n) : e;
+			let e = c(s[0]), t = n ? r.provideService(e, P, n) : e;
 			l = await this.runtime.runPromise(t);
 		} else if (n) {
 			let e = [];
-			for (let t of s) e.push(...await this.runtime.runPromise(r.provideService(c(t), F, n)));
+			for (let t of s) e.push(...await this.runtime.runPromise(r.provideService(c(t), P, n)));
 			l = e;
 		} else l = await this.runtime.runPromise(o.withTransaction(r.map(r.forEach(s, c, { concurrency: 1 }), (e) => e.flat())));
-		let u = t.map((e) => e.id), d = new Map(l.map((e) => [e.id, e])), f = u.map((e) => d.get(e)).filter((e) => e !== void 0), h = f.length === l.length ? f : [...l];
-		for (let t of h) await this.routeEvent({
+		let u = t.map((e) => e.id), d = new Map(l.map((e) => [e.id, e])), f = u.map((e) => d.get(e)).filter((e) => e !== void 0), p = f.length === l.length ? f : [...l];
+		for (let t of p) await this.routeEvent({
 			table: e,
 			op: "insert",
 			old: null,
 			new: t
 		}, i, n, a);
-		return h;
+		return p;
 	}
 	async executePatchJson(e, t, n, i, a, o, s) {
-		let c = this.sql, l = n.split("."), u = l[0], d = l.slice(1), f = d.length === 0 ? "$" : `$.${d.join(".")}`, p = JSON.stringify(i ?? null), m = c`UPDATE ${c(this.nsT(e))} SET ${c(u)} = JSON_MODIFY(${c(u)}, ${f}, JSON_QUERY(${p})) OUTPUT INSERTED.* WHERE ${c("id")} = ${t}`, h = a ? r.provideService(m, F, a) : m, g = (await this.runtime.runPromise(h))[0];
+		let c = this.sql, l = n.split("."), u = l[0], d = l.slice(1), f = d.length === 0 ? "$" : `$.${d.join(".")}`, p = JSON.stringify(i ?? null), m = c`UPDATE ${c(this.nsT(e))} SET ${c(u)} = JSON_MODIFY(${c(u)}, ${f}, JSON_QUERY(${p})) OUTPUT INSERTED.* WHERE ${c("id")} = ${t}`, h = a ? r.provideService(m, P, a) : m, g = (await this.runtime.runPromise(h))[0];
 		return g ? (await this.routeEvent({
 			table: e,
 			op: "update",
@@ -260,7 +260,7 @@ var re = {
 		}, o, a, s), g) : null;
 	}
 	async executeDelete(e, t, n, i, a) {
-		let o = this.sql, s = o`DELETE FROM ${o(this.nsT(e))} OUTPUT DELETED.* WHERE ${o("id")} = ${t}`, c = n ? r.provideService(s, F, n) : s, l = (await this.runtime.runPromise(c))[0];
+		let o = this.sql, s = o`DELETE FROM ${o(this.nsT(e))} OUTPUT DELETED.* WHERE ${o("id")} = ${t}`, c = n ? r.provideService(s, P, n) : s, l = (await this.runtime.runPromise(c))[0];
 		return l ? (await this.routeEvent({
 			table: e,
 			op: "delete",
@@ -270,17 +270,17 @@ var re = {
 	}
 	async appendInTxn(e, t, n) {
 		let i = this.sql, a = i`INSERT INTO ${i(this.nsT(e))} ${i.insert(t)}`;
-		await this.runtime.runPromise(n ? r.provideService(a, F, n) : a);
+		await this.runtime.runPromise(n ? r.provideService(a, P, n) : a);
 	}
 	async maxInTxn(e, t, n, i) {
-		let a = this.sql, o = Object.entries(n).map(([e, t]) => a`${a(e)} = ${t}`), s = a`SELECT MAX(${a(t)}) AS ${a("m")} FROM ${a(this.nsT(e))} WHERE ${a.and(o)}`, c = (await this.runtime.runPromise(i ? r.provideService(s, F, i) : s))[0]?.m;
+		let a = this.sql, o = Object.entries(n).map(([e, t]) => a`${a(e)} = ${t}`), s = a`SELECT MAX(${a(t)}) AS ${a("m")} FROM ${a(this.nsT(e))} WHERE ${a.and(o)}`, c = (await this.runtime.runPromise(i ? r.provideService(s, P, i) : s))[0]?.m;
 		return c == null ? null : Number(c);
 	}
 	async routeEvent(e, t, n = null, r) {
 		e = {
-			...f(r),
+			...p(r),
 			...e
-		}, E(e.table) && await O({
+		}, E(e.table) && await k({
 			append: (e, t) => this.appendInTxn(e, t, n),
 			maxOf: (e, t, r) => this.maxInTxn(e, t, r, n)
 		}, {
@@ -290,23 +290,23 @@ var re = {
 			prev: e.old,
 			traceId: e.traceId,
 			subjectId: e.subjectId
-		}), x(e.table) && (t === null ? this.inlineEmit && this.emitter.emit("change", e) : t.push(e));
+		}), S(e.table) && (t === null ? this.inlineEmit && this.emitter.emit("change", e) : t.push(e));
 	}
 	query(e) {
 		return this.runWithEager(e, null);
 	}
 	raw(e, t) {
-		let n = _(e, this.sql);
-		return S("mssql", "raw", () => this.runtime.runPromise(n));
+		let n = v(e, this.sql);
+		return C("mssql", "raw", () => this.runtime.runPromise(n));
 	}
 	async runWithEager(e, t) {
-		if (!ee(e)) return this.executeQuery(e, t);
-		let n = this.namespace === null ? h(e, this.sql, "mssql") : null;
+		if (!x(e)) return this.executeQuery(e, t);
+		let n = this.namespace === null ? g(e, this.sql, "mssql") : null;
 		if (n !== null) try {
-			let e = t ? r.provideService(n.fragment, F, t) : n.fragment, i = await S("mssql", "select", () => this.runtime.runPromise(e));
+			let e = t ? r.provideService(n.fragment, P, t) : n.fragment, i = await C("mssql", "select", () => this.runtime.runPromise(e));
 			return n.decode(i);
 		} catch (t) {
-			if (t instanceof l) throw t;
+			if (t instanceof u) throw t;
 			this.reportEagerFallback({
 				dialect: "mssql",
 				table: e.table,
@@ -320,31 +320,40 @@ var re = {
 			reason: "not-compilable"
 		});
 		let i = await this.executeQuery(e, t);
-		return d(i, e.eager, e.sourceTable ?? ne(e.table), (e) => this.executeQuery(e, t));
+		return f(i, e.eager, e.sourceTable ?? D(e.table), (e) => this.executeQuery(e, t));
 	}
 	getInternalRunWithEager() {
 		return this.runWithEager.bind(this);
 	}
-	localWrite(e, t) {
-		return S("mssql", e, () => j(t));
+	localWrite(e, t, n) {
+		let r = (e) => E(t) ? this.writeWithRecorders(n, e) : n(e, null, null);
+		return C("mssql", e, () => re(r));
+	}
+	async writeWithRecorders(e, t) {
+		let n = [], i = this.sql, a = await this.runtime.runPromise(i.withTransaction(r.flatMap(r.serviceOption(P), (i) => o.isNone(i) ? r.fail(/* @__PURE__ */ Error("MssqlStore.write: TransactionConnection missing.")) : r.tryPromise({
+			try: () => e(t, i.value, n),
+			catch: (e) => e
+		}))));
+		for (let e of n) this.emitChange(e);
+		return a;
 	}
 	insert(e, t) {
-		return this.localWrite("insert", (n) => this.executeInsert(e, t, null, null, n));
+		return this.localWrite("insert", e, (n, r, i) => this.executeInsert(e, t, r, i, n));
 	}
 	insertMany(e, t) {
-		return this.localWrite("insert", (n) => this.executeInsertMany(e, t, null, null, n));
+		return this.localWrite("insert", e, (n, r, i) => this.executeInsertMany(e, t, r, i, n));
 	}
 	patchJson(e, t, n, r) {
-		return this.localWrite("update", (i) => this.executePatchJson(e, t, n, r, null, null, i));
+		return this.localWrite("update", e, (i, a, o) => this.executePatchJson(e, t, n, r, a, o, i));
 	}
 	update(e, t, n) {
-		return this.localWrite("update", (r) => this.executeUpdate(e, t, n, null, null, r));
+		return this.localWrite("update", e, (r, i, a) => this.executeUpdate(e, t, n, i, a, r));
 	}
 	delete(e, t) {
-		return this.localWrite("delete", (n) => this.executeDelete(e, t, null, null, n));
+		return this.localWrite("delete", e, (n, r, i) => this.executeDelete(e, t, r, i, n));
 	}
 	async updateMany(e, t, n) {
-		let r = this.sql, i = g(n.where, r, this.namespace), a = r`UPDATE ${r(this.nsT(e))} SET ${r.update(t)} OUTPUT INSERTED.* WHERE ${i}`, o = await S("mssql", "update", () => this.runtime.runPromise(a));
+		let r = this.sql, i = _(n.where, r, this.namespace), a = r`UPDATE ${r(this.nsT(e))} SET ${r.update(t)} OUTPUT INSERTED.* WHERE ${i}`, o = await C("mssql", "update", () => this.runtime.runPromise(a));
 		for (let t of o) await this.routeEvent({
 			table: e,
 			op: "update",
@@ -354,7 +363,7 @@ var re = {
 		return o.length;
 	}
 	async deleteMany(e, t) {
-		let n = this.sql, r = g(t.where, n, this.namespace), i = n`DELETE FROM ${n(this.nsT(e))} OUTPUT DELETED.* WHERE ${r}`, a = await S("mssql", "delete", () => this.runtime.runPromise(i));
+		let n = this.sql, r = _(t.where, n, this.namespace), i = n`DELETE FROM ${n(this.nsT(e))} OUTPUT DELETED.* WHERE ${r}`, a = await C("mssql", "delete", () => this.runtime.runPromise(i));
 		for (let t of a) await this.routeEvent({
 			table: e,
 			op: "delete",
@@ -364,10 +373,10 @@ var re = {
 		return a.length;
 	}
 	upsert(e, t, n) {
-		return this.localWrite("upsert", (r) => this.executeUpsert(e, t, n, null, null, r));
+		return this.localWrite("upsert", e, (r, i, a) => this.executeUpsert(e, t, n, i, a, r));
 	}
 	insertIgnore(e, t, n) {
-		return this.localWrite("upsert", (r) => this.executeInsertIgnore(e, t, n, null, null, r));
+		return this.localWrite("upsert", e, (r, i, a) => this.executeInsertIgnore(e, t, n, i, a, r));
 	}
 	upsertPatch(e, t, n) {
 		if (typeof n.update == "function") return n.update(t);
@@ -390,7 +399,7 @@ var re = {
 		try {
 			return await this.executeInsert(e, t, r, i);
 		} catch (a) {
-			if (!W(a)) throw a;
+			if (!U(a)) throw a;
 			let o = await this.findByConflict(e, t, n.conflictColumns, r);
 			if (!o) throw a;
 			return await this.executeUpdate(e, o.id, this.upsertPatch(t, o, n), r, i) ?? o;
@@ -403,7 +412,7 @@ var re = {
       ON ${o.and(d)}
       WHEN MATCHED THEN UPDATE SET ${o.csv(m)}
       WHEN NOT MATCHED THEN INSERT (${o.csv(f)}) VALUES (${o.csv(p)})
-      OUTPUT $action AS ${o("__action")}, INSERTED.*;`, g = i ? r.provideService(h, F, i) : h, _ = (await this.runtime.runPromise(g))[0];
+      OUTPUT $action AS ${o("__action")}, INSERTED.*;`, g = i ? r.provideService(h, P, i) : h, _ = (await this.runtime.runPromise(g))[0];
 		if (!_) throw Error(`MssqlStore.upsert: MERGE returned no row for '${e}'`);
 		let v = _.__action, { __action: y, ...b } = _;
 		return this.routeEvent({
@@ -414,13 +423,13 @@ var re = {
 		}, a), b;
 	}
 	async executeInsertIgnore(e, t, n, r, i, a) {
-		t = k(e, t);
+		t = A(e, t);
 		let o = await this.findByConflict(e, t, n.conflictColumns, r);
 		if (o) return o;
 		try {
 			return await this.executeInsert(e, t, r, i);
 		} catch (i) {
-			if (!W(i)) throw i;
+			if (!U(i)) throw i;
 			let a = await this.findByConflict(e, t, n.conflictColumns, r);
 			if (!a) throw i;
 			return a;
@@ -428,7 +437,7 @@ var re = {
 	}
 	async findByConflict(e, t, n, i) {
 		if (n.length === 0) return;
-		let a = this.sql, o = n.map((e) => a`${a(e)} = ${t[e]}`), s = a`SELECT TOP 1 * FROM ${a(this.nsT(e))} WHERE ${a.and(o)}`, c = i ? r.provideService(s, F, i) : s;
+		let a = this.sql, o = n.map((e) => a`${a(e)} = ${t[e]}`), s = a`SELECT TOP 1 * FROM ${a(this.nsT(e))} WHERE ${a.and(o)}`, c = i ? r.provideService(s, P, i) : s;
 		return (await this.runtime.runPromise(c))[0];
 	}
 	emitChange(e) {
@@ -438,9 +447,9 @@ var re = {
 		if (this.cdcHandle) return;
 		this.cdcReplicaId = e.replicaId;
 		let t = e.includeTables ?? [];
-		t.length === 0 && Z.warn("cdc: no reactive tables to tail; Change Tracking reader idle. Set includeTables to enable.");
+		t.length === 0 && X.warn("cdc: no reactive tables to tail; Change Tracking reader idle. Set includeTables to enable.");
 		let n = await this.readCdcOffset(e.replicaId);
-		this.cdcHandle = await X({
+		this.cdcHandle = await Y({
 			run: this.__mssqlReplicationFriend.runEffect,
 			tables: t,
 			startVersion: n,
@@ -448,9 +457,9 @@ var re = {
 			onVersion: (e) => {
 				this.cdcPendingVersion = e;
 			},
-			onError: (e) => Z.warn("cdc: consumer error", {}, e),
+			onError: (e) => X.warn("cdc: consumer error", {}, e),
 			onResync: () => {
-				Z.warn("cdc: resynced to current CT version after retention gap — subscriptions self-heal on the next change");
+				X.warn("cdc: resynced to current CT version after retention gap — subscriptions self-heal on the next change");
 			}
 		}), this.cdcCheckpointTimer = setInterval(() => {
 			this.flushCdcOffset();
@@ -459,14 +468,14 @@ var re = {
 	async readCdcOffset(e) {
 		try {
 			let t = this.sql, n = (await this.runtime.runPromise(t`
-          SELECT TOP 1 ${t("ctVersion")} FROM ${t(s)}
+          SELECT TOP 1 ${t("ctVersion")} FROM ${t(c)}
           WHERE ${t("id")} = ${e}`))[0]?.ctVersion;
-			return n ? (Z.info("cdc: resuming from persisted CT version", {
+			return n ? (X.info("cdc: resuming from persisted CT version", {
 				replicaId: e,
 				version: n
 			}), n) : null;
 		} catch (t) {
-			return Z.warn("cdc: no readable offset; starting at current CT version", { replicaId: e }, t), null;
+			return X.warn("cdc: no readable offset; starting at current CT version", { replicaId: e }, t), null;
 		}
 	}
 	async flushCdcOffset() {
@@ -475,7 +484,7 @@ var re = {
 			this.cdcPendingVersion = null;
 			try {
 				let t = this.cdcReplicaId;
-				await this.executeUpsert(s, {
+				await this.executeUpsert(c, {
 					id: t,
 					replicaId: t,
 					streamName: this.cdcStreamName,
@@ -486,7 +495,7 @@ var re = {
 					update: ["ctVersion", "updatedAt"]
 				}, null, null);
 			} catch (t) {
-				this.cdcPendingVersion = e, Z.warn("cdc: checkpoint write failed", {}, t);
+				this.cdcPendingVersion = e, X.warn("cdc: checkpoint write failed", {}, t);
 			}
 		}
 	}
@@ -535,16 +544,16 @@ var re = {
 	}
 	async emptyTablesOn(e, t) {
 		if (e.length === 0) return;
-		let n = this.sql, i = (e) => this.nsT(e), a = (e, t) => n.unsafe(`ALTER TABLE ${w(i(e), "mssql")} ${t ? "WITH CHECK CHECK" : "NOCHECK"} CONSTRAINT ALL`), o = r.acquireUseRelease(r.forEach(e, (e) => a(e, !1), { discard: !0 }), () => r.forEach(e, (e) => n`DELETE FROM ${n(i(e))}`, { discard: !0 }), () => r.orDie(r.ignore(r.forEach(e, (e) => a(e, !0), { discard: !0 })))), s = t === null ? n.withTransaction(o) : r.provideService(o, F, t);
+		let n = this.sql, i = (e) => this.nsT(e), a = (e, t) => n.unsafe(`ALTER TABLE ${w(i(e), "mssql")} ${t ? "WITH CHECK CHECK" : "NOCHECK"} CONSTRAINT ALL`), o = r.acquireUseRelease(r.forEach(e, (e) => a(e, !1), { discard: !0 }), () => r.forEach(e, (e) => n`DELETE FROM ${n(i(e))}`, { discard: !0 }), () => r.orDie(r.ignore(r.forEach(e, (e) => a(e, !0), { discard: !0 })))), s = t === null ? n.withTransaction(o) : r.provideService(o, P, t);
 		await this.runtime.runPromise(s);
 	}
 	async transactional(e) {
 		this.inflightTxns++;
 		try {
-			return await D({
+			return await O({
 				...this.txnSpec("MssqlStore.transactional"),
 				work: e,
-				makeView: (e, t) => new oe(this, e, t)
+				makeView: (e, t) => new ce(this, e, t)
 			});
 		} finally {
 			this.inflightTxns--;
@@ -556,7 +565,7 @@ var re = {
 			dialect: "mssql",
 			withTransaction: (e) => this.sql.withTransaction(e),
 			runPromiseExit: (e) => this.runtime.runPromiseExit(e),
-			isRetryable: H,
+			isRetryable: V,
 			span: {
 				name: "store.transactional",
 				attributes: {
@@ -575,7 +584,7 @@ var re = {
 		return this.changeStrategy === "cdc" ? "fleet" : "local";
 	}
 	injectExternalChange(e) {
-		x(e.table) && this.emitter.emit("change", b(e));
+		S(e.table) && this.emitter.emit("change", ee(e));
 	}
 	run(e) {
 		return this.runtime.runPromise(e);
@@ -588,7 +597,7 @@ var re = {
 		if (this.inflightTxns > 0) {
 			let t = Date.now() + e;
 			for (; this.inflightTxns > 0 && Date.now() < t;) await new Promise((e) => setTimeout(e, 25));
-			this.inflightTxns > 0 && Z.warn("close: grace period expired with in-flight transactions", {
+			this.inflightTxns > 0 && X.warn("close: grace period expired with in-flight transactions", {
 				gracePeriodMs: e,
 				inflight: this.inflightTxns
 			});
@@ -598,7 +607,7 @@ var re = {
 	async ping() {
 		await this.runtime.runPromise(this.sql`SELECT 1`);
 	}
-}, oe = class {
+}, ce = class {
 	parent;
 	txn;
 	attr;
@@ -674,12 +683,12 @@ var re = {
 			this.events.length = 0;
 		}
 	}
-}, $ = (e) => e.__mssqlReplicationFriend ?? null, se = (e, t) => e === t ? 0 : e < t ? -1 : 1, ce = () => ({
+}, Q = (e) => e.__mssqlReplicationFriend ?? null, le = (e, t) => e === t ? 0 : e < t ? -1 : 1, $ = () => ({
 	async capturePrimaryPosition(e) {
-		let t = $(e);
+		let t = Q(e);
 		if (t === null) throw Error("mssqlReplicationAdapter: primary is not an MssqlStore (missing __mssqlReplicationFriend).");
 		return t.runEffect(r.gen(function* () {
-			return (yield* (yield* P)`
+			return (yield* (yield* N)`
           SELECT CONVERT(VARCHAR(40), end_of_log_lsn) AS lsn
           FROM sys.dm_hadr_database_replica_states
           WHERE database_id = DB_ID()
@@ -689,10 +698,10 @@ var re = {
 		}));
 	},
 	async probeReplicaPosition(e) {
-		let t = $(e);
+		let t = Q(e);
 		if (t === null) throw Error("mssqlReplicationAdapter: replica is not an MssqlStore.");
 		return t.runEffect(r.gen(function* () {
-			return (yield* (yield* P)`
+			return (yield* (yield* N)`
           SELECT CONVERT(VARCHAR(40), last_hardened_lsn) AS lsn
           FROM sys.dm_hadr_database_replica_states
           WHERE database_id = DB_ID()
@@ -701,14 +710,14 @@ var re = {
 		}));
 	},
 	compare(e, t) {
-		return se(t, e) >= 0 ? "caught-up" : "behind";
+		return le(t, e) >= 0 ? "caught-up" : "behind";
 	}
-}), le = {
+}), ue = {
 	id: "mssql",
-	makeSqlLayer: (e) => z(e),
-	makeStore: (e) => Q(e),
+	makeSqlLayer: (e) => R(e),
+	makeStore: (e) => Z(e),
 	compileContains: (e, t, n) => n`${e} LIKE ${`%${t.replace(/[\\%_[\]]/g, (e) => `\\${e}`)}%`} ESCAPE '\\'`,
-	retryFilter: G
+	retryFilter: W
 };
 //#endregion
-export { s as CDC_OFFSETS_TABLE, e as MssqlClient, u as _voltroMssqlCdcOffsetsTable, R as connectionFromConfig, q as ensureChangeTracking, I as makeMssqlSqlLayer, z as makeMssqlSqlLayerFromConfig, Q as makeMssqlStore, le as mssqlDialect, ce as mssqlReplicationAdapter, G as mssqlRetryFilter, X as startChangeTrackingCdc };
+export { c as CDC_OFFSETS_TABLE, e as MssqlClient, d as _voltroMssqlCdcOffsetsTable, L as connectionFromConfig, K as ensureChangeTracking, F as makeMssqlSqlLayer, R as makeMssqlSqlLayerFromConfig, Z as makeMssqlStore, ue as mssqlDialect, $ as mssqlReplicationAdapter, W as mssqlRetryFilter, Y as startChangeTrackingCdc };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@voltro/sql-mssql",
-  "version": "0.43.2",
+  "version": "0.44.1",
   "description": "SQL Server (mssql) dialect adapter for Voltro's cross-dialect DataStore (OUTPUT-clause returning, OFFSET/FETCH paging, inline reactivity).",
   "keywords": [
     "voltro",
@@ -35,8 +35,8 @@
   "dependencies": {
     "@effect/sql": "^0.52.0",
     "@effect/sql-mssql": "^0.53.0",
-    "@voltro/database": "0.43.2",
-    "@voltro/logger": "0.43.2"
+    "@voltro/database": "0.44.1",
+    "@voltro/logger": "0.44.1"
   },
   "peerDependencies": {
     "effect": "^3.22.0"