mirror of
https://github.com/agessaman/meshcore-bot.git
synced 2026-04-27 11:25:27 +00:00
Stacy Olivas
242e0a5df1
PR #152 introduced trailing whitespace in several command files, set message.content_lower at runtime without declaring it on the MeshMessage dataclass, and left unused/unsorted imports in test and service files. All cause ruff/mypy CI failures on branches that rebase onto dev after that merge. Linting fixes (ruff --fix): - modules/command_manager.py: restore PUBLIC_CHANNEL_KEY_HEX re-export with noqa guard (core.py imports it from here; auto-fix silently dropped it) - modules/commands/multitest_command.py: strip W291/W293 trailing whitespace - modules/commands/path_command.py: strip W293 trailing whitespace - modules/commands/prefix_command.py: strip W291 trailing whitespace - modules/commands/roll_command.py: strip W293 trailing whitespace - modules/commands/sports_command.py: strip W293 trailing whitespace - modules/core.py: strip W293 blank-line whitespace - modules/service_plugins/packet_capture_service.py: sort imports (I001) - modules/version_info.py: remove unused typing.Any import (F401) - tests/integration/test_flood_scope_reply.py: remove unused call import - tests/unit/test_log_data_scope_fields.py: remove unused asyncio import - tests/unit/test_public_channel_guard.py: sort imports, remove unused patch and validate_config imports Dataclass fix (mypy attr-defined): - modules/models.py: declare content_lower field on MeshMessage so mypy resolves the attribute set by base_command.cleanup_message_for_matching Must be merged before or alongside PRs #155–#158 to clear CI on those branches.