@adobe/spacecat-shared-utils 1.73.1 → 1.75.0

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [@adobe/spacecat-shared-utils-v1.75.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-utils-v1.74.0...@adobe/spacecat-shared-utils-v1.75.0) (2025-11-19)
+
+
+### Features
+
+* add aggregation key to get Oppty SC API ([#1148](https://github.com/adobe/spacecat-shared/issues/1148)) ([07bf485](https://github.com/adobe/spacecat-shared/commit/07bf485a5534a066899abf9488ef71301d8cb3e1))
+
+# [@adobe/spacecat-shared-utils-v1.74.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-utils-v1.73.1...@adobe/spacecat-shared-utils-v1.74.0) (2025-11-17)
+
+
+### Features
+
+* export OPPORTUNITY_TYPES constant ([#1134](https://github.com/adobe/spacecat-shared/issues/1134)) ([31ec102](https://github.com/adobe/spacecat-shared/commit/31ec102786e62bc280bf942f30c0d52e6f8ee9f1))
+
 # [@adobe/spacecat-shared-utils-v1.73.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-utils-v1.73.0...@adobe/spacecat-shared-utils-v1.73.1) (2025-11-15)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-utils",
-  "version": "1.73.1",
+  "version": "1.75.0",
   "description": "Shared modules of the Spacecat Services - utils",
   "type": "module",
   "exports": {

package/src/aggregation/aggregation-strategies.js

@@ -0,0 +1,248 @@
+/*
+ * Copyright 2025 Adobe. All rights reserved.
+ * This file is licensed to you under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License. You may obtain a copy
+ * of the License at http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software distributed under
+ * the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR REPRESENTATIONS
+ * OF ANY KIND, either express or implied. See the License for the specific language
+ * governing permissions and limitations under the License.
+ */
+
+/**
+ * Accessibility suggestion aggregation strategies
+ *
+ * Defines how HTML elements with accessibility issues are grouped into database suggestions.
+ * Each granularity level has a function that builds an aggregation key from suggestion data.
+ */
+
+/**
+ * Granularity levels for suggestion aggregation
+ * @enum {string}
+ */
+export const Granularity = {
+  /** One suggestion per HTML element - url|type|selector (e.g., page1|color-contrast|div.header) */
+  INDIVIDUAL: 'INDIVIDUAL',
+
+  /**
+   * One suggestion per issue type per page - url|type (e.g., page1|color-contrast)
+   */
+  PER_PAGE_PER_COMPONENT: 'PER_PAGE_PER_COMPONENT',
+
+  /** One suggestion per page - url (e.g., page1) */
+  PER_PAGE: 'PER_PAGE',
+
+  /**
+   * One suggestion per component type across all pages - type|selector
+   */
+  PER_COMPONENT: 'PER_COMPONENT',
+
+  /** One suggestion per issue type globally - type (e.g., color-contrast) */
+  PER_TYPE: 'PER_TYPE',
+};
+
+/**
+ * Generic key builder that concatenates non-empty values with pipe separator
+ * @param {...string} parts - Variable number of key parts to concatenate
+ * @returns {string} Concatenated key
+ * @private - exported for testing purposes
+ */
+export function buildKey(...parts) {
+  return parts.filter((part) => part != null && part !== '').join('|');
+}
+
+/**
+ * Builds aggregation key for INDIVIDUAL granularity
+ * Key format: url|type|selector|source
+ * @private - exported for testing purposes
+ */
+export function buildIndividualKey({
+  url, issueType, targetSelector, source,
+}) {
+  return buildKey(url, issueType, targetSelector, source);
+}
+
+/**
+ * Builds aggregation key for PER_PAGE_PER_COMPONENT granularity
+ * Key format: url|type|source
+ */
+function buildPerPagePerComponentKey({ url, issueType, source }) {
+  return buildKey(url, issueType, source);
+}
+
+/**
+ * Builds aggregation key for PER_PAGE granularity
+ * Key format: url|source
+ */
+function buildPerPageKey({ url, source }) {
+  return buildKey(url, source);
+}
+
+/**
+ * Builds aggregation key for COMPONENT granularity
+ * Key format: type|selector
+ */
+function buildComponentKey({ issueType, targetSelector }) {
+  return buildKey(issueType, targetSelector);
+}
+
+/**
+ * Builds aggregation key for GLOBAL granularity
+ * Key format: type
+ */
+function buildGlobalKey({ issueType }) {
+  return buildKey(issueType);
+}
+
+/**
+ * Registry of key-building functions by granularity level
+ */
+export const GRANULARITY_KEY_BUILDERS = {
+  [Granularity.INDIVIDUAL]: buildIndividualKey,
+  [Granularity.PER_PAGE_PER_COMPONENT]: buildPerPagePerComponentKey,
+  [Granularity.PER_PAGE]: buildPerPageKey,
+  [Granularity.PER_COMPONENT]: buildComponentKey,
+  [Granularity.PER_TYPE]: buildGlobalKey,
+};
+
+/**
+ * Maps issue types to their aggregation granularity
+ * Based on the nature of each issue and how they should be grouped
+ */
+export const ISSUE_GRANULARITY_MAP = {
+  'color-contrast': Granularity.INDIVIDUAL,
+  list: Granularity.PER_COMPONENT,
+  'aria-roles': Granularity.PER_PAGE_PER_COMPONENT,
+  'image-alt': Granularity.PER_PAGE_PER_COMPONENT,
+  'link-in-text-block': Granularity.PER_PAGE_PER_COMPONENT,
+  'link-name': Granularity.PER_PAGE_PER_COMPONENT,
+  'target-size': Granularity.PER_PAGE_PER_COMPONENT,
+  listitem: Granularity.PER_COMPONENT,
+  label: Granularity.PER_PAGE_PER_COMPONENT,
+  'aria-prohibited-attr': Granularity.PER_TYPE,
+  'button-name': Granularity.PER_PAGE_PER_COMPONENT,
+  'frame-title': Granularity.PER_PAGE_PER_COMPONENT,
+  'aria-valid-attr-value': Granularity.PER_PAGE_PER_COMPONENT,
+  'aria-allowed-attr': Granularity.PER_TYPE,
+  'aria-hidden-focus': Granularity.PER_PAGE_PER_COMPONENT,
+  'nested-interactive': Granularity.PER_PAGE_PER_COMPONENT,
+  'html-has-lang': Granularity.PER_PAGE,
+  'meta-viewport': Granularity.PER_PAGE,
+  'aria-required-children': Granularity.PER_PAGE_PER_COMPONENT,
+  'aria-required-parent': Granularity.PER_PAGE_PER_COMPONENT,
+  'meta-refresh': Granularity.PER_PAGE,
+  'role-img-alt': Granularity.PER_PAGE_PER_COMPONENT,
+  'aria-input-field-name': Granularity.PER_PAGE_PER_COMPONENT,
+  'scrollable-region-focusable': Granularity.PER_PAGE_PER_COMPONENT,
+  'select-name': Granularity.PER_PAGE_PER_COMPONENT,
+};
+
+/**
+ * Gets the granularity level for a specific issue type
+ *
+ * @param {string} issueType - The issue type (e.g., "color-contrast")
+ * @returns {string} The granularity level (defaults to PER_PAGE_PER_COMPONENT)
+ */
+export function getGranularityForIssueType(issueType) {
+  return ISSUE_GRANULARITY_MAP[issueType] || Granularity.PER_PAGE_PER_COMPONENT;
+}
+
+/**
+ * Builds an aggregation key for grouping HTML elements during processing
+ *
+ * @param {string} issueType - The issue type
+ * @param {string} url - Page URL
+ * @param {string} targetSelector - CSS selector for the element
+ * @param {string} source - Optional source identifier
+ * @returns {string} The aggregation key based on the issue type's granularity
+ */
+export function buildAggregationKey(issueType, url, targetSelector, source) {
+  const granularity = getGranularityForIssueType(issueType);
+  const keyBuilder = GRANULARITY_KEY_BUILDERS[granularity];
+
+  /* c8 ignore start - defensive code */
+  if (!keyBuilder) {
+    return buildIndividualKey({
+      url, issueType, targetSelector, source,
+    });
+  }
+  /* c8 ignore stop */
+
+  return keyBuilder({
+    url, issueType, targetSelector, source,
+  });
+}
+
+/**
+ * Builds an aggregation key from suggestion data.
+ * Extracts the necessary fields from a suggestion object and calls buildAggregationKey.
+ *
+ * @param {Object} suggestionData - The suggestion data object
+ * @param {string} suggestionData.url - Page URL
+ * @param {Array} suggestionData.issues - Array of issues
+ * @param {string} suggestionData.source - Optional source
+ * @returns {string|null} The aggregation key based on the issue type's granularity,
+ *   or null if no issues
+ */
+export function buildAggregationKeyFromSuggestion(suggestionData) {
+  // Handle null, undefined, or non-object inputs
+  if (!suggestionData || typeof suggestionData !== 'object') {
+    return null;
+  }
+
+  const { url, issues, source } = suggestionData;
+
+  if (!issues || issues.length === 0) {
+    return null;
+  }
+
+  const firstIssue = issues[0];
+  if (!firstIssue || !firstIssue.type) {
+    return null;
+  }
+
+  const issueType = firstIssue.type;
+  const htmlWithIssue = firstIssue.htmlWithIssues?.[0];
+  // Support both snake_case and camelCase for backwards compatibility
+  const targetSelector = htmlWithIssue?.target_selector || htmlWithIssue?.targetSelector || '';
+
+  return buildAggregationKey(issueType, url, targetSelector, source);
+}
+
+/**
+ * Builds a database-level key for matching suggestions across audit runs.
+ * Used by syncSuggestions to identify existing suggestions.
+ *
+ * This ALWAYS uses INDIVIDUAL granularity (url|type|selector|source) to ensure
+ * each HTML element gets its own suggestion in the database. This prevents
+ * incorrect merging of different HTML elements.
+ *
+ * IMPORTANT: This maintains backwards compatibility with the original buildKey logic
+ * by including a trailing pipe when selector is empty (url|type|).
+ *
+ * @param {Object} suggestionData - The suggestion data object
+ * @param {string} suggestionData.url - Page URL
+ * @param {Array} suggestionData.issues - Array of issues
+ * @param {string} suggestionData.source - Optional source
+ * @returns {string} The key for suggestion matching
+ */
+export function buildSuggestionKey(suggestionData) {
+  const { url, issues, source } = suggestionData;
+
+  if (!issues || issues.length === 0) {
+    return url;
+  }
+
+  const firstIssue = issues[0];
+  const issueType = firstIssue.type;
+  const targetSelector = firstIssue.htmlWithIssues?.[0]?.target_selector || '';
+
+  // Always build INDIVIDUAL-level key for database uniqueness
+  // Backwards compatible: url|type|selector|source or url|type| when selector is empty
+  let key = `${url}|${issueType}|${targetSelector}`;
+  if (source) {
+    key += `|${source}`;
+  }
+  return key;
+}

package/src/constants.js

@@ -10,7 +10,7 @@
  * governing permissions and limitations under the License.
  */
 
-const OPPORTUNITY_TYPES = {
+export const OPPORTUNITY_TYPES = /** @type {const} */ ({
   // Core Audit Types
   ACCESSIBILITY: 'accessibility',
   ALT_TEXT: 'alt-text',
@@ -64,8 +64,4 @@ const OPPORTUNITY_TYPES = {
 
   // Paid Cookie Consent
   PAID_COOKIE_CONSENT: 'paid-cookie-consent',
-};
-
-export {
-  OPPORTUNITY_TYPES,
-};
+});

package/src/index.d.ts

@@ -15,6 +15,8 @@ import type { ISOCalendarWeek } from './calendar-week-helper.js';
 
 export { AUTHORING_TYPES, DELIVERY_TYPES } from './aem.js';
 
+export { OPPORTUNITY_TYPES } from './constants.js';
+
 /** UTILITY FUNCTIONS */
 export function arrayEquals<T>(a: T[], b: T[]): boolean;
 

package/src/index.js

@@ -77,6 +77,8 @@ export { getStoredMetrics, storeMetrics } from './metrics-store.js';
 
 export { s3Wrapper } from './s3.js';
 
+export { OPPORTUNITY_TYPES } from './constants.js';
+
 export { fetch } from './adobe-fetch.js';
 export { tracingFetch, SPACECAT_USER_AGENT } from './tracing-fetch.js';
 export {
@@ -106,3 +108,15 @@ export * as schemas from './schemas.js';
 
 export { detectLocale } from './locale-detect/locale-detect.js';
 export { prettifyLogForwardingConfig } from './cdn-helpers.js';
+
+export {
+  buildAggregationKey,
+  buildAggregationKeyFromSuggestion,
+  buildSuggestionKey,
+  buildIndividualKey,
+  buildKey,
+  getGranularityForIssueType,
+  Granularity,
+  GRANULARITY_KEY_BUILDERS,
+  ISSUE_GRANULARITY_MAP,
+} from './aggregation/aggregation-strategies.js';