Skip to content

fix: extract zip documents without external command - #625

Open
deepanshupal wants to merge 1 commit into
camel-ai:mainfrom
deepanshupal:fix-unzip-zipfile
Open

deepanshupal wants to merge 1 commit into
camel-ai:mainfrom
deepanshupal:fix-unzip-zipfile

Conversation

@deepanshupal

Copy link
Copy Markdown

Summary

  • replace the external unzip subprocess with Python's cross-platform zipfile module
  • preserve recursive extracted-file discovery
  • add a focused regression for nested zip contents

Fixes #617.

Testing

  • .venv/bin/python -m pytest tests/test_document_toolkit.py -q (1 passed)
  • Ruff check and format check
  • py_compile and diff check

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.

DocumentProcessingToolkit uses external unzip command, breaking zip extraction on Windows

1 participant