Skip to content

Fix image converter large-file worker path#111

Draft
oguzhankir wants to merge 4 commits into
mainfrom
cursor/image-format-converter-ad90
Draft

Fix image converter large-file worker path#111
oguzhankir wants to merge 4 commits into
mainfrom
cursor/image-format-converter-ad90

Conversation

@oguzhankir
Copy link
Copy Markdown
Owner

Summary

  • Fix the Image Format Converter large-input Web Worker path by posting a structured-clone-safe plain request object instead of Svelte reactive state.
  • Clarify converter stat cards so source/output cards show actual original and converted sizes.
  • Add regression coverage for worker request structured cloning.

Type of change

  • feat
  • fix
  • docs
  • refactor
  • test
  • chore

Scope

  • JSON
  • YAML
  • XML
  • CSV
  • TOML
  • Text
  • Encode/Decode
  • Shared UI
  • Infrastructure / CI
  • Documentation

Checklist

  • I read CONTRIBUTING.md and .windsurfrules
  • I tested locally with pnpm test
  • I ran pnpm check
  • I ran pnpm lint
  • I ran pnpm build
  • I added/updated tests for changed behavior
  • I added/updated i18n keys for all locales (en,tr,de,es,fr,it) if UI text changed
  • I used dynamic import() for heavy processing libraries
  • I kept user data processing in-browser only
  • I linked the related issue (or explained why none is needed)

Screenshots / Demo (if UI changed)

Large-input worker validation:

large_image_converter_worker_stats_demo.mp4

Related issue

No linked issue; reported directly by the user.

Notes for reviewers

  • This is not a Cloudflare Worker issue; the fixed worker is a browser Web Worker.
  • Large file requests now clone safely before postMessage(), avoiding the previous worker encoding failure path.

To show artifacts inline, enable in settings.

Open in Web Open in Cursor 

cursoragent and others added 4 commits April 29, 2026 20:03
Co-authored-by: Oğuzhan KIR <oguzhankir@users.noreply.github.com>
Co-authored-by: Oğuzhan KIR <oguzhankir@users.noreply.github.com>
Co-authored-by: Oğuzhan KIR <oguzhankir@users.noreply.github.com>
Co-authored-by: Oğuzhan KIR <oguzhankir@users.noreply.github.com>
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.

2 participants