Refactor code for improved type safety and clarity; remove unused elements and update imports

Author: Sachinchaurasiya360

client/src/module/student/ats/CoverLetterPage.tsx

@@ -623,7 +623,6 @@ export default function CoverLetterPage() {
               </AnimatePresence>
             </div>
           </div>
-          </div>
 
           <CoverLetterHistoryPanel onLoad={handleLoadFromHistory} />
 

client/src/module/student/skill-verification/SkillTestPage.tsx

@@ -358,9 +358,6 @@ export default function SkillTestPage() {
               </p>
             )}
 
-          
-        </motion.div>
-      </div>
             <div className="grid grid-cols-2 gap-3">
               {[
                 {
@@ -592,6 +589,7 @@ export default function SkillTestPage() {
             )
           )}
         </div>
+        </div>
       </div>
     );
   }

server/docker-compose.yml

@@ -82,6 +82,7 @@ model user {
   roadmapEnrollments    roadmapEnrollment[]         @relation("UserRoadmapEnrollments")
   scheduledEmails       scheduledEmail[]            @relation("UserScheduledEmails")
   privateRoadmaps       roadmap[]                   @relation("UserPrivateRoadmaps")
+  generatedCoverLetters generatedCoverLetter[]      @relation("StudentCoverLetters")
 
   @@index([role])
   @@index([role, isActive])

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

@@ -44,7 +44,7 @@ function logErrorToDb(req: Request, statusCode: number, message: string, rawErr?
       userId: (req as unknown as { user?: { id: number } }).user?.id ?? null,
       ipAddress: (req.headers["x-forwarded-for"] as string)?.split(",")[0]?.trim() || req.ip || null,
       userAgent: req.headers["user-agent"] || null,
-      requestBody: sanitizeBody(req.body),
+      requestBody: sanitizeBody(req.body) ?? Prisma.DbNull,
     },
   }).catch((dbErr) => {
     console.error("[ErrorLog] Failed to write:", dbErr);

server/src/middleware/error.middleware.ts

@@ -1009,7 +1009,7 @@ export class AdminController {
 
   async getPublicExternalJobBySlug(req: Request, res: Response) {
     try {
-      const slug = req.params["slug"];
+      const slug = req.params["slug"] as string;
       if (!slug) return res.status(400).json({ message: "Slug is required" });
 
       const job = await this.adminService.getPublicExternalJobBySlug(slug);

server/src/module/admin/admin.controller.ts

@@ -1675,21 +1675,22 @@ export class AdminService {
     const html = this.renderBroadcastHtml(input.subject, input.body);
 
     if (input.testEmail) {
+      const testEmailAddr = input.testEmail;
       const sample = (s: string) =>
         s
           .replace(/\{\{?\s*username\s*\}?\}/gi, "Sachin")
           .replace(/\{\{?\s*name\s*\}?\}/gi, "Sachin")
           .replace(/\{\{?\s*firstName\s*\}?\}/gi, "Sachin")
-          .replace(/\{\{?\s*email\s*\}?\}/gi, input.testEmail);
+          .replace(/\{\{?\s*email\s*\}?\}/gi, testEmailAddr);
       await sendEmail({ to: input.testEmail, subject: `[TEST] ${sample(input.subject)}`, html: sample(html) });
       return { test: true, sent: 1, failed: 0, recipients: 1 };
     }
 
-    const where: Prisma.UserWhereInput = { isActive: true };
+    const where: Prisma.userWhereInput = { isActive: true };
     if (input.filter.role !== "ALL") where.role = input.filter.role as UserRole;
     if (typeof input.filter.isVerified === "boolean") where.isVerified = input.filter.isVerified;
     if (input.filter.subscriptionPlan !== "ALL") {
-      where.subscriptionPlan = input.filter.subscriptionPlan as Prisma.UserWhereInput["subscriptionPlan"];
+      where.subscriptionPlan = input.filter.subscriptionPlan as Prisma.userWhereInput["subscriptionPlan"];
     }
 
     const users = await prisma.user.findMany({ where, select: { email: true, name: true } });

server/src/module/admin/admin.service.ts

@@ -4,7 +4,7 @@ import type {
 } from "./cover-letter.validation.js";
 import { getProviderForService } from "../../lib/ai-provider-registry.js";
 import { logAIRequest } from "../../lib/ai-request-logger.js";
-import { prisma } from "../../lib/prisma.js";
+import { prisma } from "../../database/db.js";
 
 export class CoverLetterService {
   async generate(

server/src/module/ats/cover-letter.service.ts

@@ -44,7 +44,7 @@ export class AuthController {
           return res.status(403).json({
             message: "Please verify your email before signing in. A new verification code has been sent.",
             requiresVerification: true,
-            email: result.data.email,
+            email: (req.body as { email?: string }).email,
           });
         }
         if (error.message === "Account is deactivated") {

server/src/module/auth/auth.controller.ts

@@ -30,7 +30,7 @@ interface UpdateProfileInput {
   designation?: string;
   bio?: string;
   college?: string;
-  graduationYear?: number;
+  graduationYear?: number | null;
   skills?: string[];
   location?: string;
   linkedinUrl?: string;