Skip to content

Commit 656ca7e

Browse files
authored
Merge pull request #1870 from ksylvan/kayvan/update-web-ui-pdfjs-library
Web UI: upgrade pdfjs and add SSR-safe dynamic PDF worker init
2 parents 9c7ce4a + 0025466 commit 656ca7e

9 files changed

Lines changed: 628 additions & 865 deletions

File tree

.vscode/settings.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,7 @@
166166
"sess",
167167
"sgaunet",
168168
"shellquote",
169+
"skeletonlabs",
169170
"SSEHTTP",
170171
"storer",
171172
"Streamlit",
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
### PR [#1870](https://github.com/danielmiessler/Fabric/pull/1870) by [ksylvan](https://github.com/ksylvan): Web UI: upgrade pdfjs and add SSR-safe dynamic PDF worker init
2+
3+
- Upgrade `pdfjs-dist` to v5 with new engine requirement
4+
- Dynamically import PDF.js to avoid SSR import-time crashes
5+
- Configure PDF worker via CDN using runtime PDF.js version
6+
- Update PDF conversion pipeline to use lazy initialization
7+
- Guard chat message localStorage persistence behind browser checks

web/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,9 @@
2626
"eslint-plugin-svelte": "^2.46.1",
2727
"lucide-svelte": "^0.309.0",
2828
"mdsvex": "^0.11.2",
29-
"patch-package": "^8.0.0",
29+
"patch-package": "^8.0.1",
3030
"pdf-to-markdown-core": "github:jzillmann/pdf-to-markdown#modularize",
31-
"pdfjs-dist": "^4.2.67",
31+
"pdfjs-dist": "^5.4.449",
3232
"postcss": "^8.5.3",
3333
"postcss-load-config": "^6.0.1",
3434
"rehype-autolink-headings": "^7.1.0",

web/pnpm-lock.yaml

Lines changed: 135 additions & 414 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)