Prevent useKeyboardShortcuts from re-registering the global keydown listener every render

[RUKAYAT-CODER]

Overview
In src/hooks/useKeyboardShortcuts.ts, commandMap (lines 175-177) depends on the commands prop; when callers pass an inline commands array the map and handler change identity every render, so the document keydown listener is removed and re-added on every render. Accept a stable commands contract or key the map on command ids so the listener is attached once.

[DevALVIN-24]

@DevALVIN-24 has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Hi, I'd like to work on this issue! because I am very conversant with solving problems like this, and i can submit the pr with 48hr

ℹ️ Repo Maintainers: To accept this application, review their application or assign @DevALVIN-24 to this issue.

[Jerry-Tekh]

@Jerry-Tekh has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Hi , maintainer . Permit me work on this please. I always rate your repo with a fivestars. Issue i solves are correctly merged. i am ready to start this work. And i will fix it correctly . thank you.

ℹ️ Repo Maintainers: To accept this application, review their application or assign @Jerry-Tekh to this issue.

[israeloluwasegun293-stars]

@israeloluwasegun293-stars has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Please, kindly assign me this, maintainer.

ℹ️ Repo Maintainers: To accept this application, review their application or assign @israeloluwasegun293-stars to this issue.

[pharwasz]

@pharwasz has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Hi, I would like to work on this issue

ℹ️ Repo Maintainers: To accept this application, review their application or assign @pharwasz to this issue.

[Olamidepy]

@Olamidepy has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Hi Maintainer,

I would love to be assigned to this issue.

I’m a Full-Stack Developer and I also actively explore blockchain development and smart contracts, which has strengthened my problem-solving skills and attention to code quality.

After reviewing the issue, I believe I can contribute effectively and deliver a well-tested solution within the expected standards of the project.

ETA: 4 hours

Thank you for your consideration.

ℹ️ Repo Maintainers: To accept this application, review their application or assign @Olamidepy to this issue.

[Nacho1499]

@Nacho1499 has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Hello Maintainer,

I’m Nacho, a full-stack developer, and I’d love to contribute to the project. I’m particularly interested in working on issue #897, Prevent useKeyboardShortcuts from re-registering the global keydown listener every render.

I’ve reviewed the issue, and my approach would be:

1. Review src/hooks/useKeyboardShortcuts.ts to understand how the commands prop, commandMap, handler, and global keydown listener currently interact.

2. Identify the unstable dependency chain where an inline commands array causes commandMap and the keyboard handler to receive new references on every render.

3. Implement a stable commands strategy, either by establishing a stable commands contract for callers or by deriving the command map based on stable command IDs rather than the array/object reference.

4. Ensure the keyboard handler remains stable when the actual shortcut definitions have not changed, preventing unnecessary listener cleanup and re-registration.

5. Preserve dynamic command behavior, ensuring that when command IDs, shortcuts, or relevant command properties genuinely change, the keyboard shortcut handler still reflects those updates correctly.

6. Add or update tests covering initial listener registration, re-renders with equivalent inline commands, actual command changes, and listener cleanup on unmount.

7. Run the existing test and lint suites to verify the optimization does not introduce stale closures, broken shortcuts, or dependency issues.

I’ll keep the implementation focused and backward-compatible while ensuring the global keydown listener is not unnecessarily removed and re-added on every render.

If the issue is still available, I’d really appreciate the opportunity to be assigned to it. Thank you for your time, and I look forward to contributing to the project!

ℹ️ Repo Maintainers: To accept this application, review their application or assign @Nacho1499 to this issue.

[abdulqudus-dev]

@abdulqudus-dev has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

please ill like to work on this issue, assign it to me

ℹ️ Repo Maintainers: To accept this application, review their application or assign @abdulqudus-dev to this issue.

[drips-wave]

Congratulations, @Nacho1499! 🎉 Your application was accepted by the repo's maintainers, and the issue is due on August 31, 2026.

🧑‍💻 @Nacho1499: Please resolve the issue such that the repo's maintainers have enough time to review your contribution before the due date. You'll earn Points for completing the issue on-time, which will make you eligible for a share of the Stellar Wave Program's reward pool.

Warning

When opening a PR, please link it to this issue to ensure it gets tracked accurately. Points are awarded when this issue is marked as completed by the maintainer.

🤠 Repo maintainers: Please keep an eye on the contributor's progress and review their work before the due date. You can manage this issue, including adjusting its complexity and points, here.

🌊 Happy Wave 🌊

[grantfox-oss]

🎉 This issue has been marked as completed on GrantFox as part of the Third Campaign campaign!

@Nacho1499's PR #1241 was approved and merged by @RUKAYAT-CODER.

🏆 @Nacho1499: You earned 35 FoxPoints for this contribution! Your current tier: Explorer (637 total points). Track your full progress on GrantFox.

👏 Great work, @Nacho1499! Keep contributing to rinafcode.

[drips-wave]

This issue has been completed by @Nacho1499 as part of the Stellar Wave Program's 8th Wave 🥳

😎 @Nacho1499: You earned 200 Points for completing this issue! After the current Wave ends, you'll be eligible for a percentage of the Wave's reward pool based on the percentage of total points you've earned. Learn more here. You can also Leave a review to share your experience working on this issue.

🧑‍💻 Repo maintainers: How'd the contributor do? Leave a review to share your experience working with them.