[sebastianMindee]

PR Details

Bump most libraries and remove support for old browsers.

• add integration testing

Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

• Docs change / refactoring / dependency upgrade
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• I have added type hints (flow) to cover my changes.
• All new and existing tests passed.

[Copilot]

Copilot review overview

🟡 Changes recommended

The production library-mode worker packaging needs a consumer-level integration test before approval.

Review effort: Balanced
Findings: 1

Open (1)

• Add production library integration test for inline worker loading · New

What changed in this PR

Updates major dependencies and modernizes PDF/HEIC processing while dropping legacy runtime support.

Changes:

• Upgrades Konva and PDF.js; removes UUID.
• Bundles and lazily loads PDF workers and HEIC conversion.
• Adapts rendering and Konva APIs to newer versions.

File	Description
vite.config.ts	Configures ES module workers.
tsconfig.json	Adds Vite client types.
src/​utils/​pdfWorker.ts	Creates an inline PDF worker.
src/​utils/​pdf.ts	Lazily initializes PDF.js.
src/​utils/​image.ts	Lazily imports HEIC conversion.
src/​utils/​getPDFPageCount.ts	Uses centralized PDF loading.
src/​utils/​getImagesFromPDF.ts	Updates PDF rendering API.
src/​utils/​functions.ts	Uses updated Konva typings.
src/​common/​constants.ts	Safely handles optional stroke values.
package.json	Updates dependencies and Node requirement.
pnpm-lock.yaml	Locks upgraded dependency versions.

Files not reviewed (1)

• pnpm-lock.yaml: Generated file

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot review overview

🟡 Changes recommended

Cache-busting only the entry module leaves the lazy PDF module shared, invalidating multiple worker lifecycle tests.

Review effort: Balanced
Findings: 3

Open (3)

• Lazy PDF worker chunk is not invalidated between fresh loads · New
• Test remounts lens instead of verifying pointer-position updates · New
• Test does not verify shape updates on an existing viewer · New

Resolved since last review (1)

• Add production library integration test for inline worker loading

Files not reviewed (1)

• pnpm-lock.yaml: Generated file