Revert "⬆️ Bump pdfjs-dist from 5.3.31 to 6.1.200"

This reverts commit 5599550.

Author: williamchong

app/components/PDFReader.vue

@@ -286,7 +286,7 @@
 <script setup lang="ts">
 import type { DropdownMenuItem } from '@nuxt/ui'
 import type { RouteLocationRaw } from 'vue-router'
-import type { PDFDocumentLoadingTask, PDFDocumentProxy, PDFPageProxy, PageViewport } from 'pdfjs-dist'
+import type { PDFDocumentProxy, PDFPageProxy, PageViewport } from 'pdfjs-dist'
 import type { TextItem, TextMarkedContent } from 'pdfjs-dist/types/src/display/api'
 import type { ReaderLeftSidebarTab } from './ReaderLeftSidebar.props'
 import { ANNOTATION_TEXT_MAX_LENGTH } from '~~/shared/constants/annotations'
@@ -387,9 +387,6 @@ const scaleMenuItems = computed<DropdownMenuItem[]>(() => {
 })
 
 const pdfDocument = shallowRef<PDFDocumentProxy>()
-// Tracked separately from pdfDocument so an in-flight getDocument() can be
-// canceled on reload/unmount; destroying the task also destroys the document.
-const pdfLoadingTask = shallowRef<PDFDocumentLoadingTask>()
 const textContentCache = new Map<number, Awaited<ReturnType<PDFPageProxy['getTextContent']>>>()
 const pageSearchTextCache = new Map<number, { raw: string, lower: string }>()
 const renderQueue = ref<(() => Promise<void>)[]>([])
@@ -610,8 +607,7 @@ onBeforeUnmount(() => {
   renderQueue.value = []
   textContentCache.clear()
   pageSearchTextCache.clear()
-  pdfLoadingTask.value?.destroy().catch(() => {})
-  pdfLoadingTask.value = undefined
+  pdfDocument.value?.destroy()
   pdfDocument.value = undefined
 })
 
@@ -676,24 +672,20 @@ async function loadPDF() {
     return
   }
 
-  pdfLoadingTask.value?.destroy().catch(() => {})
-  pdfDocument.value = undefined
+  if (pdfDocument.value) {
+    pdfDocument.value.destroy()
+    pdfDocument.value = undefined
+  }
 
-  let loadingTask: PDFDocumentLoadingTask | undefined
   try {
-    loadingTask = pdfjsLib.value.getDocument({
+    const loadingTask = pdfjsLib.value.getDocument({
       data: props.pdfBuffer,
       wasmUrl: `https://cdn.jsdelivr.net/npm/pdfjs-dist@${pdfjsLib.value.version}/wasm/`,
       cMapUrl: `https://cdn.jsdelivr.net/npm/pdfjs-dist@${pdfjsLib.value.version}/cmaps/`,
       cMapPacked: true,
     })
-    pdfLoadingTask.value = loadingTask
-
-    const doc = await loadingTask.promise
-    // Superseded by a newer load or unmount while awaiting.
-    if (pdfLoadingTask.value !== loadingTask) return
 
-    pdfDocument.value = doc
+    pdfDocument.value = await loadingTask.promise
     totalPages.value = pdfDocument.value.numPages
 
     outlineItems.value = []
@@ -706,8 +698,6 @@ async function loadPDF() {
     renderPages()
   }
   catch (error) {
-    // Destroying a superseded task rejects its promise; not a real error.
-    if (loadingTask && pdfLoadingTask.value !== loadingTask) return
     emit('error', error as Error)
   }
 }
@@ -922,13 +912,19 @@ async function renderPageToCanvas(
   const page = await pdfDocument.value.getPage(pageNum)
   const viewport = page.getViewport({ scale: scale.value })
 
+  const context = canvas.getContext('2d')
+  if (!context) {
+    page.cleanup()
+    return
+  }
+
   canvas.height = viewport.height * pixelRatio.value
   canvas.width = viewport.width * pixelRatio.value
   canvas.style.width = `${viewport.width}px`
   canvas.style.height = `${viewport.height}px`
 
   await page.render({
-    canvas,
+    canvasContext: context,
     transform: pixelRatio.value !== 1
       ? [pixelRatio.value, 0, 0, pixelRatio.value, 0, 0]
       : undefined,

app/utils/parse-book-metadata.ts

@@ -108,9 +108,7 @@ export async function parsePDFMetadata(file: File): Promise<ParsedBookMetadata>
     return { title }
   }
   finally {
-    // Best-effort teardown; a rejected destroy() must not surface as an
-    // unhandled rejection or mask the parse result.
-    doc.loadingTask.destroy().catch(() => {})
+    doc.destroy()
   }
 }