[mryel00]

This moves get_avail_uvc out of init_camera_type.

[Copilot]

Pull request overview

This PR refactors UVC camera discovery by extracting the /dev/v4l enumeration helper (get_avail_uvc) out of init_camera_type, making the discovery logic reusable and easier to read.

Changes:

• Extracted get_avail_uvc from the local nested function into a dedicated UVC class method.
• Updated init_camera_type to call the extracted helper when building the available UVC camera map.

Suppressed comments (2)

crowsnest/camera/types/uvc.py:117

• Same as above: avoid hard-coding UVC for this helper call; use cls for consistency within a @classmethod.

                "by_id": avail_by_id.get(dev_path, None),
            }
            for dev_path, by_path in UVC.get_avail_uvc("/dev/v4l/by-path").items()
            if "usb" in by_path

crowsnest/camera/types/uvc.py:110

• init_camera_type is a @classmethod, but this call hard-codes UVC instead of using cls, which makes future renames/subclassing harder (and is inconsistent once get_avail_uvc is a static/class method).

    def init_camera_type(cls) -> Sequence[UVC]:
        avail_by_id = UVC.get_avail_uvc("/dev/v4l/by-id/")

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.