gwolves: add the rest of the lineup from G-Wolves' own web-driver config - #42
Merged
Conversation
Fetched mouse.xyz's env-models.json directly (the same allowlist their official web driver uses) and cross-referenced it against the two models already verified here (HTX Ultra, 0x5618/0x3854). Every other entry shares the same vendor id and 'IsNewProtocol' flag, so the existing model-agnostic GWolvesHidClient almost certainly speaks the same wire protocol to them — but none of these 21 new models have been hardware-tested by this project, so they're all added as verified: false, matching this file's own stated convention for unverified additions. Also switches vendors.ts's G-Wolves WebHID filter from two hardcoded PIDs to a derived list over GWOLVES_PRODUCTS' keys, so adding a model here is now purely a products.ts change. Skipped 'TEST HTS Plus' (wired 0x5428, wireless 0x3854): its name marks it a dev/test entry, and its wireless id collides with the already-verified HTX Ultra. Full test suite passes (572/572). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fetched mouse.xyz's
env-models.jsondirectly (the same allowlist their official web driver uses) and cross-referenced it against the two models already verified here (HTX Ultra, 0x5618/0x3854). Every other entry shares the same vendor id andIsNewProtocolflag, so the existing model-agnosticGWolvesHidClientalmost certainly speaks the same wire protocol to them — but none of these 21 new models have been hardware-tested by this project, so they're all added asverified: false, matching this file's own stated convention for unverified additions.Also switches
vendors.ts's G-Wolves WebHID filter from two hardcoded PIDs to a derived list overGWOLVES_PRODUCTS' keys, so adding a model going forward is purely aproducts.tschange.Skipped "TEST HTS Plus" (wired 0x5428, wireless 0x3854): its name marks it a dev/test entry, and its wireless id collides with the already-verified HTX Ultra.
Full test suite passes (572/572).