@memberjunction/metadata-sync 6.1.2 → 6.2.0-edge.0

package/dist/lib/push-outcome.js

@@ -0,0 +1,81 @@
+/**
+ * @fileoverview What a failed `mj sync push` left behind, and how to say so.
+ * @module push-outcome
+ */
+import path from 'path';
+/**
+ * Thrown by `PushService.push()` whenever a push fails after it started writing.
+ * The message is the original failure's message, so existing callers keep working;
+ * the extra fields let a caller say exactly what is and is not in the database.
+ */
+export class PushAbortedError extends Error {
+    constructor(details) {
+        super(messageOf(details.cause), { cause: details.cause });
+        this.name = 'PushAbortedError';
+        this.modes = details.modes;
+        this.rolledBack = details.rolledBack;
+        this.committedWrites = details.committedWrites;
+        this.totals = details.totals;
+        this.sqlLogPath = details.sqlLogPath;
+    }
+    /** True only when nothing from this push is left in the database. */
+    get NothingCommitted() {
+        return this.rolledBack && this.committedWrites.length === 0;
+    }
+}
+function messageOf(cause) {
+    return cause instanceof Error ? cause.message : String(cause);
+}
+/**
+ * The lines that tell the user how a failed push ended. The first line is the summary;
+ * any further lines list the committed records, grouped by file.
+ */
+export function describeRollbackOutcome(rolledBack, committedWrites, cwd) {
+    if (!rolledBack) {
+        return [
+            '❌ Database transaction rollback failed. Check the database before pushing again: ' +
+                'rows written by this push may still be locked or present.',
+            ...describeCommittedWrites(committedWrites, cwd),
+        ];
+    }
+    if (committedWrites.length === 0) {
+        return ['✓ Database transaction rolled back successfully. Nothing from this push was saved.'];
+    }
+    return [
+        `⚠️  The push transaction was rolled back, but ${committedWrites.length} created or updated ` +
+            `record${committedWrites.length === 1 ? '' : 's'} in directories using isolated transactions ` +
+            `${committedWrites.length === 1 ? 'was' : 'were'} already committed and ` +
+            `${committedWrites.length === 1 ? 'is' : 'are'} still in the database. Their files keep the pushed contents.`,
+        ...describeCommittedWrites(committedWrites, cwd),
+    ];
+}
+/** One line per file, then one indented line per committed record. */
+export function describeCommittedWrites(committedWrites, cwd) {
+    const byFile = new Map();
+    for (const write of committedWrites) {
+        const list = byFile.get(write.filePath) ?? [];
+        list.push(write);
+        byFile.set(write.filePath, list);
+    }
+    const lines = [];
+    for (const [filePath, writes] of byFile) {
+        lines.push(`   ${path.relative(cwd, filePath) || filePath}: ${writes.length} committed`);
+        for (const write of writes) {
+            lines.push(`      ${write.status} ${write.entityName} at ${write.recordPath}`);
+        }
+    }
+    return lines;
+}
+/**
+ * Message for a failed COMMIT. On PostgreSQL, deferred foreign keys are checked at commit, so
+ * the failure is about the transaction as a whole and cannot be tied to one record.
+ */
+export function describeCommitFailure(cause, platform) {
+    const base = `The database rejected the commit of the push transaction, so the transaction was not saved: ${messageOf(cause)}`;
+    if (platform === 'postgresql') {
+        return (`${base}. PostgreSQL checks deferred foreign-key constraints when the transaction commits, ` +
+            `so this error is not tied to a single record in the metadata files.`);
+    }
+    return base;
+}
+//# sourceMappingURL=push-outcome.js.map

package/dist/lib/push-outcome.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"push-outcome.js","sourceRoot":"","sources":["../../src/lib/push-outcome.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,IAAI,MAAM,MAAM,CAAC;AAsCxB;;;;GAIG;AACH,MAAM,OAAO,gBAAiB,SAAQ,KAAK;IAQzC,YAAY,OAA2B;QACrC,KAAK,CAAC,SAAS,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,EAAE,KAAK,EAAE,OAAO,CAAC,KAAK,EAAE,CAAC,CAAC;QAC1D,IAAI,CAAC,IAAI,GAAG,kBAAkB,CAAC;QAC/B,IAAI,CAAC,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC;QAC3B,IAAI,CAAC,UAAU,GAAG,OAAO,CAAC,UAAU,CAAC;QACrC,IAAI,CAAC,eAAe,GAAG,OAAO,CAAC,eAAe,CAAC;QAC/C,IAAI,CAAC,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;QAC7B,IAAI,CAAC,UAAU,GAAG,OAAO,CAAC,UAAU,CAAC;IACvC,CAAC;IAED,qEAAqE;IACrE,IAAI,gBAAgB;QAClB,OAAO,IAAI,CAAC,UAAU,IAAI,IAAI,CAAC,eAAe,CAAC,MAAM,KAAK,CAAC,CAAC;IAC9D,CAAC;CACF;AAED,SAAS,SAAS,CAAC,KAAc;IAC/B,OAAO,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;AAChE,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,uBAAuB,CAAC,UAAmB,EAAE,eAAiC,EAAE,GAAW;IACzG,IAAI,CAAC,UAAU,EAAE,CAAC;QAChB,OAAO;YACL,mFAAmF;gBACjF,2DAA2D;YAC7D,GAAG,uBAAuB,CAAC,eAAe,EAAE,GAAG,CAAC;SACjD,CAAC;IACJ,CAAC;IACD,IAAI,eAAe,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACjC,OAAO,CAAC,oFAAoF,CAAC,CAAC;IAChG,CAAC;IACD,OAAO;QACL,iDAAiD,eAAe,CAAC,MAAM,sBAAsB;YAC3F,SAAS,eAAe,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,8CAA8C;YAC9F,GAAG,eAAe,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,yBAAyB;YACzE,GAAG,eAAe,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,KAAK,+DAA+D;QAC/G,GAAG,uBAAuB,CAAC,eAAe,EAAE,GAAG,CAAC;KACjD,CAAC;AACJ,CAAC;AAED,sEAAsE;AACtE,MAAM,UAAU,uBAAuB,CAAC,eAAiC,EAAE,GAAW;IACpF,MAAM,MAAM,GAAG,IAAI,GAAG,EAA4B,CAAC;IACnD,KAAK,MAAM,KAAK,IAAI,eAAe,EAAE,CAAC;QACpC,MAAM,IAAI,GAAG,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAC;QAC9C,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACjB,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC;IACnC,CAAC;IACD,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,KAAK,MAAM,CAAC,QAAQ,EAAE,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;QACxC,KAAK,CAAC,IAAI,CAAC,MAAM,IAAI,CAAC,QAAQ,CAAC,GAAG,EAAE,QAAQ,CAAC,IAAI,QAAQ,KAAK,MAAM,CAAC,MAAM,YAAY,CAAC,CAAC;QACzF,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;YAC3B,KAAK,CAAC,IAAI,CAAC,SAAS,KAAK,CAAC,MAAM,IAAI,KAAK,CAAC,UAAU,OAAO,KAAK,CAAC,UAAU,EAAE,CAAC,CAAC;QACjF,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,qBAAqB,CAAC,KAAc,EAAE,QAA4B;IAChF,MAAM,IAAI,GAAG,+FAA+F,SAAS,CAAC,KAAK,CAAC,EAAE,CAAC;IAC/H,IAAI,QAAQ,KAAK,YAAY,EAAE,CAAC;QAC9B,OAAO,CACL,GAAG,IAAI,qFAAqF;YAC5F,qEAAqE,CACtE,CAAC;IACJ,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC"}

