@adobe/spacecat-shared-data-access 4.3.0 → 4.4.0

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## [@adobe/spacecat-shared-data-access-v4.4.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v4.3.1...@adobe/spacecat-shared-data-access-v4.4.0) (2026-07-09)
+
+### Features
+
+* **fix-entity:** add allByOpportunityIds for site-wide fix queries ([#1792](https://github.com/adobe/spacecat-shared/issues/1792)) ([25309bf](https://github.com/adobe/spacecat-shared/commit/25309bff4be7d90c0f7cdbf854c48a20adfd3e57))
+
+## [@adobe/spacecat-shared-data-access-v4.3.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v4.3.0...@adobe/spacecat-shared-data-access-v4.3.1) (2026-07-09)
+
+### Bug Fixes
+
+* **ticket-suggestion:** use createdAt as belongs_to sort key instead of updatedAt ([#1798](https://github.com/adobe/spacecat-shared/issues/1798)) ([711eecd](https://github.com/adobe/spacecat-shared/commit/711eecdc187ec9a21a79dedca808e6622bd18933)), closes [#2661](https://github.com/adobe/spacecat-shared/issues/2661)
+
 ## [@adobe/spacecat-shared-data-access-v4.3.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v4.2.0...@adobe/spacecat-shared-data-access-v4.3.0) (2026-07-09)
 
 ### Features

package/package.json

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

package/src/models/fix-entity/fix-entity.collection.js

@@ -15,6 +15,11 @@ import ValidationError from '../../errors/validation.error.js';
 import { guardId, guardArray, guardString } from '../../util/guards.js';
 import { resolveUpdates } from '../../util/util.js';
 
+// PostgREST GET requests serialize IN-filters into the URL, so large ID lists must be
+// chunked to stay well under the ~8KB URL length limit (see batchGetByKeys / llmo-brand-presence
+// for the same pattern elsewhere in this package).
+const IN_FILTER_CHUNK_SIZE = 50;
+
 /**
  * FixEntityCollection - A collection class responsible for managing FixEntities.
  * Extends the BaseCollection to provide specific methods for interacting with
@@ -226,6 +231,49 @@ class FixEntityCollection extends BaseCollection {
     }
   }
 
+  /**
+   * Gets all fixes across a set of opportunities (e.g. every opportunity for a site),
+   * by filtering on opportunityId IN (...). Chunks the IN-filter to stay under
+   * PostgREST's URL length limit.
+   *
+   * @async
+   * @param {string[]} opportunityIds - The IDs of the opportunities.
+   * @returns {Promise<FixEntity[]>} - A promise that resolves to an array of FixEntity models.
+   * @throws {DataAccessError} - Throws an error if the query fails.
+   * @throws {ValidationError} - Throws an error if opportunityIds is not an array of strings.
+   */
+  async allByOpportunityIds(opportunityIds) {
+    guardArray('opportunityIds', opportunityIds, 'FixEntityCollection', 'string');
+
+    if (opportunityIds.length === 0) {
+      return [];
+    }
+
+    try {
+      const uniqueIds = [...new Set(opportunityIds)];
+
+      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.opportunityId, chunk) },
+        )),
+      );
+
+      return results.flat();
+    } catch (error) {
+      if (error instanceof DataAccessError) {
+        throw error;
+      }
+      this.log.error('Failed to get all fixes by opportunity IDs', error);
+      throw new DataAccessError('Failed to get all fixes by opportunity IDs', this, error);
+    }
+  }
+
   async #buildFixesWithSuggestions(fixEntitySuggestions) {
     if (fixEntitySuggestions.length === 0) {
       return [];

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

@@ -29,7 +29,10 @@ const schema = new SchemaBuilder(TicketSuggestion, TicketSuggestionCollection)
     type: 'string', required: false, readOnly: true, postgrestIgnore: true,
   })
   .addAttribute('updatedBy', { type: 'string', required: false, postgrestIgnore: true })
-  .addReference('belongs_to', 'Ticket')
+  // Sort key uses createdAt (not the default updatedAt) because ticket_suggestions is
+  // append-only — the table has no updated_at column, so ORDER BY updated_at would
+  // cause PostgREST to error on allByTicketId queries.
+  .addReference('belongs_to', 'Ticket', ['createdAt'])
   .addAttribute('suggestionId', {
     type: 'string',
     required: true,