Implement fix based on CodeRabbit suggestions

Author: prakashsingha

app/(root)/page.tsx

@@ -1,11 +1,15 @@
+import { auth } from "@clerk/nextjs/server";
 import BookCard from "@/components/BookCard";
 import HeroSection from "@/components/HeroSection";
 import { getAllBooks } from "@/lib/actions/book.actions";
 import { IBook } from "@/types";
 // import { sampleBooks } from "@/lib/constants";
 
 const Page = async () => {
-  const bookResults = await getAllBooks();
+  const { userId } = await auth();
+  const bookResults = userId
+    ? await getAllBooks(userId)
+    : { success: true, data: [] };
   const books = bookResults.success && bookResults.data ? bookResults.data : [];
 
   return (

components/UploadForm.tsx

@@ -124,7 +124,7 @@ const UploadForm = () => {
 
     // Check if book already exists
     try {
-      const checkResult = await checkBookExists(formData.title);
+      const checkResult = await checkBookExists(formData.title, userId);
       if (checkResult?.exists && checkResult.data) {
         toast.info("Book with the same title already exists");
         form.reset();
@@ -134,6 +134,7 @@ const UploadForm = () => {
 
       const fileTitle = generateSlug(formData.title) || `book-${new Date().getTime()}`;
       const pdfFile = formData.pdfFile;
+      // parsePDFFile depends on browser APIs (window/document/canvas), so keep it in this client flow.
       const parsedPdf = await parsePDFFile(pdfFile);
       if(parsedPdf.content.length === 0){
         toast.error("Failed to parse PDF file. Please try again with a different file.");

lib/actions/book.actions.ts

@@ -37,7 +37,6 @@ export const createBook = async (data: CreateBook) => {
 export const saveBookSegments = async (bookId: string, clerkId:string, segments: TextSegment[]) => {
     try {
         await connectToDb();
-        console.log("Saving book segments", bookId, clerkId, segments);
 
         const segmentsToSave = segments.map(segment => ({
             ...segment,
@@ -105,10 +104,10 @@ export const getBookBySlug = async (slug: string, clerkId: string) => {
     }
 }
 
-export const getAllBooks = async () => {
+export const getAllBooks = async (clerkId: string) => {
     try {
         await connectToDb();
-        const books = await Book.find().sort({createdAt: -1}).lean();
+        const books = await Book.find({ clerkId }).sort({createdAt: -1}).lean();
         return {
             success: true,
             data: serializeData(books)

lib/utils.ts

@@ -92,6 +92,10 @@ export const formatDuration = (seconds: number): string => {
 
 export async function parsePDFFile(file: File) {
   try {
+    if (typeof window === 'undefined' || typeof document === 'undefined') {
+      throw new Error('parsePDFFile must run in a browser runtime (window/document unavailable)');
+    }
+
     // Use the legacy build for broader browser compatibility in client-side parsing.
     const pdfjsLib = await import('pdfjs-dist/legacy/build/pdf.mjs');