@adobe/spacecat-shared-data-access 1.33.1 → 1.34.1-config

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [@adobe/spacecat-shared-data-access-v1.34.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v1.34.0...@adobe/spacecat-shared-data-access-v1.34.1) (2024-06-29)
+
+
+### Bug Fixes
+
+* **deps:** update external fixes ([#279](https://github.com/adobe/spacecat-shared/issues/279)) ([4710607](https://github.com/adobe/spacecat-shared/commit/471060797fac55649ff0c0ea24129bdd421d46b2))
+
+# [@adobe/spacecat-shared-data-access-v1.34.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v1.33.1...@adobe/spacecat-shared-data-access-v1.34.0) (2024-06-28)
+
+
+### Features
+
+* add helix config to site/candidates ([#277](https://github.com/adobe/spacecat-shared/issues/277)) ([5ed489d](https://github.com/adobe/spacecat-shared/commit/5ed489dc84cb594689092e1f0019dd83d3647039))
+
 # [@adobe/spacecat-shared-data-access-v1.33.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v1.33.0...@adobe/spacecat-shared-data-access-v1.33.1) (2024-06-27)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-data-access",
-  "version": "1.33.1",
+  "version": "1.34.1-config",
   "description": "Shared modules of the Spacecat Services - Data Access",
   "type": "module",
   "main": "src/index.js",
@@ -31,8 +31,8 @@
   "dependencies": {
     "@adobe/spacecat-shared-dynamo": "1.2.5",
     "@adobe/spacecat-shared-utils": "1.2.0",
-    "@aws-sdk/client-dynamodb": "3.602.0",
-    "@aws-sdk/lib-dynamodb": "3.602.0",
+    "@aws-sdk/client-dynamodb": "3.606.0",
+    "@aws-sdk/lib-dynamodb": "3.606.0",
     "@types/joi": "17.2.3",
     "joi": "17.13.3",
     "uuid": "10.0.0"

package/src/dto/configuration.js

@@ -26,6 +26,7 @@ export const ConfigurationDto = {
       version: dynamoItem.version,
       queues: dynamoItem.queues,
       jobs: dynamoItem.jobs,
+      handlers: dynamoItem.handlers,
     };
 
     return createConfiguration(configurationData);
@@ -41,5 +42,6 @@ export const ConfigurationDto = {
     version: configuration.getVersion(),
     queues: configuration.getQueues(),
     jobs: configuration.getJobs(),
+    handlers: configuration.getHandlers(),
   }),
 };

package/src/dto/site-candidate.js

@@ -23,6 +23,7 @@ export const SiteCandidateDto = {
    */
   toDynamoItem: (siteCandidate) => ({
     baseURL: siteCandidate.getBaseURL(),
+    hlxConfig: siteCandidate.getHlxConfig(),
     siteId: siteCandidate.getSiteId(),
     source: siteCandidate.getSource(),
     status: siteCandidate.getStatus(),
@@ -39,6 +40,7 @@ export const SiteCandidateDto = {
   fromDynamoItem: (dynamoItem) => {
     const siteCandidateData = {
       baseURL: dynamoItem.baseURL,
+      hlxConfig: dynamoItem.hlxConfig,
       siteId: dynamoItem.siteId,
       source: dynamoItem.source,
       status: dynamoItem.status,

package/src/dto/site.js

@@ -11,7 +11,6 @@
  */
 
 import { createSite } from '../models/site.js';
-import AuditConfig from '../models/site/audit-config.js';
 import { Config } from '../models/site/config.js';
 
 /**
@@ -28,13 +27,13 @@ export const SiteDto = {
     baseURL: site.getBaseURL(),
     deliveryType: site.getDeliveryType(),
     gitHubURL: site.getGitHubURL() || '',
+    hlxConfig: site.getHlxConfig(),
     organizationId: site.getOrganizationId() || '',
     isLive: site.isLive(),
     isLiveToggledAt: site.getIsLiveToggledAt(),
     createdAt: site.getCreatedAt(),
     updatedAt: site.getUpdatedAt(),
     GSI1PK: 'ALL_SITES',
-    auditConfig: AuditConfig.toDynamoItem(site.getAuditConfig()),
     config: Config.toDynamoItem(site.getConfig()),
   }),
 
@@ -49,12 +48,12 @@ export const SiteDto = {
       baseURL: dynamoItem.baseURL,
       deliveryType: dynamoItem.deliveryType,
       gitHubURL: dynamoItem.gitHubURL,
+      hlxConfig: dynamoItem.hlxConfig,
       organizationId: dynamoItem.organizationId,
       isLive: dynamoItem.isLive,
       isLiveToggledAt: dynamoItem.isLiveToggledAt,
       createdAt: dynamoItem.createdAt,
       updatedAt: dynamoItem.updatedAt,
-      auditConfig: dynamoItem.auditConfig,
       config: dynamoItem.config,
     };
 

package/src/index.d.ts

@@ -84,18 +84,6 @@ export interface Audit {
   getScores: () => object;
 }
 
-/**
- * AuditConfigType defines the structure for specific audit type configurations.
- */
-export interface AuditConfigType {
-  /**
-   * Returns true if the audit type is disabled for the site. If an audit type is disabled, no
-   * audits of that type will be scheduled for the site.
-   * @returns {boolean} True if the audit type is disabled for the site.
-   */
-  disabled: () => boolean;
-}
-
 export interface Config {
 
 }
@@ -104,31 +92,6 @@ export interface FulfillableItems {
   items: string[];
 }
 
-/**
- * AuditConfig defines the structure for the overall audit configuration of a site.
- */
-export interface AuditConfig {
-  /**
-   * Returns true if audits are disabled for the site. If audits are disabled, no audits will be
-   * scheduled for the site. Overrides any audit type specific configurations.
-   * @returns {boolean} True if audits are disabled for the site.
-   */
-  auditsDisabled: () => boolean;
-
-  /**
-   * Returns the audit config for a specific audit type. The audit type is the key.
-   * @param {string} auditType The audit type to get the config for.
-   * @returns {AuditConfigType} The audit config for the audit type.
-   */
-  getAuditTypeConfig: (auditType: string) => AuditConfigType;
-
-  /**
-   * Returns the audit configs for all audit types. The keys are the audit types.
-   * @returns {object} The audit configs for all audit types.
-   */
-  getAuditTypeConfigs: () => object;
-}
-
 /**
  * Represents a key event.
  */
@@ -222,12 +185,6 @@ export interface Site {
    */
   getUpdatedAt: () => string;
 
-  /**
-   * Retrieves the current audit configuration for the site.
-   * @returns {AuditConfig} The current audit configuration.
-   */
-  getAuditConfig: () => AuditConfig;
-
   /**
    * Retrieves the current configuration for the site.
    * @returns {Config} The current configuration.
@@ -435,6 +392,60 @@ export interface Configuration {
    */
   getJobs: () => Array<object>;
 
+    /**
+     * Retrieves the handlers configuration.
+     * @returns {object} The handlers configuration.
+     */
+  getHandlers: () => object;
+
+  /**
+   * Retrieves the handler configuration for handler type.
+   * @returns {object} The handler type configuration.
+   */
+  getHandler: (type) => object;
+
+  /**
+   * Return true if a handler type is enabled for an organization.
+   * @param type handler type
+   * @param org organization
+   */
+  isHandlerEnabledForOrg: (type: string, org: Organization) => boolean;
+
+  /**
+   * Return true if a handler type is enabled for a site.
+   * @param type handler type
+   * @param site site
+   */
+  isHandlerEnabledForSite: (type: string, site: Site) => boolean;
+
+  /**
+   * Enables a handler type for an site.
+   * @param type handler type
+   * @param site site
+   */
+  enableHandlerForSite: (type: string, site: Site) => void;
+
+  /**
+   * Enables a handler type for an organization.
+   * @param type handler type
+   * @param org organization
+   */
+  enableHandlerForOrg: (type: string, org: Organization) => void;
+
+    /**
+     * Disables a handler type for an site.
+     * @param type handler type
+     * @param site site
+     */
+  disableHandlerForSite: (type: string, site: Site) => void;
+
+  /**
+   * Disables a handler type for an organization.
+   * @param type handler type
+   * @param org organization
+   */
+  disableHandlerForOrg: (type:string, org: Organization) => void;
+
 }
 
 export interface ImportJob {

package/src/models/configuration.js

@@ -13,18 +13,151 @@
 import Joi from 'joi';
 
 const Configuration = (data = {}) => {
-  const self = { ...data };
-  self.getJobs = () => self.jobs;
-  self.getVersion = () => self.version;
-  self.getQueues = () => self.queues;
+  const state = { ...data };
+  const self = { state };
+  self.getJobs = () => self.state.jobs;
+  self.getVersion = () => self.state.version;
+  self.getQueues = () => self.state.queues;
+  self.getHandlers = () => self.state.handlers;
+  self.getHandler = (type) => self.state.handlers[type];
+  self.addHandler = (type, handlerData) => {
+    state.handlers = state.handlers || {};
+    state.handlers[type] = { ...handlerData };
+  };
+  self.isHandlerEnabledForSite = (type, site) => {
+    const handler = state.handlers[type];
+    if (!handler) return false;
+
+    const siteId = site.getId();
+    const orgId = site.getOrganizationId();
+
+    if (handler.enabled) {
+      return handler.enabled.sites.includes(siteId) || handler.enabled.orgs.includes(orgId);
+    }
+
+    if (handler.disabled) {
+      return !((handler.disabled.sites && handler.disabled.sites.includes(siteId))
+          || (handler.disabled.orgs && handler.disabled.orgs.includes(orgId)));
+    }
+
+    return handler.enabledByDefault;
+  };
+
+  self.isHandlerEnabledForOrg = (type, org) => {
+    const handler = state.handlers[type];
+    if (!handler) return false;
+
+    const orgId = org.getId();
+
+    if (handler.enabled) {
+      return handler.enabled.orgs.includes(orgId);
+    }
+
+    if (handler.disabled) {
+      return !handler.disabled.orgs.includes(orgId);
+    }
+
+    return handler.enabledByDefault;
+  };
+
+  const updateHandlerOrgs = (type, orgId, enabled) => {
+    const handler = state.handlers[type];
+    if (!handler) return;
+
+    if (enabled) {
+      if (handler.enabledByDefault) {
+        handler.disabled.orgs = handler.disabled.orgs?.filter((id) => id !== orgId) || [];
+      } else {
+        handler.enabled = handler.enabled || { orgs: [] };
+        handler.enabled.orgs = [...(handler.enabled?.orgs || []), orgId];
+      }
+    } else if (handler.enabledByDefault) {
+      handler.disabled = handler.disabled || { orgs: [] };
+      handler.disabled.orgs = [...(handler.disabled?.orgs || []), orgId];
+    } else {
+      handler.enabled.orgs = handler.enabled.orgs?.filter((id) => id !== orgId) || [];
+    }
+  };
+
+  const updateHandlerSites = (type, siteId, enabled) => {
+    const handler = state.handlers[type];
+    if (!handler) return;
+
+    if (enabled) {
+      if (handler.enabledByDefault) {
+        handler.disabled.sites = handler.disabled.sites?.filter((id) => id !== siteId) || [];
+      } else {
+        handler.enabled = handler.enabled || { sites: [] };
+        handler.enabled.sites = [...(handler.enabled.sites || []), siteId];
+      }
+    } else if (handler.enabledByDefault) {
+      handler.disabled = handler.disabled || { sites: [] };
+      handler.disabled.sites = [...(handler.disabled.sites || []), siteId];
+    } else {
+      handler.enabled.sites = handler.enabled.sites?.filter((id) => id !== siteId) || [];
+    }
+  };
+
+  self.enableHandlerForSite = (type, site) => {
+    const siteId = site.getId();
+    if (self.isHandlerEnabledForSite(type, site)) return;
+
+    updateHandlerSites(type, siteId, true);
+  };
+
+  self.enableHandlerForOrg = (type, org) => {
+    const orgId = org.getId();
+    if (self.isHandlerEnabledForOrg(type, org)) return;
+
+    updateHandlerOrgs(type, orgId, true);
+  };
+
+  self.disableHandlerForSite = (type, site) => {
+    const siteId = site.getId();
+    if (!self.isHandlerEnabledForSite(type, site)) return;
+
+    updateHandlerSites(type, siteId, false);
+  };
+
+  self.disableHandlerForOrg = (type, org) => {
+    const orgId = org.getId();
+    if (!self.isHandlerEnabledForOrg(type, org)) return;
+
+    updateHandlerOrgs(type, orgId, false);
+  };
 
   return Object.freeze(self);
 };
 
+/**
+ *
+ * @param configuration
+ * @returns {any}
+ */
+
 export const checkConfiguration = (configuration) => {
   const schema = Joi.object({
     version: Joi.string().required(),
     queues: Joi.object().required(),
+    handlers: Joi.object().pattern(Joi.string(), Joi.object(
+      {
+        enabled: Joi.object({
+          sites: Joi.array().items(Joi.string()),
+          orgs: Joi.array().items(Joi.string()),
+        }),
+        disabled: Joi.object({
+          sites: Joi.array().items(Joi.string()),
+          orgs: Joi.array().items(Joi.string()),
+        }),
+        enabledByDefault: Joi.boolean().required(),
+        dependencies: Joi.array().items(Joi.object(
+          {
+            handler: Joi.string(),
+            actions: Joi.array().items(Joi.string()),
+          },
+        )),
+      },
+    )),
     jobs: Joi.array().required(),
   }).unknown(true);
   const { error, value } = schema.validate(configuration);

package/src/models/organization.js

@@ -26,24 +26,11 @@ export const DEFAULT_ORGANIZATION_ID = 'default';
 const Organization = (data = {}) => {
   const self = Base(data);
 
-  self.getAuditConfig = () => self.state.config.audits;
   self.getConfig = () => self.state.config;
   self.getName = () => self.state.name;
   self.getImsOrgId = () => self.state.imsOrgId;
   self.getFulfillableItems = () => self.state.fulfillableItems;
 
-  self.setAllAuditsDisabled = (disabled) => {
-    self.state.config.audits.updateAuditsDisabled(disabled);
-    self.touch();
-    return self;
-  };
-
-  self.updateAuditTypeConfig = (type, config) => {
-    self.state.config.audits.updateAuditTypeConfig(type, config);
-    self.touch();
-    return self;
-  };
-
   /**
      * Updates the IMS Org ID belonging to the organization.
      * @param {string} imsOrgId - The IMS Org ID.

package/src/models/site/config.js

@@ -11,7 +11,6 @@
  */
 
 import Joi from 'joi';
-import AuditConfig from './audit-config.js';
 
 export const configSchema = Joi.object({
   slack: Joi.object({
@@ -19,35 +18,24 @@ export const configSchema = Joi.object({
     channel: Joi.string(),
     invitedUserCount: Joi.number().integer().min(0),
   }),
-  alerts: Joi.array().items(Joi.object({
-    type: Joi.string().required(),
-    byOrg: Joi.boolean(),
-    mentions: Joi.array().items(Joi.object({ slack: Joi.array().items(Joi.string()) })),
-  }).unknown(true)),
-  audits: Joi.object({
-    auditsDisabled: Joi.boolean().optional(),
-    auditTypeConfigs: Joi.object().pattern(
-      Joi.string(),
-      Joi.object({
-        disabled: Joi.boolean().optional(),
-        excludedURLs: Joi.array().items(Joi.string()).optional(),
-        manualOverwrites: Joi.array().items(Joi.object({
-          brokenTargetURL: Joi.string().optional(),
-          targetURL: Joi.string().optional(),
-        })).optional(),
-        fixedURLs: Joi.array().items(Joi.object({
-          brokenTargetURL: Joi.string().optional(),
-          targetURL: Joi.string().optional(),
-        })).optional(),
-      }).unknown(true),
-    ).unknown(true),
-  }).unknown(true),
+  handlers: Joi.object().pattern(Joi.string(), Joi.object({
+    mentions: Joi.object().pattern(Joi.string(), Joi.array().items(Joi.string())),
+    excludedURLs: Joi.array().items(Joi.string()),
+    manualOverwrites: Joi.array().items(Joi.object({
+      brokenTargetURL: Joi.string().optional(),
+      targetURL: Joi.string().optional(),
+    })).optional(),
+    fixedURLs: Joi.array().items(Joi.object({
+      brokenTargetURL: Joi.string().optional(),
+      targetURL: Joi.string().optional(),
+    })).optional(),
+  }).unknown(true)).unknown(true),
 }).unknown(true);
 
 export const DEFAULT_CONFIG = {
   slack: {},
-  alerts: [],
-  audits: {},
+  handlers: {
+  },
 };
 
 // Function to validate incoming configuration
@@ -63,11 +51,49 @@ function validateConfiguration(config) {
 
 export const Config = (data = {}) => {
   const validConfig = validateConfiguration(data);
-  validConfig.audits = AuditConfig(validConfig.audits);
 
   const state = { ...validConfig };
+  const self = { state };
+  self.getSlackConfig = () => state.slack;
+  self.getSlackMentions = (type) => state?.handlers[type]?.mentions?.slack;
+  self.getHandlerConfig = (type) => state?.handlers[type];
+  self.getHandlers = () => state.handlers;
+  self.getExcludedURLs = (type) => state?.handlers[type]?.excludedURLs;
+  self.getManualOverrides = (type) => state?.handlers[type]?.manualOverwrites;
+  self.getFixedURLs = (type) => state?.handlers[type]?.fixedURLs;
 
-  const self = { ...state };
+  self.updateSlackConfig = (channel, workspace, invitedUserCount) => {
+    state.slack = {
+      channel,
+      workspace,
+      invitedUserCount,
+    };
+  };
+
+  self.updateSlackMentions = (type, mentions) => {
+    state.handlers = state.handlers || {};
+    state.handlers[type] = state.handlers[type] || {};
+    state.handlers[type].mentions = state.handlers[type].mentions || {};
+    state.handlers[type].mentions.slack = mentions;
+  };
+
+  self.updateExcludeURLs = (type, excludedURLs) => {
+    state.handlers = state.handlers || {};
+    state.handlers[type] = state.handlers[type] || {};
+    state.handlers[type].excludedURLs = excludedURLs;
+  };
+
+  self.updateManualOverrides = (type, manualOverwrites) => {
+    state.handlers = state.handlers || {};
+    state.handlers[type] = state.handlers[type] || {};
+    state.handlers[type].manualOverwrites = manualOverwrites;
+  };
+
+  self.updateFixedURLs = (type, fixedURLs) => {
+    state.handlers = state.handlers || {};
+    state.handlers[type] = state.handlers[type] || {};
+    state.handlers[type].fixedURLs = fixedURLs;
+  };
 
   return Object.freeze(self);
 };
@@ -75,6 +101,6 @@ export const Config = (data = {}) => {
 Config.fromDynamoItem = (dynamoItem) => Config(dynamoItem);
 
 Config.toDynamoItem = (config) => ({
-  ...config,
-  audits: AuditConfig.toDynamoItem(config.audits),
+  slack: config.getSlackConfig(),
+  handlers: config.getHandlers(),
 });

package/src/models/site-candidate.js

@@ -10,7 +10,7 @@
  * governing permissions and limitations under the License.
  */
 
-import { hasText, isValidUrl } from '@adobe/spacecat-shared-utils';
+import { hasText, isObject, isValidUrl } from '@adobe/spacecat-shared-utils';
 
 import { Base } from './base.js';
 
@@ -43,6 +43,7 @@ const SiteCandidate = (data = {}) => {
   delete self.state.id; // no id property used in SiteCandidate modal
 
   self.getBaseURL = () => self.state.baseURL;
+  self.getHlxConfig = () => self.state.hlxConfig;
   self.getSiteId = () => self.state.siteId;
   self.getSource = () => self.state.source;
   self.getStatus = () => self.state.status;
@@ -54,6 +55,12 @@ const SiteCandidate = (data = {}) => {
     return self;
   };
 
+  self.setHlxConfig = (hlxConfig) => {
+    self.state.hlxConfig = hlxConfig;
+    self.touch();
+    return self;
+  };
+
   self.setSource = (source) => {
     self.state.source = source;
     self.touch();
@@ -88,6 +95,14 @@ export const createSiteCandidate = (data) => {
     throw new Error(`Base URL must be a valid URL: ${newState.baseURL}`);
   }
 
+  if (newState.hlxConfig && !isObject(newState.hlxConfig)) {
+    throw new Error(`HLX Config must be an object: ${newState.hlxConfig}`);
+  }
+
+  if (!newState.hlxConfig) {
+    newState.hlxConfig = {};
+  }
+
   if (!hasText(newState.updatedBy)) {
     newState.updatedBy = DEFAULT_UPDATED_BY;
   }

package/src/models/site.js

@@ -13,7 +13,6 @@
 import { hasText, isObject, isValidUrl } from '@adobe/spacecat-shared-utils';
 
 import { Base } from './base.js';
-import AuditConfig from './site/audit-config.js';
 import { Config, DEFAULT_CONFIG } from './site/config.js';
 import { DEFAULT_ORGANIZATION_ID } from './organization.js';
 
@@ -34,12 +33,12 @@ export const DEFAULT_DELIVERY_TYPE = DELIVERY_TYPES.AEM_EDGE;
 const Site = (data = {}) => {
   const self = Base(data);
 
-  self.getAuditConfig = () => self.state.auditConfig;
   self.getAudits = () => self.state.audits;
   self.getBaseURL = () => self.state.baseURL;
   self.getConfig = () => self.state.config;
   self.getDeliveryType = () => self.state.deliveryType;
   self.getGitHubURL = () => self.state.gitHubURL;
+  self.getHlxConfig = () => self.state.hlxConfig;
   self.getOrganizationId = () => self.state.organizationId;
   self.isLive = () => self.state.isLive;
   self.getIsLiveToggledAt = () => self.state.isLiveToggledAt;
@@ -81,18 +80,6 @@ const Site = (data = {}) => {
     return self;
   }; */
 
-  self.setAllAuditsDisabled = (disabled) => {
-    self.state.auditConfig.updateAuditsDisabled(disabled);
-    self.touch();
-    return self;
-  };
-
-  self.updateAuditTypeConfig = (type, config) => {
-    self.state.auditConfig.updateAuditTypeConfig(type, config);
-    self.touch();
-    return self;
-  };
-
   /**
    * Updates the site config.
    * @param {string} config - The Site config.
@@ -136,6 +123,23 @@ const Site = (data = {}) => {
     return self;
   };
 
+  /**
+   * Updates the Helix Configuration for this site.
+   * @param {object} hlxConfig - The Helix Configuration.
+   * @throws {Error} If the provided Helix Configuration is not an object.
+   * @return {Base} The updated site.
+   */
+  self.updateHlxConfig = (hlxConfig) => {
+    if (!isObject(hlxConfig)) {
+      throw new Error('HLX Config must be an object');
+    }
+
+    self.state.hlxConfig = hlxConfig;
+    self.touch();
+
+    return self;
+  };
+
   /**
    * Updates the organizationId the site belongs to.
    * @param {string} organizationId - The Org ID.
@@ -179,6 +183,14 @@ export const createSite = (data) => {
     throw new Error(`Base URL must be a valid URL: ${newState.baseURL}`);
   }
 
+  if (newState.hlxConfig && !isObject(newState.hlxConfig)) {
+    throw new Error(`HLX Config must be an object: ${newState.hlxConfig}`);
+  }
+
+  if (!newState.hlxConfig) {
+    newState.hlxConfig = {};
+  }
+
   if (!hasText(newState.organizationId)) {
     newState.organizationId = DEFAULT_ORGANIZATION_ID;
   }
@@ -196,15 +208,6 @@ export const createSite = (data) => {
     newState.audits = [];
   }
 
-  if (!isObject(newState.auditConfig)) {
-    newState.auditConfig = {
-      auditsDisabled: false,
-      auditTypeConfigs: {},
-    };
-  }
-
-  newState.auditConfig = AuditConfig(newState.auditConfig);
-
   if (!isObject(newState.config)) {
     newState.config = { ...DEFAULT_CONFIG };
   }

package/src/service/configurations/accessPatterns.js

@@ -110,8 +110,7 @@ export const updateConfiguration = async (
   const newConfigurationData = { ...configurationData };
   const latestConfiguration = await getConfiguration(dynamoClient, config);
 
-  newConfigurationData.version = incrementVersion(latestConfiguration?.version);
-  newConfigurationData.PK = config.pkAllConfigurations;
+  newConfigurationData.version = incrementVersion(latestConfiguration?.getVersion());
 
   const newConfiguration = createConfiguration(newConfigurationData);
 

package/src/service/sites/accessPatterns.js

@@ -336,7 +336,6 @@ export const getSiteByID = async (
 
   return isObject(dynamoItem) ? SiteDto.fromDynamoItem(dynamoItem) : null;
 };
-
 /**
  * Adds a site.
  *

package/src/models/site/audit-config-type.js

@@ -1,60 +0,0 @@
-/*
- * Copyright 2023 Adobe. All rights reserved.
- * This file is licensed to you under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License. You may obtain a copy
- * of the License at http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software distributed under
- * the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR REPRESENTATIONS
- * OF ANY KIND, either express or implied. See the License for the specific language
- * governing permissions and limitations under the License.
- */
-
-const AuditConfigType = (data = {}) => {
-  const state = {
-    disabled: data.disabled || false,
-    excludedURLs: data.excludedURLs || [],
-    manualOverwrites: data.manualOverwrites || [],
-    fixedURLs: data.fixedURLs || [],
-  };
-
-  const self = {
-    disabled: () => state.disabled,
-    getExcludedURLs: () => state.excludedURLs,
-    updateExcludedURLs: (excludeURLs) => {
-      state.excludedURLs = excludeURLs;
-    },
-    getManualOverwrites: () => state.manualOverwrites,
-    updateManualOverwrites: (manualOverwrites) => {
-      state.manualOverwrites = manualOverwrites;
-    },
-    getFixedURLs: () => state.fixedURLs,
-    updateFixedURLs: (fixedURLs) => {
-      state.fixedURLs = fixedURLs;
-    },
-    updateDisabled: (newValue) => {
-      state.disabled = newValue;
-    },
-  };
-
-  return Object.freeze(self);
-};
-
-AuditConfigType.fromDynamoItem = (dynamoItem) => {
-  const auditConfigTypeData = {
-    disabled: dynamoItem.disabled,
-    excludedURLs: dynamoItem.excludedURLs,
-    manualOverwrites: dynamoItem.manualOverwrites,
-    fixedURLs: dynamoItem.fixedURLs,
-  };
-  return AuditConfigType(auditConfigTypeData);
-};
-
-AuditConfigType.toDynamoItem = (auditConfigType) => ({
-  disabled: auditConfigType.disabled(),
-  excludedURLs: auditConfigType.getExcludedURLs(),
-  manualOverwrites: auditConfigType.getManualOverwrites(),
-  fixedURLs: auditConfigType.getFixedURLs(),
-});
-
-export default AuditConfigType;

package/src/models/site/audit-config.js

@@ -1,85 +0,0 @@
-/*
- * Copyright 2023 Adobe. All rights reserved.
- * This file is licensed to you under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License. You may obtain a copy
- * of the License at http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software distributed under
- * the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR REPRESENTATIONS
- * OF ANY KIND, either express or implied. See the License for the specific language
- * governing permissions and limitations under the License.
- */
-
-import AuditConfigType from './audit-config-type.js';
-import {
-  AUDIT_TYPE_BROKEN_BACKLINKS,
-  AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY,
-  AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY,
-  AUDIT_TYPE_ORGANIC_KEYWORDS,
-  AUDIT_TYPE_ORGANIC_TRAFFIC,
-} from '../audit.js';
-
-const AUDIT_TYPE_DISABLED_DEFAULTS = {
-  [AUDIT_TYPE_BROKEN_BACKLINKS]: true,
-  [AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY]: true,
-  [AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY]: true,
-  [AUDIT_TYPE_ORGANIC_KEYWORDS]: true,
-  [AUDIT_TYPE_ORGANIC_TRAFFIC]: true,
-};
-
-function getAuditTypeConfigs(auditTypeConfigs, auditsDisabled) {
-  if (!auditTypeConfigs || Object.keys(auditTypeConfigs).length === 0) {
-    return {
-      [AUDIT_TYPE_BROKEN_BACKLINKS]: AuditConfigType({ disabled: true }),
-      [AUDIT_TYPE_EXPERIMENTATION_ESS_DAILY]: AuditConfigType({ disabled: true }),
-      [AUDIT_TYPE_EXPERIMENTATION_ESS_MONTHLY]: AuditConfigType({ disabled: true }),
-      [AUDIT_TYPE_ORGANIC_KEYWORDS]: AuditConfigType({ disabled: true }),
-      [AUDIT_TYPE_ORGANIC_TRAFFIC]: AuditConfigType({ disabled: true }),
-    };
-  }
-  return Object.entries(auditTypeConfigs || {}).reduce((acc, [key, value]) => {
-    const disabled = value.disabled !== undefined
-      ? value.disabled : (AUDIT_TYPE_DISABLED_DEFAULTS[key] || auditsDisabled || false);
-    acc[key] = AuditConfigType(
-      {
-        ...value,
-        disabled,
-      },
-    );
-    return acc;
-  }, {});
-}
-
-const AuditConfig = (data = {}) => {
-  const state = {
-    auditsDisabled: data.auditsDisabled || false,
-    auditTypeConfigs: getAuditTypeConfigs(data.auditTypeConfigs, data.auditsDisabled),
-  };
-
-  const self = {
-    auditsDisabled: () => state.auditsDisabled,
-    getAuditTypeConfigs: () => state.auditTypeConfigs,
-    getAuditTypeConfig: (type) => state.auditTypeConfigs[type],
-    updateAuditsDisabled: (newValue) => {
-      state.auditsDisabled = newValue;
-    },
-    updateAuditTypeConfig: (type, config) => {
-      state.auditTypeConfigs[type] = AuditConfigType(config);
-    },
-  };
-
-  return Object.freeze(self);
-};
-
-AuditConfig.fromDynamoItem = (dynamoItem) => AuditConfig(dynamoItem);
-
-AuditConfig.toDynamoItem = (auditConfig) => ({
-  auditsDisabled: auditConfig.auditsDisabled(),
-  auditTypeConfigs: Object.entries(auditConfig.getAuditTypeConfigs())
-    .reduce((acc, [key, value]) => {
-      acc[key] = AuditConfigType.toDynamoItem(value);
-      return acc;
-    }, {}),
-});
-
-export default AuditConfig;