Pre-commit updates

Author: Your Name

benchmark/benchmark.py

@@ -292,9 +292,7 @@ def simple_namespace_to_dict(obj):
 
     if not dry and "CECLI_DOCKER" not in os.environ:
         logger.warning("Warning: Benchmarking runs unvetted code. Run in a docker container.")
-        logger.warning(
-            "Set CECLI_DOCKER in the environment to by-pass this check at your own risk."
-        )
+        logger.warning("Set CECLI_DOCKER in the environment to bypass this check at your own risk.")
         return
 
     # Check dirs exist

cecli/coders/base_coder.py

@@ -1131,8 +1131,9 @@ def _include_in_map(abs_path):
                     "other_files": other_files,
                     "mentioned_fnames": mentioned_fnames,
                     "all_abs_files": all_abs_files,
-                    "read_only_count": len(set(self.abs_read_only_fnames)) + len(
-                        set(self.abs_read_only_stubs_fnames)
+                    "read_only_count": (
+                        len(set(self.abs_read_only_fnames))
+                        + len(set(self.abs_read_only_stubs_fnames))
                     ),
                 }
             )

cecli/onboarding.py

@@ -165,7 +165,7 @@ def start_openrouter_oauth_flow(io):
 
     class OAuthCallbackHandler(http.server.SimpleHTTPRequestHandler):
         def do_GET(self):
-            nonlocal auth_code, server_error
+            nonlocal auth_code, server_error  # noqa
             parsed_path = urlparse(self.path)
             if parsed_path.path == "/callback/cecli":
                 query_params = parse_qs(parsed_path.query)
@@ -193,7 +193,7 @@ def log_message(self, format, *args):
             pass
 
     def run_server():
-        nonlocal server_error
+        nonlocal server_error  # noqa
         try:
             with socketserver.TCPServer(("localhost", port), OAuthCallbackHandler) as httpd:
                 io.tool_output(f"Temporary server listening on {callback_url}", log_only=True)
@@ -203,7 +203,7 @@ def run_server():
                     time.sleep(0.1)
                 io.tool_output("Shutting down temporary server.", log_only=True)
         except Exception as e:
-            server_error = f"Failed to start or run temporary server: {e}"
+            server_error = f"Failed to start or run temporary server: {e}"  # noqa
             server_started.set()
             shutdown_server.set()
 

cecli/report.py

@@ -78,22 +78,22 @@ def format_args_for_reporting(args):
 
 
 def get_args_error_data():
-    global resolved_args_data
+    global resolved_args_data  # noqa
     return resolved_args_data
 
 
 def set_args_error_data(args):
-    global resolved_args_data
+    global resolved_args_data  # noqa
     resolved_args_data = args
 
 
 def get_error_prefix():
-    global error_prefix
+    global error_prefix  # noqa
     return error_prefix
 
 
 def update_error_prefix(prefix):
-    global error_prefix
+    global error_prefix  # noqa
     error_prefix.append(f"{prefix}\n")
     error_prefix = error_prefix[-10:]
 

cecli/sessions.py

@@ -71,9 +71,10 @@ def list_sessions(self) -> List[Dict]:
                     "file": session_file,
                     "model": session_data.get("model", "unknown"),
                     "edit_format": session_data.get("edit_format", "unknown"),
-                    "num_messages": len(
-                        session_data.get("chat_history", {}).get("done_messages", [])
-                    ) + len(session_data.get("chat_history", {}).get("cur_messages", [])),
+                    "num_messages": (
+                        len(session_data.get("chat_history", {}).get("done_messages", []))
+                        + len(session_data.get("chat_history", {}).get("cur_messages", []))
+                    ),
                     "num_files": (
                         len(session_data.get("files", {}).get("editable", []))
                         + len(session_data.get("files", {}).get("read_only", []))
@@ -149,11 +150,11 @@ def _build_session_data(self, session_name) -> Dict:
             "editor_edit_format": self.coder.main_model.editor_edit_format,
             "edit_format": self.coder.edit_format,
             "chat_history": {
-                "done_messages": ConversationService.get_manager(self.coder).get_messages_dict(
-                    MessageTag.DONE
+                "done_messages": (
+                    ConversationService.get_manager(self.coder).get_messages_dict(MessageTag.DONE)
                 ),
-                "cur_messages": ConversationService.get_manager(self.coder).get_messages_dict(
-                    MessageTag.CUR
+                "cur_messages": (
+                    ConversationService.get_manager(self.coder).get_messages_dict(MessageTag.CUR)
                 ),
             },
             "files": {

scripts/30k-image.py

@@ -135,14 +135,18 @@ def generate_celebration_svg(output_path=None, width=DEFAULT_WIDTH, height=DEFAU
 
     # Font embedding
     font_data = embed_font()
-    font_face = f"""
+    font_face = (
+        f"""
     @font-face {{
         font-family: 'GlassTTYVT220';
         src: url(data:font/truetype;charset=utf-8;base64,{font_data}) format('truetype');
         font-weight: normal;
         font-style: normal;
     }}
-    """ if font_data else ""
+    """
+        if font_data
+        else ""
+    )
 
     # Generate confetti elements
     confetti = generate_confetti(count=150, width=width, height=height)

tests/basic/test_sessions.py

@@ -106,8 +106,10 @@ async def test_cmd_load_session_basic(self):
                     "read_only_stubs": [],
                 },
                 "settings": {"auto_commits": True, "auto_lint": False, "auto_test": False},
-                "todo_list": """Restored tasks
-- item""",
+                "todo_list": (
+                    """Restored tasks
+- item"""
+                ),
             }
             session_file = Path(handle_core_files(".cecli")) / "sessions" / "test_session.json"
             session_file.parent.mkdir(parents=True, exist_ok=True)

tests/hooks/test_base.py

@@ -1,6 +1,5 @@
 """Tests for base hook classes."""
 
-
 import pytest
 
 from cecli.hooks import BaseHook, CommandHook, HookType