npm - ghost - Versions diffs - 6.54.1 → 6.55.0 - Mend

ghost 6.54.1 → 6.55.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (247) hide show

package/core/built/admin/index.html CHANGED Viewed

@@ -2,8 +2,8 @@
 <html lang="en">
   <head>
     <link rel="stylesheet" href="./assets/vendor-3bf602dfa6463c3400ffd81697767274.css">
-    <link rel="stylesheet" href="./assets/ghost-3063eae29e6c9cc6d6e7131928a27da0.css">
-    <meta name="ghost-admin/config/environment" content="%7B%22modulePrefix%22%3A%22ghost-admin%22%2C%22environment%22%3A%22production%22%2C%22editorUrl%22%3A%22%22%2C%22rootURL%22%3A%22%22%2C%22locationType%22%3A%22trailing-hash%22%2C%22EmberENV%22%3A%7B%22FEATURES%22%3A%7B%7D%2C%22EXTEND_PROTOTYPES%22%3A%7B%22Date%22%3Afalse%2C%22Array%22%3Atrue%2C%22String%22%3Atrue%2C%22Function%22%3Afalse%7D%2C%22_APPLICATION_TEMPLATE_WRAPPER%22%3Afalse%2C%22_JQUERY_INTEGRATION%22%3Atrue%2C%22_TEMPLATE_ONLY_GLIMMER_COMPONENTS%22%3Atrue%7D%2C%22APP%22%3A%7B%22version%22%3A%226.54%22%2C%22name%22%3A%22ghost-admin%22%7D%2C%22ember-simple-auth%22%3A%7B%7D%2C%22%40sentry%2Fember%22%3A%7B%22disablePerformance%22%3Atrue%2C%22sentry%22%3A%7B%7D%7D%2C%22ember-cli-mirage%22%3A%7B%22usingProxy%22%3Afalse%2C%22useDefaultPassthroughs%22%3Atrue%7D%2C%22exportApplicationGlobal%22%3Afalse%2C%22ember-load%22%3A%7B%22loadingIndicatorClass%22%3A%22ember-load-indicator%22%7D%2C%22editorFilename%22%3A%22koenig-lexical.umd.js%22%2C%22editorHash%22%3A%22293973f65e%22%2C%22activitypubFilename%22%3A%22activitypub.js%22%2C%22activitypubHash%22%3A%22b9d03a4632%22%2C%22activitypubRemoteConfigUrl%22%3A%22%2F.ghost%2Factivitypub%2Fstable%2Fclient-config%22%7D">
+    <link rel="stylesheet" href="./assets/ghost-a9c788f6f0b1ea6cf0f14ca4b13f4496.css">
+    <meta name="ghost-admin/config/environment" content="%7B%22modulePrefix%22%3A%22ghost-admin%22%2C%22environment%22%3A%22production%22%2C%22editorUrl%22%3A%22%22%2C%22rootURL%22%3A%22%22%2C%22locationType%22%3A%22trailing-hash%22%2C%22EmberENV%22%3A%7B%22FEATURES%22%3A%7B%7D%2C%22EXTEND_PROTOTYPES%22%3A%7B%22Date%22%3Afalse%2C%22Array%22%3Atrue%2C%22String%22%3Atrue%2C%22Function%22%3Afalse%7D%2C%22_APPLICATION_TEMPLATE_WRAPPER%22%3Afalse%2C%22_JQUERY_INTEGRATION%22%3Atrue%2C%22_TEMPLATE_ONLY_GLIMMER_COMPONENTS%22%3Atrue%7D%2C%22APP%22%3A%7B%22version%22%3A%226.55%22%2C%22name%22%3A%22ghost-admin%22%7D%2C%22ember-simple-auth%22%3A%7B%7D%2C%22%40sentry%2Fember%22%3A%7B%22disablePerformance%22%3Atrue%2C%22sentry%22%3A%7B%7D%7D%2C%22ember-cli-mirage%22%3A%7B%22usingProxy%22%3Afalse%2C%22useDefaultPassthroughs%22%3Atrue%7D%2C%22exportApplicationGlobal%22%3Afalse%2C%22ember-load%22%3A%7B%22loadingIndicatorClass%22%3A%22ember-load-indicator%22%7D%2C%22editorFilename%22%3A%22koenig-lexical.umd.js%22%2C%22editorHash%22%3A%22e35f1aa34a%22%2C%22activitypubFilename%22%3A%22activitypub.js%22%2C%22activitypubHash%22%3A%22d7e993cdd1%22%2C%22activitypubRemoteConfigUrl%22%3A%22%2F.ghost%2Factivitypub%2Fstable%2Fclient-config%22%7D">
     <meta charset="UTF-8" />
