feat(cli): support .xls, .xlsm, and .xlsb spreadsheet formats - #14078
feat(cli): support .xls, .xlsm, and .xlsb spreadsheet formats#14078Sayandevxyz wants to merge 1 commit into
Conversation
| }, | ||
| "trustedDependencies": [ | ||
| "web-tree-sitter", | ||
| "esbuild", |
There was a problem hiding this comment.
SUGGESTION: Unrelated lockfile churn
This PR only adds spreadsheet format support in packages/opencode/src/kilocode/tool/xlsx.ts and does not add, remove, or change any dependency. The reordering of trustedDependencies and patchedDependencies in bun.lock is incidental (same entries, different order, likely written by a different Bun version) and only adds unrelated diff/merge surface. Consider reverting the bun.lock changes to keep this PR focused.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge The change is small and correct. Files Reviewed (3 files)
Previous Review Summary (commit be10708)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit be10708)Status: 1 Issue Found | Recommendation: Address before merge The functional change is small and correct: Overview
Issue Details (click to expand)SUGGESTION
Files Reviewed (4 files)
Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0 Review guidance: REVIEW.md from base branch |
be10708 to
c24f6e8
Compare
What
Extends the spreadsheet reader tool to support three additional Excel formats beyond
.xlsx:.xls- legacy Excel 97-2003 format.xlsm- macro-enabled Excel workbook.xlsb- Excel Binary WorkbookWhy
Users working with older Excel files were getting an unsupported-format error. This change makes the read tool handle all common Excel variants consistently.
Changes
packages/opencode/src/kilocode/tool/xlsx.ts- added.xls,.xlsm,.xlsbto accepted extensionspackages/opencode/test/kilocode/read-xlsx.test.ts- added test cases for new formats.changeset/add-xls-xlsm-xlsb-spreadsheet-support.md- minor bump changeset