ppfeiffer/meshcore-bot
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

cleanup: Inline-Imports nach Modulebene verschieben (issue #6, #7) #11

Merged
ppfeiffer merged 1 commit from cleanup/remove-inline-imports into main 2026-03-13 11:05:26 +01:00
Conversation 0 Commits 1 Files changed 2 +1 -5
ppfeiffer commented 2026-03-13 10:30:11 +01:00
Owner
Copy link

Problem

Zwei Module enthielten überflüssige import-Aufrufe innerhalb von Funktionen, obwohl die Module bereits auf Modulebene importiert waren.

#6 – import copy in message_handler.py

copy wird auf Modulebene (Zeile 11) importiert, aber in drei Funktionen nochmals:

Zeile Funktion
107 handle_contact_message()
619 handle_raw_data()
1448 handle_channel_message()

#7 – import inspect in command_manager.py

inspect wurde zweimal innerhalb der inneren Schleife von execute_commands() importiert – einer Methode, die bei jeder eingehenden Nachricht durchlaufen wird.

Fix

message_handler.py: Die drei redundanten import copy-Zeilen entfernt.

command_manager.py: import inspect auf Modulebene verschoben (Zeile 9), die beiden Inline-Importe entfernt.

Tests

426 Tests grün (2 vorab bereits fehlgeschlagene test_config_validation-Tests sind unabhängig).

Schließt #6, schließt #7

## Problem Zwei Module enthielten überflüssige `import`-Aufrufe innerhalb von Funktionen, obwohl die Module bereits auf Modulebene importiert waren. ### #6 – `import copy` in `message_handler.py` `copy` wird auf Modulebene (Zeile 11) importiert, aber in drei Funktionen nochmals: | Zeile | Funktion | |-------|----------| | 107 | `handle_contact_message()` | | 619 | `handle_raw_data()` | | 1448 | `handle_channel_message()` | ### #7 – `import inspect` in `command_manager.py` `inspect` wurde zweimal innerhalb der inneren Schleife von `execute_commands()` importiert – einer Methode, die bei **jeder eingehenden Nachricht** durchlaufen wird. ## Fix **`message_handler.py`:** Die drei redundanten `import copy`-Zeilen entfernt. **`command_manager.py`:** `import inspect` auf Modulebene verschoben (Zeile 9), die beiden Inline-Importe entfernt. ## Tests 426 Tests grün (2 vorab bereits fehlgeschlagene `test_config_validation`-Tests sind unabhängig). Schließt #6, schließt #7
cleanup: Inline-Imports nach Modulebene verschieben (#6, #7)
Some checks failed
Build and Push Docker Image / build (pull_request) Failing after 40s
Details
Tests / test (pull_request) Failing after 22s
Details
62fda5aff7 
message_handler.py: `import copy` wurde in drei Funktionen
(handle_contact_message, handle_raw_data, handle_channel_message)
redundant importiert, obwohl es bereits auf Modulebene (Zeile 11)
vorhanden ist. Die drei überflüssigen Zeilen werden entfernt.

command_manager.py: `import inspect` wurde zweimal innerhalb der
inneren Schleife von execute_commands aufgerufen, die bei jeder
eingehenden Nachricht durchlaufen wird. `inspect` wird nun auf
Modulebene importiert; die beiden Inline-Importe entfallen.

Schließt #6, schließt #7
ppfeiffer force-pushed cleanup/remove-inline-imports from 62fda5aff7
Some checks failed
Build and Push Docker Image / build (pull_request) Failing after 40s
Details
Tests / test (pull_request) Failing after 22s
Details
to 48bf5759bf
Some checks failed
Build and Push Docker Image / build (pull_request) Failing after 39s
Details
Tests / test (pull_request) Failing after 22s
Details
2026-03-13 11:04:34 +01:00 Compare
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ppfeiffer/meshcore-bot!11
No description provided.