Skip to content

fix(keychron): give the Launcher 8k and 1k families their sleep and debounce cards - #514

Merged
snekxs merged 1 commit into
OpenMouse-Project:mainfrom
ydw1904:fix/keychron-launcher-traits
Oct 3, 2026
Merged

snekxs merged 1 commit into
OpenMouse-Project:mainfrom
ydw1904:fix/keychron-launcher-traits

Conversation

@ydw1904

@ydw1904 ydw1904 commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Protocol 0.22.0 reports ui.family as keychron-8k (the M6 and most Keychron mice, Keychron8kHidClient) and keychron-1k (Keychron1kHidClient), but BY_FAMILY in traits.ts still only knew the old keychron-m6 family, which no driver emits any more. On main right now:

  • the 8k mice (M6 and the rest) lose the advanced section, so no sleep or debounce card
  • the 1k mice lose the advanced section, so no debounce card

This is the traits.ts half of #426, which was closed as superseded by #440. #440 carried the controller wiring and the bump but not this hunk.

  • keychron-8k replaces the dead keychron-m6 entry: advanced section, sleep, debounce
  • keychron-1k is new: advanced section and debounce only, since its protocol has no sleep command

Tested

  • new test in availability.test.ts covering both families; it fails on main and passes with this change
  • tsc --noEmit clean and the full suite passes (255 tests) against @openmouse/protocol 0.22.0

Not tested on a real mouse.

…ebounce cards

Protocol 0.22.0 reports ui.family "keychron-8k" (the M6 and most Keychron
mice) and "keychron-1k", but BY_FAMILY still only knew the old
"keychron-m6" family. Those mice lost their advanced section, and the 8k
mice lost sleep and debounce. This is the traits half of the closed #426
that #440 did not carry over.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@snekxs
snekxs merged commit 364ced8 into OpenMouse-Project:main Oct 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants