Skip to content

fix(ios-provider): filter non-iOS USB serials in iOS observer#406

Open
mdub1na wants to merge 3 commits into
VKCOM:developfrom
mdub1na:codex/ios-observer-filter-non-ios-usb-serials
Open

fix(ios-provider): filter non-iOS USB serials in iOS observer#406
mdub1na wants to merge 3 commits into
VKCOM:developfrom
mdub1na:codex/ios-observer-filter-non-ios-usb-serials

Conversation

@mdub1na
Copy link
Copy Markdown

@mdub1na mdub1na commented Apr 14, 2026

Summary

Filter non-iOS USB serials in IOSObserver before emitting iOS attach events.

Changes

  • Added serial validation for iOS UDID formats (24/40 hex, XXXXXXXX-XXXXXXXXXXXXXXXX, simulator UUID).
  • Applied validation in both initial USB scan and hotplug event handling in lib/units/ios-provider/IOSObserver.ts.

Validation

  • npm run lint (no errors, existing repo warnings)
  • npm run typecheck

Context

More details about the issue are available in the VK development support chat by keyword 7423J07.

DroBov1Tya and others added 3 commits January 12, 2026 16:09
Without these arguments, the `processor` and `groups-engine` services crash on startup, causing devices to be invisible in the interface.
fix: add missing ZeroMQ endpoints to docker-compose (macos)
@mdub1na mdub1na requested a review from a team as a code owner April 14, 2026 18:52
@mdub1na mdub1na changed the base branch from master to develop April 17, 2026 06:45
return
}

if (event.eventType === 'Connected' && event.device?.vendorId === 1452) {
Copy link
Copy Markdown
Collaborator

@e-khalilov e-khalilov Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently, devices are filtered by device.vendorId. Have you tried adding filtering by device.deviceClass? The mouse/keyboard must be marked as Human Interface Device, so regular expressions won't be necessary.

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.

4 participants