fix: add caching for charts

Author: FreekBes

src/routes/admin/charts.ts

@@ -3,9 +3,18 @@ import { PrismaClient } from '@prisma/client';
 import { CURSUS_ID } from '../../env';
 import { ChartConfiguration } from 'chart.js';
 import { getBlocAtDate } from '../../utils';
+import NodeCache from 'node-cache';
+
+const adminChartDataCache = new NodeCache({ stdTTL: 60 * 5, checkperiod: 60 * 5 });
 
 export const setupAdminChartsRoutes = function(app: Express, prisma: PrismaClient): void {
 	app.get('/admin/charts/coalitions/users/distribution', async (req, res) => {
+		// Check cache first
+		const cachedData = adminChartDataCache.get<ChartConfiguration>('coalitionsUsersDistribution');
+		if (cachedData) {
+			return res.json(cachedData);
+		}
+
 		// Get the distribution of users per coalition
 		const distribution = await prisma.intraCoalitionUser.groupBy({
 			by: ['coalition_id'],
@@ -77,12 +86,23 @@ export const setupAdminChartsRoutes = function(app: Express, prisma: PrismaClien
 				},
 			},
 		}
+
+		// Cache and return the data
+		adminChartDataCache.set('coalitionsUsersDistribution', chartJSData);
+
 		return res.json(chartJSData);
 	});
 
 	app.get('/admin/charts/coalitions/:coalitionId/scores/distribution', async (req, res) => {
 		try {
 			const coalitionId = parseInt(req.params.coalitionId);
+
+			// Check cache first
+			const cachedData = adminChartDataCache.get<ChartConfiguration>(`coalitionScoresDistribution_${coalitionId}`);
+			if (cachedData) {
+				return res.json(cachedData);
+			}
+
 			const coalition = await prisma.intraCoalition.findFirst({
 				where: {
 					id: coalitionId,
@@ -100,6 +120,7 @@ export const setupAdminChartsRoutes = function(app: Express, prisma: PrismaClien
 			if (!currentBloc) {
 				throw new Error('No current bloc found');
 			}
+
 			const scores = await prisma.codamCoalitionScore.groupBy({
 				by: ['user_id'],
 				where: {
@@ -182,6 +203,9 @@ export const setupAdminChartsRoutes = function(app: Express, prisma: PrismaClien
 				}
 			};
 
+			// Cache and return the data
+			adminChartDataCache.set(`coalitionScoresDistribution_${coalitionId}`, chartJSData);
+
 			return res.json(chartJSData);
 		}
 		catch (err) {

src/routes/charts.ts

@@ -2,10 +2,19 @@ import { PrismaClient } from '@prisma/client';
 import { ChartConfiguration } from 'chart.js';
 import { Express } from 'express';
 import { CoalitionScore, getBlocAtDate, getCoalitionScore } from '../utils';
+import NodeCache from 'node-cache';
+
+const chartDataCache = new NodeCache({ stdTTL: 60 * 5, checkperiod: 60 * 5 });
 
 export const setupChartRoutes = function(app: Express, prisma: PrismaClient): void {
 	app.get('/charts/coalitions/scores/history', async (req, res) => {
 		try {
+			// Check cache first
+			const cachedData = chartDataCache.get<ChartConfiguration>('coalitionsScoresHistory');
+			if (cachedData) {
+				return res.json(cachedData);
+			}
+
 			const now = new Date();
 			const currentBloc = await getBlocAtDate(prisma, now);
 			if (!currentBloc) {
@@ -82,6 +91,9 @@ export const setupChartRoutes = function(app: Express, prisma: PrismaClient): vo
 				});
 			}
 
+			// Cache and return the data
+			chartDataCache.set('coalitionsScoresHistory', chartJSData);
+
 			return res.json(chartJSData);
 		}
 		catch (err) {
@@ -92,12 +104,18 @@ export const setupChartRoutes = function(app: Express, prisma: PrismaClient): vo
 
 	app.get('/charts/coalitions/:coalitionId/scores/history', async (req, res) => {
 		try {
+			// Check cache first
+			const coalitionId = parseInt(req.params.coalitionId);
+			const cachedData = chartDataCache.get<ChartConfiguration>(`coalitionsScoresHistory_${coalitionId}`);
+			if (cachedData) {
+				return res.json(cachedData);
+			}
+
 			const now = new Date();
 			const currentBloc = await getBlocAtDate(prisma, now);
 			if (!currentBloc) {
 				throw new Error('No season is currently ongoing');
 			}
-			const coalitionId = parseInt(req.params.coalitionId);
 			const coalition = await prisma.intraCoalition.findFirst({
 				where: {
 					id: coalitionId,
@@ -200,6 +218,9 @@ export const setupChartRoutes = function(app: Express, prisma: PrismaClient): vo
 				}
 			};
 
+			// Cache and return the data
+			chartDataCache.set(`coalitionsScoresHistory_${coalitionId}`, chartJSData);
+
 			return res.json(chartJSData);
 		}
 		catch (err) {
@@ -232,6 +253,13 @@ export const setupChartRoutes = function(app: Express, prisma: PrismaClient): vo
 			if (!user || !user.coalition_users || user.coalition_users.length === 0) {
 				return res.status(404).send('User not found or not in a coalition');
 			}
+
+			// Check cache first
+			const cachedData = chartDataCache.get<ChartConfiguration>(`coalitionsScoresHistory_${user.id}`);
+			if (cachedData) {
+				return res.json(cachedData);
+			}
+
 			const now = new Date();
 			const currentBloc = await getBlocAtDate(prisma, now);
 			if (!currentBloc) {
@@ -309,6 +337,9 @@ export const setupChartRoutes = function(app: Express, prisma: PrismaClient): vo
 				}
 			};
 
+			// Cache and return the data
+			chartDataCache.set(`coalitionsScoresHistory_${user.id}`, chartJSData);
+
 			return res.json(chartJSData);
 		}
 		catch (err) {
@@ -342,6 +373,12 @@ export const setupChartRoutes = function(app: Express, prisma: PrismaClient): vo
 				return res.status(404).send('User not found or not in a coalition');
 			}
 
+			// Check cache first
+			const cachedData = chartDataCache.get<ChartConfiguration>(`coalitionsScoresHistorySplit_${user.id}`);
+			if (cachedData) {
+				return res.json(cachedData);
+			}
+
 			const now = new Date();
 			const currentBloc = await getBlocAtDate(prisma, now);
 			if (!currentBloc) {
@@ -436,6 +473,9 @@ export const setupChartRoutes = function(app: Express, prisma: PrismaClient): vo
 				}
 			};
 
+			// Cache and return the data
+			chartDataCache.set(`coalitionsScoresHistorySplit_${user.id}`, chartJSData);
+
 			return res.json(chartJSData);
 		}
 		catch (err) {