Merge branch 'main' into pass

Author: Aryan0819

backend/config/passportConfig.js

@@ -7,7 +7,7 @@ passport.use(
         { usernameField: "email" },
         async (email, password, done) => {
             try {
-                const user = await User.findOne({ email }).select("+password");
+                const user = await User.findOne( {email} );
                 if (!user) {
                     return done(null, false, { message: 'Email is invalid ' });
                 }
@@ -38,13 +38,6 @@ passport.serializeUser((user, done) => {
 passport.deserializeUser(async (id, done) => {
     try {
         const user = await User.findById(id);
-        
-        // 🛡️ Safety check: If the user record no longer exists in MongoDB, exit safely
-        // This prevents the application from throwing an unhandled TypeError downstream
-        if (!user) {
-            return done(null, false); // Gracefully invalidates the cookie and ends the request loop
-        }
-        
         done(null, user);
     } catch (err) {
         done(err, null);

backend/models/User.js

@@ -19,20 +19,16 @@ const UserSchema = new mongoose.Schema({
 });
 
 // ✅ FIXED: no next()
-UserSchema.pre('save', async function (next) {
-  if (!this.isModified('password')) return next();
-  try {
-      const salt = await bcrypt.genSalt(10);
-      this.password = await bcrypt.hash(this.password, salt);
-      next(); // Tells Mongoose hashing is done, save the document now
-  } catch (err) {
-    next(err); // Safely passes any encryption errors to the database handler
-  }
+UserSchema.pre('save', async function () {
+  if (!this.isModified('password')) return;
+
+  const salt = await bcrypt.genSalt(10);
+  this.password = await bcrypt.hash(this.password, salt);
 });
 
 // ✅ password comparison
 UserSchema.methods.comparePassword = async function (enteredPassword) {
   return bcrypt.compare(enteredPassword, this.password);
 };
 
-module.exports = mongoose.model("User", UserSchema);
+module.exports = mongoose.model("User", UserSchema);

src/components/ScrollProgressBar.tsx

@@ -61,21 +61,18 @@ const ScrollProgressBar = () => {
       {/* Scroll progress bar after animation ends */}
       {!isAnimating && (
         <div
-        style={{
-          position: "fixed",
-          top: 0,
-          left: 0,
-          // Dynamically swaps between full width loading animation and real-time scroll updates
-          width: isAnimating ? "0" : `${scrollWidth}%`,
-          height: isAnimating ? "2px" : "3px",
-          backgroundColor: isAnimating ? "blue" : "#3b82f6",
-          zIndex: 100,
-          // Applies the loading animation curve ONLY during the initial loading phase
-          animation: isAnimating ? "slideIn 2s ease-in-out forwards" : "none",
-          // Applies smooth tracking ticks once scroll tracking takes over
-          transition: isAnimating ? "none" : "width 0.1s ease",
-        }}
-      />
+          style={{
+            position: "fixed",
+            top: 0,
+            left: 0,
+            width: `${scrollWidth}%`,
+            height: "3px",
+            backgroundColor: "#3b82f6",
+            zIndex: 100,
+            transition: "width 0.1s ease",
+          }}
+        />
+      )}
 
       {/* Animation Keyframes */}
       <style>
@@ -93,3 +90,5 @@ const ScrollProgressBar = () => {
     </>
   );
 };
+
+export default ScrollProgressBar;

src/hooks/useDebounce.ts

@@ -1,19 +1,17 @@
-import { useState, useEffect, useRef } from "react";
+import { useState, useEffect } from 'react';
 
-// Inside your custom useDebounce hook function:
-const isMounted = useRef(true); 
+export function useDebounce<T>(value: T, delay: number): T {
+  const [debouncedValue, setDebouncedValue] = useState<T>(value);
 
-useEffect(() => {
-  isMounted.current = true; 
-
-  const handler = setTimeout(() => {
-    if (isMounted.current) {
+  useEffect(() => {
+    const handler = setTimeout(() => {
       setDebouncedValue(value);
-    }
-  }, delay);
+    }, delay);
+
+    return () => {
+      clearTimeout(handler);
+    };
+  }, [value, delay]);
 
-  return () => {
-    clearTimeout(handler);   
-    isMounted.current = false; 
-  };
-}, [value, delay]);
+  return debouncedValue;
+}