revert(deps): pdfjs-dist back to 4.3.136 (#1697)

ahorowitz123 · claude · mrscobbler · web-flow · commit fa5605f62ddb · 2026-07-09T16:16:47.000-07:00
Revert the pdfjs-dist 6.0.227 bump (#1666) while v6 regressions are investigated. Restores engines.node >=20.0.0 (the bump only existed to satisfy pdfjs 6) and removes the annotationLayer z-index workaround from #1694, which compensated for v6's earlier page-layer insertion and is unnecessary on v4. Co-authored-by: Claude Fable 5 <noreply@anthropic.com> Co-authored-by: Sophia <1317004+mrscobbler@users.noreply.github.com>
diff --git a/package.json b/package.json
@@ -19,19 +19,14 @@
         "./styles.css": "./dist/lib/index.css",
         "./package.json": "./package.json"
     },
-    "files": [
-        "dist/lib"
-    ],
-    "sideEffects": [
-        "*.css",
-        "*.scss"
-    ],
+    "files": ["dist/lib"],
+    "sideEffects": ["*.css", "*.scss"],
     "publishConfig": {
         "access": "public"
     },
     "dependencies": {
         "axios": "^0.24.0",
-        "pdfjs-dist": "6.0.227"
+        "pdfjs-dist": "4.3.136"
     },
     "peerDependencies": {
         "@box/blueprint-web": "^14.32.1",
@@ -137,7 +132,7 @@
         "webpack-dev-server": "^5.2.4"
     },
     "engines": {
-        "node": ">=22.13.0",
+        "node": ">=20.0.0",
         "yarn": ">=1.19.x"
     },
     "scripts": {
diff --git a/src/lib/viewers/doc/_docBase.scss b/src/lib/viewers/doc/_docBase.scss
@@ -363,12 +363,6 @@ $thumbnail-sidebar-width: 191px; // Extra pixel to account for sidebar border
     }
 
     .annotationLayer {
-        // pdf.js 6 inserts its layers before the box-annotations layers, so the
-        // region creation overlay (pointer-events: auto) paints above the link
-        // annotations and swallows their clicks. Lift the annotation layer above
-        // the overlay but below the region/highlight targets (z-index: 3).
-        z-index: 2;
-
         h1,
         h2,
         h3,
diff --git a/yarn.lock b/yarn.lock
