fix: Resolve all ruff lint errors and add CONFIG_SCHEMA

[markus-lassfolk]

Summary

Fix all 27 ruff lint errors and add missing CONFIG_SCHEMA for Home Assistant 2024.x compatibility.

Changes

__init__.py

• Add CONFIG_SCHEMA = cv.empty_config_schema(DOMAIN) — prevents HA from logging warnings about missing config schema
• Add import homeassistant.helpers.config_validation as cv
• Add noqa: E402 for intentional late imports (after safety-net _lib_init_error_reporting call)

discovery.py

• Add from typing import Any
• Add type annotations to on_connect() and on_message() callbacks (ANN001/ANN202)

coordinator.py

• Wrap long comment and expression to stay within 100-char limit (E501)

tests/test_init.py

• Add -> None return type annotations to test functions (ANN201)
• Replace assert False with raise AssertionError() (B011)
• Auto-fix import sorting (I001)

Result

ruff check . → All checks passed! (was 27 errors)

---

Note

Low Risk
Low risk: primarily lint/type/formatting changes plus a minimal CONFIG_SCHEMA addition and a safer (try/except) guard around disabling python-yarbo’s error reporting.

Overview
Adds CONFIG_SCHEMA = cv.empty_config_schema(DOMAIN) to the integration to satisfy HA’s expected config schema and updates __init__.py to disable python-yarbo’s library-level Sentry init via a try/except ImportError guard (with late imports annotated using # noqa: E402).

Cleans up remaining ruff violations across the integration (line wrapping, callback type annotations, minor simplifications) and updates tests to stub yarbo.error_reporting, mark DHCP discovery tests with @pytest.mark.allow_net_connect, and patch async_discover_endpoints to keep config-flow tests deterministic.

^{Written by Cursor Bugbot for commit 5e582e2. This will update automatically on new commits. Configure here.}

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[Copilot]

Pull request overview

This PR fixes all 27 ruff lint errors across the Yarbo Home Assistant integration codebase and adds the CONFIG_SCHEMA constant required for HA 2024.x compatibility.

Changes:

• Adds CONFIG_SCHEMA = cv.empty_config_schema(DOMAIN) and corresponding cv import to __init__.py, along with noqa: E402 annotations on intentionally late imports
• Adds type annotations (Any and -> None) to paho-mqtt callback functions in discovery.py and removes unnecessary noqa directives for rule sets (ARG, BLE) that aren't enabled
• Wraps long lines in coordinator.py and fixes test function signatures, import ordering, and assert False → raise AssertionError(...) in tests/test_init.py

Reviewed changes

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

File	Description
custom_components/yarbo/__init__.py	Added CONFIG_SCHEMA for HA compatibility, cv import, and noqa: E402 on late imports
custom_components/yarbo/discovery.py	Added Any type annotations to MQTT callbacks, removed unnecessary noqa directives
custom_components/yarbo/coordinator.py	Wrapped long comment and expression to stay within 100-char line limit
tests/test_init.py	Added -> None return types, fixed import sorting, replaced assert False with raise AssertionError(...)

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix prepared a fix for the issue found in the latest run.

• ✅ Fixed: Import changed from library to local, breaking safety net
  • Restored the import from yarbo.error_reporting (library) instead of .error_reporting (local module) so the safety net properly disables library-level Sentry auto-init.