@plusscommunities/pluss-core-aws 2.0.25-beta.7 → 2.0.25-beta.8

package/helper/hqPublishing.js

@@ -144,6 +144,23 @@ const fanOutHqSource = async (hqSourceRow, tableName) => {
 	if (targets.length === 0) {
 		return [];
 	}
+	return writeCopiesForTargets(hqSourceRow, targets, tableName);
+};
+
+/**
+ * Write one community-copy row per site in `targetSites` (all of the source's
+ * PublishedToSites on INSERT fan-out, or just the newly-added subset on a
+ * MODIFY target-set change). Shared by fanOutHqSource and syncHqTargets. The
+ * deterministic RowId makes each write idempotent AND makes re-adding a
+ * previously-removed (soft-deleted) site a lossless resurrection — the PUT
+ * overwrites the row and drops the Deleted flag.
+ *
+ * @param {object} hqSourceRow — the HQ-source row.
+ * @param {string[]} targetSites — sites to write copies for.
+ * @param {string} tableName — entity's table name.
+ * @returns {Promise<object[]>} the copy rows written.
+ */
+const writeCopiesForTargets = (hqSourceRow, targetSites, tableName) => {
 	const copyId = `hqcopy-${hqSourceRow.Id}`;
 	// Preserve the HQ-source row's own UnixTimestamp on each copy rather than
 	// stamping "now". For an immediate publish the source timestamp already IS
@@ -163,7 +180,7 @@ const fanOutHqSource = async (hqSourceRow, tableName) => {
 	const now = moment.utc().valueOf();
 	const publishTs = hqSourceRow.UnixTimestamp || now;
 
-	const writes = targets.map((targetSiteId) => {
+	const writes = targetSites.map((targetSiteId) => {
 		// Spread + override pattern: keep all content fields, swap identity
 		// fields, drop HQ-only fields. Spread is shallow — nested objects
 		// (Author, etc.) share references with the source row, which is fine
@@ -192,6 +209,63 @@ const fanOutHqSource = async (hqSourceRow, tableName) => {
 	return Promise.all(writes);
 };
 
+/**
+ * Reconcile community copies when an HQ-source MODIFY changes PublishedToSites:
+ * fan out copies for newly-ADDED sites, soft-delete copies for REMOVED sites.
+ * Content propagation to surviving copies is propagateHqEdit's job; this only
+ * reconciles the target SET. Re-adding a removed site resurrects its copy
+ * losslessly (deterministic RowId + PUT — see writeCopiesForTargets).
+ *
+ * MUST run sequentially relative to propagateHqEdit (never concurrently): both
+ * use editRef (get-merge-put), and two concurrent writes to the same removed-
+ * copy RowId would race (one clobbering the other's Deleted flag).
+ *
+ * @param {object} prevRow — OldImage of the HQ-source row.
+ * @param {object} nextRow — NewImage of the HQ-source row.
+ * @param {string} tableName — entity's table name.
+ * @param {object} [options]
+ * @param {string} [options.indexName="NewsletterEntriesHqSourceIdIndex"]
+ * @returns {Promise<{added: number, removed: number}>}
+ */
+const syncHqTargets = async (prevRow, nextRow, tableName, options = {}) => {
+	const indexName = options.indexName || "NewsletterEntriesHqSourceIdIndex";
+	if (!nextRow || nextRow.Site !== "hq" || nextRow.Deleted) {
+		return { added: 0, removed: 0 };
+	}
+	const prev = Array.isArray(prevRow?.PublishedToSites)
+		? prevRow.PublishedToSites
+		: [];
+	const next = Array.isArray(nextRow.PublishedToSites)
+		? nextRow.PublishedToSites
+		: [];
+	const added = next.filter((s) => !prev.includes(s));
+	const removed = prev.filter((s) => !next.includes(s));
+
+	if (added.length > 0) {
+		await writeCopiesForTargets(nextRow, added, tableName);
+	}
+	if (removed.length > 0) {
+		const copies = await findCopiesByHqSourceId(
+			nextRow.Id,
+			tableName,
+			indexName,
+		);
+		const toRemove = copies.filter((c) => removed.includes(c.Site));
+		// Per-call fresh payload — editRef sync-mutates updates[keyCol]; sharing
+		// across Promise.all(map) would leak RowId between iterations (same
+		// rationale as propagateHqEdit / cascadeHqRetract).
+		await Promise.all(
+			toRemove.map((c) =>
+				editRef(tableName, "RowId", c.RowId, {
+					Deleted: true,
+					Changed: moment.utc().unix(),
+				}),
+			),
+		);
+	}
+	return { added: added.length, removed: removed.length };
+};
+
 /**
  * Propagate an HQ-source MODIFY to every community copy via the GSI lookup.
  * Skips the retract case (where `Deleted: true`) — that flows through
@@ -370,6 +444,8 @@ module.exports = {
 	// Story B
 	findCopiesByHqSourceId,
 	fanOutHqSource,
+	writeCopiesForTargets,
+	syncHqTargets,
 	propagateHqEdit,
 	cascadeHqRetract,
 	incrementTemplateUseCount,

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@plusscommunities/pluss-core-aws",
-	"version": "2.0.25-beta.7",
+	"version": "2.0.25-beta.8",
 	"description": "Core extension package for Pluss Communities platform",
 	"scripts": {
 		"betapatch": "npm version prepatch --preid=beta",