[xbsheng]

manage.py show printed only the first picture of a switch's row in docs/switches.md, while the page lets a row carry more than one and holds every picture in a row to its rules: show vpn_pill named vpn_pill.jpg and never vpn_pill_rest.jpg, the pill at rest the page draws beside it, and icon_only_vpn lost icon_only_vpn_on.jpg the same way. switches() now reads every <img src> of a row in page order (img_srcs()), the item keeps them as screenshots while screenshot stays the first path so everything else that reads an item is unchanged, and show prints one screenshot: line per picture - a key with one picture prints exactly what it printed before.

Tested on: no TV; tools/manage.py against the checkout's own docs/switches.md, unit tests only

If this adds or changes a TV list

• not a list change: nothing in examples/ or a tv.json is touched

If this touches the launcher

• not a launcher change: no Java, no config.xml, no script on the install or Home path

If this adds or changes a theme

• not a theme change: no theme folder, config.json or themes/README.md row is touched

If this adds a theme key (a switch)

• not a new key: docs/switches.md is read, not edited (a new row would need a screenshot from a real TV)

If this adds an add-on

• not an add-on change

Two files: tools/manage.py (+30 -6) and tests/test_manage_show_screenshots.py (new, 13 tests). No launcher Java, list, tv.json, generated table, count, workflow, PROMPT.md, disable.sh/check.py logic, Home guard or .github file touched.

Counter-proof, no TV: the new test file alone on origin/main 294d3d5, nothing else changed -> FAILED (failures=4, errors=7), e.g. AssertionError: 'vpn_pill_rest.jpg' not found in '... screenshot: .../vpn_pill.jpg ...' : show printed one of the two pictures docs/switches.md gives vpn_pill, and AssertionError: ['two.jpg', 'two_rest.jpg'] != ['two.jpg'] : show printed one picture of a row that carries two. On this branch the same file is 13 tests OK.

Tests: CLEANYOURTV_STRICT=1 tools/venv/bin/python -m unittest discover -s tests -> 3222 OK (skipped=36) = origin/main 294d3d5's own 3209 + 13; tools/venv/bin/python -I tools/gen_tables.py --check exit 0; no shell changed.

Collision: of the 54 open PRs, none touches tools/manage.py, tests/test_manage_show_screenshots.py or docs/switches.md except #2954, which appends its own row and picture to that page - a file this PR does not edit.

[github-actions]

maintainer-bot review · suggests merge (p3)

Thanks for catching that show vpn_pill left out the picture of the pill at rest, and for the tests that check every picture the page gives a key. This looks ready to merge as it is: keys with one picture print exactly what they did before.

Checks

• Size: size/M (+190 -6, 2 files)

This is a suggestion; a maintainer makes the final call. A maintainer can comment /bot review to run it again.

_{Reviewed by maintainer-bot · runs on 0G Private Computer}

[Gathin23]

Thanks @xbsheng, manage.py show now prints every picture a switch's row carries, so vpn_pill and icon_only_vpn show both of theirs, while 'screenshot' stays the first path for other readers.