@@ -20,13 +20,13 @@
     <link rel="shortcut icon" href="./assets/favicon-C_q1KHzv.ico" />
     <link rel="apple-touch-icon" href="./assets/apple-touch-icon-DhO18_4B.png" />
-    <script type="module" crossorigin src="./assets/index-ukW_0Uie.js"></script>
+    <script type="module" crossorigin src="./assets/index-BGG7NckO.js"></script>
     <link rel="modulepreload" crossorigin href="./assets/rolldown-runtime-DAXXjFlN.js">
     <link rel="modulepreload" crossorigin href="./assets/react-Bb3IsNbi.js">
     <link rel="modulepreload" crossorigin href="./assets/jsx-runtime-DX3WBWVO.js">
     <link rel="modulepreload" crossorigin href="./assets/framework-provider-CmtnPXIT.js">
     <link rel="modulepreload" crossorigin href="./assets/dist-Ndy-PuGN.js">
-    <link rel="modulepreload" crossorigin href="./assets/hooks-BApwwBE0.js">
+    <link rel="modulepreload" crossorigin href="./assets/hooks-CixO01lL.js">
     <link rel="modulepreload" crossorigin href="./assets/dist-DGzPJ_bO.js">
     <link rel="modulepreload" crossorigin href="./assets/dist-CQJY6SVY.js">
     <link rel="modulepreload" crossorigin href="./assets/dist-P_ohUlTi.js">
@@ -40,8 +40,8 @@
     <link rel="modulepreload" crossorigin href="./assets/dist-2Tnz2Yfq.js">
     <link rel="modulepreload" crossorigin href="./assets/input-surface-BpAjZXY5.js">
     <link rel="modulepreload" crossorigin href="./assets/update-queries-C4CnvJaW.js">
-    <link rel="modulepreload" crossorigin href="./assets/users-CcSCQFcQ.js">
-    <link rel="stylesheet" crossorigin href="./assets/index-D8PbzuT9.css">
+    <link rel="modulepreload" crossorigin href="./assets/users-Dkv8NQZe.js">
+    <link rel="stylesheet" crossorigin href="./assets/index-9Cw8z3rA.css">
   </head>
   <body class="react-admin">
     <div id="ember-alerts-wormhole"></div>
@@ -62,8 +62,8 @@
     <div id="ember-notifications-wormhole"></div>
     <script src="./assets/vendor-25f0bb2b2de85265afdf33da79f14a44.js"></script>
     <script src="./assets/chunk.211.6f7328d63c41536451b3.js"></script>
-    <script src="./assets/chunk.361.3cdb584b973959e52ccc.js"></script>
-    <script src="./assets/chunk.524.8365cd9280405c0f6216.js"></script>
-    <script src="./assets/ghost-05f9a7182ed56903dba05878a5b8422b.js"></script>
+    <script src="./assets/chunk.481.6c7395a49484bb4583aa.js"></script>
+    <script src="./assets/chunk.524.cffd399783a519e404a9.js"></script>
+    <script src="./assets/ghost-c4057474943effc568c5659ccc4909d7.js"></script>
   </body>
 </html>

package/core/frontend/services/proxy.js CHANGED Viewed

