Add MENU_PICK command support

[henrikwidlund]

Same as CONSUMER_MENU_PICK in Bluetooth.

Depending on context:

• Exit screensaver command
• Play command
• OK command

[zehnm]

That sounds very promising!
The current used remote_control.select() function has some weird behaviour. I'll test it within the next few days.

[henrikwidlund]

Do you think we should use this command as the default Play/pause command instead if it works as intended?

[zehnm]

The MENU_PICK command is missing in the media player options list.
I'll try to add it if I have modify rights for your PR.

The command works, but it doesn't exit the screensaver on tvOS 26.1. It only shows the current time within the screensaver.
I'll have to do some more testing, if it makes sense to remap the default commands.

[zehnm]

I've added the command to the simple command list. Now it's available in the web-configurator.

[zehnm]

I have the impression that the new tvOS 26 changed things for the worse when the screensaver is active. Only the back key seems to work, which brings up the new account selection screen (but not always, sometimes the home screen is shown). Some commands now even time out when the screensaver is active.
Maybe this only happens with multiple accounts and the new account selection screen?

There's more testing required, and hopefully there is some "dpad enter" command that works the same as the original ATV remote to get a consistent behaviour.
Otherwise, _playpause_in_screensaver has to be enhanced for other commands too, like cursor_enter, menu, etc.

@henrikwidlund do you have other ideas?

[henrikwidlund]

I've also noticed that with the screensaver and account picking, quite annoying. To me the safest way would probably be to call something similar to _playpause_in_screensaver whenever a button is pressed. Maybe it would be enough to send a back command if the screensaver is active. I don't know what kind of overhead that would add though.

[zehnm]

This is getting interesting: found some old comments in the code that the screensaver state is no longer accessible with tvOS >= 18.4.
Testing with 26.1 while the screensaver is active, the _atv.apps.main_instance.api.fetch_attention_state() call returns SystemStatus.Awake and not Screensaver :-(

[henrikwidlund]

Damn, that sucks :( I wonder if they just don't send that info, or if the library used by this integration needs updating.

Edit: Nothing in the postlund repo about it, but it also hasn't been very active since 2024.