Move UART device selection into uart_io.__connect()

[Copilot]

Device selection logic was duplicated across main.py, test_ultrasonic.py, and uart_raw.py. Moved it into uart_io.__connect() where it belongs.

Changes

• uart_io.connect(): Made port parameter optional (defaults to None)

  • Auto-selects first USB device (/dev/ttyUSB* or /dev/ttyACM*) when port=None
  • Falls back to any available UART device
  • Raises RuntimeError if no devices found

• Callers: Simplified to uart_dev.connect(baud_rate, timeout)

Before:

uart_dev = uart_io()
uart_devices = uart_dev.list_ports()
usb_devices = [d for d in uart_devices if 'USB' in d.device or 'ACM' in d.device]
selected = usb_devices[0] if usb_devices else uart_devices[0]
uart_dev.connect(selected.device, consts.UART_BAUD_RATE, consts.UART_TIMEOUT)

After:

uart_dev = uart_io()
uart_dev.connect(consts.UART_BAUD_RATE, consts.UART_TIMEOUT)

Port can still be specified explicitly: uart_dev.connect(baud_rate, timeout, port='/dev/ttyUSB0')

Original prompt

I think the uart io is currently choosing a device on main.py, but can you make it choose on the uart_io's __connect function?

Created from Copilot CLI via the copilot delegate command.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Copilot]

Pull request overview

This pull request refactors UART device selection by centralizing the auto-selection logic into the uart_io.__connect() method, eliminating code duplication across multiple entry points. The change makes the API cleaner by making the port parameter optional and handling device selection internally.

Changes:

• Moved USB device auto-selection logic from callers into uart_io.__connect()
• Made port parameter optional (defaults to None) with auto-selection when not specified
• Updated parameter order to (baud_rate, timeout, port=None) for better ergonomics
• Simplified all three callers (main.py, test_ultrasonic.py, uart_raw.py)

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
modules/robot.py	Added auto-selection logic in __connect() method; changed connect() signature to make port optional with default None
main.py	Removed duplicate device selection code; simplified to call connect() with only baud_rate and timeout
test_ultrasonic.py	Removed duplicate device selection code and unused sys import; simplified connection call
uart_raw.py	Removed duplicate device selection code and unused sys import; simplified connection call

[rotarymars]

Squash merged manually from local