@adobe/spacecat-shared-data-access 3.6.0 → 3.6.2

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## [@adobe/spacecat-shared-data-access-v3.6.2](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v3.6.1...@adobe/spacecat-shared-data-access-v3.6.2) (2026-03-04)
+
+### Bug Fixes
+
+* **data-access:** surface PG error details in logs ([#1401](https://github.com/adobe/spacecat-shared/issues/1401)) ([ddd3041](https://github.com/adobe/spacecat-shared/commit/ddd3041322a1468ee911f49f06c240a6697b8e66))
+
+## [@adobe/spacecat-shared-data-access-v3.6.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v3.6.0...@adobe/spacecat-shared-data-access-v3.6.1) (2026-03-03)
+
+### Bug Fixes
+
+* stage changes wrt to spacecat-shared ([#1394](https://github.com/adobe/spacecat-shared/issues/1394)) ([7145fb0](https://github.com/adobe/spacecat-shared/commit/7145fb037e5b809d4552889c291b6f8688655b88))
+
 ## [@adobe/spacecat-shared-data-access-v3.6.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v3.5.0...@adobe/spacecat-shared-data-access-v3.6.0) (2026-03-02)
 
 ### Features

package/CLAUDE.md

@@ -21,7 +21,8 @@ Lambda/ECS service
 | File | Purpose |
 |------|---------|
 | `src/index.js` | Default export: `dataAccessWrapper(fn)` for Helix/Lambda handlers |
-| `src/service/index.js` | `createDataAccess(config, log?, client?)` factory |
+| `src/service/index.js` | `createDataAccess(config, log?, client?)` factory — returns entity collections + `services.postgrestClient` |
+| `src/service/index.d.ts` | `DataAccess` and `DataAccessServices` type definitions |
 | `src/models/base/schema.builder.js` | DSL for defining entity schemas (attributes, references, indexes) |
 | `src/models/base/base.model.js` | Base entity class (auto-generated getters/setters, save, remove) |
 | `src/models/base/base.collection.js` | Base collection class (findById, all, query, count) |
@@ -152,6 +153,22 @@ npm run test:it
 - PostgREST calls are stubbed via sinon
 - Each entity model and collection has its own test file
 
+## Direct PostgREST Queries
+
+`dataAccess.services.postgrestClient` exposes the raw `@supabase/postgrest-js` `PostgrestClient` for querying tables that don't have entity models (e.g. analytics views, reporting tables):
+
+```js
+const { postgrestClient } = context.dataAccess.services;
+
+const { data, error } = await postgrestClient
+  .from('brand_presence_executions')
+  .select('*')
+  .eq('site_id', siteId)
+  .limit(100);
+```
+
+This is the same client instance used internally by entity collections — same URL, auth, and schema.
+
 ## Common Patterns
 
 ### Collection query with WHERE clause

package/README.md

@@ -16,9 +16,10 @@ npm install @adobe/spacecat-shared-data-access
 ## What You Get
 
 The package provides:
-- `createDataAccess(config, log?, client?)`
+- `createDataAccess(config, log?, client?)` — returns entity collections + `services.postgrestClient`
 - `dataAccessWrapper(fn)` (default export) for Helix/Lambda style handlers
 - Entity collections/models with stable external API shape for services
+- `services.postgrestClient` for direct PostgREST queries against non-entity tables
 
 ## Quick Start
 
@@ -89,6 +90,29 @@ The wrapper reads from `context.env`:
 - `S3_CONFIG_BUCKET`
 - `AWS_REGION`
 
+## Direct PostgREST Queries
+
+For querying PostgREST tables that are not modeled as entities (e.g. analytics views, reporting tables), the `postgrestClient` is available under `dataAccess.services`:
+
+```js
+const { Site } = dataAccess;                             // entity collections
+const { postgrestClient } = dataAccess.services;         // raw PostgREST client
+
+// Use entity collections as usual
+const site = await Site.findById(siteId);
+
+// Direct queries against non-entity tables
+const { data, error } = await postgrestClient
+  .from('brand_presence_executions')
+  .select('execution_date, visibility_score, sentiment')
+  .eq('site_id', siteId)
+  .gte('execution_date', '2025-01-01')
+  .order('execution_date', { ascending: false })
+  .limit(100);
+```
+
+This is the same `@supabase/postgrest-js` `PostgrestClient` instance used internally by the entity collections. Full IDE autocomplete is available for the query builder chain.
+
 ## Field Mapping Behavior
 
 Public model API remains camelCase while Postgres/PostgREST tables are snake_case.
@@ -383,7 +407,10 @@ export MYSTICAT_DATA_SERVICE_REPOSITORY=682033462621.dkr.ecr.us-east-1.amazonaws
 
 Type definitions are shipped from:
 - `src/index.d.ts`
-- `src/models/**/index.d.ts`
+- `src/service/index.d.ts` — `DataAccess` and `DataAccessServices` interfaces
+- `src/models/**/index.d.ts` — per-entity collection and model interfaces
+
+The `DataAccess` interface provides full typing for all entity collections and `services.postgrestClient` (typed as `PostgrestClient` from `@supabase/postgrest-js`).
 
 Use the package directly in TS projects; no extra setup required.
 

package/package.json

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

package/src/models/base/base.collection.js

@@ -104,21 +104,29 @@ class BaseCollection {
   #isInvalidInputError(error) {
     let current = error;
     while (current) {
-      if (current?.code === '22P02') {
-        return true;
-      }
+      if (current?.code === '22P02') return true;
       current = current.cause;
     }
     return false;
   }
 
   #logAndThrowError(message, cause) {
-    const error = new DataAccessError(message, this, cause);
-    this.log.error(`Base Collection Error [${this.entityName}]`, error);
-    if (isNonEmptyArray(error.cause?.fields)) {
-      this.log.error(`Validation errors: ${JSON.stringify(error.cause.fields)}`);
+    const parts = [message];
+    if (cause?.code) parts.push(`[${cause.code}] ${cause.message}`);
+    if (cause?.details) parts.push(cause.details);
+    if (cause?.hint) parts.push(`hint: ${cause.hint}`);
+
+    this.log.error(`[${this.entityName}] ${parts.join(' - ')}`);
+
+    if (isNonEmptyArray(cause?.fields)) {
+      this.log.error(`Validation errors: ${JSON.stringify(cause.fields)}`);
     }
-    throw error;
+
+    throw new DataAccessError(
+      message,
+      { entityName: this.entityName, tableName: this.tableName },
+      cause,
+    );
   }
 
   #initializeCollectionMethods() {
@@ -547,7 +555,7 @@ class BaseCollection {
       const instances = this.#createInstances(allRows);
       return shouldReturnCursor ? { data: instances, cursor } : instances;
     } catch (error) {
-      if (error instanceof DataAccessError) {
+      if (error.constructor === DataAccessError) {
         throw error;
       }
       return this.#logAndThrowError('Failed to query', error);
@@ -601,6 +609,7 @@ class BaseCollection {
     return isNonEmptyObject(item);
   }
 
+  // eslint-disable-next-line consistent-return
   async batchGetByKeys(keys, options = {}) {
     guardArray('keys', keys, this.entityName, 'any');
 
@@ -677,8 +686,11 @@ class BaseCollection {
         unprocessed: [],
       };
     } catch (error) {
-      this.log.error(`Failed to batch get by keys [${this.entityName}]`, error);
-      throw new DataAccessError('Failed to batch get by keys', this, error);
+      /* c8 ignore next 3 -- re-throw guard (exact match; excludes ValidationError subclass) */
+      if (error.constructor === DataAccessError) {
+        throw error;
+      }
+      this.#logAndThrowError('Failed to batch get by keys', error);
     }
   }
 
@@ -721,6 +733,8 @@ class BaseCollection {
       await this.#onCreate(instance);
       return instance;
     } catch (error) {
+      /* c8 ignore next -- re-throw guard (exact match; excludes ValidationError subclass) */
+      if (error.constructor === DataAccessError) throw error;
       return this.#logAndThrowError('Failed to create', error);
     }
   }
@@ -833,6 +847,8 @@ class BaseCollection {
       await this.#onCreateMany({ createdItems, errorItems });
       return { createdItems, errorItems };
     } catch (error) {
+      /* c8 ignore next -- re-throw guard (exact match; excludes ValidationError subclass) */
+      if (error.constructor === DataAccessError) throw error;
       return this.#logAndThrowError('Failed to create many', error);
     }
   }
@@ -856,7 +872,7 @@ class BaseCollection {
 
     const { error } = await query.select().maybeSingle();
     if (error) {
-      throw new DataAccessError('Failed to update entity', this, error);
+      this.#logAndThrowError('Failed to update entity', error);
     }
   }
 
@@ -917,6 +933,8 @@ class BaseCollection {
       this.#invalidateCache();
       return undefined;
     } catch (error) {
+      /* c8 ignore next -- re-throw guard (exact match; excludes ValidationError subclass) */
+      if (error.constructor === DataAccessError) throw error;
       return this.#logAndThrowError('Failed to save many', error);
     }
   }
@@ -947,6 +965,8 @@ class BaseCollection {
       this.#invalidateCache();
       return undefined;
     } catch (error) {
+      /* c8 ignore next -- re-throw guard (exact match; excludes ValidationError subclass) */
+      if (error.constructor === DataAccessError) throw error;
       return this.#logAndThrowError('Failed to remove by IDs', error);
     }
   }

package/src/models/site/config.js

@@ -344,6 +344,12 @@ export const configSchema = Joi.object({
   edgeOptimizeConfig: Joi.object({
     enabled: Joi.boolean().optional(),
     opted: Joi.number().optional(),
+    stagingDomains: Joi.array().items(
+      Joi.object({
+        domain: Joi.string().required(),
+        id: Joi.string().required(),
+      }),
+    ).optional(),
   }).optional(),
   contentAiConfig: Joi.object({
     index: Joi.string().optional(),