@adobe/spacecat-shared-data-access 4.6.0 → 4.8.0

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## [@adobe/spacecat-shared-data-access-v4.8.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v4.7.0...@adobe/spacecat-shared-data-access-v4.8.0) (2026-07-14)
+
+### Features
+
+* add temporary replaceHandlerEnabledDisabled method for cleanup. ([#1328](https://github.com/adobe/spacecat-shared/issues/1328)) ([be9234f](https://github.com/adobe/spacecat-shared/commit/be9234fa67dc9960d4abbb0b0425e621b7ad33c7))
+
+## [@adobe/spacecat-shared-data-access-v4.7.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v4.6.0...@adobe/spacecat-shared-data-access-v4.7.0) (2026-07-13)
+
+### Features
+
+* **data-access:** add TicketSuggestion bulk accessors allBySuggestionIds and allByTicketIds (SITES-44690) ([#1804](https://github.com/adobe/spacecat-shared/issues/1804)) ([99387f6](https://github.com/adobe/spacecat-shared/commit/99387f64423c822c0f377014d245549bed68cd40))
+
 ## [@adobe/spacecat-shared-data-access-v4.6.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v4.5.0...@adobe/spacecat-shared-data-access-v4.6.0) (2026-07-12)
 
 ### Features

package/package.json

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

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

@@ -554,6 +554,84 @@ class Configuration {
     this.setHandlers(handlers);
   }
 
+  /**
+   * Replaces enabled/disabled lists for a handler.
+   * This method replaces (not merges) the provided arrays.
+   * Only the arrays provided in the data object will be replaced.
+   *
+   * TEMPORARY: This method was created to support a temporary API endpoint for
+   * cleaning up enabled and disabled lists (SITES-40312). It will be removed once the cleanup
+   * task is completed. SITES-40878 is the ticket to remove this temporary endpoint.
+   *
+   * @deprecated Temporary method for cleanup. See SITES-40312. Will be removed (SITES-40878).
+   * @param {string} type - The handler type to update
+   * @param {object} data - Object containing enabled/disabled arrays to replace
+   * @param {object} [data.enabled] - Enabled lists to replace (use null-safe check)
+   * @param {string[]} [data.enabled.sites] - Sites array to replace (if provided, must be array)
+   * @param {string[]} [data.enabled.orgs] - Orgs array to replace (if provided, must be array)
+   * @param {object} [data.disabled] - Disabled lists to replace (use null-safe check)
+   * @param {string[]} [data.disabled.sites] - Sites array to replace (if provided, must be array)
+   * @param {string[]} [data.disabled.orgs] - Orgs array to replace (if provided, must be array)
+   * @throws {Error} If data is empty, handler not found, or non-array value provided for sites/orgs
+   */
+  replaceHandlerEnabledDisabled(type, data) {
+    if (!isNonEmptyObject(data)) {
+      throw new Error('Data cannot be empty');
+    }
+
+    this.log.warn('replaceHandlerEnabledDisabled invoked (temporary method - see SITES-40312)');
+
+    const handlers = this.getHandlers();
+    if (!handlers[type]) {
+      throw new Error(`Handler "${type}" not found in configuration`);
+    }
+
+    const handler = handlers[type];
+
+    // Initialize enabled/disabled objects if they don't exist
+    if (!isNonEmptyObject(handler.enabled)) {
+      handler.enabled = { orgs: [], sites: [] };
+    }
+    if (!isNonEmptyObject(handler.disabled)) {
+      handler.disabled = { orgs: [], sites: [] };
+    }
+
+    // Replace enabled arrays if provided (use != null to catch both null and undefined)
+    if (data.enabled != null) {
+      if (data.enabled.sites !== undefined) {
+        if (!Array.isArray(data.enabled.sites)) {
+          throw new Error('enabled.sites must be an array');
+        }
+        handler.enabled.sites = data.enabled.sites;
+      }
+      if (data.enabled.orgs !== undefined) {
+        if (!Array.isArray(data.enabled.orgs)) {
+          throw new Error('enabled.orgs must be an array');
+        }
+        handler.enabled.orgs = data.enabled.orgs;
+      }
+    }
+
+    // Replace disabled arrays if provided (use != null to catch both null and undefined)
+    if (data.disabled != null) {
+      if (data.disabled.sites !== undefined) {
+        if (!Array.isArray(data.disabled.sites)) {
+          throw new Error('disabled.sites must be an array');
+        }
+        handler.disabled.sites = data.disabled.sites;
+      }
+      if (data.disabled.orgs !== undefined) {
+        if (!Array.isArray(data.disabled.orgs)) {
+          throw new Error('disabled.orgs must be an array');
+        }
+        handler.disabled.orgs = data.disabled.orgs;
+      }
+    }
+
+    handlers[type] = handler;
+    this.setHandlers(handlers);
+  }
+
   /**
    * Updates the configuration by merging changes into existing sections.
    * This is a flexible update method that allows updating one or more sections at once.

package/src/models/configuration/index.d.ts

@@ -40,6 +40,10 @@ export interface Configuration {
   isHandlerEnabledForOrg(type: string, org: Organization): boolean;
   isHandlerEnabledForSite(type: string, site: Site): boolean;
   registerAudit(type: string, enabledByDefault?: boolean, interval?: string, productCodes?: string[]): void;
+  replaceHandlerEnabledDisabled(
+    type: string,
+    data: { enabled?: { sites?: string[]; orgs?: string[] }; disabled?: { sites?: string[]; orgs?: string[] } }
+  ): void;
   save(): Promise<Configuration>;
   setHandlers(handlers: object): void;
   setJobs(jobs: object[]): void;

package/src/models/ticket-suggestion/ticket-suggestion.collection.js

@@ -10,8 +10,12 @@
  * governing permissions and limitations under the License.
  */
 
+import { DataAccessError } from '../../errors/index.js';
+import { guardArray } from '../../util/guards.js';
 import BaseCollection from '../base/base.collection.js';
 
+const IN_FILTER_CHUNK_SIZE = 50;
+
 /**
  * TicketSuggestionCollection — manages TicketSuggestion bridge records.
  *
@@ -24,6 +28,80 @@ import BaseCollection from '../base/base.collection.js';
  */
 class TicketSuggestionCollection extends BaseCollection {
   static COLLECTION_NAME = 'TicketSuggestionCollection';
+
+  /**
+   * Returns all bridge records whose suggestion_id is in the given array.
+   * Used for bulk pre-flight checks before ticket creation.
+   *
+   * @param {string[]} suggestionIds
+   * @returns {Promise<import('./ticket-suggestion.model.js').default[]>}
+   */
+  async allBySuggestionIds(suggestionIds) {
+    guardArray('suggestionIds', suggestionIds, 'TicketSuggestionCollection', 'string');
+
+    if (suggestionIds.length === 0) {
+      return [];
+    }
+
+    try {
+      const uniqueIds = [...new Set(suggestionIds)];
+      const chunks = [];
+      for (let i = 0; i < uniqueIds.length; i += IN_FILTER_CHUNK_SIZE) {
+        chunks.push(uniqueIds.slice(i, i + IN_FILTER_CHUNK_SIZE));
+      }
+
+      const results = await Promise.all(
+        chunks.map((chunk) => this.all(
+          {},
+          { where: (attrs, op) => op.in(attrs.suggestionId, chunk) },
+        )),
+      );
+
+      return results.flat();
+    } catch (error) {
+      if (error instanceof DataAccessError) {
+        throw error;
+      }
+      throw new DataAccessError('Failed to load ticket suggestions by suggestion IDs', this, error);
+    }
+  }
+
+  /**
+   * Returns all bridge records whose ticket_id is in the given array.
+   * Used to bulk-load suggestion mappings when listing tickets.
+   *
+   * @param {string[]} ticketIds
+   * @returns {Promise<import('./ticket-suggestion.model.js').default[]>}
+   */
+  async allByTicketIds(ticketIds) {
+    guardArray('ticketIds', ticketIds, 'TicketSuggestionCollection', 'string');
+
+    if (ticketIds.length === 0) {
+      return [];
+    }
+
+    try {
+      const uniqueIds = [...new Set(ticketIds)];
+      const chunks = [];
+      for (let i = 0; i < uniqueIds.length; i += IN_FILTER_CHUNK_SIZE) {
+        chunks.push(uniqueIds.slice(i, i + IN_FILTER_CHUNK_SIZE));
+      }
+
+      const results = await Promise.all(
+        chunks.map((chunk) => this.all(
+          {},
+          { where: (attrs, op) => op.in(attrs.ticketId, chunk) },
+        )),
+      );
+
+      return results.flat();
+    } catch (error) {
+      if (error instanceof DataAccessError) {
+        throw error;
+      }
+      throw new DataAccessError('Failed to load ticket suggestions by ticket IDs', this, error);
+    }
+  }
 }
 
 export default TicketSuggestionCollection;