feat: implement project caching by ID and refactor data deletion

- Add setProjectById and getProjectById to redisCaching service

- Use project ID caching in getSingleProject controller

- Refactor deleteData to use updateOne (Note: verify logic for  vs )

- Remove console.time logs

Author: yash-pouranik

backend/controllers/data.controller.js

@@ -178,8 +178,11 @@ module.exports.deleteSingleDoc = async (req, res) => {
         await Model.deleteOne({ _id: id });
 
         if (!project.resources.db.isExternal) {
-            project.databaseUsed = Math.max(0, (project.databaseUsed || 0) - docSize);
-            await project.save();
+            let databaseUsed = Math.max(0, (project.databaseUsed || 0) - docSize);
+            await Project.updateOne(
+                { _id: project._id },
+                { $inc: { databaseUsed } }
+            );
         }
 
         res.json({ message: "Document deleted", id });

backend/controllers/project.controller.js

@@ -11,7 +11,7 @@ const { URL } = require('url');
 const { getConnection } = require("../utils/connection.manager");
 const { getCompiledModel } = require("../utils/injectModel")
 const { storageRegistry } = require("../utils/registry");
-const { deleteProjectByApiKeyCache } = require("../services/redisCaching");
+const { deleteProjectByApiKeyCache, setProjectById, getProjectById } = require("../services/redisCaching");
 
 
 
@@ -68,10 +68,17 @@ module.exports.getAllProject = async (req, res) => {
 
 module.exports.getSingleProject = async (req, res) => {
     try {
-        const project = await Project.findOne({ _id: req.params.projectId, owner: req.user._id }).select('-apiKey -jwtSecret');
-        if (!project) return res.status(404).json({ error: "Project not found." });
+        let project;
+        project = await getProjectById(req.params.projectId);
+        let projectObj;
+        if (!project) {
+            project = await Project.findOne({ _id: req.params.projectId, owner: req.user._id }).select('-apiKey -jwtSecret');
+            if (!project) return res.status(404).json({ error: "Project not found." });
+            projectObj = project.toObject();
+            await setProjectById(req.params.projectId, project);
+        }
 
-        const projectObj = project.toObject();
+        projectObj = project;
         delete projectObj.apiKey;
         delete projectObj.jwtSecret;
         res.json(projectObj);
@@ -85,6 +92,10 @@ module.exports.regenerateApiKey = async (req, res) => {
         const newApiKey = generateApiKey();
         const hashed = hashApiKey(newApiKey);
 
+        const oldApiProj = await Project.findOne({ _id: req.params.projectId, owner: req.user._id }).select('apiKey');
+        if (!oldApiProj) return res.status(404).json({ error: "Project not found." });
+        await deleteProjectByApiKeyCache(oldApiProj.apiKey);
+
 
         const project = await Project.findOneAndUpdate(
             { _id: req.params.projectId, owner: req.user._id },

backend/services/redisCaching.js

@@ -2,9 +2,7 @@ const redis = require("../config/redis");
 
 async function setProjectByApiKeyCache(api, project) {
     try {
-        console.time("json");
         const data = JSON.stringify(project);
-        console.timeEnd("json");
         await redis.set(
             `project:apikey:${api}`,
             data,
@@ -20,9 +18,8 @@ async function getProjectByApiKeyCache(api) {
     try {
         const data = await redis.get(`project:apikey:${api}`);
         if (!data) return null;
-        console.time("json parse")
+
         const parsedData = JSON.parse(data)
-        console.timeEnd("json parse")
         return parsedData;
     } catch (err) {
         console.log(err);
@@ -38,8 +35,39 @@ async function deleteProjectByApiKeyCache(api) {
     }
 }
 
+
+async function setProjectById(id, project) {
+    try {
+        const data = JSON.stringify(project);
+        await redis.set(
+            `project:id:${id}`,
+            data,
+            'EX',
+            60 * 60 * 2
+        );
+    } catch (err) {
+        console.log(err);
+    }
+}
+
+
+async function getProjectById(id) {
+    try {
+        const data = await redis.get(`project:id:${id}`);
+        if (!data) return null;
+        const parsedData = JSON.parse(data)
+        return parsedData;
+    } catch (err) {
+        console.log(err);
+        return null;
+    }
+}
+
+
 module.exports = {
     setProjectByApiKeyCache,
     getProjectByApiKeyCache,
-    deleteProjectByApiKeyCache
+    deleteProjectByApiKeyCache,
+    setProjectById,
+    getProjectById
 };