Include sheet state in listWorksheetInfo

[christian-fries]

The method listWorksheetInfo is great for analyzing an Excel workbook without loading it (and therefore consuming a lot of memory). Sadly, it is missing the sheet state of the worksheets. Adding the sheet state to the output of listWorksheetInfo would make reasoning about workbooks and their worksheets more flexibel.

[oleibman]

Would you need sheetState returned all the time, or just in cases where it isn't visible? The latter would probably be easier for me, and less likely to be viewed as a breaking change (existing results would change, though it's hard for me to imagine what might break as a result, but sometimes I don't have enough imagination).

[christian-fries]

For our use cases, we would need a reliable way to detect if a worksheet is visible or not. If the sheetState property is only set if the sheet is not visible, it means that whenever the property is not available, the sheet is visible. Doesn't sound like the best design but it would work for us.

[oleibman]

Thank you for the input. I am still considering the best approach. The fact that four different input types (csv, html, slk, and xml) don't support any state other than 'visible' is a factor.