@adobe/spacecat-shared-tokowaka-client 1.4.6 → 1.5.0

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [@adobe/spacecat-shared-tokowaka-client-v1.5.0](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tokowaka-client-v1.4.7...@adobe/spacecat-shared-tokowaka-client-v1.5.0) (2026-01-15)
+
+
+### Features
+
+* add TokowakaKVClient for fetching stale suggestions from edge o… ([#1262](https://github.com/adobe/spacecat-shared/issues/1262)) ([e8d440b](https://github.com/adobe/spacecat-shared/commit/e8d440bcc462419b4c82d3415f3a83dfde5195f1))
+
+# [@adobe/spacecat-shared-tokowaka-client-v1.4.7](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tokowaka-client-v1.4.6...@adobe/spacecat-shared-tokowaka-client-v1.4.7) (2026-01-14)
+
+
+### Bug Fixes
+
+* update tokowaka s3 config with patches ([#1263](https://github.com/adobe/spacecat-shared/issues/1263)) ([59605e9](https://github.com/adobe/spacecat-shared/commit/59605e9b75b7289029bd14e1832e594ea565296d))
+
 # [@adobe/spacecat-shared-tokowaka-client-v1.4.6](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tokowaka-client-v1.4.5...@adobe/spacecat-shared-tokowaka-client-v1.4.6) (2026-01-13)
 
 

package/README.md

@@ -64,10 +64,12 @@ Creates a client instance from a context object.
 
 #### `deploySuggestions(site, opportunity, suggestions)`
 
-Generates configuration and uploads to S3 **per URL**. **Automatically fetches existing configuration for each URL and merges** new suggestions with it. Invalidates CDN cache after upload.
+Generates configuration and uploads to S3 **per URL**. **Automatically fetches existing configuration for each URL and merges** new suggestions with it. Invalidates CDN cache after upload. **Updates the metaconfig's `patches` field** to track deployed endpoints.
 
 **Architecture Change:** Creates one S3 file per URL instead of a single file with all URLs. This prevents files from growing too large over time.
 
+**Metaconfig Tracking:** After successful deployment, the method updates the domain-level metaconfig's `patches` object with the normalized paths of all deployed endpoints (e.g., `{ "/products/item": true }`). This provides a centralized registry of all deployed endpoints for the domain.
+
 **Returns:** `Promise<DeploymentResult>` with:
 - `s3Paths` - Array of S3 keys where configs were uploaded (one per URL)
 - `cdnInvalidations` - Array of CDN invalidation results (one per URL per provider)
@@ -212,7 +214,17 @@ Domain-level metaconfig (created once per domain, shared by all URLs):
 {
   "siteId": "abc-123",
   "apiKeys": ["tokowaka-api-key-1"],
-  "prerender": true
+  "tokowakaEnabled": true,
+  "enhancements": true,
+  "prerender": {
+    "allowList": [],
+    "denyList": ["/*"]
+  },
+  "patches": {
+    "/products/item": true,
+    "/about": true,
+    "/contact": true
+  }
 }
 ```
 

package/package.json

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

package/src/fastly-kv-client.js

@@ -0,0 +1,197 @@
+/*
+ * Copyright 2026 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.
+ */
+
+import {
+  tracingFetch as fetch,
+  hasText,
+  isNonEmptyArray,
+} from '@adobe/spacecat-shared-utils';
+
+const FASTLY_KV_API_BASE = 'https://api.fastly.com/resources/stores/kv';
+const DEFAULT_PAGE_SIZE = 100;
+const DEFAULT_TIMEOUT = 30000;
+
+/**
+ * Client for interacting with Fastly KV Store used by Tokowaka.
+ * Used to fetch stale suggestion IDs for cleanup.
+ *
+ * Key format: `${suggestionId}`
+ * Value format: { url: string, status: 'stale' | 'live' }
+ */
+export class FastlyKVClient {
+  /**
+   * Creates a new FastlyKVClient instance.
+   * @param {object} env - Environment variables containing FASTLY_KV_STORE_ID and FASTLY_API_TOKEN
+   * @param {object} log - Logger instance
+   */
+  constructor(env, log) {
+    if (!hasText(env?.FASTLY_KV_STORE_ID)) {
+      throw new Error('FASTLY_KV_STORE_ID environment variable is required');
+    }
+    if (!hasText(env?.FASTLY_API_TOKEN)) {
+      throw new Error('FASTLY_API_TOKEN environment variable is required');
+    }
+
+    this.storeId = env.FASTLY_KV_STORE_ID;
+    this.apiToken = env.FASTLY_API_TOKEN;
+    this.log = log;
+    this.timeout = env.FASTLY_KV_TIMEOUT || DEFAULT_TIMEOUT;
+  }
+
+  /**
+   * Creates the authorization headers for Fastly API requests.
+   * @returns {object} Headers object
+   */
+  #getHeaders() {
+    return {
+      'Fastly-Key': this.apiToken,
+      Accept: 'application/json',
+    };
+  }
+
+  /**
+   * Retrieves the value for a specific key from the KV store.
+   *
+   * @param {string} key - The key to retrieve
+   * @returns {Promise<{url: string, status: string}|null>} The parsed value or null if not found
+   */
+  async #getValue(key) {
+    const url = `${FASTLY_KV_API_BASE}/${this.storeId}/keys/${encodeURIComponent(key)}`;
+
+    const response = await fetch(url, {
+      method: 'GET',
+      headers: this.#getHeaders(),
+      timeout: this.timeout,
+    });
+
+    if (response.status === 404) {
+      return null;
+    }
+
+    if (!response.ok) {
+      const errorText = await response.text();
+      throw new Error(`Failed to get value from KV Store: ${response.status} - ${errorText}`);
+    }
+
+    const text = await response.text();
+
+    try {
+      return JSON.parse(text);
+    } catch {
+      this.log.warn(`Failed to parse value for key ${key} as JSON`);
+      return { url: text, status: 'unknown' };
+    }
+  }
+
+  /**
+   * Lists stale keys from a single page of the KV store.
+   *
+   * @param {object} [options] - Options for listing keys
+   * @param {number} [options.pageSize=100] - Number of keys to fetch per page
+   * @param {string} [options.cursor] - Cursor for pagination
+   * @returns {Promise<{keys: Array<object>, cursor: string|null}>}
+   */
+  async #listStaleKeysPage(options = {}) {
+    const { pageSize = DEFAULT_PAGE_SIZE, cursor } = options;
+    const staleEntries = [];
+
+    const url = new URL(`${FASTLY_KV_API_BASE}/${this.storeId}/keys`);
+    url.searchParams.set('limit', pageSize.toString());
+    if (cursor) {
+      url.searchParams.set('cursor', cursor);
+    }
+
+    const response = await fetch(url.toString(), {
+      method: 'GET',
+      headers: this.#getHeaders(),
+      timeout: this.timeout,
+    });
+
+    if (!response.ok) {
+      const errorText = await response.text();
+      throw new Error(`Failed to list keys from KV Store: ${response.status} - ${errorText}`);
+    }
+
+    const result = await response.json();
+    const { data: keys = [], meta = {} } = result;
+
+    for (const keyName of keys) {
+      try {
+        // eslint-disable-next-line no-await-in-loop
+        const value = await this.#getValue(keyName);
+
+        const normalizedStatus = typeof value?.status === 'string'
+          ? value.status.trim().toLowerCase()
+          : '';
+
+        if (normalizedStatus === 'stale' && hasText(keyName)) {
+          staleEntries.push({
+            key: keyName,
+            suggestionId: keyName,
+            url: value.url,
+          });
+        }
+      } catch (error) {
+        this.log.warn(`Failed to fetch value for key ${keyName}: ${error.message}`);
+      }
+    }
+
+    return {
+      keys: staleEntries,
+      cursor: meta.next_cursor || null,
+    };
+  }
+
+  /**
+   * Lists all stale suggestion IDs from the KV store, handling pagination automatically.
+   *
+   * @param {object} [options] - Options for listing keys
+   * @param {number} [options.pageSize=100] - Number of keys to fetch per page
+   * @param {number} [options.maxPages=100] - Maximum number of pages to fetch (safety limit)
+   * @returns {Promise<Array<{key: string, suggestionId: string, url: string}>>}
+   */
+  async listAllStaleKeys(options = {}) {
+    const { pageSize = DEFAULT_PAGE_SIZE, maxPages = 100 } = options;
+    const allStaleKeys = [];
+    let cursor = null;
+    let pageCount = 0;
+
+    this.log.info('Starting to fetch stale keys from Fastly KV Store');
+
+    do {
+      // eslint-disable-next-line no-await-in-loop
+      const result = await this.#listStaleKeysPage({ pageSize, cursor });
+
+      if (isNonEmptyArray(result.keys)) {
+        allStaleKeys.push(...result.keys);
+      }
+
+      cursor = result.cursor;
+      pageCount += 1;
+
+      this.log.debug(
+        `Fetched page ${pageCount}, found ${result.keys.length} stale keys, total: ${allStaleKeys.length}`,
+      );
+
+      if (pageCount >= maxPages) {
+        this.log.warn(`Reached maximum page limit (${maxPages}), stopping pagination`);
+        break;
+      }
+    } while (cursor);
+
+    this.log.info(`Completed fetching stale keys: ${allStaleKeys.length} total from ${pageCount} pages`);
+
+    return allStaleKeys;
+  }
+}
+
+export default FastlyKVClient;

package/src/index.d.ts

@@ -128,6 +128,25 @@ export interface Suggestion {
   getUpdatedAt(): string;
 }
 
+export interface FastlyKVEntry {
+  key: string;
+  suggestionId: string;
+  url: string;
+}
+
+export class FastlyKVClient {
+  constructor(env: {
+    FASTLY_KV_STORE_ID: string;
+    FASTLY_API_TOKEN: string;
+    FASTLY_KV_TIMEOUT?: number;
+  }, log: any);
+
+  listAllStaleKeys(options?: {
+    pageSize?: number;
+    maxPages?: number;
+  }): Promise<FastlyKVEntry[]>;
+}
+
 /**
  * Base class for opportunity mappers
  * Extend this class to create custom mappers for new opportunity types

package/src/index.js

@@ -17,11 +17,18 @@ import { v4 as uuidv4 } from 'uuid';
 import MapperRegistry from './mappers/mapper-registry.js';
 import CdnClientRegistry from './cdn/cdn-client-registry.js';
 import { mergePatches } from './utils/patch-utils.js';
-import { getTokowakaConfigS3Path, getTokowakaMetaconfigS3Path, getHostName } from './utils/s3-utils.js';
+import {
+  getTokowakaConfigS3Path,
+  getTokowakaMetaconfigS3Path,
+  getHostName,
+  normalizePath,
+} from './utils/s3-utils.js';
 import { groupSuggestionsByUrlPath, filterEligibleSuggestions } from './utils/suggestion-utils.js';
 import { getEffectiveBaseURL } from './utils/site-utils.js';
 import { fetchHtmlWithWarmup, calculateForwardedHost } from './utils/custom-html-utils.js';
 
+export { FastlyKVClient } from './fastly-kv-client.js';
+
 const HTTP_BAD_REQUEST = 400;
 const HTTP_INTERNAL_SERVER_ERROR = 500;
 const HTTP_NOT_IMPLEMENTED = 501;
@@ -94,6 +101,44 @@ class TokowakaClient {
     return error;
   }
 
+  /**
+   * Updates the metaconfig with deployed endpoint paths
+   * @param {Object} metaconfig - Existing metaconfig object
+   * @param {Array<string>} deployedUrls - Array of successfully deployed URLs
+   * @param {string} baseUrl - Base URL for uploading metaconfig
+   * @returns {Promise<void>}
+   * @private
+   */
+  async #updateMetaconfigWithDeployedPaths(metaconfig, deployedUrls, baseUrl) {
+    if (!Array.isArray(deployedUrls) || deployedUrls.length === 0) {
+      return;
+    }
+
+    try {
+      // Initialize patches field if it doesn't exist
+      const updatedMetaconfig = {
+        ...metaconfig,
+        patches: { ...(metaconfig.patches || {}) },
+      };
+
+      // Extract normalized paths from deployed URLs and add to patches object
+      deployedUrls.forEach((url) => {
+        const urlObj = new URL(url);
+        const normalizedPath = normalizePath(urlObj.pathname);
+        updatedMetaconfig.patches[normalizedPath] = true;
+      });
+
+      await this.uploadMetaconfig(baseUrl, updatedMetaconfig);
+      this.log.info(`Updated metaconfig with ${deployedUrls.length} deployed endpoint(s)`);
+    } catch (error) {
+      this.log.error(`Failed to update metaconfig with deployed paths: ${error.message}`, error);
+      throw this.#createError(
+        `Failed to update metaconfig with deployed paths: ${error.message}`,
+        HTTP_INTERNAL_SERVER_ERROR,
+      );
+    }
+  }
+
   /**
    * Gets the list of CDN providers from environment configuration
    * Supports both single provider (string) and multiple providers (comma-separated string or array)
@@ -597,6 +642,9 @@ class TokowakaClient {
 
     this.log.info(`Uploaded Tokowaka configs for ${s3Paths.length} URLs`);
 
+    // Update metaconfig with deployed paths
+    await this.#updateMetaconfigWithDeployedPaths(metaconfig, deployedUrls, baseURL);
+
     // Invalidate CDN cache for all deployed URLs at once
     const cdnInvalidations = await this.invalidateCdnCache({ urls: deployedUrls });
 

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

@@ -0,0 +1,254 @@
+/*
+ * Copyright 2026 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.
+ */
+
+/* eslint-env mocha */
+
+import { expect } from 'chai';
+import sinon from 'sinon';
+import nock from 'nock';
+import { FastlyKVClient } from '../src/fastly-kv-client.js';
+
+describe('FastlyKVClient', () => {
+  let sandbox;
+  let log;
+  let env;
+
+  const FASTLY_KV_API_BASE = 'https://api.fastly.com/resources/stores/kv';
+  const TEST_STORE_ID = 'test-store-id';
+  const TEST_API_TOKEN = 'test-api-token';
+
+  beforeEach(() => {
+    sandbox = sinon.createSandbox();
+    log = {
+      debug: sandbox.spy(),
+      info: sandbox.spy(),
+      warn: sandbox.spy(),
+      error: sandbox.spy(),
+    };
+    env = {
+      FASTLY_KV_STORE_ID: TEST_STORE_ID,
+      FASTLY_API_TOKEN: TEST_API_TOKEN,
+    };
+  });
+
+  afterEach(() => {
+    sandbox.restore();
+    nock.cleanAll();
+  });
+
+  describe('constructor', () => {
+    it('should create a client with valid configuration', () => {
+      const client = new FastlyKVClient(env, log);
+      expect(client.storeId).to.equal(TEST_STORE_ID);
+      expect(client.apiToken).to.equal(TEST_API_TOKEN);
+    });
+
+    it('should throw error if FASTLY_KV_STORE_ID is missing', () => {
+      delete env.FASTLY_KV_STORE_ID;
+      expect(() => new FastlyKVClient(env, log))
+        .to.throw('FASTLY_KV_STORE_ID environment variable is required');
+    });
+
+    it('should throw error if FASTLY_API_TOKEN is missing', () => {
+      delete env.FASTLY_API_TOKEN;
+      expect(() => new FastlyKVClient(env, log))
+        .to.throw('FASTLY_API_TOKEN environment variable is required');
+    });
+
+    it('should throw error if env is null', () => {
+      expect(() => new FastlyKVClient(null, log))
+        .to.throw('FASTLY_KV_STORE_ID environment variable is required');
+    });
+  });
+
+  describe('listAllStaleKeys', () => {
+    it('should fetch all pages of stale keys', async () => {
+      const client = new FastlyKVClient(env, log);
+      const keysPage1 = ['sugg-1'];
+      const keysPage2 = ['sugg-2'];
+      const staleValue = { url: 'https://example.com', status: 'stale' };
+
+      // Page 1
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys`)
+        .query({ limit: '100' })
+        .reply(200, { data: keysPage1, meta: { next_cursor: 'cursor-1' } });
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys/${encodeURIComponent(keysPage1[0])}`)
+        .reply(200, JSON.stringify(staleValue));
+
+      // Page 2
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys`)
+        .query({ limit: '100', cursor: 'cursor-1' })
+        .reply(200, { data: keysPage2, meta: {} });
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys/${encodeURIComponent(keysPage2[0])}`)
+        .reply(200, JSON.stringify(staleValue));
+
+      const result = await client.listAllStaleKeys();
+
+      expect(result).to.have.lengthOf(2);
+      expect(result[0].suggestionId).to.equal('sugg-1');
+      expect(result[1].suggestionId).to.equal('sugg-2');
+    });
+
+    it('should filter out non-stale keys', async () => {
+      const client = new FastlyKVClient(env, log);
+      const keys = ['sugg-stale', 'sugg-live'];
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys`)
+        .query({ limit: '100' })
+        .reply(200, { data: keys, meta: {} });
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys/${encodeURIComponent(keys[0])}`)
+        .reply(200, JSON.stringify({ url: 'https://example.com/stale', status: 'stale' }));
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys/${encodeURIComponent(keys[1])}`)
+        .reply(200, JSON.stringify({ url: 'https://example.com/live', status: 'live' }));
+
+      const result = await client.listAllStaleKeys();
+
+      expect(result).to.have.lengthOf(1);
+      expect(result[0].suggestionId).to.equal('sugg-stale');
+      expect(result[0].url).to.equal('https://example.com/stale');
+    });
+
+    it('should respect maxPages limit', async () => {
+      const client = new FastlyKVClient(env, log);
+      const staleValue = { url: 'https://example.com', status: 'stale' };
+
+      // Always return a next cursor to simulate infinite pages
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys`)
+        .query(true)
+        .times(3)
+        .reply(200, { data: ['sugg-1'], meta: { next_cursor: 'next' } });
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys/${encodeURIComponent('sugg-1')}`)
+        .times(3)
+        .reply(200, JSON.stringify(staleValue));
+
+      const result = await client.listAllStaleKeys({ maxPages: 3 });
+
+      expect(result).to.have.lengthOf(3);
+      expect(log.warn.calledWith(sinon.match(/Reached maximum page limit/))).to.be.true;
+    });
+
+    it('should return empty array if no stale keys found', async () => {
+      const client = new FastlyKVClient(env, log);
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys`)
+        .query({ limit: '100' })
+        .reply(200, { data: [], meta: {} });
+
+      const result = await client.listAllStaleKeys();
+
+      expect(result).to.have.lengthOf(0);
+    });
+
+    it('should continue processing if individual key fetch fails', async () => {
+      const client = new FastlyKVClient(env, log);
+      const keys = ['sugg-fail', 'sugg-success'];
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys`)
+        .query({ limit: '100' })
+        .reply(200, { data: keys, meta: {} });
+
+      // First key fails
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys/${encodeURIComponent(keys[0])}`)
+        .reply(500, 'Error');
+
+      // Second key succeeds
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys/${encodeURIComponent(keys[1])}`)
+        .reply(200, JSON.stringify({ url: 'https://example.com', status: 'stale' }));
+
+      const result = await client.listAllStaleKeys();
+
+      expect(result).to.have.lengthOf(1);
+      expect(result[0].suggestionId).to.equal('sugg-success');
+      expect(log.warn.called).to.be.true;
+    });
+
+    it('should handle non-JSON response gracefully', async () => {
+      const client = new FastlyKVClient(env, log);
+      const keys = ['sugg-1'];
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys`)
+        .query({ limit: '100' })
+        .reply(200, { data: keys, meta: {} });
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys/${encodeURIComponent(keys[0])}`)
+        .reply(200, 'plain-text-value');
+
+      const result = await client.listAllStaleKeys();
+
+      // Non-JSON response gets status 'unknown', which is not 'stale', so filtered out
+      expect(result).to.have.lengthOf(0);
+      expect(log.warn.called).to.be.true;
+    });
+
+    it('should skip keys that return 404', async () => {
+      const client = new FastlyKVClient(env, log);
+      const keys = ['sugg-deleted', 'sugg-exists'];
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys`)
+        .query({ limit: '100' })
+        .reply(200, { data: keys, meta: {} });
+
+      // First key returns 404
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys/${encodeURIComponent(keys[0])}`)
+        .reply(404, 'Not Found');
+
+      // Second key exists and is stale
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys/${encodeURIComponent(keys[1])}`)
+        .reply(200, JSON.stringify({ url: 'https://example.com', status: 'stale' }));
+
+      const result = await client.listAllStaleKeys();
+
+      // Only the existing stale key should be returned
+      expect(result).to.have.lengthOf(1);
+      expect(result[0].suggestionId).to.equal('sugg-exists');
+    });
+
+    it('should throw error when listing keys fails', async () => {
+      const client = new FastlyKVClient(env, log);
+
+      nock(FASTLY_KV_API_BASE)
+        .get(`/${TEST_STORE_ID}/keys`)
+        .query({ limit: '100' })
+        .reply(500, 'Internal Server Error');
+
+      try {
+        await client.listAllStaleKeys();
+        expect.fail('Should have thrown an error');
+      } catch (error) {
+        expect(error.message).to.include('Failed to list keys from KV Store');
+      }
+    });
+  });
+});

package/test/index.test.js

@@ -1030,6 +1030,161 @@ describe('TokowakaClient', () => {
       expect(result.failedSuggestions[0].suggestion.getId()).to.equal('sugg-2');
     });
 
+    it('should update metaconfig patches field with deployed endpoints', async () => {
+      mockSuggestions = [
+        {
+          getId: () => 'sugg-1',
+          getUpdatedAt: () => '2025-01-15T10:00:00.000Z',
+          getData: () => ({
+            url: 'https://example.com/page1',
+            recommendedAction: 'Page 1 Heading',
+            checkType: 'heading-empty',
+            transformRules: {
+              action: 'replace',
+              selector: 'h1',
+            },
+          }),
+        },
+        {
+          getId: () => 'sugg-2',
+          getUpdatedAt: () => '2025-01-15T10:00:00.000Z',
+          getData: () => ({
+            url: 'https://example.com/page2',
+            recommendedAction: 'Page 2 Heading',
+            checkType: 'heading-empty',
+            transformRules: {
+              action: 'replace',
+              selector: 'h1',
+            },
+          }),
+        },
+      ];
+
+      const result = await client.deploySuggestions(
+        mockSite,
+        mockOpportunity,
+        mockSuggestions,
+      );
+
+      expect(result.succeededSuggestions).to.have.length(2);
+      // Verify uploadMetaconfig was called to update the metaconfig
+      expect(client.uploadMetaconfig).to.have.been.called;
+      // Check that the last call included the patches field
+      const { lastCall } = client.uploadMetaconfig;
+      expect(lastCall.args[1]).to.have.property('patches');
+      expect(lastCall.args[1].patches).to.deep.equal({
+        '/page1': true,
+        '/page2': true,
+      });
+    });
+
+    it('should add to existing patches in metaconfig when deploying new endpoints', async () => {
+      // Set up metaconfig with existing patches
+      // Reset the stub to provide consistent behavior
+      client.fetchMetaconfig.reset();
+      client.fetchMetaconfig.resolves({
+        siteId: 'site-123',
+        prerender: true,
+        patches: {
+          '/existing-page': true,
+        },
+      });
+
+      mockSuggestions = [
+        {
+          getId: () => 'sugg-1',
+          getUpdatedAt: () => '2025-01-15T10:00:00.000Z',
+          getData: () => ({
+            url: 'https://example.com/new-page',
+            recommendedAction: 'New Heading',
+            checkType: 'heading-empty',
+            transformRules: {
+              action: 'replace',
+              selector: 'h1',
+            },
+          }),
+        },
+      ];
+
+      await client.deploySuggestions(
+        mockSite,
+        mockOpportunity,
+        mockSuggestions,
+      );
+
+      // Verify the updated metaconfig includes both existing and new patches
+      const { lastCall } = client.uploadMetaconfig;
+      expect(lastCall.args[1].patches).to.deep.equal({
+        '/existing-page': true,
+        '/new-page': true,
+      });
+    });
+
+    it('should throw error when metaconfig update fails', async () => {
+      // Make uploadMetaconfig fail during the update
+      client.uploadMetaconfig.rejects(new Error('S3 upload error'));
+
+      try {
+        await client.deploySuggestions(
+          mockSite,
+          mockOpportunity,
+          mockSuggestions,
+        );
+        expect.fail('Should have thrown error');
+      } catch (error) {
+        expect(error.message).to.include('Failed to update metaconfig with deployed paths');
+        expect(error.status).to.equal(500);
+      }
+    });
+
+    it('should return early when no eligible suggestions to deploy', async () => {
+      // All suggestions are ineligible
+      mockSuggestions = [
+        {
+          getId: () => 'sugg-1',
+          getData: () => ({
+            url: 'https://example.com/page1',
+            recommendedAction: 'New Heading',
+            checkType: 'heading-missing', // Not eligible
+          }),
+        },
+      ];
+
+      const result = await client.deploySuggestions(
+        mockSite,
+        mockOpportunity,
+        mockSuggestions,
+      );
+
+      // No suggestions deployed - returns early before metaconfig check
+      expect(result.succeededSuggestions).to.have.length(0);
+      expect(result.failedSuggestions).to.have.length(1);
+      // fetchMetaconfig should not be called at all (returns before that point)
+      expect(client.fetchMetaconfig).to.not.have.been.called;
+      // uploadMetaconfig should not be called at all
+      expect(client.uploadMetaconfig).to.not.have.been.called;
+    });
+
+    it('should not update metaconfig when all URLs fail to generate configs', async () => {
+      // Stub generateConfig to return null (no config generated)
+      sinon.stub(client, 'generateConfig').returns(null);
+
+      const result = await client.deploySuggestions(
+        mockSite,
+        mockOpportunity,
+        mockSuggestions,
+      );
+
+      // Suggestions are marked as succeeded (eligible) but no configs uploaded
+      expect(result.succeededSuggestions).to.have.length(2);
+      expect(result.s3Paths).to.have.length(0); // No configs uploaded
+      // fetchMetaconfig called once for initial check, but not for update
+      // since no URLs were actually deployed (deployedUrls is empty)
+      expect(client.fetchMetaconfig).to.have.been.calledOnce;
+      // uploadMetaconfig should not be called at all
+      expect(client.uploadMetaconfig).to.not.have.been.called;
+    });
+
     it('should skip URL when generateConfig returns no patches', async () => {
       // Stub mapper to return empty patches for the first call, normal for subsequent calls
       const mapper = client.mapperRegistry.getMapper('headings');