package/dist/lib/push-write-mode.d.ts

@@ -0,0 +1,47 @@
+/**
+ * @fileoverview How `mj sync push` writes the creates and updates of one entity directory.
+ * @module push-write-mode
+ *
+ * - `shared` (default): every save runs on the host provider inside the push transaction, one
+ *   JSON-root graph at a time. A failure anywhere rolls back everything that directory wrote,
+ *   along with every other shared directory in the push.
+ * - `isolated` (`isolatedTransactions: true`, or `--isolated-transactions`): sibling graphs run in
+ *   parallel, each on its own provider instance and therefore its own connection and transaction.
+ *   Each save commits on its own, so a failure does NOT roll those creates and updates back.
+ *
+ * Isolation is chosen **per entity directory**, because entities differ: metadata is simple and
+ * wants atomicity, while an entity that manages its own transaction scopes (an order with journal
+ * entries and payments) may want the parallelism. It cannot be finer than a directory: mixing the
+ * host connection with per-graph connections inside one file is the deadlock the graph pool exists
+ * to prevent.
+ */
+/** How one entity directory's creates and updates are written. */
+export type PushWriteMode = 'shared' | 'isolated';
+/** Default number of JSON-root graphs run at once in an isolated directory. */
+export declare const DEFAULT_PARALLEL_BATCH_SIZE = 10;
+export interface PushWriteModeInput {
+    /** CLI `--isolated-transactions` / `--no-isolated-transactions`. Undefined when not passed. */
+    isolatedFlag?: boolean;
+    /** `push.isolatedTransactions` from this directory's `.mj-sync.json`. */
+    entityIsolated?: boolean;
+    /** `push.isolatedTransactions` from the root `.mj-sync.json`. */
+    rootIsolated?: boolean;
+}
+/** Where a directory's mode came from, for the log line. */
+export type PushWriteModeSource = 'flag' | 'entity' | 'root' | 'default';
+export interface PushDirectoryMode {
+    mode: PushWriteMode;
+    source: PushWriteModeSource;
+}
+/**
+ * The mode for one entity directory. The CLI flag wins over every file, so a run can force either
+ * mode without editing metadata; then the directory's own config, then the root's, then shared.
+ */
+export declare function resolveDirectoryMode(input: PushWriteModeInput): PushDirectoryMode;
+/** Graphs to run at once in a directory of this mode. */
+export declare function graphBatchSizeFor(mode: PushWriteMode, parallelBatchSize?: number): number;
+/** The warning shown once per push that writes any isolated directory to the database. */
+export declare function isolatedModeWarning(directories: string[], graphBatchSize: number): string;
+/** Told once when `--parallel-batch-size` cannot apply, so the flag does not look effective. */
+export declare function unusedBatchSizeWarning(size: number): string;
+//# sourceMappingURL=push-write-mode.d.ts.map

package/dist/lib/push-write-mode.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"push-write-mode.d.ts","sourceRoot":"","sources":["../../src/lib/push-write-mode.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,kEAAkE;AAClE,MAAM,MAAM,aAAa,GAAG,QAAQ,GAAG,UAAU,CAAC;AAElD,+EAA+E;AAC/E,eAAO,MAAM,2BAA2B,KAAK,CAAC;AAE9C,MAAM,WAAW,kBAAkB;IACjC,+FAA+F;IAC/F,YAAY,CAAC,EAAE,OAAO,CAAC;IACvB,yEAAyE;IACzE,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,iEAAiE;IACjE,YAAY,CAAC,EAAE,OAAO,CAAC;CACxB;AAED,4DAA4D;AAC5D,MAAM,MAAM,mBAAmB,GAAG,MAAM,GAAG,QAAQ,GAAG,MAAM,GAAG,SAAS,CAAC;AAEzE,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,aAAa,CAAC;IACpB,MAAM,EAAE,mBAAmB,CAAC;CAC7B;AAED;;;GAGG;AACH,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,kBAAkB,GAAG,iBAAiB,CAWjF;AAED,yDAAyD;AACzD,wBAAgB,iBAAiB,CAAC,IAAI,EAAE,aAAa,EAAE,iBAAiB,CAAC,EAAE,MAAM,GAAG,MAAM,CAEzF;AAED,0FAA0F;AAC1F,wBAAgB,mBAAmB,CAAC,WAAW,EAAE,MAAM,EAAE,EAAE,cAAc,EAAE,MAAM,GAAG,MAAM,CAOzF;AAED,gGAAgG;AAChG,wBAAgB,sBAAsB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAM3D"}

package/dist/lib/push-write-mode.js

