Skip to content

[WIP] Add per-sheet include/exclude controls for incremental workbook encoding - #42

Draft
kingkillery with Claude wants to merge 1 commit into
mainfrom
claude/add-per-sheet-include-exclude-controls
Draft

[WIP] Add per-sheet include/exclude controls for incremental workbook encoding#42
kingkillery with Claude wants to merge 1 commit into
mainfrom
claude/add-per-sheet-include-exclude-controls

Conversation

@Claude

@Claude Claude AI commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>Add per-sheet include/exclude controls for incremental workbook encoding</issue_title>
<issue_description>## Problem
Some real workbooks contain many sheets with very different sizes. Users may need to encode only a relevant subset instead of paying the cost of every sheet in one run.

Why this matters

During smoke testing, one workbook contained many small sheets plus a very large 3214x110 sheet. Per-sheet controls would allow users to inspect or process the useful sheets immediately while deferring the expensive sheet.

Proposed work

  • Add CLI flags such as --include-sheet, --exclude-sheet, or glob/regex variants.
  • Add equivalent Python API parameters.
  • Record included/excluded sheet names in output metadata.
  • Make evaluation and downstream prompt code tolerate deliberately omitted sheets.

Acceptance criteria

Comments on the Issue (you are @claude[agent] in this section)

@kingkillery @claude @kingkillery @copilot

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.

Add per-sheet include/exclude controls for incremental workbook encoding

2 participants