Skip to content

traits: open the advanced cards for the MCHOSE A7 V3 - #267

Merged
snekxs merged 1 commit into
OpenMouse-Project:mainfrom
jamezrin:mchose-v3-writes-traits
Sep 14, 2026
Merged

snekxs merged 1 commit into
OpenMouse-Project:mainfrom
jamezrin:mchose-v3-writes-traits

Conversation

@jamezrin

Copy link
Copy Markdown
Contributor

One line, plus the comment it replaces.

Depends on OpenMouse-Project/mouse-protocol#101. Until that lands the V3 driver has no setters, and these flags would open cards with nothing behind them — which is exactly what the comment being removed here said.

The V3 driver now writes its settings block, so mchose-v3 takes the same flags as the mchose entry above it, for the same reason: it is not a direct-mode (CompX) driver and publishes its own sleep and debounce option lists.

Worth knowing before merging: the protocol PR's writes are built to the shape MCHOSE's own software uses and the framing under them is hardware-proven, but no write has been sent to a real V3 yet. The driver's status note says so where a user will see it. If you would rather wait for a tester, hold this one with it.

npm test (151), tsc and vite build pass against a local build of the protocol branch.

🤖 Generated with Claude Code

The V3 driver now writes its settings block, so the sleep, debounce and
processing cards have something behind them. It takes the same flags as the A7
V2 entry above it, and for the same reason: it is not a direct-mode (CompX)
driver and publishes its own sleep and debounce option lists.

This replaces the comment that explained why the entry was deliberately absent
while the driver could only read.

Needs the A7 V3 writes in @openmouse/protocol first.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@snekxs
snekxs merged commit 05f3401 into OpenMouse-Project:main Sep 14, 2026
3 checks passed
snekxs added a commit that referenced this pull request Sep 14, 2026
Pulls in the mouse-protocol PRs just merged to main: attackshark X11
polling/DPI/battery, MCHOSE A7 V3 settings writes + product-string
naming (the driver PR #267's traits work depends on), Windows
read-restriction fixes, and the README export list. Raises the JS
bundle budget to 1,570 kB to match.

Verified: npm run build, npm test, and check-bundle-size all pass.

Claude-Session: https://claude.ai/code/session_01QTpkq2i9qmNz8RN1MwXoNc
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