fix(tokowaka-client): keep edgeOptimizeEnabled as raw routing signal for now

Don't fold `blocked` into edgeOptimizeEnabled yet — persistEdgeConfig's
routingEnabled stamping (import-worker) reads edgeOptimizeEnabled, so
gating it on WAF status today would mean a routing-confirmed-but-WAF-
blocked site never gets routingEnabled stamped, which would misdirect
the frontend's "Configure CDN" vs "running final checks" messaging for
that exact case.

blocked/statusCode are still classified from the same fetched response
and returned + logged, for the frontend to start consuming later, with
retiring the separate checkWafConnectivity call as a further follow-up.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Author: jindaliiita

packages/spacecat-shared-tokowaka-client/src/index.js

@@ -1345,9 +1345,14 @@ class TokowakaClient {
    * Fetches the customer's own domain directly (no proxy) and checks the response for
    * BOTH the routing signal (x-tokowaka-request-id / x-edgeoptimize-request-id header)
    * and a WAF/bot-block (via classifyProbeResponse, on the same fetched response — no
-   * second request). `edgeOptimizeEnabled` folds both together: it is only `true` when
-   * the request-id header is present AND the response isn't classified as blocked —
-   * routing alone is not sufficient.
+   * second request).
+   *
+   * `edgeOptimizeEnabled` is still the raw routing signal only (unchanged from before —
+   * `blocked` does NOT factor into it yet), to stay fully backward compatible with
+   * existing consumers (e.g. the import-worker's routingEnabled stamping). `blocked`/
+   * `statusCode` are returned (and logged) purely for visibility for now; consuming them
+   * on the frontend, and eventually retiring the separate checkWafConnectivity call in
+   * favor of this, is follow-up work.
    *
    * @param {Object} site - Site entity
    * @param {string} path - Path to check (e.g., '/products/chair')
@@ -1407,11 +1412,15 @@ class TokowakaClient {
         this.log.info(`[edge-optimize-status] Edge optimize headers found: ${requestIdPresent} for ${targetUrl}`);
 
         // Classify the same response already fetched above for a WAF/bot-block — no second
-        // request. edgeOptimizeEnabled folds both signals together: routing alone isn't
-        // enough if the customer's own WAF is blocking us.
+        // request. Logged/returned for visibility only for now — edgeOptimizeEnabled stays
+        // the raw routing signal (does NOT fold in `blocked` yet) so this stays fully
+        // backward compatible: persistEdgeConfig's routingEnabled stamping (and everything
+        // downstream of it) is untouched. Consuming `blocked`/`statusCode` on the frontend,
+        // and eventually retiring the separate checkWafConnectivity call, is follow-up work.
         // eslint-disable-next-line no-await-in-loop
         const { blocked, statusCode } = await classifyProbeResponse(response, targetHost, this.log);
-        const edgeOptimizeEnabled = requestIdPresent && !blocked;
+        this.log.info(`[edge-optimize-status] WAF classification for ${targetUrl}: blocked=${blocked}, statusCode=${statusCode}`);
+        const edgeOptimizeEnabled = requestIdPresent;
 
         return {
           edgeOptimizeEnabled,

packages/spacecat-shared-tokowaka-client/test/index.test.js

@@ -5783,7 +5783,7 @@ describe('TokowakaClient', () => {
       });
     });
 
-    describe('WAF classification merged into routing check', () => {
+    describe('WAF classification alongside routing check (logged/returned, not gating yet)', () => {
       let site;
 
       beforeEach(() => {
@@ -5795,7 +5795,7 @@ describe('TokowakaClient', () => {
         };
       });
 
-      it('routing confirmed but hard-blocked (403) → edgeOptimizeEnabled: false, blocked: true', async () => {
+      it('routing confirmed but hard-blocked (403) → edgeOptimizeEnabled stays true (raw routing signal), blocked: true reported separately', async () => {
         const headersMap = new Map([
           ['x-tokowaka-request-id', 'abc123'],
         ]);
@@ -5809,13 +5809,13 @@ describe('TokowakaClient', () => {
         const result = await esmockClient.checkEdgeOptimizeStatus(site, '/');
 
         expect(result).to.deep.equal({
-          edgeOptimizeEnabled: false,
+          edgeOptimizeEnabled: true,
           blocked: true,
           statusCode: 403,
         });
       });
 
-      it('routing confirmed but soft-blocked (challenge page body) → edgeOptimizeEnabled: false, blocked: true', async () => {
+      it('routing confirmed but soft-blocked (challenge page body) → edgeOptimizeEnabled stays true (raw routing signal), blocked: true reported separately', async () => {
         const headersMap = new Map([
           ['x-tokowaka-request-id', 'abc123'],
           ['content-type', 'text/html'],
@@ -5831,7 +5831,7 @@ describe('TokowakaClient', () => {
         const result = await esmockClient.checkEdgeOptimizeStatus(site, '/');
 
         expect(result).to.deep.equal({
-          edgeOptimizeEnabled: false,
+          edgeOptimizeEnabled: true,
           blocked: true,
           statusCode: 200,
         });