style(soul,tests): address CodeRabbit type/hygiene nits

- slash.py: annotate the new accept_edits handler with -> None (public src rule).
- test_pythinkersoul_steer.py: annotate the finish_reason test-double property.
- test_agent_tool.py: replace the hardcoded /tmp agent_file with the in-file
  runtime.subagent_store.root pattern, and split the combined assertion for
  clearer failure messages.

Author: elkaix

src/pythinker_code/soul/slash.py

@@ -207,7 +207,7 @@ async def auto(soul: PythinkerSoul, args: str):
 
 
 @registry.command(name="accept-edits")
-async def accept_edits(soul: PythinkerSoul, args: str):
+async def accept_edits(soul: PythinkerSoul, args: str) -> None:
     """Toggle accept-edits mode (auto-approve reversible in-workspace file edits)"""
     approval = soul.runtime.approval
     if approval.is_accept_edits():

tests/core/test_pythinkersoul_steer.py

@@ -452,7 +452,7 @@ def usage(self):
         return None
 
     @property
-    def finish_reason(self):
+    def finish_reason(self) -> str | None:
         return None
 
 

tests/tools/test_agent_tool.py

@@ -2,7 +2,6 @@
 
 import asyncio
 import re
-from pathlib import Path
 from types import SimpleNamespace
 
 import pytest
@@ -72,11 +71,12 @@ def test_check_required_mcp_servers_gates_absent_server(runtime: Runtime) -> Non
     from pythinker_code.subagents import AgentTypeDefinition, ToolPolicy
     from pythinker_code.tools.agent import AgentTool
 
+    assert runtime.subagent_store is not None  # populated by the runtime fixture
     runtime.labor_market.add_builtin_type(
         AgentTypeDefinition(
             name="needs_db",
             description="needs the db MCP server",
-            agent_file=Path("/tmp/needs_db.yaml"),
+            agent_file=runtime.subagent_store.root / "needs_db.yaml",
             tool_policy=ToolPolicy(mode="inherit"),
             required_mcp_servers=("db",),
         )
@@ -85,7 +85,8 @@ def test_check_required_mcp_servers_gates_absent_server(runtime: Runtime) -> Non
 
     runtime.mcp_status = lambda: None  # no MCP configured -> required server absent
     err = tool.check_required_mcp_servers("needs_db")
-    assert err is not None and "db" in err.message
+    assert err is not None
+    assert "db" in err.message
 
     runtime.mcp_status = lambda: _mcp_snapshot(True, [])  # still loading -> allow
     assert tool.check_required_mcp_servers("needs_db") is None