fix(excel): restore split-by-column fallback - #178
Merged
CoderWanFeng merged 1 commit intoAug 19, 2026
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
office.excel.split_excel_by_column()and the matching Excel Skill fail before processing any workbook when used with the currently publishedpoexcel0.0.22 package:The repository already contains a maintained split-by-column implementation under
office.lib.excel.SplitExcel, but the public wrapper does not use it.Root Cause
The wrapper unconditionally calls
poexcel.split_excel_by_column. The currentpoexcelrelease does not expose that name from its top-level package, so attribute lookup fails before the bundled implementation can run.Solution
poexcel.split_excel_by_columnwhen a future or alternatepoexcelversion exposes it.office.lib.excel.SplitExcel.split_excel_by_columnimplementation when the dependency does not expose the function.Testing
Also verified the fallback with
poexcel0.0.22 and a generated.xlsxworkbook. It created the split output with the expectedAlphaandBetaworksheets, and the input/output files could be renamed afterward (no open file handles remained).Compatibility
No public API or parameter behavior changes. Environments where
poexcelalready exports the function keep the existing delegation path; affected environments regain the documented functionality through the bundled implementation.Related Issue
No dedicated issue. This is a follow-up to the bundled split implementation fixes merged in #161.