Matches prefix to verify presence of DOCX,PPTX,XLSX files instead of standard file names

[srisudarsan]

Instead of looking for presence of word/document.xml , ppt/presentation.xml and xl/workbook.xml to identify DOCX,PPTX and XLSX files, we look for prefix word/document*.xml, ppt/presentation*.xml and xl/workbook*.xml as certain files generated from office365 has files with different names.
Fixes #3937

[srisudarsan]

Potentially looked at another fix, to read the contents of [Content_Types].xml in the ZIP and identify the content type present in it. But this would involve reading the XML and parsing it to identify the necessary info.

Let me know your thought on the same

[badGarnet]

Hi @srisudarsan, thanks for the contribution. The fix looks reasonable. Could you please update the changelog and bump the version (convention is to bump from devn to dev(n+1)). Then I can let the ci run (ci would stop if detect changelog workflow fails).

[scanny]

@badGarnet looks good to me :)

[scanny]

I think the final word on is_docx will be a package part with the ContentType you highlighted in the .rels file, but I'm fine with this as a step toward that :)

[srisudarsan]

Aligned too. Will raise another PR with changes incorporating the content type identified from [Content_Types].xml

[srisudarsan]

Hi @srisudarsan, thanks for the contribution. The fix looks reasonable. Could you please update the changelog and bump the version (convention is to bump from devn to dev(n+1)). Then I can let the ci run (ci would stop if detect changelog workflow fails).

Done

[srisudarsan]

@badGarnet , seems like the previous run failed, I have updated the CHANGELOG with the recent number, can you approve this flow now ?

[badGarnet]

@badGarnet , seems like the previous run failed, I have updated the CHANGELOG with the recent number, can you approve this flow now ?

seem you forgot to bump the version in __version__.py: https://github.com/Unstructured-IO/unstructured/actions/runs/13950355943/job/39072924215?pr=3959#step:5:1012

[srisudarsan]

@badGarnet , seems like the previous run failed, I have updated the CHANGELOG with the recent number, can you approve this flow now ?

seem you forgot to bump the version in __version__.py: https://github.com/Unstructured-IO/unstructured/actions/runs/13950355943/job/39072924215?pr=3959#step:5:1012

Thanks for highlighting, wasn't aware, done now

[badGarnet]

@badGarnet , seems like the previous run failed, I have updated the CHANGELOG with the recent number, can you approve this flow now ?

seem you forgot to bump the version in __version__.py: https://github.com/Unstructured-IO/unstructured/actions/runs/13950355943/job/39072924215?pr=3959#step:5:1012

Thanks for highlighting, wasn't aware, done now

Linting is now complaining of missing empty line. I would suggest you install pre-commit tool in your env via pip install pre-commit && pre-commit install. The repo has a pre-commit configuration to automatically fix most of the linting issues on changed files and it is a real life saver. Once you've done the install just edit and save the version.py file by adding a new line at the end of the file and commit. The pre-commit tool will fix any remaining issues.

[srisudarsan]

@badGarnet , seems like the previous run failed, I have updated the CHANGELOG with the recent number, can you approve this flow now ?

seem you forgot to bump the version in __version__.py: https://github.com/Unstructured-IO/unstructured/actions/runs/13950355943/job/39072924215?pr=3959#step:5:1012

Thanks for highlighting, wasn't aware, done now

Linting is now complaining of missing empty line. I would suggest you install pre-commit tool in your env via pip install pre-commit && pre-commit install. The repo has a pre-commit configuration to automatically fix most of the linting issues on changed files and it is a real life saver. Once you've done the install just edit and save the version.py file by adding a new line at the end of the file and commit. The pre-commit tool will fix any remaining issues.

I don't think I am getting it right. pre-commit runs and removes the line if added, can you please help making this change and getting this through ?