@adobe/spacecat-shared-data-access 2.65.1 → 2.65.2

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [@adobe/spacecat-shared-data-access-v2.65.2](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v2.65.1...@adobe/spacecat-shared-data-access-v2.65.2) (2025-10-01)
+
+
+### Bug Fixes
+
+* remove all debug logs from data access ([#994](https://github.com/adobe/spacecat-shared/issues/994)) ([453d55b](https://github.com/adobe/spacecat-shared/commit/453d55bde5373519b9102ffdc8ca1ea523338078))
+
 # [@adobe/spacecat-shared-data-access-v2.65.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-data-access-v2.65.0...@adobe/spacecat-shared-data-access-v2.65.1) (2025-10-01)
 
 

package/package.json

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

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

@@ -260,26 +260,12 @@ class BaseCollection {
                                  || (options.fetchAllPages !== false && !options.limit);
 
       if (shouldFetchAllPages) {
-        let pageCount = 1;
-        const startTime = Date.now();
-
         while (result.cursor) {
-          pageCount += 1;
           queryOptions.cursor = result.cursor;
           // eslint-disable-next-line no-await-in-loop
           result = await query.go(queryOptions);
           allData = allData.concat(result.data);
         }
-
-        // Debug logging to track pagination behavior
-        if (pageCount > 1) {
-          this.log.debug(`Pagination completed for ${this.entityName}`, {
-            entityName: this.entityName,
-            totalPages: pageCount,
-            totalRecords: allData.length,
-            queryDuration: Date.now() - startTime,
-          });
-        }
       }
 
       if (options.limit === 1) {
@@ -415,8 +401,6 @@ class BaseCollection {
 
       this.#invalidateCache();
 
-      this.log.debug(`Created item for [${this.entityName}]`); // ~100k logs
-
       await this.#onCreate(instance);
 
       return instance;
@@ -495,8 +479,6 @@ class BaseCollection {
 
       this.#invalidateCache();
 
-      this.log.debug(`Created ${createdItems.length} items for [${this.entityName}]`);
-
       await this.#onCreateMany({ createdItems, errorItems });
 
       return { createdItems, errorItems };
@@ -537,8 +519,6 @@ class BaseCollection {
         this.log.error(`Failed to process all items in batch write for [${this.entityName}]: ${JSON.stringify(response.unprocessed)}`);
       }
 
-      this.log.debug(`Updated ${items.length} items for [${this.entityName}]`);
-
       return this.#invalidateCache();
     } catch (error) {
       return this.#logAndThrowError('Failed to save many', error);
@@ -565,8 +545,6 @@ class BaseCollection {
 
       await this.entity.delete(ids.map((id) => ({ [this.idName]: id }))).go();
 
-      this.log.debug(`Removed ${ids.length} items for [${this.entityName}]`);
-
       return this.#invalidateCache();
     } catch (error) {
       return this.#logAndThrowError('Failed to remove by IDs', error);

package/src/models/base/base.model.js

@@ -267,8 +267,6 @@ class BaseModel {
         }
       });
 
-      this.log.debug(`Removing entity ${this.entityName} with ID ${this.getId()} and ${dependents.length} dependents`);
-
       await Promise.all(removePromises);
 
       await this.entity.remove({ [this.idName]: this.getId() }).go();
@@ -297,8 +295,6 @@ class BaseModel {
   async save() {
     // todo: validate associations
     try {
-      this.log.debug(`Saving entity ${this.entityName} with ID ${this.getId()}`);
-
       await this.patcher.save();
       this.#invalidateCache();
 

package/src/models/base/entity.registry.js

@@ -97,18 +97,6 @@ class EntityRegistry {
       const collection = new Collection(this.service, this, schema, this.log);
       this.collections.set(Collection.name, collection);
     });
-
-    this.#logIndexes();
-  }
-
-  #logIndexes() {
-    // reduce collection schema indexes into object
-    const indexes = Object.values(EntityRegistry.entities).reduce((acc, { schema }) => {
-      acc[schema.getEntityName()] = schema.indexes;
-      return acc;
-    }, {});
-
-    this.log.debug('Indexes:', JSON.stringify(indexes, null, 2));
   }
 
   /**

package/src/models/base/reference.js

@@ -45,7 +45,6 @@ const createSortKeyAccessorConfigs = (
 
   const sortKeys = belongsToRef.getSortKeys();
   if (!isNonEmptyArray(sortKeys)) {
-    log.debug(`No sort keys defined for ${entity.schema.getModelName()} to ${target}`);
     return configs;
   }
 

package/src/models/base/schema.js

@@ -315,16 +315,15 @@ class Schema {
    * @throws {SchemaError} - Throws an error if the entity is not a BaseModel or BaseCollection.
    * @return {Object[]}
    */
-  toAccessorConfigs(entity, log = console) {
+  toAccessorConfigs(entity) {
     if (!(entity instanceof BaseModel) && !(entity instanceof BaseCollection)) {
       throw new SchemaError(this, 'Entity must extend BaseModel or BaseCollection');
     }
 
     const indexAccessors = this.getIndexAccessors();
     const accessorConfigs = [];
-    const createdAccessors = [];
 
-    indexAccessors.forEach(({ indexName, keySets }) => {
+    indexAccessors.forEach(({ keySets }) => {
       // generate a method for each prefix of the keySets array
       // for example, if keySets = ['opportunityId', 'status'], we create:
       //   allByOpportunityId(...)
@@ -346,14 +345,9 @@ class Schema {
           name: keyNamesToMethodName(subset, 'findBy'),
           requiredKeys: subset,
         });
-        createdAccessors.push(`index [${indexName}] with keys [${subset.join(', ')}]`);
       });
     });
 
-    if (createdAccessors.length > 0) {
-      log.debug(`Created accessors:\n${createdAccessors.map((accessor) => `  - ${accessor}`).join('\n')}`);
-    }
-
     return accessorConfigs;
   }
 

package/src/models/site-top-form/site-top-form.collection.js

@@ -150,7 +150,7 @@ class SiteTopFormCollection extends BaseCollection {
           });
         } catch (legacyError) {
           // If the null search also fails, ignore and return the original null result
-          this.log.debug(`Legacy null formSource search failed: ${legacyError.message}`);
+          this.log.error(`Legacy null formSource search failed: ${legacyError.message}`);
         }
       }
 

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

@@ -48,8 +48,6 @@ class SuggestionCollection extends BaseCollection {
 
     await this._saveMany(suggestions);
 
-    this.log.debug(`Bulk updated ${suggestions.length} suggestions to status: ${status}`);
-
     return suggestions;
   }
 }