fix(msg): add new `msg_show.shell_out` to default routes for messages

[dpetka2001]

Description

Add new shell_out for Neovim 0.11 onwards as kind for external commands messages

Related Issue(s)

Fixes #1097

Screenshots

[lljbash]

It seems that just adding shell_out is not enough.

[dpetka2001]

I added shell_err and shell_ret, that seem related to the aforementioned issue. Not sure about the rest. Folke can decide what he further wants to include or not.

[lljbash]

I guess this fixes #1096 as well.

[dpetka2001]

No, it doesn't from my testing. Something else must be amiss there, not sure what.

[lljbash]

BTW, do you think it is a mistake that :Noice filters out almost everything from msg_show by default? https://github.com/folke/noice.nvim/blob/0427460c2d7f673ad60eb02b35f5e9926cf67c59/lua/noice/config/init.lua#L60-L106

At least with :Noice telescope I would like to see everything and filter it manually. Maybe using the all filter config for telescope? https://github.com/folke/noice.nvim/blob/0427460c2d7f673ad60eb02b35f5e9926cf67c59/lua/telescope/_extensions/noice.lua#L36-L40

[dpetka2001]

Sorry, but you're asking the wrong person about this. I have limited programming experience and am not familiar with Noice codebase at all.

This PR was just a simple fix. You should direct your question towards Folke.

[dpetka2001]

I found something interesting regarding #1096. If you add print(#self._messages > 0) here before line 157 https://github.com/folke/noice.nvim/blob/0427460c2d7f673ad60eb02b35f5e9926cf67c59/lua/noice/view/init.lua#L156-L159, then this will be printed in the popup view.

It seems like the token from command lua require("noice").redirect("echo 3") is somehow being consumed?? No idea what might be causing this. Just mentioning this in case you might be able to figure out something that I fail to think of.

[github-actions[bot]]

This PR is stale because it has been open 30 days with no activity.

[Avsilver]

Helped with a lot of headache. Hopefully this gets merged.

[github-actions[bot]]

This PR is stale because it has been open 30 days with no activity.