@@ -16,9 +16,10 @@ const FRONTEND_SUBSCRIBABLE_EVENTS = ['site.changed', 'url.added', 'url.removed'
 // Require from the handlebars framework
 const {SafeString} = require('./handlebars');
-const createDOMPurify = require('dompurify');
-const {JSDOM} = require('jsdom');
-const DOMPurify = createDOMPurify(new JSDOM('').window);
+// sanitize-html over DOMPurify here: DOMPurify needs a DOM, and jsdom costs
+// ~116MB RSS to load. This module is on the boot path, so that was paid by
+// every process — including ones that never render a caption.
+const sanitizeHtml = require('sanitize-html');
 module.exports = {
     getFrontendKey: async () => {
@@ -48,11 +49,9 @@ module.exports = {
         (Array.isArray(data) ? data : [data]).forEach((resource) => {
             // feature_image_caption contains HTML, making it a SafeString spares theme devs from triple-curlies
             if (resource.feature_image_caption) {
-                const sanitizedCaption = DOMPurify.sanitize(resource.feature_image_caption, {
-                    ALLOWED_TAGS: ['a', 'b', 'i', 'span'],
-                    ALLOWED_ATTR: ['href', 'style'],
-                    ALLOW_DATA_ATTR: false,
-                    ALLOW_ARIA_ATTR: false
+                const sanitizedCaption = sanitizeHtml(resource.feature_image_caption, {
+                    allowedTags: ['a', 'b', 'i', 'span'],
+                    allowedAttributes: {'*': ['href', 'style']}
                 });
                 resource.feature_image_caption = new SafeString(sanitizedCaption);
             }

package/core/frontend/services/routing/router-manager.js CHANGED Viewed

@@ -1,5 +1,4 @@
 const debug = require('@tryghost/debug')('frontend:routing');
-const _ = require('lodash');
 const StaticRoutesRouter = require('./static-routes-router');
 const StaticPagesRouter = require('./static-pages-router');
 const CollectionRouter = require('./collection-router');
@@ -111,30 +110,30 @@ class RouterManager {
         this.siteRouter.mountRouter(previewRouter.router());
         this.registry.setRouter('previewRouter', previewRouter);
-        _.each(routerSettings.routes, (value, key) => {
-            const staticRoutesRouter = new StaticRoutesRouter(key, value, this.routerCreated.bind(this));
+        for (const route of routerSettings.routes) {
+            const staticRoutesRouter = new StaticRoutesRouter(route.path, route, this.routerCreated.bind(this));
             this.siteRouter.mountRouter(staticRoutesRouter.router());
             this.registry.setRouter(staticRoutesRouter.identifier, staticRoutesRouter);
-        });
+        }
-        _.each(routerSettings.collections, (value, key) => {
-            const collectionRouter = new CollectionRouter(key, value, RESOURCE_CONFIG, this.routerCreated.bind(this));
+        for (const collection of routerSettings.collections) {
+            const collectionRouter = new CollectionRouter(collection.path, collection, RESOURCE_CONFIG, this.routerCreated.bind(this));
             this.siteRouter.mountRouter(collectionRouter.router());
             this.registry.setRouter(collectionRouter.identifier, collectionRouter);
-        });
+        }
         const staticPagesRouter = new StaticPagesRouter(RESOURCE_CONFIG, this.routerCreated.bind(this));
         this.siteRouter.mountRouter(staticPagesRouter.router());
         this.registry.setRouter('staticPagesRouter', staticPagesRouter);
-        _.each(routerSettings.taxonomies, (value, key) => {
-            const taxonomyRouter = new TaxonomyRouter(key, value, RESOURCE_CONFIG, this.routerCreated.bind(this));
+        for (const taxonomy of routerSettings.taxonomies) {
+            const taxonomyRouter = new TaxonomyRouter(taxonomy.key, taxonomy.permalink, RESOURCE_CONFIG, this.routerCreated.bind(this));
             this.siteRouter.mountRouter(taxonomyRouter.router());
             this.registry.setRouter(taxonomyRouter.identifier, taxonomyRouter);
-        });
+        }
         const appRouter = new ParentRouter('AppsRouter');
         this.siteRouter.mountRouter(appRouter.router());
@@ -186,10 +185,11 @@ module.exports = RouterManager;
  */
 /**
- * @typedef {Object} RouteSettings
- * @property {Object} routes
- * @property {Object} collections
- * @property {Object} taxonomies
+ * The shape RouterManager consumes — produced by the activation bridge. This is
+ * a temporary reference into server code; when the bridge is removed (HKG-1898)
+ * it becomes the domain `RouteSettings` from @tryghost/adapter-base-route-settings.
+ *
+ * @typedef {import('../../../server/services/route-settings/activation-bridge').RouterSettings} RouteSettings
  */
 /**

package/core/frontend/services/routing/static-routes-router.js CHANGED Viewed

@@ -30,12 +30,12 @@ class StaticRoutesRouter extends ParentRouter {
             this.limit = object.limit;
             this.order = object.order;
-            this.controller = object.controller;
+            this.type = object.type;
             debug(this.route.value, this.templates, this.filter, this.data);
             this._registerChannelRoutes();
         } else {
-            this.contentType = object.content_type;
+            this.contentType = object.contentType;
             debug(this.route.value, this.templates);
             this._registerStaticRoute();
         }
@@ -56,11 +56,11 @@ class StaticRoutesRouter extends ParentRouter {
         }
         // REGISTER: channel route
-        this.mountRoute(this.route.value, controllers[this.controller]);
+        this.mountRoute(this.route.value, controllers[this.type]);
         // REGISTER: pagination
         this.router().param('page', middleware.pageParam);
-        this.mountRoute(urlUtils.urlJoin(this.route.value, 'page', ':page(\\d+)'), controllers[this.controller]);
+        this.mountRoute(urlUtils.urlJoin(this.route.value, 'page', ':page(\\d+)'), controllers[this.type]);
         this.routerCreated(this);
     }
@@ -74,7 +74,7 @@ class StaticRoutesRouter extends ParentRouter {
      */
     _prepareChannelContext(req, res, next) {
         res.routerOptions = {
-            type: this.controller,
+            type: this.type,
             name: this.routerName,
             context: [this.routerName],
             filter: this.filter,
@@ -131,11 +131,9 @@ class StaticRoutesRouter extends ParentRouter {
      * @returns {boolean}
      */
     isChannel(object) {
-        if (object && object.controller && object.controller === 'channel') {
-            return true;
-        }
-        return this.controller === 'channel';
+        // buildRouterSettings always sets `type` ('channel' | 'template') on
+        // every route, so a channel is simply `type === 'channel'`.
+        return object?.type === 'channel';
     }
 }

package/core/server/api/endpoints/automation-action-links.js ADDED Viewed

@@ -0,0 +1,68 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+const automationsApi = __importStar(require("../../services/automations/automations-api"));
+const controller = {
+    docName: 'automation_action_links',
+    browse: {
+        headers: {
+            cacheInvalidate: false
+        },
+        options: [
+            'automation_id',
+            'action_id'
+        ],
+        validation: {
+            options: {
+                automation_id: {
+                    required: true
+                },
+                action_id: {
+                    required: true
+                }
+            }
+        },
+        permissions: {
+            docName: 'automations',
+            method: 'read'
+        },
+        async query(frame) {
+            return {
+                data: await automationsApi.browseActionLinks(frame.options.automation_id, frame.options.action_id)
+            };
+        }
+    }
+};
+module.exports = controller;

package/core/server/api/endpoints/automation-action-links.ts ADDED Viewed

@@ -0,0 +1,46 @@
+import * as automationsApi from '../../services/automations/automations-api';
+type BrowseFrame = {
+    options: {
+        automation_id: string;
+        action_id: string;
+    };
+};
+const controller = {
+    docName: 'automation_action_links',
+    browse: {
+        headers: {
+            cacheInvalidate: false
+        },
+        options: [
+            'automation_id',
+            'action_id'
+        ],
+        validation: {
+            options: {
+                automation_id: {
+                    required: true
+                },
+                action_id: {
+                    required: true
+                }
+            }
+        },
+        permissions: {
+            docName: 'automations',
+            method: 'read'
+        },
+        async query(frame: BrowseFrame) {
+            return {
+                data: await automationsApi.browseActionLinks(
+                    frame.options.automation_id,
+                    frame.options.action_id
+                )
+            };
+        }
+    }
+};
+module.exports = controller;

package/core/server/api/endpoints/index.js CHANGED Viewed

@@ -11,6 +11,10 @@ module.exports = {
         return apiFramework.pipeline(require('./automations'), localUtils);
     },
+    get automationActionLinks() {
+        return apiFramework.pipeline(require('./automation-action-links'), localUtils);
+    },
     get automationEmailPreviews() {
         return apiFramework.pipeline(require('./automation-email-previews'), localUtils);
     },

package/core/server/api/endpoints/settings.js CHANGED Viewed

@@ -174,8 +174,6 @@ const controller = {
         async query(frame) {
             const content = await fs.readFile(frame.file.path, 'utf8');
             await routeSettings.api.upload(content);
-            const getRoutesHash = () => routeSettings.api.getCurrentHash();
-            await settingsService.syncRoutesHash(getRoutesHash);
         }
     },

package/core/server/api/endpoints/utils/serializers/output/utils/url.js CHANGED Viewed

@@ -24,7 +24,17 @@ const forPost = (id, attrs, frame, type = 'posts') => {
         return attrs;
     }
-    attrs.url = urlService.facade.getUrlForResource({...attrs, id, type}, {absolute: true});
+    // Diagnostic only: hand the compare-mode URL service the producing api
+    // endpoint. The parity/thin-resource logs capture a caller stack, but it
+    // truncates at the async api-framework boundary and never names the
+    // endpoint; the api-framework Frame carries it, so pass it through. Omitted
+    // for non-api callers (mentions, email) whose frame has no endpoint fields.
+    const options = {absolute: true};
+    if (frame && (frame.docName || frame.method)) {
+        options.serializerContext = {apiType: frame.apiType, docName: frame.docName, method: frame.method};
+    }
+    attrs.url = urlService.facade.getUrlForResource({...attrs, id, type}, options);
     /**
      * CASE: admin api should serve preview urls

package/core/server/data/migrations/versions/6.55/2026-07-27-11-25-44-add-to-hash-to-redirects.js ADDED Viewed

@@ -0,0 +1,51 @@
+const logging = require('@tryghost/logging');
+const {combineNonTransactionalMigrations, createAddColumnMigration, createNonTransactionalMigration} = require('../../utils');
+const {addIndex, addUnique, dropIndex, dropUnique, getIndexes} = require('../../../schema/commands');
+const uniqueColumns = ['automation_action_revision_id', 'to_hash'];
+const leftoverIndexName = 'redirects_automation_action_revision_id_index';
+module.exports = combineNonTransactionalMigrations(
+    createAddColumnMigration('redirects', 'to_hash', {
+        type: 'binary',
+        maxlength: 32,
+        nullable: true
+    }, {algorithm: 'auto'}),
+    createNonTransactionalMigration(
+        async function up(knex) {
+            await addUnique('redirects', uniqueColumns, knex);
+            // A previous down() leaves a standalone index behind (see below).
+            // InnoDB drops the index it created for the foreign key itself once
+            // the composite unique can serve the constraint, but not one we
+            // created, so drop it here to keep a rolled-back-and-remigrated
+            // database in the same shape as every other one.
+            const indexes = await getIndexes('redirects', knex);
+            if (indexes.includes(leftoverIndexName)) {
+                await dropIndex('redirects', ['automation_action_revision_id'], knex);
+            }
+        },
+        async function down(knex) {
+            try {
+                await dropUnique('redirects', uniqueColumns, knex);
+            } catch (err) {
+                // The composite unique ends up as the only index covering the
+                // automation_action_revision_id FK: fresh installs create it
+                // that way, and on upgrades InnoDB drops the FK's own index
+                // once the composite can serve the constraint. MySQL won't
+                // drop the last index a foreign key depends on, so re-add a
+                // single-column one when it objects.
+                if (err.code === 'ER_DROP_INDEX_FK') {
+                    logging.warn('Cannot drop unique constraint over redirects(automation_action_revision_id, to_hash) while it backs the foreign key, re-adding index for automation_action_revision_id');
+                    await addIndex('redirects', ['automation_action_revision_id'], knex);
+                    await dropUnique('redirects', uniqueColumns, knex);
+                    return;
+                }
+                throw err;
+            }
+        }
+    )
+);

package/core/server/data/migrations/versions/6.55/2026-07-28-12-08-00-remove-routes-hash-from-settings.js ADDED Viewed

@@ -0,0 +1,5 @@
+const {combineTransactionalMigrations, removeSetting} = require('../../utils');
+module.exports = combineTransactionalMigrations(
+    removeSetting('routes_hash')
+);

package/core/server/data/schema/commands.js CHANGED Viewed

@@ -25,6 +25,11 @@ function addTableColumn(tableName, tableBuilder, columnName, columnSpec = schema
     // creation distinguishes between text with fieldtype, string with maxlength and all others
     if (columnSpec.type === 'text' && Object.prototype.hasOwnProperty.call(columnSpec, 'fieldtype')) {
         column = tableBuilder[columnSpec.type](columnName, columnSpec.fieldtype);
+    } else if (columnSpec.type === 'binary' && Object.prototype.hasOwnProperty.call(columnSpec, 'maxlength')) {
+        // knex emits an unbounded `blob` for a length-less binary column, which
+        // MySQL can only index with a prefix. Passing the length gives us
+        // `varbinary(N)`, which is indexable in full.
+        column = tableBuilder[columnSpec.type](columnName, columnSpec.maxlength);
     } else if (columnSpec.type === 'string') {
         if (Object.prototype.hasOwnProperty.call(columnSpec, 'maxlength')) {
             column = tableBuilder[columnSpec.type](columnName, columnSpec.maxlength);

package/core/server/data/schema/default-settings/default-settings.json CHANGED Viewed

@@ -8,10 +8,6 @@
             "defaultValue": null,
             "type": "string"
         },
-        "routes_hash": {
-            "defaultValue": null,
-            "type": "string"
-        },
         "next_update_check": {
             "defaultValue": null,
             "type": "number"

package/core/server/data/schema/schema.js CHANGED Viewed

@@ -1263,8 +1263,12 @@ module.exports = {
         to: {type: 'string', maxlength: 2000, nullable: false},
         post_id: {type: 'string', maxlength: 24, nullable: true, unique: false, references: 'posts.id', setNullDelete: true},
         automation_action_revision_id: {type: 'string', maxlength: 24, nullable: true, references: 'automation_action_revisions.id', setNullDelete: true},
+        to_hash: {type: 'binary', maxlength: 32, nullable: true},
         created_at: {type: 'dateTime', nullable: false},
-        updated_at: {type: 'dateTime', nullable: true}
+        updated_at: {type: 'dateTime', nullable: true},
+        '@@UNIQUE_CONSTRAINTS@@': [
+            ['automation_action_revision_id', 'to_hash']
+        ]
     },
     members_click_events: {
         id: {type: 'string', maxlength: 24, nullable: false, primary: true},

package/core/server/services/automations/automations-api.js CHANGED Viewed

@@ -5,6 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.browse = browse;
 exports.read = read;
+exports.browseActionLinks = browseActionLinks;
 exports.edit = edit;
 exports.requestPoll = requestPoll;
 exports.trigger = trigger;
@@ -15,6 +16,7 @@ exports.retryStep = retryStep;
 exports.recordEmailSent = recordEmailSent;
 exports.getAutomatedEmailRecipientsByMailgunIds = getAutomatedEmailRecipientsByMailgunIds;
 exports.trackEmailDeliveredAndOpened = trackEmailDeliveredAndOpened;
+exports.trackEmailClicked = trackEmailClicked;
 const errors_1 = __importDefault(require("@tryghost/errors"));
 const tpl_1 = __importDefault(require("@tryghost/tpl"));
 const bson_objectid_1 = __importDefault(require("bson-objectid"));
@@ -30,6 +32,7 @@ const StartAutomationsPollEvent = require('./events/start-automations-poll-event
 const MAX_AUTOMATION_ACTIONS = 20;
 const messages = {
     automationNotFound: 'Automation not found.',
+    automationActionNotFound: 'Automation action not found.',
     invalidAutomationPayload: 'Automation edit payload must include status, actions, and edges.',
     invalidAutomationStatus: 'Automation status must be one of: active, inactive.',
     duplicateAutomationActionIdentity: 'Automation action identifiers must be unique.',
@@ -86,6 +89,15 @@ async function read(automationId) {
     }
     return automation;
 }
+async function browseActionLinks(automationId, actionId) {
+    const links = await repository.getAutomationActionLinks(automationId, actionId);
+    if (!links) {
+        throw new errors_1.default.NotFoundError({
+            message: (0, tpl_1.default)(messages.automationActionNotFound)
+        });
+    }
+    return links;
+}
 async function edit(automationId, data) {
     const parsedData = await validateEditData(data);
     const automation = await repository.edit(automationId, parsedData);
@@ -311,3 +323,6 @@ async function getAutomatedEmailRecipientsByMailgunIds(...args) {
 async function trackEmailDeliveredAndOpened(...args) {
     return await repository.trackEmailDeliveredAndOpened(...args);
 }
+async function trackEmailClicked(...args) {
+    await repository.trackEmailClicked(...args);
+}

package/core/server/services/automations/automations-api.ts CHANGED Viewed

@@ -20,6 +20,7 @@ const MAX_AUTOMATION_ACTIONS = 20;
 const messages = {
     automationNotFound: 'Automation not found.',
+    automationActionNotFound: 'Automation action not found.',
     invalidAutomationPayload: 'Automation edit payload must include status, actions, and edges.',
     invalidAutomationStatus: 'Automation status must be one of: active, inactive.',
     duplicateAutomationActionIdentity: 'Automation action identifiers must be unique.',
@@ -87,6 +88,18 @@ export async function read(automationId: string) {
     return automation;
 }
+export async function browseActionLinks(automationId: string, actionId: string) {
+    const links = await repository.getAutomationActionLinks(automationId, actionId);
+    if (!links) {
+        throw new errors.NotFoundError({
+            message: tpl(messages.automationActionNotFound)
+        });
+    }
+    return links;
+}
 export async function edit(automationId: string, data: unknown) {
     const parsedData = await validateEditData(data);
@@ -375,3 +388,9 @@ export async function trackEmailDeliveredAndOpened(
 ) {
     return await repository.trackEmailDeliveredAndOpened(...args);
 }
+export async function trackEmailClicked(
+    ...args: Parameters<AutomationsRepository['trackEmailClicked']>
+) {
+    await repository.trackEmailClicked(...args);
+}

package/core/server/services/automations/automations-repository.ts CHANGED Viewed

@@ -1,4 +1,5 @@
 import type {ReadonlyDeep} from 'type-fest';
+import type {Knex} from 'knex';
 export interface Pagination {
     page: number;
@@ -32,6 +33,11 @@ export interface AutomationEmailStats {
     clicked_rate: number | null;
 }
+export interface AutomationActionLink {
+    url: string;
+    clicked_count: number;
+}
 export interface SendEmailAction {
     id: string;
     type: 'send_email';
@@ -131,6 +137,7 @@ export type AutomationStepTerminalStatus =
 export interface AutomationsRepository {
     browse(): Promise<Page<AutomationSummary>>;
     getById(id: string): Promise<Automation | null>;
+    getAutomationActionLinks(automationId: string, actionId: string): Promise<AutomationActionLink[] | null>;
     edit(id: string, data: EditAutomationData): Promise<Automation | null>;
     trigger(options: {
         memberEmail: string;
@@ -195,4 +202,15 @@ export interface AutomationsRepository {
     trackEmailDeliveredAndOpened(
         eventsByAutomatedEmailRecipientId: ReadonlyDeep<Map<string, AutomatedEmailEvents>>
     ): Promise<void>;
+    /**
+     * Track the first click for an automated email recipient.
+     *
+     */
+    trackEmailClicked(options: {
+        automationActionRevisionId: string;
+        memberId: string;
+        clickedAt: Readonly<Date>;
+    }, transactionOptions?: {
+        transacting?: Knex.Transaction;
+    }): Promise<void>;
 }