Refactor user model relations and enhance caching logic in roadmap controller

Author: Sachinchaurasiya360

client/tsconfig.app.json

@@ -7,7 +7,7 @@
     "module": "ESNext",
     "skipLibCheck": true,
     "forceConsistentCasingInFileNames": true,
-    "ignoreDeprecations": "6.0",
+    "ignoreDeprecations": "5.0",
 
     /* Bundler mode */
     "moduleResolution": "bundler",

server/src/database/prisma/schema/base.prisma

@@ -82,12 +82,9 @@ model user {
   roadmapEnrollments    roadmapEnrollment[]      @relation("UserRoadmapEnrollments")
   scheduledEmails       scheduledEmail[]         @relation("UserScheduledEmails")
   privateRoadmaps       roadmap[]                @relation("UserPrivateRoadmaps")
-  generatedResumes generatedResume[] @relation("UserGeneratedResumes")
-  leetcodeImports       leetcodeImportLog[]      @relation("UserLeetcodeImports")
-  roadmapEnrollments    roadmapEnrollment[]         @relation("UserRoadmapEnrollments")
-  scheduledEmails       scheduledEmail[]            @relation("UserScheduledEmails")
-  privateRoadmaps       roadmap[]                   @relation("UserPrivateRoadmaps")
-  generatedCoverLetters generatedCoverLetter[]      @relation("StudentCoverLetters")
+  generatedResumes      generatedResume[]           @relation("UserGeneratedResumes")
+  leetcodeImports       leetcodeImportLog[]          @relation("UserLeetcodeImports")
+  generatedCoverLetters generatedCoverLetter[]       @relation("StudentCoverLetters")
 
   @@index([role])
   @@index([role, isActive])

server/src/middleware/cache.middleware.ts

@@ -18,7 +18,7 @@ export const cacheMiddleware = (ttl: number = 300, keyPrefix: string = "cache")
 
     const originalJson = res.json;
     res.json = function (data) {
-      if (res.statusCode >= 200 && res.statusCode < 300) {
+      if (res.statusCode >= 200 && res.statusCode < 300 && !res.locals["skipCache"]) {
         appCache.set(key, data, ttl);
       }
       res.setHeader("X-Cache", "MISS");

server/src/module/dsa/dsa-import.service.ts

@@ -136,14 +136,16 @@ async function loadCatalog() {
   const byLcId = new Map<number, typeof problems[0]>();
   const bySlug = new Map<string, typeof problems[0]>();
   const byNormTitle = new Map<string, typeof problems[0]>();
+  const byId = new Map<number, typeof problems[0]>();
 
   for (const p of problems) {
     if (p.leetcodeId) byLcId.set(p.leetcodeId, p);
     bySlug.set(p.slug, p);
     byNormTitle.set(normalizeTitle(p.title), p);
+    byId.set(p.id, p);
   }
 
-  return { byLcId, bySlug, byNormTitle };
+  return { byLcId, bySlug, byNormTitle, byId };
 }
 
 function normalizeTitle(t: string) {
@@ -217,9 +219,7 @@ async function buildPreview(
 
   // Build preview items (first 50 for display)
   const preview: PreviewItem[] = newRows.slice(0, 50).map((r) => {
-    // Reverse-lookup the problem info from the catalog maps
-    const allCatalogByIdArr = [...catalog.bySlug.values()];
-    const problem = allCatalogByIdArr.find((p) => p.id === r.problemId)!;
+    const problem = catalog.byId.get(r.problemId)!;
     return {
       problemId: r.problemId,
       title: problem?.title ?? "Unknown",

server/src/module/roadmap/roadmap.controller.ts

@@ -72,6 +72,7 @@ export async function getRoadmap(req: Request, res: Response, next: NextFunction
         res.status(404).json({ message: "Roadmap not found" });
         return;
       }
+      res.locals["skipCache"] = true;
     }
 
     res.json({ roadmap });

server/src/module/roadmap/roadmap.routes.ts

@@ -21,6 +21,7 @@ export const roadmapRouter = Router();
 roadmapRouter.post("/ai/generate", authMiddleware, aiRoadmapLimiter, postAiGenerate);
 roadmapRouter.get("/me/enrollments", authMiddleware, getMyEnrollments);
 roadmapRouter.get("/me/enrollments/:id", authMiddleware, getMyEnrollment);
+roadmapRouter.get("/me/enrollments/:id/pdf", authMiddleware, downloadPdf);
 roadmapRouter.get("/me/enrollments/:id/certificate", authMiddleware, downloadCertificate);
 roadmapRouter.patch(
   "/me/enrollments/:id/topics/:topicId",