Unable to trigger shell reload using Ctrl+Alt-R combination on MacOS

[euphemism]

Describe the bug

Any keypress ordering of the combination just ends up triggering reverse-i-search via Ctrl+R.

To reproduce

1. Start a devenv shell
2. Make change to devenv configuration
3. Wait for evaluation to complete
4. Attempt to trigger reload

Version

devenv 2.0.3+2105b1a (aarch64-darwin)

[sandydoo]

On my mac, a lot of these defaults are taken by Rectangle (window resizing). We need better defaults and a way to configure them.

[euphemism]

For me, Rectangle doesn't seem to have claimed Ctrl+Alt+R, but I'm not sure what the solution here is.

[sandydoo]

Ah, interesting. ctrl+alt+D is the default for first third for me. That's used by devenv to pause/unpause watching.

[davidkern]

Also an issue for KDE Plasma.

TIL how to record a screencast! :) KDE spectacle uses Ctrl-Alt-R to start recording.

[domenkozar]

What's the best way to fix this?

[euphemism]

I'm unsure about the best way to fix this. It seems there are a few ways to mitigate it though:

• Configuration option to define custom key combination
• Configuration option to enable auto-reload on completion of evaluation

For me, I wouldn't mind having the latter. I got very used to direnv automatically reloading my shell, and it would make it so that I don't have to worry about the key combo not working.

[euphemism]

For what it's worth, Ctrl + Shift + R does not get preempted by reverse-i-search.

[davidkern]

I also agree on the configuration approach - it seems like regardless on what key combination is used it is going to conflict for someone.

But also for our use, being able to auto-reload on evaluation completion would also be welcome. We also use direnv and have a pretool hook for claude using devenv. Both of these broke with 2.0 and we had to revert back to 1.x since both use cases are expecting the shell to be updated with the most recent evaluation. In both cases these are infrastructure repos for us where nix is the primary content, so the auto-reload functionality was being used much like a frontend developer relies on auto-reload of their browser while developing.

[domenkozar]

I'm going to change the default to just auto-reload, let's see how that goes.