@adobe/spacecat-shared-data-access 2.6.0 → 2.7.0

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [@adobe/spacecat-shared-data-access-v2.7.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v2.6.0...@adobe/spacecat-shared-data-access-v2.7.0) (2025-02-18)
+
+
+### Features
+
+* Implement dependency check for handlers ([#608](https://github.com/adobe/spacecat-shared/issues/608)) ([0ada515](https://github.com/adobe/spacecat-shared/commit/0ada5157a3cf50bf67129e296e719a14ee2c451c)), closes [/github.com/adobe/spacecat-shared/blob/main/packages/spacecat-shared-data-access/src/models/configuration/configuration.schema.js#L35-L39](https://github.com//github.com/adobe/spacecat-shared/blob/main/packages/spacecat-shared-data-access/src/models/configuration/configuration.schema.js/issues/L35-L39)
+
 # [@adobe/spacecat-shared-data-access-v2.6.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v2.5.0...@adobe/spacecat-shared-data-access-v2.6.0) (2025-02-18)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-data-access",
-  "version": "2.6.0",
+  "version": "2.7.0",
   "description": "Shared modules of the Spacecat Services - Data Access",
   "type": "module",
   "engines": {

package/src/models/configuration/configuration.model.js

@@ -10,7 +10,7 @@
  * governing permissions and limitations under the License.
  */
 
-import { isNonEmptyObject } from '@adobe/spacecat-shared-utils';
+import { isNonEmptyObject, isNonEmptyArray } from '@adobe/spacecat-shared-utils';
 
 import { sanitizeIdAndAuditFields } from '../../util/util.js';
 import BaseModel from '../base/base.model.js';
@@ -146,12 +146,58 @@ class Configuration extends BaseModel {
     const siteId = site.getId();
     if (this.isHandlerEnabledForSite(type, site)) return;
 
+    const deps = this.isHandlerDependencyMetForSite(type, site);
+    if (deps !== true) {
+      throw new Error(`Cannot enable handler ${type} for site ${siteId} because of missing dependencies: ${deps}`);
+    }
+
     this.updateHandlerSites(type, siteId, true);
   }
 
+  /**
+   * Check if all dependencies for a handler of given type are met for the given org.
+   *
+   * @param {string} type handler type
+   * @param {object} org org object
+   * @returns true if all dependencies are met, array with missing dependencies otherwise
+   */
+  isHandlerDependencyMetForOrg(type, org) {
+    const handler = this.getHandler(type);
+
+    if (!handler || !isNonEmptyArray(handler?.dependencies)) return true;
+
+    const unmetDependencies = handler.dependencies
+      .filter(({ handler: depHandler }) => !this.isHandlerEnabledForOrg(depHandler, org))
+      .map(({ handler: depHandler }) => depHandler);
+
+    return isNonEmptyArray(unmetDependencies) ? unmetDependencies : true;
+  }
+
+  /**
+   * Check if all dependencies for a handler of given type are met for the given site.
+   *
+   * @param {string} type handler type
+   * @param {object} site site object
+   * @returns true if all dependencies are met, array with missing dependencies otherwise
+   */
+  isHandlerDependencyMetForSite(type, site) {
+    const handler = this.getHandler(type);
+    if (!handler || !isNonEmptyArray(handler?.dependencies)) return true;
+
+    const unmetDependencies = handler.dependencies
+      .filter(({ handler: depHandler }) => !this.isHandlerEnabledForSite(depHandler, site))
+      .map(({ handler: depHandler }) => depHandler);
+
+    return isNonEmptyArray(unmetDependencies) ? unmetDependencies : true;
+  }
+
   enableHandlerForOrg(type, org) {
     const orgId = org.getId();
     if (this.isHandlerEnabledForOrg(type, org)) return;
+    const deps = this.isHandlerDependencyMetForOrg(type, org);
+    if (deps !== true) {
+      throw new Error(`Cannot enable handler ${type} for org ${orgId} because of missing dependencies: ${deps}`);
+    }
 
     this.updateHandlerOrgs(type, orgId, true);
   }