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

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
@@ -326,8 +400,8 @@ const cascadeHqRetract = async (hqSourceRow, tableName, options = {}) => {
  * @param {string} templateId — the `Id` of the contentLibrary row.
  * @param {string} contentLibraryTable — entity-agnostic; caller passes the
  *   table name (without prefix).
- * @returns {Promise<boolean>} `true` if increment applied; `false` if table
- *   doesn't exist (graceful skip).
+ * @returns {Promise<boolean>} `true` if increment applied; `false` on graceful
+ *   skip (table not shipped yet, or no template with that Id exists).
  */
 const incrementTemplateUseCount = async (templateId, contentLibraryTable) => {
 	if (!templateId || !contentLibraryTable) {
@@ -336,11 +410,16 @@ const incrementTemplateUseCount = async (templateId, contentLibraryTable) => {
 	const params = {
 		TableName: `${process.env.tablePrefix}${contentLibraryTable}`,
 		Key: { Id: templateId },
-		UpdateExpression:
-			"ADD UseCount :inc SET LastUsedTimestamp = :now",
+		UpdateExpression: "ADD UseCount :inc SET LastUsedTimestamp = :now",
+		// Guard against minting a phantom row: ADD on a missing key would CREATE
+		// it with UseCount=1. Only increment a template that actually exists; a
+		// stale/bogus TemplateId is skipped (ConditionalCheckFailed below).
+		ConditionExpression: "attribute_exists(Id)",
 		ExpressionAttributeValues: {
 			":inc": 1,
-			":now": moment.utc().unix(),
+			// epoch ms — the contentLibrary timestamp convention (matches
+			// UpdatedAt). Was .unix() (seconds); reconciled to .valueOf().
+			":now": moment().utc().valueOf(),
 		},
 	};
 	try {
@@ -358,6 +437,12 @@ const incrementTemplateUseCount = async (templateId, contentLibraryTable) => {
 			);
 			return false;
 		}
+		if (error?.code === "ConditionalCheckFailedException") {
+			// TemplateId references no existing template — skip rather than
+			// create a phantom row. Not an error; continue the stream handler.
+			log("incrementTemplateUseCount", "NoSuchTemplate", { templateId });
+			return false;
+		}
 		// Anything else is genuinely unexpected — surface it.
 		log("incrementTemplateUseCount", "Error", error);
 		throw error;
@@ -370,6 +455,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.9",
 	"description": "Core extension package for Pluss Communities platform",
 	"scripts": {
 		"betapatch": "npm version prepatch --preid=beta",