macos: apply key-remap to menu key equivalents

[knu]

Refs #12293
Related: #3493 (duplicates: #4374, #5091), #7013
Supersedes: #12597

Apply key-remap before keybind lookup and when reporting keybind triggers for AppKit menu key equivalents.

Previously, menu shortcuts were registered from the configured trigger without considering modifier remaps. This let AppKit consume the physical shortcut before Ghostty could match the remapped keybind.

Config.keyEventIsBinding now remaps event modifiers before lookup, so remapped key equivalents match the same keybinds as normal key events.
CApi.config_trigger_ returns the physical trigger that maps back to the configured logical modifiers, and leaves triggers unset when no remapped physical shortcut can produce them.

RemapSet.unapply performs the reverse lookup and is checked with apply so ambiguous or impossible remaps do not register incorrect menu shortcuts.

Inspired by: @bo2themax (#12597 (review))

AI usage: OpenAI Codex helped investigate, implement, test, and refine this change. I reviewed and tested the resulting code.

[bo2themax]

Not exactly what I thought. I think you can check mappings and remap if needed in Config.keyEventIsBinding and CApi.config_trigger_ , it should also make the menu shortcut correct again, and you don't need to change the apprt anymore.

[knu]

@bo2themax Thanks for guiding me in this direction!

I updated the implementation to apply key-remap in Config.keyEventIsBinding and to expose remapped physical menu key equivalents through CApi.config_trigger_, without changing the AppKit/apprt event flow.

I also tested this manually on macOS with:

• macos-option-as-alt = left
• key-remap = left_alt=command
• key-remap = command=left_alt

With that config, the menu item for New Window displays ⌥N, and pressing ⌥N triggers New Window, while ⌘N is sent to the terminal as ESC N.

[knu]

@bo2themax How about this?

[bo2themax]

@bo2themax How about this?

It looks good to me in general, but we should wait for terminal maintainers to review this when they're available, since zig isn't really my profession here.

Be patient plz, you can use your local build for now with -Doptimize=ReleaseFast

[knu]

@bo2themax So, can I assume that the issue you marked as “Changes requested” has been resolved? If so, I’d like to wait for other reviewers to review this as is.