@@ -0,0 +1,53 @@
+/**
+ * @fileoverview How `mj sync push` writes the creates and updates of one entity directory.
+ * @module push-write-mode
+ *
+ * - `shared` (default): every save runs on the host provider inside the push transaction, one
+ *   JSON-root graph at a time. A failure anywhere rolls back everything that directory wrote,
+ *   along with every other shared directory in the push.
+ * - `isolated` (`isolatedTransactions: true`, or `--isolated-transactions`): sibling graphs run in
+ *   parallel, each on its own provider instance and therefore its own connection and transaction.
+ *   Each save commits on its own, so a failure does NOT roll those creates and updates back.
+ *
+ * Isolation is chosen **per entity directory**, because entities differ: metadata is simple and
+ * wants atomicity, while an entity that manages its own transaction scopes (an order with journal
+ * entries and payments) may want the parallelism. It cannot be finer than a directory: mixing the
+ * host connection with per-graph connections inside one file is the deadlock the graph pool exists
+ * to prevent.
+ */
+/** Default number of JSON-root graphs run at once in an isolated directory. */
+export const DEFAULT_PARALLEL_BATCH_SIZE = 10;
+/**
+ * The mode for one entity directory. The CLI flag wins over every file, so a run can force either
+ * mode without editing metadata; then the directory's own config, then the root's, then shared.
+ */
+export function resolveDirectoryMode(input) {
+    if (input.isolatedFlag !== undefined) {
+        return { mode: input.isolatedFlag ? 'isolated' : 'shared', source: 'flag' };
+    }
+    if (input.entityIsolated !== undefined) {
+        return { mode: input.entityIsolated ? 'isolated' : 'shared', source: 'entity' };
+    }
+    if (input.rootIsolated !== undefined) {
+        return { mode: input.rootIsolated ? 'isolated' : 'shared', source: 'root' };
+    }
+    return { mode: 'shared', source: 'default' };
+}
+/** Graphs to run at once in a directory of this mode. */
+export function graphBatchSizeFor(mode, parallelBatchSize) {
+    return mode === 'isolated' ? parallelBatchSize ?? DEFAULT_PARALLEL_BATCH_SIZE : 1;
+}
+/** The warning shown once per push that writes any isolated directory to the database. */
+export function isolatedModeWarning(directories, graphBatchSize) {
+    const list = directories.join(', ');
+    return (`Isolated transactions (${graphBatchSize} graphs in parallel) for: ${list}. ` +
+        `Each create and update in those directories is committed as soon as it is saved. If the push fails, ` +
+        `those records stay in the database; only the shared directories, the deletes and the deferred records roll back.`);
+}
+/** Told once when `--parallel-batch-size` cannot apply, so the flag does not look effective. */
+export function unusedBatchSizeWarning(size) {
+    return (`--parallel-batch-size=${size} is ignored: no entity directory in this push uses isolated transactions, ` +
+        `so records are saved one graph at a time in a single transaction. ` +
+        `Set push.isolatedTransactions on an entity's .mj-sync.json, or pass --isolated-transactions, to run graphs in parallel.`);
+}
+//# sourceMappingURL=push-write-mode.js.map

package/dist/lib/push-write-mode.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"push-write-mode.js","sourceRoot":"","sources":["../../src/lib/push-write-mode.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAKH,+EAA+E;AAC/E,MAAM,CAAC,MAAM,2BAA2B,GAAG,EAAE,CAAC;AAmB9C;;;GAGG;AACH,MAAM,UAAU,oBAAoB,CAAC,KAAyB;IAC5D,IAAI,KAAK,CAAC,YAAY,KAAK,SAAS,EAAE,CAAC;QACrC,OAAO,EAAE,IAAI,EAAE,KAAK,CAAC,YAAY,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;IAC9E,CAAC;IACD,IAAI,KAAK,CAAC,cAAc,KAAK,SAAS,EAAE,CAAC;QACvC,OAAO,EAAE,IAAI,EAAE,KAAK,CAAC,cAAc,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,QAAQ,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC;IAClF,CAAC;IACD,IAAI,KAAK,CAAC,YAAY,KAAK,SAAS,EAAE,CAAC;QACrC,OAAO,EAAE,IAAI,EAAE,KAAK,CAAC,YAAY,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;IAC9E,CAAC;IACD,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC;AAC/C,CAAC;AAED,yDAAyD;AACzD,MAAM,UAAU,iBAAiB,CAAC,IAAmB,EAAE,iBAA0B;IAC/E,OAAO,IAAI,KAAK,UAAU,CAAC,CAAC,CAAC,iBAAiB,IAAI,2BAA2B,CAAC,CAAC,CAAC,CAAC,CAAC;AACpF,CAAC;AAED,0FAA0F;AAC1F,MAAM,UAAU,mBAAmB,CAAC,WAAqB,EAAE,cAAsB;IAC/E,MAAM,IAAI,GAAG,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACpC,OAAO,CACL,0BAA0B,cAAc,6BAA6B,IAAI,IAAI;QAC7E,sGAAsG;QACtG,kHAAkH,CACnH,CAAC;AACJ,CAAC;AAED,gGAAgG;AAChG,MAAM,UAAU,sBAAsB,CAAC,IAAY;IACjD,OAAO,CACL,yBAAyB,IAAI,4EAA4E;QACzG,oEAAoE;QACpE,yHAAyH,CAC1H,CAAC;AACJ,CAAC"}

package/dist/lib/record-primary-key.d.ts

