Remove unused timeout parameter from django_shell (#11)

Author: joshuadavidthomas

CHANGELOG.md

@@ -18,11 +18,15 @@ and this project attempts to adhere to [Semantic Versioning](https://semver.org/
 
 ## [Unreleased]
 
+### Removed
+
+- Removed unused `timeout` parameter from django_shell tool, to prevent potential robot confusion.
+
 ## [0.3.0]
 
 ### Changed
 
-- Removed custom formatting for QuerySets and iterables in shell output. QuerySets now display as `<QuerySet [...]>` and lists show their standard `repr()` instead of truncated displays with "... and X more items". This makes output consistent with standard Django/Python shell behavior and should hopefully not confused the robots.
+- Removed custom formatting for QuerySets and iterables in shell output. QuerySets now display as `<QuerySet [...]>` and lists show their standard `repr()` instead of truncated displays with "... and X more items". This makes output consistent with standard Django/Python shell behavior and should hopefully not confuse the robots.
 
 ### Fixed
 

src/mcp_django_shell/server.py

@@ -17,7 +17,7 @@
 
 
 @mcp.tool
-async def django_shell(code: str, ctx: Context, timeout: int | None = None) -> str:
+async def django_shell(code: str, ctx: Context) -> str:
     """Execute Python code in a stateful Django shell session.
 
     Django is pre-configured and ready to use with your project. You can import and use any Django
@@ -42,7 +42,7 @@ async def django_shell(code: str, ctx: Context, timeout: int | None = None) -> s
     )
 
     try:
-        result = await shell.execute(code, timeout=timeout)
+        result = await shell.execute(code)
 
         logger.debug(
             "django_shell execution completed - request_id: %s, result type: %s",

src/mcp_django_shell/shell.py

@@ -43,7 +43,7 @@ def reset(self):
         self.globals = {}
         self.history = []
 
-    async def execute(self, code: str, timeout: int | None = None) -> Result:
+    async def execute(self, code: str) -> Result:
         """Execute Python code in the Django shell context (async wrapper).
 
         This async wrapper enables use from FastMCP and other async contexts.
@@ -55,9 +55,9 @@ async def execute(self, code: str, timeout: int | None = None) -> Result:
         errors.
         """
 
-        return await sync_to_async(self._execute)(code, timeout)
+        return await sync_to_async(self._execute)(code)
 
-    def _execute(self, code: str, timeout: int | None = None) -> Result:
+    def _execute(self, code: str) -> Result:
         """Execute Python code in the Django shell context (synchronous).
 
         Attempts to evaluate code as an expression first (returning a value),