[PR #12885] fix: document extractor node incorrectly handles doc and ppt files #27767

Closed
opened 2026-02-21 20:42:10 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/12885

State: closed
Merged: No


Summary

Fixes #12889

  1. In the current code, document extractor node was using python-docx library to handle doc files, which it cannot process. this commit is trying to use unstructured API to handle doc files.
  2. In the current code, document extractor node is using partititon_ppt to handle ppt files, probably a leftover when unstructured[ppt] was still in pyproject.toml. This commit is trying to use unstructured API to handle ppt files.

Tip

Close issue syntax: Fixes #<issue number> or Resolves #<issue number>, see documentation for more details.

Screenshots

Before After
... ...

Checklist

Important

Please review the checklist below before submitting your pull request.

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods
**Original Pull Request:** https://github.com/langgenius/dify/pull/12885 **State:** closed **Merged:** No --- # Summary Fixes #12889 1. In the current code, document extractor node was using `python-docx` library to handle doc files, which it cannot process. this commit is trying to use unstructured API to handle doc files. 2. In the current code, document extractor node is using `partititon_ppt` to handle ppt files, probably a leftover when `unstructured[ppt]` was still in `pyproject.toml`. This commit is trying to use unstructured API to handle ppt files. > [!Tip] > Close issue syntax: `Fixes #<issue number>` or `Resolves #<issue number>`, see [documentation](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword) for more details. # Screenshots | Before | After | |--------|-------| | ... | ... | # Checklist > [!IMPORTANT] > Please review the checklist below before submitting your pull request. - [ ] This change requires a documentation update, included: [Dify Document](https://github.com/langgenius/dify-docs) - [x] I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!) - [x] I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change. - [x] I've updated the documentation accordingly. - [x] I ran `dev/reformat`(backend) and `cd web && npx lint-staged`(frontend) to appease the lint gods
yindo added the pull-request label 2026-02-21 20:42:10 -05:00
yindo closed this issue 2026-02-21 20:42:10 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#27767