I see there is some mixup here. Are the new methods supposed to handle only custom shortcuts, or all shortcuts? If the former, the built-in actions in get_shortcut() should be ignored and has_shortcut() should only do shortcuts.has(p_path).

Are the new methods supposed to handle only custom shortcuts, or all shortcuts?

Personally, I'm in favor of allowing the user to do whatever they want, in the same vein of them being able to remove any UI element in the editor. An example may be someone creates a custom GridMap plugin that is supposed to basically replace the built-in one. Let's say the plugin doesn't have a Next Floor shortcut for whatever reason (they came up an alternative way to handle this), this gives them the ability to remove it to avoid confusion.

Feedback addressed.

Is this planned for 4.6?

Thanks!