Skip to content

fix: unique ZIP folders for same-named batch PDFs - #630

Merged
Durgeshwar-AI merged 2 commits into
Durgeshwar-AI:mainfrom
nyxsky404:fix/606-unique-batch-zip-folders
Aug 10, 2026
Merged

fix: unique ZIP folders for same-named batch PDFs#630
Durgeshwar-AI merged 2 commits into
Durgeshwar-AI:mainfrom
nyxsky404:fix/606-unique-batch-zip-folders

Conversation

@nyxsky404

Copy link
Copy Markdown
Contributor

Pull Request

Issue Reference

Closes #606


Summary

  • allocate unique ZIP folder names when batch inputs share a filename
  • keep first folder as the base name and suffix later collisions
  • add unit coverage for the folder-name allocator

Screenshots (if applicable)

NA


Checklist

  • My code follows the project's coding conventions
  • I have tested all impacted features
  • I have updated or added necessary documentation

Related Issues / PRs

NA


Open Source Program Participation

Program Name: GSSoC


Additional Notes

Focused Vitest checks, typecheck, and production build pass.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

@nyxsky404 is attempting to deploy a commit to the Durgeshwar's projects Team on Vercel.

A member of the Team first needs to authorize it.

Keep the unique batch ZIP folder README note alongside upstream
merge-same-name wording.

Co-authored-by: Cursor <cursoragent@cursor.com>
@nyxsky404

Copy link
Copy Markdown
Contributor Author

@Durgeshwar-AI Merge conflicts with main are resolved (README kept both the unique batch ZIP-folder note and upstream’s same-name merge wording).

The remaining backend CI failure (test_add_watermark_* unpacking error in blueprints/watermark.py) is already present on upstream main, and is unrelated to this PR. The Vercel check is failing on team authorization, not on this PR’s code.

@Durgeshwar-AI
Durgeshwar-AI merged commit 9e73b33 into Durgeshwar-AI:main Aug 10, 2026
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Batch PDF-to-PNG ZIP overwrites results from same-named files

2 participants