Follow-up from the PR #70 review (Minor #5).
_scan_cameras discards the distinction between probe verdicts: None (could not probe: not in the video group, non-Linux, odd udev permissions) and False (probed fine, not a color-capture node). When no node probes True, the wizard silently lists every entry, including metadata nodes, and the "only N by-id entries for M detected cameras" hint computes M from raw entry counts, overcounting in fallback mode.
Suggested improvement: thread the per-entry verdicts up to _camera_section so the wizard can print a one-line yellow hint such as "couldn't probe /dev/v4l nodes (not in the video group?); listing everything", and base the count hint on probed cameras when verdicts are available. Also consider listing None nodes alongside True nodes when both occur in one directory (currently a per-node permission oddity hides the unprobeable node).
Low priority: all paths have a manual-entry escape hatch and the fallback errs toward showing more, not less.
🤖 Generated with Claude Code
Follow-up from the PR #70 review (Minor #5).
_scan_camerasdiscards the distinction between probe verdicts:None(could not probe: not in thevideogroup, non-Linux, odd udev permissions) andFalse(probed fine, not a color-capture node). When no node probesTrue, the wizard silently lists every entry, including metadata nodes, and the "only N by-id entries for M detected cameras" hint computes M from raw entry counts, overcounting in fallback mode.Suggested improvement: thread the per-entry verdicts up to
_camera_sectionso the wizard can print a one-line yellow hint such as "couldn't probe /dev/v4l nodes (not in thevideogroup?); listing everything", and base the count hint on probed cameras when verdicts are available. Also consider listingNonenodes alongsideTruenodes when both occur in one directory (currently a per-node permission oddity hides the unprobeable node).Low priority: all paths have a manual-entry escape hatch and the fallback errs toward showing more, not less.
🤖 Generated with Claude Code