@@ -0,0 +1,35 @@
+/**
+ * Primary-key extraction for pulled records.
+ *
+ * Pull must key every database record by its real primary-key value: the key decides whether a
+ * record updates an existing file entry or is appended as a new one. Reading the key through the
+ * entity's typed property (`record.ID`) only works when the entity's generated subclass is
+ * registered in this process. When it is not — an Open App whose server package did not load, a
+ * host's generated entities package missing from the CLI — the ClassFactory hands back a bare
+ * `BaseEntity`, which has no typed properties, so every record read `undefined`. All records then
+ * shared one key, each overwrote the last in the write batch, and a pull of N new records wrote
+ * exactly one, with an empty `primaryKey` that duplicated on the next pull (#3415).
+ *
+ * `BaseEntity.Get()` is metadata-driven and works on a bare `BaseEntity` as well as on a subclass.
+ */
+import type { BaseEntity, EntityInfo } from '@memberjunction/core';
+import type { RecordData } from './sync-engine.js';
+/** Primary-key values keyed by field name — the shape `RecordData.primaryKey` carries. */
+export type PrimaryKeyValues = NonNullable<RecordData['primaryKey']>;
+/**
+ * Reads every primary-key field of `record` through `BaseEntity.Get()`.
+ *
+ * @throws when a key field has no value. A record without its key cannot be matched to a file
+ *         entry and would overwrite other records, so refusing is the only safe outcome.
+ */
+export declare function extractPrimaryKeyValues(record: BaseEntity, entityInfo: EntityInfo): PrimaryKeyValues;
+/** True when `primaryKey` has at least one field and every field holds a value. */
+export declare function hasCompletePrimaryKey(primaryKey: RecordData['primaryKey']): boolean;
+/**
+ * Builds the string pull uses to match a database record to a file entry: `field:value` segments,
+ * sorted by field name, joined by `|`. `\` and `|` inside a value are escaped, so a value that
+ * contains the separator can't make two different keys produce the same string. The string is only
+ * ever compared in memory — never parsed or persisted.
+ */
+export declare function createPrimaryKeyLookup(primaryKey: RecordData['primaryKey']): string;
+//# sourceMappingURL=record-primary-key.d.ts.map

package/dist/lib/record-primary-key.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"record-primary-key.d.ts","sourceRoot":"","sources":["../../src/lib/record-primary-key.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AACH,OAAO,KAAK,EAAE,UAAU,EAAE,UAAU,EAAE,MAAM,sBAAsB,CAAC;AACnE,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAEhD,0FAA0F;AAC1F,MAAM,MAAM,gBAAgB,GAAG,WAAW,CAAC,UAAU,CAAC,YAAY,CAAC,CAAC,CAAC;AAErE;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,MAAM,EAAE,UAAU,EAAE,UAAU,EAAE,UAAU,GAAG,gBAAgB,CAapG;AAUD,mFAAmF;AACnF,wBAAgB,qBAAqB,CAAC,UAAU,EAAE,UAAU,CAAC,YAAY,CAAC,GAAG,OAAO,CAGnF;AAED;;;;;GAKG;AACH,wBAAgB,sBAAsB,CAAC,UAAU,EAAE,UAAU,CAAC,YAAY,CAAC,GAAG,MAAM,CAMnF"}

package/dist/lib/record-primary-key.js

@@ -0,0 +1,44 @@
+/**
+ * Reads every primary-key field of `record` through `BaseEntity.Get()`.
+ *
+ * @throws when a key field has no value. A record without its key cannot be matched to a file
+ *         entry and would overwrite other records, so refusing is the only safe outcome.
+ */
+export function extractPrimaryKeyValues(record, entityInfo) {
+    const values = {};
+    for (const pk of entityInfo.PrimaryKeys) {
+        const value = record.Get(pk.Name);
+        if (!isPresentKeyValue(value)) {
+            throw new Error(`Cannot read primary key field '${pk.Name}' on a '${entityInfo.Name}' record. ` +
+                `Refusing to write it: a record without its key would overwrite other records in the same file.`);
+        }
+        values[pk.Name] = value;
+    }
+    return values;
+}
+/**
+ * True when a key field holds a value. Only a missing value counts as absent: an empty string, or
+ * the text `null`, is a real key value.
+ */
+function isPresentKeyValue(value) {
+    return value !== undefined && value !== null;
+}
+/** True when `primaryKey` has at least one field and every field holds a value. */
+export function hasCompletePrimaryKey(primaryKey) {
+    const values = Object.values(primaryKey ?? {});
+    return values.length > 0 && values.every(isPresentKeyValue);
+}
+/**
+ * Builds the string pull uses to match a database record to a file entry: `field:value` segments,
+ * sorted by field name, joined by `|`. `\` and `|` inside a value are escaped, so a value that
+ * contains the separator can't make two different keys produce the same string. The string is only
+ * ever compared in memory — never parsed or persisted.
+ */
+export function createPrimaryKeyLookup(primaryKey) {
+    const values = primaryKey ?? {};
+    return Object.keys(values)
+        .sort()
+        .map((field) => `${field}:${String(values[field]).replace(/[\\|]/g, '\\$&')}`)
+        .join('|');
+}
+//# sourceMappingURL=record-primary-key.js.map

package/dist/lib/record-primary-key.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"record-primary-key.js","sourceRoot":"","sources":["../../src/lib/record-primary-key.ts"],"names":[],"mappings":"AAoBA;;;;;GAKG;AACH,MAAM,UAAU,uBAAuB,CAAC,MAAkB,EAAE,UAAsB;IAChF,MAAM,MAAM,GAAqB,EAAE,CAAC;IACpC,KAAK,MAAM,EAAE,IAAI,UAAU,CAAC,WAAW,EAAE,CAAC;QACxC,MAAM,KAAK,GAAG,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC;QAClC,IAAI,CAAC,iBAAiB,CAAC,KAAK,CAAC,EAAE,CAAC;YAC9B,MAAM,IAAI,KAAK,CACb,kCAAkC,EAAE,CAAC,IAAI,WAAW,UAAU,CAAC,IAAI,YAAY;gBAC7E,gGAAgG,CACnG,CAAC;QACJ,CAAC;QACD,MAAM,CAAC,EAAE,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC;IAC1B,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC;AAED;;;GAGG;AACH,SAAS,iBAAiB,CAAC,KAA+B;IACxD,OAAO,KAAK,KAAK,SAAS,IAAI,KAAK,KAAK,IAAI,CAAC;AAC/C,CAAC;AAED,mFAAmF;AACnF,MAAM,UAAU,qBAAqB,CAAC,UAAoC;IACxE,MAAM,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,UAAU,IAAI,EAAE,CAAC,CAAC;IAC/C,OAAO,MAAM,CAAC,MAAM,GAAG,CAAC,IAAI,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,CAAC;AAC9D,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,sBAAsB,CAAC,UAAoC;IACzE,MAAM,MAAM,GAAG,UAAU,IAAI,EAAE,CAAC;IAChC,OAAO,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC;SACvB,IAAI,EAAE;SACN,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,GAAG,KAAK,IAAI,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC,QAAQ,EAAE,MAAM,CAAC,EAAE,CAAC;SAC7E,IAAI,CAAC,GAAG,CAAC,CAAC;AACf,CAAC"}

