fix(defineShortcuts): add alt key guard

[KazariEX]

🔗 Linked issue

fix #5982

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

After this fix, the situation in #5982 still require the layoutIndependent option to be enabled.

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[coderabbitai]

📝 Walkthrough

Walkthrough

The change adds a modifier key validation check in the shortcut matching logic of defineShortcuts. Specifically, it introduces an additional condition requiring e.altKey to match shortcut.altKey, alongside existing checks for metaKey and ctrlKey. This ensures that shortcuts defined with the Alt modifier (e.g., alt_g) only trigger when the Alt key state matches the configured requirement, preventing unintended activation when the Alt key is not pressed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding an alt key guard to the defineShortcuts function to fix the bug where alt shortcuts were firing without the Alt key.
Linked Issues check	✅ Passed	The PR directly addresses issue #5982 by adding an altKey guard in defineShortcuts.ts, ensuring alt_* shortcuts only activate when the Alt modifier is actually pressed.
Out of Scope Changes check	✅ Passed	The three-line change is entirely scoped to fixing the alt key guard issue in defineShortcuts, with no unrelated modifications present.
Description check	✅ Passed	The PR description clearly references the linked issue #5982 and explains the bug fix context regarding the alt key modifier behavior.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[pkg-pr-new]

npm i https://pkg.pr.new/@nuxt/ui@6020

commit: 9613fde