Excluding sheets works differently for QSEoW and QS Cloud apps, with an additional bug for QS Cloud apps

[mountaindude]

What version of Butler Sheet Icons are you using?

3.4.1

What version of Node.js are you using? Not applicable if you use the standalone version of Butler Sheet Icons.

19

What command did you use to start Butler Sheet Icons?

...

What operating system are you using?

macOS

What CPU architecture are you using?

x86_64

What Qlik Sense versions are you using?

2024-Feb

Describe the Bug

Let's say an unpublished QS Cloud app has 5 public sheets and 1 private sheet:

If running BSI with the --exclude-sheet-status private option the private sheet should not be updated.
The result is this:

This is a bug in at least one way, possible more, depending on how BSI is supposed to work.
This is currently not clearly defined - should thus be clarified!

If run again with --exclude-sheet-status public the public sheets won't be updated - but they are instead left broken:

Scenario one

BSI should clear all sheet icons and then update the correct ones.
This is how BSI works today for QS Cloud apps, except that the clearing of sheet icons results in broken icons.

Scenario two

BSI should first determine which sheets will be updated and clear only those sheet icons before updating the icons.
This is how QSEoW apps are handled currently.

Expected Behavior

No response

To Reproduce

No response