@adobe/spacecat-shared-data-access 4.21.1 → 4.22.1

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## [@adobe/spacecat-shared-data-access-v4.22.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v4.22.0...@adobe/spacecat-shared-data-access-v4.22.1) (2026-08-19)
+
+### Bug Fixes
+
+* **data-access:** remove deprecated brand pendingSemrushProvisioning attribute (SITES-49448) ([#1886](https://github.com/adobe/spacecat-shared/issues/1886)) ([3b09e0b](https://github.com/adobe/spacecat-shared/commit/3b09e0b613526030f317db5e1f74324fdeea7ff0)), closes [#5](https://github.com/adobe/spacecat-shared/issues/5) [1867/#1880](https://github.com/1867/spacecat-shared/issues/1880)
+
+## [@adobe/spacecat-shared-data-access-v4.22.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v4.21.1...@adobe/spacecat-shared-data-access-v4.22.0) (2026-08-14)
+
+### Features
+
+* schema changes for trial users entity to allow searching by external user id ([#1871](https://github.com/adobe/spacecat-shared/issues/1871)) ([3ecf546](https://github.com/adobe/spacecat-shared/commit/3ecf5466b41dcd32cf5dd591f8762fefd714af1f))
+
 ## [@adobe/spacecat-shared-data-access-v4.21.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v4.21.0...@adobe/spacecat-shared-data-access-v4.21.1) (2026-08-14)
 
 ### Bug Fixes

package/package.json

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

package/src/models/brand/brand.model.js

@@ -34,6 +34,14 @@ import BaseModel from '../base/base.model.js';
  * `Organization.semrushWorkspaceId` is a DISTINCT field and stays — do not
  * reintroduce a brand mirror by symbol-name sweep.
  *
+ * NOTE: there is also no `pendingSemrushProvisioning` attribute. That
+ * "Save as pending" staging blob (`{primaryUrl, markets, generatePrompts}`,
+ * mapped to `brands.pending_semrush_provisioning`) was removed in SITES-49448
+ * once the brand/market management model made pending brands carry no
+ * markets, models chosen per market, and the primary URL a `site_id`
+ * selection at create time — the blob's shape no longer matches anything the
+ * product writes.
+ *
  * @class Brand
  * @extends BaseModel
  */

package/src/models/brand/brand.schema.js

@@ -49,15 +49,6 @@ const schema = new SchemaBuilder(Brand, BrandCollection)
     type: 'string',
     validate: (value) => value == null || hasText(value),
   })
-  // Nullable JSONB blob holding deferred Semrush provisioning data for a
-  // pending (draft) brand. Maps to brands.pending_semrush_provisioning
-  // (migration 20260618120000). Validated loosely here (object-or-null); the
-  // DB CHECK enforces the object shape and the controller owns field-level
-  // validation and the activate-flow consumption semantics.
-  .addAttribute('pendingSemrushProvisioning', {
-    type: 'any',
-    validate: (value) => value == null || (typeof value === 'object' && !Array.isArray(value)),
-  })
   // Uniqueness guarantee on the write-of-record column
   // (brands.semrush_sub_workspace_id, mysticat-data-service migration
   // 20260702091920), so findBySemrushSubWorkspaceId returns at most one row.

package/src/models/trial-user/index.d.ts

@@ -43,9 +43,29 @@ export interface TrialUserCollection extends BaseCollection<TrialUser> {
   allByProviderAndExternalUserId(provider: ProviderType, externalId: string): Promise<TrialUser[]>;
   allByOrganizationId(organizationId: string): Promise<TrialUser[]>;
   allByEmailId(emailId: string): Promise<TrialUser[]>;
+  allByExternalUserId(externalUserId: string): Promise<TrialUser[]>;
+  allByExternalUserIdAndUpdatedAt(
+    externalUserId: string,
+    updatedAt: string
+  ): Promise<TrialUser[]>;
   findByProvider(provider: ProviderType): Promise<TrialUser | null>;
   findByProviderAndExternalUserId(provider: ProviderType, externalId: string):
     Promise<TrialUser | null>;
   findByOrganizationId(organizationId: string): Promise<TrialUser | null>;
   findByEmailId(emailId: string): Promise<TrialUser | null>;
+  /**
+   * externalUserId is optional and not enforced unique (no DB unique constraint), so this
+   * returns one arbitrary match when multiple trial users share the same externalUserId.
+   * Use allByExternalUserId if you need every matching record.
+   */
+  findByExternalUserId(externalUserId: string): Promise<TrialUser | null>;
+  /**
+   * externalUserId is optional and not enforced unique (no DB unique constraint), so this
+   * returns one arbitrary match when multiple trial users share the same externalUserId.
+   * Use allByExternalUserIdAndUpdatedAt if you need every matching record.
+   */
+  findByExternalUserIdAndUpdatedAt(
+    externalUserId: string,
+    updatedAt: string
+  ): Promise<TrialUser | null>;
 }

package/src/models/trial-user/trial-user.schema.js

@@ -53,6 +53,11 @@ const schema = new SchemaBuilder(TrialUser, TrialUserCollection)
     { composite: ['organizationId'] },
     { composite: ['updatedAt'] },
   )
+  // externalUserId is not enforced unique (no DB unique constraint) and is optional, so
+  // findByExternalUserId only returns one arbitrary match. Prefer allByExternalUserId
+  // unless the caller can guarantee uniqueness for their use case.
+  // Note: this schema is now at 4 of the 5-index cap enforced by SchemaBuilder
+  // (#buildIndexes) - budget the next lookup index accordingly.
   .addIndex(
     { composite: ['externalUserId'] },
     { composite: ['updatedAt'] },