Renamed TopContentStatsService to ContentStatsService (#23148)

no refs

Renaming this service and its test files to align with the pattern in
the other stats services. There are likely to be other content endpoints
that aren't necessarily `top-*`, which can be added to this service and
given their own endpoints.

Author: cmraible

ghost/core/core/server/services/stats/TopContentStatsService.js renamed to ghost/core/core/server/services/stats/ContentStatsService.js

@@ -9,7 +9,7 @@ const logging = require('@tryghost/logging');
  * @property {string} [title] - Page title
  */
 
-class TopContentStatsService {
+class ContentStatsService {
     /**
      * @param {object} deps
      * @param {import('knex').Knex} deps.knex - Database client
@@ -38,25 +38,25 @@ class TopContentStatsService {
             if (!this.tinybirdClient) {
                 return {data: []};
             }
-            
+
             // Step 1: Get raw data from Tinybird
             const rawData = await this.fetchRawTopContentData(options);
-            
+
             if (!rawData || !rawData.length) {
                 return {data: []};
             }
-            
+
             // Step 2: Enrich the data with titles
             const enrichedData = await this.enrichTopContentData(rawData);
-            
+
             return {data: enrichedData};
         } catch (error) {
             logging.error('Error fetching top content:');
             logging.error(error);
             return {data: []};
         }
     }
-    
+
     /**
      * Fetch raw top pages data from Tinybird
      * @param {Object} options - Query options with snake_case keys
@@ -71,10 +71,10 @@ class TopContentStatsService {
             memberStatus: options.member_status,
             tbVersion: options.tb_version
         };
-        
+
         return await this.tinybirdClient.fetch('api_top_pages', tinybirdOptions);
     }
-    
+
     /**
      * Extract post UUIDs from page data (internal method)
      * @param {Array<TopContentDataItem>} data - Raw page data
@@ -88,7 +88,7 @@ class TopContentStatsService {
             })
             .filter(Boolean);
     }
-    
+
     /**
      * Lookup post titles in the database
      * @param {Array<string>} uuids - Post UUIDs to look up
@@ -98,11 +98,11 @@ class TopContentStatsService {
         if (!uuids.length) {
             return {};
         }
-        
+
         const posts = await this.knex.select('uuid', 'title', 'id')
             .from('posts')
             .whereIn('uuid', uuids);
-            
+
         return posts.reduce((map, post) => {
             map[post.uuid] = {
                 title: post.title,
@@ -111,7 +111,7 @@ class TopContentStatsService {
             return map;
         }, {});
     }
-    
+
     /**
      * Get resource title using UrlService
      * @param {string} pathname - Path to look up
@@ -121,10 +121,10 @@ class TopContentStatsService {
         if (!this.urlService) {
             return null;
         }
-        
+
         try {
             const resource = this.urlService.getResource(pathname);
-            
+
             if (resource && resource.data) {
                 if (resource.data.title) {
                     return {
@@ -144,10 +144,10 @@ class TopContentStatsService {
                 logging.warn(`Error looking up resource for ${pathname}: ${err.message}`);
             }
         }
-        
+
         return null;
     }
-    
+
     /**
      * Enrich top pages data with titles
      * @param {Array<TopContentDataItem>} data - Raw page data
@@ -157,11 +157,11 @@ class TopContentStatsService {
         if (!data || !data.length) {
             return [];
         }
-        
+
         // Extract post UUIDs and lookup titles
         const postUuids = this.extractPostUuids(data);
         const titleMap = await this.lookupPostTitles(postUuids);
-        
+
         // Enrich the data with post titles or UrlService lookups
         return Promise.all(data.map(async (item) => {
             // Check if post_uuid is available directly
@@ -172,7 +172,7 @@ class TopContentStatsService {
                     post_id: titleMap[item.post_uuid].id
                 };
             }
-            
+
             // Use UrlService for pages without post_uuid
             const resourceInfo = this.getResourceTitle(item.pathname);
             if (resourceInfo) {
@@ -182,7 +182,7 @@ class TopContentStatsService {
                     resourceType: resourceInfo.resourceType
                 };
             }
-            
+
             // Otherwise fallback to pathname (removing leading/trailing slashes)
             const formattedPath = item.pathname.replace(/^\/|\/$/g, '') || 'Home';
             return {
@@ -193,4 +193,4 @@ class TopContentStatsService {
     }
 }
 
-module.exports = TopContentStatsService; 
+module.exports = ContentStatsService;

ghost/core/core/server/services/stats/StatsService.js

@@ -2,8 +2,8 @@ const MRRService = require('./MrrStatsService');
 const MembersService = require('./MembersStatsService');
 const SubscriptionStatsService = require('./SubscriptionStatsService');
 const ReferrersStatsService = require('./ReferrersStatsService');
-const TopContentStatsService = require('./TopContentStatsService');
 const PostsStatsService = require('./PostsStatsService');
+const ContentStatsService = require('./ContentStatsService');
 const tinybird = require('./utils/tinybird');
 
 class StatsService {
@@ -13,16 +13,16 @@ class StatsService {
      * @param {MembersService} deps.members
      * @param {SubscriptionStatsService} deps.subscriptions
      * @param {ReferrersStatsService} deps.referrers
-     * @param {TopContentStatsService} deps.topContent
      * @param {PostsStatsService} deps.posts
+     * @param {ContentStatsService} deps.content
      **/
     constructor(deps) {
         this.mrr = deps.mrr;
         this.members = deps.members;
         this.subscriptions = deps.subscriptions;
         this.referrers = deps.referrers;
-        this.topContent = deps.topContent;
-        this.topPosts = deps.posts;
+        this.posts = deps.posts;
+        this.content = deps.content;
     }
 
     async getMRRHistory() {
@@ -41,7 +41,7 @@ class StatsService {
             startDate: options.dateFrom
         };
         delete mappedOptions.dateFrom;
-        
+
         return this.members.getCountHistory(mappedOptions);
     }
 
@@ -67,7 +67,7 @@ class StatsService {
      * @param {Object} options
      */
     async getTopContent(options = {}) {
-        return await this.topContent.getTopContent(options);
+        return await this.content.getTopContent(options);
     }
 
     /**
@@ -77,7 +77,7 @@ class StatsService {
      */
     async getTopPosts(options = {}) {
         // Return the original { data: results } structure
-        const result = await this.topPosts.getTopPosts(options);
+        const result = await this.posts.getTopPosts(options);
         return result;
     }
 
@@ -111,8 +111,8 @@ class StatsService {
             members: new MembersService(deps),
             subscriptions: new SubscriptionStatsService(deps),
             referrers: new ReferrersStatsService(deps),
-            topContent: new TopContentStatsService(depsWithTinybird),
-            posts: new PostsStatsService(deps)
+            posts: new PostsStatsService(deps),
+            content: new ContentStatsService(depsWithTinybird)
         });
     }
 }