@adobe/spacecat-shared-tokowaka-client 1.13.2 → 1.13.3

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## [@adobe/spacecat-shared-tokowaka-client-v1.13.3](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tokowaka-client-v1.13.2...@adobe/spacecat-shared-tokowaka-client-v1.13.3) (2026-04-09)
+
+### Bug Fixes
+
+* skip HTTP check when edge optimize already enabled in site config ([#1519](https://github.com/adobe/spacecat-shared/issues/1519)) ([057608d](https://github.com/adobe/spacecat-shared/commit/057608da86fef2538b68a425d96fa351a4231076))
+
 ## [@adobe/spacecat-shared-tokowaka-client-v1.13.2](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tokowaka-client-v1.13.1...@adobe/spacecat-shared-tokowaka-client-v1.13.2) (2026-04-06)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-tokowaka-client",
-  "version": "1.13.2",
+  "version": "1.13.3",
   "description": "Tokowaka Client for SpaceCat - Edge optimization config management",
   "type": "module",
   "engines": {

package/src/index.js

@@ -1129,6 +1129,13 @@ class TokowakaClient {
       throw this.#createError('Path is required', HTTP_BAD_REQUEST);
     }
 
+    const currentConfig = site.getConfig();
+    const existingEdgeConfig = currentConfig?.getEdgeOptimizeConfig();
+    const isAlreadyEnabled = existingEdgeConfig?.enabled ?? false;
+    if (isAlreadyEnabled) {
+      return { edgeOptimizeEnabled: true };
+    }
+
     const baseURL = getEffectiveBaseURL(site);
     const targetUrl = new URL(path, baseURL).toString();
 
@@ -1341,8 +1348,12 @@ class TokowakaClient {
 
           if (regexPatterns.length > 0) {
             const covered = allSuggestions.filter((s) => {
-              if (s.getId() === suggestion.getId()) return false;
-              if (skippedInBatchIds.has(s.getId())) return false;
+              if (s.getId() === suggestion.getId()) {
+                return false;
+              }
+              if (skippedInBatchIds.has(s.getId())) {
+                return false;
+              }
               if (!isEdgeDeployableSuggestionStatus(s.getStatus())) {
                 return false;
               }

package/src/mappers/commerce-page-enrichment-mapper.js

@@ -58,26 +58,34 @@ function renderList(items, cls) {
 }
 
 function renderValue(key, value, cls) {
-  if (value == null) return '';
+  if (value == null) {
+    return '';
+  }
 
   if (key === 'facts.facets.category_path' || key === 'category') {
     return `<p class="${cls}">${renderCategoryPath(value)}</p>`;
   }
 
   if (Array.isArray(value)) {
-    if (value.length === 0) return '';
+    if (value.length === 0) {
+      return '';
+    }
     return renderList(value, cls);
   }
 
   if (typeof value === 'object') {
     const entries = Object.entries(value).filter(([, v]) => v != null && v !== '');
-    if (entries.length === 0) return '';
+    if (entries.length === 0) {
+      return '';
+    }
     const items = entries.map(([k, v]) => `${escapeHtml(k)}: ${escapeHtml(String(v))}`);
     return renderList(items, cls);
   }
 
   const str = String(value);
-  if (!str) return '';
+  if (!str) {
+    return '';
+  }
   return `<p class="${cls}">${escapeHtml(str)}</p>`;
 }
 

package/src/mappers/generic-mapper.js

@@ -36,8 +36,12 @@ export default class GenericMapper extends BaseOpportunityMapper {
 
   // eslint-disable-next-line class-methods-use-this
   #resolveValue(data) {
-    if (data.format === 'html') return htmlToHast(data.patchValue);
-    if (data.format === 'hast' || data.format === 'json') return JSON.parse(data.patchValue);
+    if (data.format === 'html') {
+      return htmlToHast(data.patchValue);
+    }
+    if (data.format === 'hast' || data.format === 'json') {
+      return JSON.parse(data.patchValue);
+    }
     return data.patchValue;
   }
 

package/test/cdn/base-cdn-client.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import BaseCdnClient from '../../src/cdn/base-cdn-client.js';
 

package/test/cdn/cdn-client-registry.test.js

@@ -10,7 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
 /* eslint-disable max-classes-per-file */
 
 import { expect, use } from 'chai';

package/test/cdn/cloudfront-cdn-client.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect, use } from 'chai';
 import sinon from 'sinon';
 import sinonChai from 'sinon-chai';

package/test/cdn/fastly-cdn-client.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import sinon from 'sinon';
 import FastlyCdnClient from '../../src/cdn/fastly-cdn-client.js';

package/test/fastly-kv-client.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import sinon from 'sinon';
 import nock from 'nock';

package/test/index.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect, use } from 'chai';
 import sinon from 'sinon';
 import sinonChai from 'sinon-chai';
@@ -3844,7 +3842,7 @@ describe('TokowakaClient', () => {
         const site = {
           getId: () => 'site-id',
           getBaseURL: () => 'https://example.com',
-          getConfig: () => ({}),
+          getConfig: () => ({ getEdgeOptimizeConfig: () => undefined }),
           getDeliveryType: () => 'aem_edge',
         };
 
@@ -3861,7 +3859,7 @@ describe('TokowakaClient', () => {
         const site = {
           getId: () => 'site-id',
           getBaseURL: () => 'https://example.com',
-          getConfig: () => ({}),
+          getConfig: () => ({ getEdgeOptimizeConfig: () => undefined }),
           getDeliveryType: () => 'aem_edge',
         };
 
@@ -3874,6 +3872,76 @@ describe('TokowakaClient', () => {
       });
     });
 
+    describe('Early Return (Already Enabled via Site Config)', () => {
+      it('should return edgeOptimizeEnabled: true immediately when enabled is boolean true, without making HTTP request', async () => {
+        const site = {
+          getId: () => 'site-id',
+          getBaseURL: () => 'https://example.com',
+          getConfig: () => ({ getEdgeOptimizeConfig: () => ({ enabled: true }) }),
+          getDeliveryType: () => 'aem_edge',
+        };
+
+        const result = await esmockClient.checkEdgeOptimizeStatus(site, '/');
+
+        expect(result).to.deep.equal({ edgeOptimizeEnabled: true });
+        expect(tracingFetchStub).to.not.have.been.called;
+      });
+
+      it('should return edgeOptimizeEnabled: true immediately when enabled is a timestamp (number), without making HTTP request', async () => {
+        const site = {
+          getId: () => 'site-id',
+          getBaseURL: () => 'https://example.com',
+          getConfig: () => ({ getEdgeOptimizeConfig: () => ({ enabled: 1772531669121 }) }),
+          getDeliveryType: () => 'aem_edge',
+        };
+
+        const result = await esmockClient.checkEdgeOptimizeStatus(site, '/products');
+
+        expect(result).to.deep.equal({ edgeOptimizeEnabled: true });
+        expect(tracingFetchStub).to.not.have.been.called;
+      });
+
+      it('should NOT return early and should make HTTP request when enabled is false', async () => {
+        const site = {
+          getId: () => 'site-id',
+          getBaseURL: () => 'https://example.com',
+          getConfig: () => ({ getEdgeOptimizeConfig: () => ({ enabled: false }) }),
+          getDeliveryType: () => 'aem_edge',
+        };
+
+        const mockResponse = {
+          status: 200,
+          headers: { get: () => null },
+        };
+        tracingFetchStub.resolves(mockResponse);
+
+        const result = await esmockClient.checkEdgeOptimizeStatus(site, '/');
+
+        expect(result.edgeOptimizeEnabled).to.be.false;
+        expect(tracingFetchStub).to.have.been.called;
+      });
+
+      it('should NOT return early and should make HTTP request when edgeOptimizeConfig is undefined', async () => {
+        const site = {
+          getId: () => 'site-id',
+          getBaseURL: () => 'https://example.com',
+          getConfig: () => ({ getEdgeOptimizeConfig: () => undefined }),
+          getDeliveryType: () => 'aem_edge',
+        };
+
+        const mockResponse = {
+          status: 200,
+          headers: { get: () => null },
+        };
+        tracingFetchStub.resolves(mockResponse);
+
+        const result = await esmockClient.checkEdgeOptimizeStatus(site, '/');
+
+        expect(result.edgeOptimizeEnabled).to.be.false;
+        expect(tracingFetchStub).to.have.been.called;
+      });
+    });
+
     describe('Direct Response (No Redirect)', () => {
       let site;
 
@@ -3881,7 +3949,7 @@ describe('TokowakaClient', () => {
         site = {
           getId: () => 'site-id',
           getBaseURL: () => 'https://example.com',
-          getConfig: () => ({}),
+          getConfig: () => ({ getEdgeOptimizeConfig: () => undefined }),
           getDeliveryType: () => 'aem_edge',
         };
       });
@@ -4006,7 +4074,7 @@ describe('TokowakaClient', () => {
         site = {
           getId: () => 'site-id',
           getBaseURL: () => 'https://example.com',
-          getConfig: () => ({}),
+          getConfig: () => ({ getEdgeOptimizeConfig: () => undefined }),
           getDeliveryType: () => 'aem_edge',
         };
         clock = sinon.useFakeTimers();
@@ -4141,7 +4209,7 @@ describe('TokowakaClient', () => {
         site = {
           getId: () => 'site-id',
           getBaseURL: () => 'https://example.com',
-          getConfig: () => ({}),
+          getConfig: () => ({ getEdgeOptimizeConfig: () => undefined }),
           getDeliveryType: () => 'aem_edge',
         };
       });
@@ -4178,7 +4246,7 @@ describe('TokowakaClient', () => {
         site = {
           getId: () => 'site-id',
           getBaseURL: () => 'https://example.com/',
-          getConfig: () => ({}),
+          getConfig: () => ({ getEdgeOptimizeConfig: () => undefined }),
           getDeliveryType: () => 'aem_edge',
         };
 
@@ -4199,7 +4267,7 @@ describe('TokowakaClient', () => {
         site = {
           getId: () => 'site-id',
           getBaseURL: () => 'https://example.com',
-          getConfig: () => ({}),
+          getConfig: () => ({ getEdgeOptimizeConfig: () => undefined }),
           getDeliveryType: () => 'aem_edge',
         };
 
@@ -4224,7 +4292,7 @@ describe('TokowakaClient', () => {
         site = {
           getId: () => 'site-id',
           getBaseURL: () => 'https://example.com',
-          getConfig: () => ({}),
+          getConfig: () => ({ getEdgeOptimizeConfig: () => undefined }),
           getDeliveryType: () => 'aem_edge',
         };
       });

package/test/mappers/base-mapper.test.js

@@ -10,7 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
 /* eslint-disable max-classes-per-file, class-methods-use-this */
 
 import { expect } from 'chai';

package/test/mappers/commerce-page-enrichment-mapper.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import sinon from 'sinon';
 import CommercePageEnrichmentMapper from '../../src/mappers/commerce-page-enrichment-mapper.js';
@@ -20,11 +18,15 @@ import CommercePageEnrichmentMapper from '../../src/mappers/commerce-page-enrich
  * Recursively finds the first HAST element node matching a predicate.
  */
 function findNode(node, predicate) {
-  if (predicate(node)) return node;
+  if (predicate(node)) {
+    return node;
+  }
   if (node.children) {
     for (const child of node.children) {
       const found = findNode(child, predicate);
-      if (found) return found;
+      if (found) {
+        return found;
+      }
     }
   }
   return null;
@@ -35,7 +37,9 @@ function findNode(node, predicate) {
  */
 function findAllNodes(node, predicate) {
   const results = [];
-  if (predicate(node)) results.push(node);
+  if (predicate(node)) {
+    results.push(node);
+  }
   if (node.children) {
     for (const child of node.children) {
       results.push(...findAllNodes(child, predicate));
@@ -48,8 +52,12 @@ function findAllNodes(node, predicate) {
  * Extracts concatenated text content from a HAST subtree.
  */
 function textContent(node) {
-  if (node.type === 'text') return node.value;
-  if (node.children) return node.children.map(textContent).join('');
+  if (node.type === 'text') {
+    return node.value;
+  }
+  if (node.children) {
+    return node.children.map(textContent).join('');
+  }
   return '';
 }
 

package/test/mappers/content-mapper.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import ContentMapper from '../../src/mappers/content-summarization-mapper.js';
 

package/test/mappers/faq-mapper.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import FaqMapper from '../../src/mappers/faq-mapper.js';
 

package/test/mappers/generic-mapper.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { readFileSync } from 'fs';
 import { dirname, join } from 'path';
 import { fileURLToPath } from 'url';

package/test/mappers/headings-mapper.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import HeadingsMapper from '../../src/mappers/headings-mapper.js';
 

package/test/mappers/mapper-registry.test.js

@@ -10,7 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
 /* eslint-disable max-classes-per-file */
 
 import { expect, use } from 'chai';

package/test/mappers/prerender-mapper.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import PrerenderMapper from '../../src/mappers/prerender-mapper.js';
 

package/test/mappers/readability-mapper.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import ReadabilityMapper from '../../src/mappers/readability-mapper.js';
 

package/test/mappers/semantic-value-visibility-mapper.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import { readFileSync } from 'fs';
 import { dirname, join } from 'path';

package/test/mappers/toc-mapper.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import TocMapper from '../../src/mappers/toc-mapper.js';
 

package/test/utils/html-utils.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import sinon from 'sinon';
 import { fetchHtmlWithWarmup, calculateForwardedHost } from '../../src/utils/custom-html-utils.js';
@@ -656,8 +654,12 @@ describe('HTML Utils', () => {
         statusText: 'OK',
         headers: {
           get: (name) => {
-            if (name === 'x-edgeoptimize-cache') return 'HIT';
-            if (name === 'x-edgeoptimize-proxy') return 'true';
+            if (name === 'x-edgeoptimize-cache') {
+              return 'HIT';
+            }
+            if (name === 'x-edgeoptimize-proxy') {
+              return 'true';
+            }
             return null;
           },
         },
@@ -794,8 +796,12 @@ describe('HTML Utils', () => {
         statusText: 'OK',
         headers: {
           get: (name) => {
-            if (name === 'x-edgeoptimize-cache') return 'HIT';
-            if (name === 'x-edgeoptimize-proxy') return 'true';
+            if (name === 'x-edgeoptimize-cache') {
+              return 'HIT';
+            }
+            if (name === 'x-edgeoptimize-proxy') {
+              return 'true';
+            }
             return null;
           },
         },

package/test/utils/htmlToHast.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import { htmlToHast } from '../../src/utils/html-utils.js';
 

package/test/utils/patch-utils.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import { mergePatches, removePatchesBySuggestionIds } from '../../src/utils/patch-utils.js';
 

package/test/utils/s3-utils.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import {
   normalizePath,

package/test/utils/site-utils.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import { getEffectiveBaseURL } from '../../src/utils/site-utils.js';
 

package/test/utils/suggestion-utils.test.js

@@ -10,8 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/* eslint-env mocha */
-
 import { expect } from 'chai';
 import { groupSuggestionsByUrlPath, filterEligibleSuggestions } from '../../src/utils/suggestion-utils.js';