@adobe/spacecat-shared-utils 1.82.0 → 1.82.1

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [@adobe/spacecat-shared-utils-v1.82.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-utils-v1.82.0...@adobe/spacecat-shared-utils-v1.82.1) (2025-12-05)
+
+
+### Bug Fixes
+
+* 0 week valid month on  numSeries ([#1220](https://github.com/adobe/spacecat-shared/issues/1220)) ([7670da9](https://github.com/adobe/spacecat-shared/commit/7670da9691b2623aa149b1b07c36c97585112ca7))
+
 # [@adobe/spacecat-shared-utils-v1.82.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-utils-v1.81.1...@adobe/spacecat-shared-utils-v1.82.0) (2025-12-03)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-utils",
-  "version": "1.82.0",
+  "version": "1.82.1",
   "description": "Shared modules of the Spacecat Services - utils",
   "type": "module",
   "exports": {

package/src/calendar-week-helper.js

@@ -208,17 +208,17 @@ export function getTemporalCondition({
   year,
   numSeries = 1,
 } = {}) {
-  const hasWeek = Number.isInteger(week) && Number.isInteger(year);
-  const hasMonth = Number.isInteger(month) && Number.isInteger(year);
+  const hasValidWeek = isValidWeek(week, year);
+  const hasValidMonth = isValidMonth(month, year);
 
   if (numSeries > 1) {
-    if (!hasWeek && !hasMonth) {
+    if (!hasValidWeek && !hasValidMonth) {
       throw new Error('Missing required parameters: week or month');
     }
 
     const conditions = [];
     for (let i = 0; i < numSeries; i += 1) {
-      if (hasWeek) {
+      if (hasValidWeek) {
         let currentWeek = week - i;
         let currentYear = year;
         if (currentWeek < 1) {
@@ -226,7 +226,7 @@ export function getTemporalCondition({
           currentYear -= 1;
         }
         conditions.push(getWeekInfo(currentWeek, currentYear).temporalCondition);
-      } else if (hasMonth) {
+      } else if (hasValidMonth) {
         let currentMonth = month - i;
         let currentYear = year;
         if (currentMonth < 1) {
@@ -239,22 +239,27 @@ export function getTemporalCondition({
     return conditions.join(' OR ');
   }
 
-  if (hasWeek && isValidWeek(week, year)) {
+  if (hasValidWeek) {
     return getWeekInfo(week, year).temporalCondition;
   }
 
-  if (hasMonth && isValidMonth(month, year)) {
+  if (hasValidMonth) {
     return getMonthInfo(month, year).temporalCondition;
   }
 
   // Fallbacks
-  if (Number.isInteger(week) || (!hasWeek && !hasMonth)) {
-    // default last full week
+  // If week was provided (even if invalid), default to last full week
+  if (Number.isInteger(week)) {
     return getWeekInfo().temporalCondition;
   }
 
-  // Otherwise fall back to last full month
-  return getMonthInfo().temporalCondition;
+  // If month was provided (even if invalid), default to last full month
+  if (Number.isInteger(month)) {
+    return getMonthInfo().temporalCondition;
+  }
+
+  // If neither was provided, default to last full week
+  return getWeekInfo().temporalCondition;
 }
 
 // Note: This function binds week exclusively to one year