Skip to content

[PB-6834]: add embedded JPEG scanner and image test builders - #2152

Open
terrerox wants to merge 1 commit into
masterfrom
feature/image-preview-jpeg-scanner
Open

terrerox wants to merge 1 commit into
masterfrom
feature/image-preview-jpeg-scanner

Conversation

@terrerox

@terrerox terrerox commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds the utif2 dependency (package.json + yarn.lock) used by the upcoming TIFF decoder.
  • Adds embeddedJpeg.ts: scans a camera RAW file and extracts the largest browser-decodable JPEG embedded in it.
  • Adds src/testUtils/imageBuilders.ts: in-code JPEG/TIFF builders so tests do not need binary fixtures.
  • Adds node unit tests for the scanner.
  • First of three stacked branches for RAW/TIFF file preview; no user-facing change on its own.

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

Adds the utif2 dependency, a scanner that extracts the largest
browser-decodable JPEG embedded in camera RAW files, and in-code
JPEG/TIFF builders for tests.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8c4b382
Status: ✅  Deploy successful!
Preview URL: https://19e16013.drive-web.pages.dev
Branch Preview URL: https://feature-image-preview-jpeg-s.drive-web.pages.dev

View logs

@sonarqubecloud

Copy link
Copy Markdown

@terrerox terrerox changed the title feat(preview): add embedded JPEG scanner and image test builders [PB-6834]: add embedded JPEG scanner and image test builders Sep 14, 2026
@terrerox terrerox self-assigned this Sep 14, 2026
@terrerox
terrerox requested a review from xabg2 September 16, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant