fix(core): add max chord length and duplicate keybinding warnings

[RtlZeroMemory]

Summary

• Reject key chord sequences longer than 8 keys (prevents unbounded trie memory and impossible-to-press combos)
• Warn in dev mode when keybinding sequences are registered multiple times and silently replaced

Files changed

• keybindings/parser.ts — MAX_CHORD_LENGTH = 8 enforcement
• keybindings/manager.ts — duplicate sequence warning in mergeBindingsReplacingSequences()

[coderabbitai]

Warning

Rate limit exceeded

@RtlZeroMemory has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 4 minutes and 15 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between ff26087 and a6c2e55.

📒 Files selected for processing (2)

• packages/core/src/keybindings/manager.ts
• packages/core/src/keybindings/parser.ts

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch guardrails/keybinding-limits

---

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