merge with staging

abhitrueprogrammer · abhitrueprogrammer · commit f189ee7017a0 · 2026-02-14T22:12:45.000+05:30
diff --git a/.env.example b/.env.example
@@ -30,4 +30,4 @@ KV_REST_API_READ_ONLY_TOKEN=""      # The read-only REST API token for your Verc
 
 # Upstash_Redis
 UPSTASH_REDIS_REST_URL=""           # REST URL of your Upstash Redis database
-UPSTASH_REDIS_REST_TOKEN=""         # REST API token for Upstash Redis
+UPSTASH_REDIS_REST_TOKEN=""         # REST API token for Upstash Redis
diff --git a/README.md b/README.md
@@ -70,7 +70,7 @@ Before getting started, please ensure that the .env file is properly configured.
 Dev names and stats be seen in [github stats](https://github.com/CodeChefVIT/papers-codechef/graphs/contributors) page
 
 ## Maintainers
-Our top maintainers, making sure the Database of papers is maintainered and upto-date.
+Our top maintainers, making sure the database of papers is maintained and upto-date.
 <table>
   <tr>
     <td align="center">
diff --git a/src/app/api/papers/route.ts b/src/app/api/papers/route.ts
@@ -1,43 +1,25 @@
 import { NextResponse, type NextRequest } from "next/server";
-import { connectToDatabase } from "@/lib/database/mongoose";
-import Paper from "@/db/papers";
-import { type IPaper } from "@/interface";
-import { escapeRegExp } from "@/lib/utils/regex";
-import { extractUniqueValues } from "@/lib/utils/paper-aggregation";
+import { getPapersBySubject } from "@/lib/services/paper";
 
 export const dynamic = "force-dynamic";
 
 export async function GET(req: NextRequest) {
   try {
-    await connectToDatabase();
     const url = req.nextUrl.searchParams;
-    const subject = url.get("subject");
-
-    if (!subject) {
+    const sub = url.get("subject");
+    if (!sub) {
       return NextResponse.json(
         { message: "Subject query parameter is required" },
         { status: 400 },
       );
     }
+    const paper = await getPapersBySubject(sub);
 
-    const escapedSubject = escapeRegExp(subject);
-    const papers: IPaper[] = await Paper.find({
-      subject: { $regex: new RegExp(`${escapedSubject}`, "i") },
-    });
-
-    const uniqueValues = extractUniqueValues(papers);
-
-    return NextResponse.json(
-      {
-        papers,
-        ...uniqueValues,
-      },
-      { status: 200 },
-    );
+    return NextResponse.json(paper, { status: 200 });
   } catch (error) {
     return NextResponse.json(
       { message: "Failed to fetch papers", error },
       { status: 500 },
     );
   }
-}
+}
diff --git a/src/app/api/report-tag/route.ts b/src/app/api/report-tag/route.ts
@@ -3,30 +3,48 @@ import { connectToDatabase } from "@/lib/database/mongoose";
 import TagReport from "@/db/tagReport";
 import { Ratelimit } from "@upstash/ratelimit";
 import { redis } from "@/lib/utils/redis";
-import { exams } from "@/components/select_options";
+
+const exams: string[] = ["CAT-1", "CAT-2", "FAT", "Model CAT-1", "Model CAT-2", "Model FAT"]
 
 interface ReportedFieldInput {
   field: string;
   value?: string;
 }
+interface ReportTagBody {
+  paperId?: string;
+  reportedFields?: unknown;
+  comment?: unknown;
+  reporterEmail?: unknown;
+  reporterId?: unknown;
+}
+
 const ALLOWED_FIELDS = ["subject", "courseCode", "exam", "slot", "year"];
 
-const ratelimit = new Ratelimit({
-  redis,
-  limiter: Ratelimit.slidingWindow(3, "1 h"),//per id - 3 request - per hour
-  analytics: true,
+function getRateLimit(){
+  return new Ratelimit({
+    redis,
+    limiter: Ratelimit.slidingWindow(3, "1 h"),//per id - 3 request - per hour
+    analytics: true,
 });
-
+}
 function getClientIp(req: Request & { ip?: string}): string {
-  return req.ip || "127.0.0.1";
+  const xff = req.headers.get("x-forwarded-for");
+  if (typeof xff === "string" && xff.length > 0) {
+    return xff.split(",")[0]?.trim()??"";
+  }
+  const xri = req.headers.get("x-real-ip");
+  if (typeof xri === "string" && xri.length > 0) {
+    return xri;
+  }
+  return "0.0.0.0";
 }
 
 export async function POST(req: Request & { ip?: string }) {
   try {
     await connectToDatabase();
-
-    const body = await req.json();
-    const { paperId } = body;
+    const ratelimit = getRateLimit();
+    const body = (await req.json()) as ReportTagBody;
+    const paperId = typeof body.paperId === "string" ? body.paperId : undefined;
 
     if (!paperId) {
       return NextResponse.json(
@@ -55,12 +73,14 @@ export async function POST(req: Request & { ip?: string }) {
     }
       const reportedFields: ReportedFieldInput[] = Array.isArray(body.reportedFields)
         ? body.reportedFields
-            .map((r:Partial<ReportedFieldInput>) => ({
-              field: typeof r.field === "string" ? r.field.trim() : "",
-              value: typeof r.value === "string" ? r.value.trim() : undefined,
-            }))
-            .filter((r:Partial<ReportedFieldInput>) => r.field)
-        : [];
+            .map((r): ReportedFieldInput | null => {
+               if (!r || typeof r !== "object") return null;
+
+              const field = typeof (r as { field?: unknown }).field === "string" ? (r as { field: string }).field.trim() : "";
+              const value = typeof (r as { value?: unknown }).value === "string" ? (r as { value: string }).value.trim() : undefined;
+             return field ? { field, value } : null;
+            })
+             .filter((r): r is ReportedFieldInput => r !== null):[];
 
     for (const rf of reportedFields) {
       if (!ALLOWED_FIELDS.includes(rf.field)) {
@@ -82,9 +102,10 @@ export async function POST(req: Request & { ip?: string }) {
     const newReport = await TagReport.create({
       paperId,
       reportedFields,
-      comment: body.comment,
-      reporterEmail: body.reporterEmail,
-      reporterId: body.reporterId,
+      comment: typeof body.comment === "string" ? body.comment : undefined,
+      reporterEmail: typeof body.reporterEmail === "string" ? body.reporterEmail : undefined,
+      reporterId: typeof body.reporterId === "string" ? body.reporterId : undefined,
+
     });
 
     return NextResponse.json(
diff --git a/src/components/Footer.tsx b/src/components/Footer.tsx
@@ -15,6 +15,12 @@ import {
 } from "react-icons/fa6";
 import { Bold, Mail } from "lucide-react";
 import toast from "react-hot-toast";
+
+type SubscribeResponse = {
+  success?: boolean;
+  error?: string;
+};
+
 export default function Footer() {
   const { theme } = useTheme();
   const [isDarkMode, setIsDarkMode] = useState<boolean | null>(true);
@@ -37,8 +43,9 @@ export default function Footer() {
         body: JSON.stringify({ email }),
       })     
       .then(async (res) => {
-        if (!res.ok) throw new Error("Network response was not ok.");
-        return res.json();
+        const data = (await res.json()) as SubscribeResponse;
+        if (!res.ok) throw new Error(data.error ?? "Something went wrong.");
+        return data;
       }),
       {
         loading: "Subscribing...",
@@ -97,7 +104,7 @@ export default function Footer() {
         {/* Events */}
         <div className="flex w-full flex-col gap-2 text-black dark:text-white  lg:w-[15%]">
           <h3 className="font-jost text-xl font-semibold">Events</h3>
-          <Link href="https://devsoc25.codechefvit.com" target="_blank">DevSoc</Link>
+          <Link href="https://devsoc25.codechefvit.com" target="_blank">DevSOC</Link>
           <Link href="https://gravitas.codechefvit.com" target="_blank">CookOff</Link>
           <Link href="https://gravitas.codechefvit.com" target="_blank">Clueminati</Link>
         </div>
diff --git a/src/components/ReportTagModal.tsx b/src/components/ReportTagModal.tsx
@@ -14,9 +14,11 @@ import { Button } from "@/components/ui/button";
 import { MultiSelect } from "@/components/multi-select";
 import LabeledInput from "@/components/ui/LabeledInput";
 import LabeledSelect from "@/components/ui/LabeledSelect";
-import axios from "axios";
+import axios, { AxiosError } from "axios";
 import toast from "react-hot-toast";  
 
+type ReportResponse = { error?: string; message?: string };
+
 interface ReportTagModalProps {
   paperId: string;
   subject?: string;
@@ -40,8 +42,8 @@ const ReportTagModal = ({
   const [internalOpen, setInternalOpen] = useState(false);
   const isControlled = open !== undefined && setOpen !== undefined;
 
-  const modalOpen = isControlled ? open! : internalOpen;
-  const modalSetOpen = isControlled ? setOpen! : setInternalOpen;
+  const modalOpen = isControlled ? open : internalOpen;
+  const modalSetOpen = isControlled ? setOpen : setInternalOpen;
   const [comment, setComment] = useState("");
   const [email, setEmail] = useState("");
   const [selectedCategories, setSelectedCategories] = useState<string[]>([]);
@@ -64,14 +66,14 @@ const ReportTagModal = ({
   const isDirty = useMemo(() => {
     if (selectedCategories.length === 0) return false;
     for (const c of selectedCategories) {
-      const curr = (categoryValues[c] || "").trim();
-      const orig = (originalCategoryValues[c] || "").trim();
+      const curr = (categoryValues[c] ?? "").trim();
+      const orig = (originalCategoryValues[c] ?? "").trim();
       if (curr !== orig) return true;
     }
 
     if (selectedCategories.includes("subject")) {
-      const currCode = (categoryValues["courseCode"] || "").trim();
-      const origCode = (originalCategoryValues["courseCode"] || "").trim();
+      const currCode = (categoryValues.courseCode ?? "").trim();
+      const origCode = (originalCategoryValues.courseCode ?? "").trim();
       if (currCode !== origCode) return true;
     }
 
@@ -83,7 +85,8 @@ const ReportTagModal = ({
     for (const c of selectedCategories) {
       if (categoryValues[c]) continue;
       if (c === "subject" && subject) {
-        const m = subject.match(/^(.*)\s*\[([^\]]+)\]\s*$/);
+        const subjectRegex = /^(.*)\s*\[([^\]]+)\]\s*$/;
+        const m = subjectRegex.exec(subject);
         if (m?.[1] && m?.[2]) {
           const name = m[1].trim();
           const code = m[2].trim();
@@ -104,17 +107,18 @@ const ReportTagModal = ({
     if (open) {
       const base: Record<string, string> = {};
       if (subject) {
-        const m = subject.match(/^(.*)\s*\[([^\]]+)\]\s*$/);
+        const subjectRegex = /^(.*)\s*\[([^\]]+)\]\s*$/;
+        const m = subjectRegex.exec(subject);
         if (m?.[1] && m?.[2]) {
-          base["subject"] = m[1].trim();
-          base["courseCode"] = m[2].trim();
+          base.subject = m[1].trim();
+          base.courseCode = m[2].trim();
         } else {
-          base["subject"] = subject;
+          base.subject = subject;
         }
       }
-      if (exam) base["exam"] = exam;
-      if (slot) base["slot"] = slot;
-      if (year) base["year"] = year;
+      if (exam) base.exam = exam;
+      if (slot) base.slot = slot;
+      if (year) base.year = year;
       setOriginalCategoryValues(base);
       setOriginalEmail("");
     } else {
@@ -134,23 +138,23 @@ const ReportTagModal = ({
   }
 
   if (selectedCategories.includes("subject")) {
-  const sub = (categoryValues.subject || "").trim();
+  const sub = (categoryValues.subject ?? "").trim();
   if (!sub) {
     toast.error("Subject name cannot be empty.");
     return;
   }
 }
 
   if (selectedCategories.includes("slot")) {
-    const v = (categoryValues.slot || "").trim();
+    const v = (categoryValues.slot ?? "").trim();
     const slotRegex = /^[A-G][1-2]$/;
     if (!slotRegex.test(v)) {
       toast.error("Slot must be from A1 to G2 (e.g., D1, B2).");
       return;
     }
   }
   if (selectedCategories.includes("year")) {
-    const y = (categoryValues.year || "").trim();
+    const y = (categoryValues.year ?? "").trim();
     const yearRegex = /^\d{4}(-\d{4})?$/;
     if (!yearRegex.test(y)) {
       toast.error("Year must be a valid format (e.g., 2024 or 2024-2025).");
@@ -180,15 +184,15 @@ const ReportTagModal = ({
 
   for (const c of selectedCategories) {
     if (c === "subject") {
-      const newSub = (categoryValues.subject || "").trim();
-      const oldSub = (originalCategoryValues.subject || "").trim();
+      const newSub = (categoryValues.subject ?? "").trim();
+      const oldSub = (originalCategoryValues.subject ?? "").trim();
 
       if (newSub !== oldSub) {
         reportedFields.push({ field: "subject", value: newSub });
       }
 
-      const newCode = (categoryValues.courseCode || "").trim();
-      const oldCode = (originalCategoryValues.courseCode || "").trim();
+      const newCode = (categoryValues.courseCode ?? "").trim();
+      const oldCode = (originalCategoryValues.courseCode ?? "").trim();
 
       if (newCode !== oldCode) {
         reportedFields.push({ field: "courseCode", value: newCode });
@@ -197,8 +201,8 @@ const ReportTagModal = ({
       continue;
     }
 
-    const newVal = (categoryValues[c] || "").trim();
-    const oldVal = (originalCategoryValues[c] || "").trim();
+    const newVal = (categoryValues[c] ?? "").trim();
+    const oldVal = (originalCategoryValues[c] ?? "").trim();
 
     if (newVal !== oldVal) {
       reportedFields.push({ field: c, value: newVal });
@@ -213,20 +217,23 @@ if (reportedFields.length === 0 && comment.trim().length === 0) {
     paperId,
     reportedFields,
     comment,
-    reporterEmail: email || undefined,
+    reporterEmail: email ?? undefined,
   };
   setLoading(true);
   await toast.promise(
-    axios.post("/api/report-tag", payload),
+    axios.post<ReportResponse>("/api/report-tag", payload),
     {
       loading: "Submitting your report...",
       success: "Reported successfully. Thank you, We will work on that",
-      error: (err)=>{
-        return (
-        err?.response?.data?.error ||
-        err?.message ||
-        "Failed to submit report."
-        )
+      error: (err: unknown)=>{
+          if (axios.isAxiosError<ReportResponse>(err)) {
+            return ( 
+              err.response?.data?.error ??
+              err.message ??
+              "Failed to submit report."
+            );
+        }
+        return err instanceof Error ? err.message : "Failed to submit report.";
       },
     }
   )
diff --git a/src/components/ui/LabeledSelect.tsx b/src/components/ui/LabeledSelect.tsx
@@ -17,7 +17,7 @@ const LabeledSelect = ({ label, value, onChange, options, placeholder }: Labeled
     <Field label={label}>
       <Select value={value} onValueChange={onChange}>
         <SelectTrigger className="w-full">
-          <SelectValue placeholder={placeholder || "Select"} />
+          <SelectValue placeholder={placeholder ?? "Select"} />
         </SelectTrigger>
         <SelectContent>
           {options.map((opt) => (
diff --git a/src/components/ui/field.tsx b/src/components/ui/field.tsx
@@ -6,7 +6,7 @@ export function Field({ label, children, className }: {
   className?: string;
 }) {
   return (
-    <div className={`space-y-1 w-full ${className || ""}`}>
+    <div className={`space-y-1 w-full ${className ?? ""}`}>
       <Label>{label}</Label>
       {children}
     </div>
diff --git a/src/context/filterContext.tsx b/src/context/filterContext.tsx
diff --git a/src/lib/services/paper.ts b/src/lib/services/paper.ts
