refactor: separate Event and Project chart data APIs

- Event.chartData: uses 'days' parameter, fetches data only from MongoDB
- Project.chartData: uses 'startDate', 'endDate', 'groupBy' parameters, fetches from Redis with MongoDB fallback
- Add getChartDataFromMongo method for Event-specific chart data
- Fix redisHelper comment to reflect TS.INCRBY usage
- Remove unused 'projectId' parameter from getChartDataFromMongo

Author: pavelzotikov

src/models/eventsFactory.js

@@ -402,6 +402,17 @@ class EventsFactory extends Factory {
     };
   }
 
+  /**
+   * Get chart data for projects (uses Redis with fallback to MongoDB)
+   *
+   * @param {string} startDate - start date (ISO string or Unix timestamp)
+   * @param {string} endDate - end date (ISO string or Unix timestamp)
+   * @param {number} groupBy - grouping interval in minutes
+   * @param {number} timezoneOffset - user's local timezone offset in minutes
+   * @param {string} projectId - project ID
+   * @param {string} groupHash - event's group hash (empty for project-level data)
+   * @returns {Promise<Array>}
+   */
   async getChartData(startDate, endDate, groupBy = 60, timezoneOffset = 0, projectId = '', groupHash = '') {
     try {
       const redisData = await this.redis.getChartDataFromRedis(
@@ -431,6 +442,18 @@ class EventsFactory extends Factory {
     }
   }
 
+  /**
+   * Get chart data from MongoDB only (for events)
+   *
+   * @param {number} days - how many days to fetch
+   * @param {number} timezoneOffset - user's local timezone offset in minutes
+   * @param {string} groupHash - event's group hash
+   * @returns {Promise<Array>}
+   */
+  async getChartDataFromMongo(days, timezoneOffset = 0, groupHash = '') {
+    return this.findChartData(days, timezoneOffset, groupHash);
+  }
+
   /**
    * Fetch timestamps and total count of errors (or target error) for each day since
    *

src/redisHelper.ts

@@ -101,8 +101,8 @@ export default class RedisHelper {
 
     let result: [string, string][] = [];
     try {
-      // Use aggregation to sum events within each bucket
-      // Since we now use TS.ADD (not TS.INCRBY), each sample is 1, so SUM gives us count
+      // Use aggregation to sum values within each bucket
+      // TS.INCRBY creates one point per time period with accumulated count
       result = (await this.redisClient.sendCommand([
         'TS.RANGE',
         key,

src/resolvers/event.js

@@ -83,10 +83,10 @@ module.exports = {
      * @param {number} timezoneOffset - user's local timezone offset in minutes
      * @returns {Promise<ProjectChartItem[]>}
      */
-    async chartData({ projectId, groupHash }, { startDate, endDate, groupBy, timezoneOffset }, context) {
+    async chartData({ projectId, groupHash }, { days, timezoneOffset }, context) {
       const factory = getEventsFactory(context, projectId);
 
-      return factory.getChartData(startDate, endDate, groupBy, timezoneOffset, projectId, groupHash);
+      return factory.getChartDataFromMongo(days, timezoneOffset, groupHash);
     },
 
     /**

src/typeDefs/event.ts

@@ -282,19 +282,9 @@ type Event {
   """
   chartData(
     """
-    Start date (ISO string or Unix timestamp in seconds)
+    How many days we need to fetch for displaying in a chart
     """
-    startDate: String!
-
-    """
-    End date (ISO string or Unix timestamp in seconds)
-    """
-    endDate: String!
-
-    """
-    Grouping interval in minutes (1=minute, 60=hour, 1440=day)
-    """
-    groupBy: Int! = 60
+    days: Int! = 0
 
     """
     User's local timezone offset in minutes