- session_management.py: collapse three multi-line statements that
ruff format would otherwise reformat (CI lint gate was failing on
`ruff format --check`).
- features/tui_settings_session_screens.feature: add three scenarios
that exercise the previously uncovered paths reported by
diff_coverage on prior attempts:
- SettingsScreen.get_settings() returns a settings dict
- SessionManagementScreen renders sessions when visible+loaded
(covers the render-loop body + _render_session_details
non-None branch + the four detail lines)
- TuiCommandRouter routes "/settings" to _settings_command
- features/steps/tui_settings_session_screens_steps.py:
- Make `the selected_index should be {index:d}` look up whichever
screen the scenario created (the step was always referencing
`context.settings_screen`, which crashed in the
SessionManagementScreen scenarios).
- Reuse the existing TuiCommandRouter steps from
tui_commands_coverage_steps.py for the /settings scenario.
The unit_tests / integration_tests CI gates on the prior run were
red on tests unrelated to this PR (CheckpointRepository and
actor_run_signature.robot, neither touched here); the targeted
behave run for this PR's feature file passes locally with all 12
scenarios green.
controller-ci-rerun
HAL9000