fix: remove unused imports by default (#39)

* fix: remove unused imports by default

* style: run updated pre-commit

Author: maciejmajek

.pre-commit-config.yaml

@@ -23,6 +23,12 @@ repos:
     hooks:
       - id: shellcheck
 
+  - repo: https://github.com/pycqa/autoflake
+    rev: v2.3.1
+    hooks:
+      - id: autoflake
+        args: ["--remove-all-unused-imports", "--in-place"]
+
   - repo: https://github.com/pycqa/isort
     rev: 5.13.2
     hooks:
@@ -33,8 +39,3 @@ repos:
     rev: 24.4.2
     hooks:
       - id: black
-
-  - repo: https://github.com/pycqa/autoflake
-    rev: v2.3.1
-    hooks:
-      - id: autoflake

src/rai/scenario_engine/__init__.py

@@ -1 +0,0 @@
-from .scenario_engine import *

src/rai/scenario_engine/tool_runner.py

@@ -1,13 +1,7 @@
 from typing import Any, Dict, List, Literal, Sequence
 
 from langchain.tools import BaseTool
-from langchain_core.messages import (
-    AIMessage,
-    BaseMessage,
-    HumanMessage,
-    ToolCall,
-    ToolMessage,
-)
+from langchain_core.messages import AIMessage, BaseMessage, ToolCall, ToolMessage
 
 from rai.scenario_engine.messages import ToolMultimodalMessage
 

src/rai/tools/ros/tools.py

@@ -38,8 +38,6 @@ def _run(self, x: float, y: float, z: float = 0.0, text: str = ""):
         global marker_it
         """Sets a waypoint on the map."""
         import rclpy
-        from builtin_interfaces.msg import Time
-        from geometry_msgs.msg import Point
         from rclpy.node import Node
         from visualization_msgs.msg import Marker