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

package/helper/hqPublishing.js

@@ -400,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) {
@@ -410,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 {
@@ -432,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;

package/package.json

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