package/dist/lib/transaction-manager.d.ts

@@ -5,13 +5,23 @@
  * This module provides transaction support for push operations to ensure
  * all related entities are saved atomically with rollback capability.
  */
+import type { DatabaseProviderBase } from '@memberjunction/core';
 import { SQLLogger } from './sql-logger.js';
 export interface TransactionOptions {
 }
 export declare class TransactionManager {
     private inTransaction;
     private sqlLogger?;
-    constructor(sqlLogger?: SQLLogger);
+    private readonly explicitProvider?;
+    /**
+     * @param sqlLogger - optional SQL logger
+     * @param provider - the provider that owns the transaction. Pass it whenever the caller runs
+     *   work on a specific provider (an atomic `mj sync push` runs every save on it), so the
+     *   transaction and the work cannot end up on different instances. Defaults to the provider
+     *   `initializeProvider()` set up.
+     */
+    constructor(sqlLogger?: SQLLogger, provider?: DatabaseProviderBase);
+    private resolveProvider;
     /**
      * Begin a new transaction.
      *

package/dist/lib/transaction-manager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"transaction-manager.d.ts","sourceRoot":"","sources":["../../src/lib/transaction-manager.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAGH,OAAO,EAAE,SAAS,EAAE,MAAM,cAAc,CAAC;AAEzC,MAAM,WAAW,kBAAkB;CAClC;AAED,qBAAa,kBAAkB;IAC7B,OAAO,CAAC,aAAa,CAAS;IAC9B,OAAO,CAAC,SAAS,CAAC,CAAY;gBAElB,SAAS,CAAC,EAAE,SAAS;IAIjC;;;;;;;;;;;;;;;;OAgBG;IACG,gBAAgB,CAAC,OAAO,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,IAAI,CAAC;IAkBnE;;OAEG;IACG,iBAAiB,IAAI,OAAO,CAAC,IAAI,CAAC;IAkBxC;;;OAGG;IACG,mBAAmB,IAAI,OAAO,CAAC,OAAO,CAAC;IAqB7C;;OAEG;IACG,oBAAoB,CAAC,CAAC,EAC1B,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,EACpB,OAAO,CAAC,EAAE,kBAAkB,GAC3B,OAAO,CAAC,CAAC,CAAC;IAab;;OAEG;IACH,IAAI,eAAe,IAAI,OAAO,CAE7B;CACF"}
+{"version":3,"file":"transaction-manager.d.ts","sourceRoot":"","sources":["../../src/lib/transaction-manager.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAEjE,OAAO,EAAE,SAAS,EAAE,MAAM,cAAc,CAAC;AAEzC,MAAM,WAAW,kBAAkB;CAClC;AAED,qBAAa,kBAAkB;IAC7B,OAAO,CAAC,aAAa,CAAS;IAC9B,OAAO,CAAC,SAAS,CAAC,CAAY;IAC9B,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAC,CAAuB;IAEzD;;;;;;OAMG;gBACS,SAAS,CAAC,EAAE,SAAS,EAAE,QAAQ,CAAC,EAAE,oBAAoB;IAKlE,OAAO,CAAC,eAAe;IAIvB;;;;;;;;;;;;;;;;OAgBG;IACG,gBAAgB,CAAC,OAAO,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,IAAI,CAAC;IAkBnE;;OAEG;IACG,iBAAiB,IAAI,OAAO,CAAC,IAAI,CAAC;IAkBxC;;;OAGG;IACG,mBAAmB,IAAI,OAAO,CAAC,OAAO,CAAC;IA0B7C;;OAEG;IACG,oBAAoB,CAAC,CAAC,EAC1B,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,EACpB,OAAO,CAAC,EAAE,kBAAkB,GAC3B,OAAO,CAAC,CAAC,CAAC;IAab;;OAEG;IACH,IAAI,eAAe,IAAI,OAAO,CAE7B;CACF"}

package/dist/lib/transaction-manager.js

@@ -7,9 +7,20 @@
  */
 import { getDataProvider } from './provider-utils.js';
 export class TransactionManager {
-    constructor(sqlLogger) {
+    /**
+     * @param sqlLogger - optional SQL logger
+     * @param provider - the provider that owns the transaction. Pass it whenever the caller runs
+     *   work on a specific provider (an atomic `mj sync push` runs every save on it), so the
+     *   transaction and the work cannot end up on different instances. Defaults to the provider
+     *   `initializeProvider()` set up.
+     */
+    constructor(sqlLogger, provider) {
         this.inTransaction = false;
         this.sqlLogger = sqlLogger;
+        this.explicitProvider = provider;
+    }
+    resolveProvider() {
+        return this.explicitProvider ?? getDataProvider();
     }
     /**
      * Begin a new transaction.
@@ -32,7 +43,7 @@ export class TransactionManager {
         if (this.inTransaction) {
             throw new Error('Transaction already in progress');
         }
-        const provider = getDataProvider();
+        const provider = this.resolveProvider();
         if (!provider) {
             throw new Error('No data provider available');
         }
@@ -51,7 +62,7 @@ export class TransactionManager {
         if (!this.inTransaction) {
             return; // No transaction to commit
         }
-        const provider = getDataProvider();
+        const provider = this.resolveProvider();
         if (!provider) {
             throw new Error('No data provider available');
         }
@@ -71,7 +82,7 @@ export class TransactionManager {
         if (!this.inTransaction) {
             return true; // No transaction to rollback
         }
-        const provider = getDataProvider();
+        const provider = this.resolveProvider();
         if (!provider) {
             throw new Error('No data provider available');
         }
@@ -83,6 +94,11 @@ export class TransactionManager {
         catch (error) {
             // Log but don't throw - we're already in an error state
             console.error('Failed to rollback transaction:', error);
+            if (provider.TransactionDepth === 0) {
+                // The provider already dropped its handle, so there is nothing left to retry. The rollback
+                // is still reported as failed: the server-side transaction was not confirmed as rolled back.
+                this.inTransaction = false;
+            }
             return false;
         }
     }

package/dist/lib/transaction-manager.js.map

@@ -1 +1 @@
-{"version":3,"file":"transaction-manager.js","sourceRoot":"","sources":["../../src/lib/transaction-manager.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAC;AAMnD,MAAM,OAAO,kBAAkB;IAI7B,YAAY,SAAqB;QAHzB,kBAAa,GAAG,KAAK,CAAC;QAI5B,IAAI,CAAC,SAAS,GAAG,SAAS,CAAC;IAC7B,CAAC;IAED;;;;;;;;;;;;;;;;OAgBG;IACH,KAAK,CAAC,gBAAgB,CAAC,OAA4B;QACjD,IAAI,IAAI,CAAC,aAAa,EAAE,CAAC;YACvB,MAAM,IAAI,KAAK,CAAC,iCAAiC,CAAC,CAAC;QACrD,CAAC;QAED,MAAM,QAAQ,GAAG,eAAe,EAAE,CAAC;QACnC,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,MAAM,IAAI,KAAK,CAAC,4BAA4B,CAAC,CAAC;QAChD,CAAC;QAED,IAAI,CAAC;YACH,MAAM,QAAQ,CAAC,gBAAgB,EAAE,CAAC;YAClC,IAAI,CAAC,aAAa,GAAG,IAAI,CAAC;QAC5B,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,IAAI,KAAK,CAAC,gCAAgC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QAC5G,CAAC;IACH,CAAC;IAED;;OAEG;IACH,KAAK,CAAC,iBAAiB;QACrB,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,CAAC;YACxB,OAAO,CAAC,2BAA2B;QACrC,CAAC;QAED,MAAM,QAAQ,GAAG,eAAe,EAAE,CAAC;QACnC,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,MAAM,IAAI,KAAK,CAAC,4BAA4B,CAAC,CAAC;QAChD,CAAC;QAED,IAAI,CAAC;YACH,MAAM,QAAQ,CAAC,iBAAiB,EAAE,CAAC;YACnC,IAAI,CAAC,aAAa,GAAG,KAAK,CAAC;QAC7B,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,IAAI,KAAK,CAAC,iCAAiC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QAC7G,CAAC;IACH,CAAC;IAED;;;OAGG;IACH,KAAK,CAAC,mBAAmB;QACvB,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,CAAC;YACxB,OAAO,IAAI,CAAC,CAAC,6BAA6B;QAC5C,CAAC;QAED,MAAM,QAAQ,GAAG,eAAe,EAAE,CAAC;QACnC,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,MAAM,IAAI,KAAK,CAAC,4BAA4B,CAAC,CAAC;QAChD,CAAC;QAED,IAAI,CAAC;YACH,MAAM,QAAQ,CAAC,mBAAmB,EAAE,CAAC;YACrC,IAAI,CAAC,aAAa,GAAG,KAAK,CAAC;YAC3B,OAAO,IAAI,CAAC;QACd,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,wDAAwD;YACxD,OAAO,CAAC,KAAK,CAAC,iCAAiC,EAAE,KAAK,CAAC,CAAC;YACxD,OAAO,KAAK,CAAC;QACf,CAAC;IACH,CAAC;IAED;;OAEG;IACH,KAAK,CAAC,oBAAoB,CACxB,EAAoB,EACpB,OAA4B;QAE5B,MAAM,IAAI,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC;QAErC,IAAI,CAAC;YACH,MAAM,MAAM,GAAG,MAAM,EAAE,EAAE,CAAC;YAC1B,MAAM,IAAI,CAAC,iBAAiB,EAAE,CAAC;YAC/B,OAAO,MAAM,CAAC;QAChB,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,IAAI,CAAC,mBAAmB,EAAE,CAAC;YACjC,MAAM,KAAK,CAAC;QACd,CAAC;IACH,CAAC;IAED;;OAEG;IACH,IAAI,eAAe;QACjB,OAAO,IAAI,CAAC,aAAa,CAAC;IAC5B,CAAC;CACF"}
+{"version":3,"file":"transaction-manager.js","sourceRoot":"","sources":["../../src/lib/transaction-manager.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAGH,OAAO,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAC;AAMnD,MAAM,OAAO,kBAAkB;IAK7B;;;;;;OAMG;IACH,YAAY,SAAqB,EAAE,QAA+B;QAX1D,kBAAa,GAAG,KAAK,CAAC;QAY5B,IAAI,CAAC,SAAS,GAAG,SAAS,CAAC;QAC3B,IAAI,CAAC,gBAAgB,GAAG,QAAQ,CAAC;IACnC,CAAC;IAEO,eAAe;QACrB,OAAO,IAAI,CAAC,gBAAgB,IAAI,eAAe,EAAE,CAAC;IACpD,CAAC;IAED;;;;;;;;;;;;;;;;OAgBG;IACH,KAAK,CAAC,gBAAgB,CAAC,OAA4B;QACjD,IAAI,IAAI,CAAC,aAAa,EAAE,CAAC;YACvB,MAAM,IAAI,KAAK,CAAC,iCAAiC,CAAC,CAAC;QACrD,CAAC;QAED,MAAM,QAAQ,GAAG,IAAI,CAAC,eAAe,EAAE,CAAC;QACxC,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,MAAM,IAAI,KAAK,CAAC,4BAA4B,CAAC,CAAC;QAChD,CAAC;QAED,IAAI,CAAC;YACH,MAAM,QAAQ,CAAC,gBAAgB,EAAE,CAAC;YAClC,IAAI,CAAC,aAAa,GAAG,IAAI,CAAC;QAC5B,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,IAAI,KAAK,CAAC,gCAAgC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QAC5G,CAAC;IACH,CAAC;IAED;;OAEG;IACH,KAAK,CAAC,iBAAiB;QACrB,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,CAAC;YACxB,OAAO,CAAC,2BAA2B;QACrC,CAAC;QAED,MAAM,QAAQ,GAAG,IAAI,CAAC,eAAe,EAAE,CAAC;QACxC,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,MAAM,IAAI,KAAK,CAAC,4BAA4B,CAAC,CAAC;QAChD,CAAC;QAED,IAAI,CAAC;YACH,MAAM,QAAQ,CAAC,iBAAiB,EAAE,CAAC;YACnC,IAAI,CAAC,aAAa,GAAG,KAAK,CAAC;QAC7B,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,IAAI,KAAK,CAAC,iCAAiC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QAC7G,CAAC;IACH,CAAC;IAED;;;OAGG;IACH,KAAK,CAAC,mBAAmB;QACvB,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,CAAC;YACxB,OAAO,IAAI,CAAC,CAAC,6BAA6B;QAC5C,CAAC;QAED,MAAM,QAAQ,GAAG,IAAI,CAAC,eAAe,EAAE,CAAC;QACxC,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,MAAM,IAAI,KAAK,CAAC,4BAA4B,CAAC,CAAC;QAChD,CAAC;QAED,IAAI,CAAC;YACH,MAAM,QAAQ,CAAC,mBAAmB,EAAE,CAAC;YACrC,IAAI,CAAC,aAAa,GAAG,KAAK,CAAC;YAC3B,OAAO,IAAI,CAAC;QACd,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,wDAAwD;YACxD,OAAO,CAAC,KAAK,CAAC,iCAAiC,EAAE,KAAK,CAAC,CAAC;YACxD,IAAI,QAAQ,CAAC,gBAAgB,KAAK,CAAC,EAAE,CAAC;gBACpC,2FAA2F;gBAC3F,6FAA6F;gBAC7F,IAAI,CAAC,aAAa,GAAG,KAAK,CAAC;YAC7B,CAAC;YACD,OAAO,KAAK,CAAC;QACf,CAAC;IACH,CAAC;IAED;;OAEG;IACH,KAAK,CAAC,oBAAoB,CACxB,EAAoB,EACpB,OAA4B;QAE5B,MAAM,IAAI,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC;QAErC,IAAI,CAAC;YACH,MAAM,MAAM,GAAG,MAAM,EAAE,EAAE,CAAC;YAC1B,MAAM,IAAI,CAAC,iBAAiB,EAAE,CAAC;YAC/B,OAAO,MAAM,CAAC;QAChB,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,IAAI,CAAC,mBAAmB,EAAE,CAAC;YACjC,MAAM,KAAK,CAAC;QACd,CAAC;IACH,CAAC;IAED;;OAEG;IACH,IAAI,eAAe;QACjB,OAAO,IAAI,CAAC,aAAa,CAAC;IAC5B,CAAC;CACF"}

package/dist/plugins/index.d.ts

@@ -24,6 +24,7 @@ export declare class SyncPushPlugin extends BaseCLIPlugin {
         ci: Interfaces.BooleanFlag<boolean>;
         'no-validate': Interfaces.BooleanFlag<boolean>;
         'delete-db-only': Interfaces.BooleanFlag<boolean>;
+        'isolated-transactions': Interfaces.BooleanFlag<boolean>;
         'parallel-batch-size': Interfaces.OptionFlag<number, Interfaces.CustomOptions>;
         include: Interfaces.OptionFlag<string, Interfaces.CustomOptions>;
         exclude: Interfaces.OptionFlag<string, Interfaces.CustomOptions>;

package/dist/plugins/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/plugins/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AACH,OAAO,EAAS,KAAK,UAAU,EAAE,MAAM,aAAa,CAAC;AAGrD,OAAO,EAAE,aAAa,EAAE,KAAK,WAAW,EAAyB,KAAK,WAAW,EAAE,MAAM,0BAA0B,CAAC;AAqCpH,qBACa,cAAe,SAAQ,aAAa;IAC/C,MAAM,CAAC,WAAW,SAA6C;IAE/D,MAAM,CAAC,QAAQ,WAKb;IAEF,MAAM,CAAC,KAAK;;;;;;;;;;;MAyBV;IAEF,MAAM,CAAC,KAAK,EAAE,WAAW,CAgBvB;cAEc,OAAO,IAAI,OAAO,CAAC,WAAW,CAAC;IAuK/C,0FAA0F;IAC1F,OAAO,CAAC,qBAAqB;IAqD7B,OAAO,CAAC,IAAI;IAIZ,OAAO,CAAC,SAAS;IAKjB;;;;OAIG;cACa,OAAO,CAAC,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC;CAU5D;AAMD,qBACa,cAAe,SAAQ,aAAa;IAC/C,MAAM,CAAC,WAAW,SAAgD;IAElE,MAAM,CAAC,QAAQ,WAIb;IAEF,MAAM,CAAC,KAAK;;;;;;;;;;;;;;;;MAgBV;IAEF,MAAM,CAAC,KAAK,EAAE,WAAW,CAevB;cAEc,OAAO,IAAI,OAAO,CAAC,WAAW,CAAC;YAwJjC,qBAAqB;IAoBnC,OAAO,CAAC,iCAAiC;cAezB,OAAO,CAAC,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC;CAU5D;AAED,gFAAgF;AAChF,wBAAgB,uBAAuB,IAAI,IAAI,CAE9C"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/plugins/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AACH,OAAO,EAAS,KAAK,UAAU,EAAE,MAAM,aAAa,CAAC;AAGrD,OAAO,EAAE,aAAa,EAAE,KAAK,WAAW,EAAyB,KAAK,WAAW,EAAE,MAAM,0BAA0B,CAAC;AAgEpH,qBACa,cAAe,SAAQ,aAAa;IAC/C,MAAM,CAAC,WAAW,SAA6C;IAE/D,MAAM,CAAC,QAAQ,WAKb;IAEF,MAAM,CAAC,KAAK;;;;;;;;;;;;MAgCV;IAEF,MAAM,CAAC,KAAK,EAAE,WAAW,CAkBvB;cAEc,OAAO,IAAI,OAAO,CAAC,WAAW,CAAC;IAkK/C,0FAA0F;IAC1F,OAAO,CAAC,qBAAqB;IAqD7B,OAAO,CAAC,IAAI;IAIZ,OAAO,CAAC,SAAS;IAKjB;;;;OAIG;cACa,OAAO,CAAC,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC;CAU5D;AAMD,qBACa,cAAe,SAAQ,aAAa;IAC/C,MAAM,CAAC,WAAW,SAAgD;IAElE,MAAM,CAAC,QAAQ,WAIb;IAEF,MAAM,CAAC,KAAK;;;;;;;;;;;;;;;;MAgBV;IAEF,MAAM,CAAC,KAAK,EAAE,WAAW,CAevB;cAEc,OAAO,IAAI,OAAO,CAAC,WAAW,CAAC;YAwJjC,qBAAqB;IAoBnC,OAAO,CAAC,iCAAiC;cAezB,OAAO,CAAC,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC;CAU5D;AAED,gFAAgF;AAChF,wBAAgB,uBAAuB,IAAI,IAAI,CAE9C"}

package/dist/plugins/index.js

@@ -26,6 +26,8 @@ import { BaseCLIPlugin } from '@memberjunction/cli-core';
 import fs from 'fs';
 import path from 'path';
 import { writeFileSync } from 'fs';
+// PushAbortedError lives in a dependency-free module, so importing it does not load the engine.
+import { PushAbortedError } from '../lib/push-outcome.js';
 /** Shared loader for the heavy sync engine. Dynamic so the plugin module stays light. */
 async function loadEngine() {
     return import('../index.js');
@@ -43,6 +45,29 @@ function flushAndExit(code) {
     hardExit.unref();
     process.stdout.write('', () => process.exit(code));
 }
+/**
+ * The `data` block for a failed push. `PushAbortedError` carries the counts the push reached and
+ * the SQL log path; anything else failed before the push started and has nothing to report.
+ */
+function failureData(error, dryRun) {
+    if (!(error instanceof PushAbortedError)) {
+        return undefined;
+    }
+    const { totals } = error;
+    return {
+        created: totals.created,
+        updated: totals.updated,
+        unchanged: totals.unchanged,
+        deleted: totals.deleted,
+        skipped: totals.skipped,
+        deferred: totals.deferred,
+        errorCount: totals.errors,
+        dryRun,
+        sqlLogPath: error.sqlLogPath,
+        rolledBack: error.rolledBack,
+        committedOutsideTransaction: error.committedWrites.length,
+    };
+}
 // ─────────────────────────────────────────────────────────────────────────────
 // mj sync push
 // ─────────────────────────────────────────────────────────────────────────────
@@ -63,9 +88,15 @@ let SyncPushPlugin = class SyncPushPlugin extends BaseCLIPlugin {
             description: 'Delete database-only records that reference records being deleted (prevents FK errors)',
             default: false,
         }),
+        'isolated-transactions': Flags.boolean({
+            description: 'Give every JSON-root graph its own connection and transaction, so siblings are written in parallel and each ' +
+                'create/update commits as it is saved (a failure does not roll those back). Off by default: the whole push runs ' +
+                'in one transaction, one graph at a time. Overrides push.isolatedTransactions in every .mj-sync.json, in both ' +
+                'directions — use --no-isolated-transactions to force one run back to a single transaction',
+            allowNo: true,
+        }),
         'parallel-batch-size': Flags.integer({
-            description: 'JSON-root graphs to process in parallel (default: 10, not 1)',
-            default: 10,
+            description: 'JSON-root graphs to process in parallel in a directory using isolated transactions (default: 10)',
             min: 1,
             max: 50,
         }),
@@ -91,6 +122,8 @@ let SyncPushPlugin = class SyncPushPlugin extends BaseCLIPlugin {
             { name: '--ci', type: 'boolean', description: 'No prompts; non-zero exit on error' },
             { name: '--no-validate', type: 'boolean', description: 'Skip pre-push validation' },
             { name: '--incremental', type: 'boolean', description: 'Skip unchanged files using stored checksums' },
+            { name: '--isolated-transactions', type: 'boolean', description: 'Per-graph connections; creates/updates commit as they go and are not rolled back' },
+            { name: '--parallel-batch-size', type: 'number', description: 'Graphs at once in an isolated directory (default 10)' },
             { name: '--format', type: 'text|json|md', description: 'Output format (json for machine-readable result)' },
         ],
         examples: ['mj sync push --dir=ai-agents', 'mj sync push --ci --format=json'],
@@ -164,6 +197,7 @@ let SyncPushPlugin = class SyncPushPlugin extends BaseCLIPlugin {
                 noValidate: flags['no-validate'],
                 deleteDbOnly: flags['delete-db-only'],
                 parallelBatchSize: flags['parallel-batch-size'],
+                isolatedTransactions: flags['isolated-transactions'],
                 include: includeFilter,
                 exclude: excludeFilter,
                 incremental: flags.incremental,
@@ -198,20 +232,9 @@ let SyncPushPlugin = class SyncPushPlugin extends BaseCLIPlugin {
             for (const w of result.warnings)
                 if (!warnings.includes(w))
                     warnings.push(w);
-            // Recovery decision: with non-fatal errors, the original CLI asked whether to
-            // keep the successfully-committed changes. Preserve that for interactive text
-            // mode. CI / non-interactive keeps the failure (no prompt possible).
-            let success = result.errors === 0 && errors.length === 0;
-            if (!success && result.errors > 0 && !flags.ci && isText) {
-                const commit = await confirm({
-                    message: 'Push completed with errors. Do you want to commit the successful changes?',
-                    default: false,
-                });
-                if (commit)
-                    success = true;
-                else
-                    warnings.push('Push cancelled due to errors.');
-            }
+            // A push that fails rolls back and throws (see the catch below), so a result with
+            // errors only comes from a dry run. There is nothing to commit, so no prompt.
+            const success = result.errors === 0 && errors.length === 0;
             if (isText) {
                 this.renderPushTextSummary(formatter, result, flags['change-detail'], startTime, endTime);
             }
@@ -246,7 +269,10 @@ let SyncPushPlugin = class SyncPushPlugin extends BaseCLIPlugin {
             this.Host.FailStep('Push failed', message);
             if (errors.length === 0)
                 errors.push({ message });
-            return { success: false, command: 'sync:push', durationSeconds: (Date.now() - startTime) / 1000, errors, warnings };
+            // A failed push still reports what it did. Without this the JSON consumer loses the counts
+            // and the SQL log path on exactly the runs that need them.
+            const data = failureData(error, flags['dry-run'] === true);
+            return { success: false, command: 'sync:push', durationSeconds: (Date.now() - startTime) / 1000, data, errors, warnings };
         }
     }
     /** Reproduces the original rich text summary (recap, change-detail file, box, status). */