feat: optimize create and update record operations

Author: ivnvxd

CHANGELOG.md

@@ -5,6 +5,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.2.2] - 2025-08-04
+
+### Changed
+- **Minimal Response Fields**: Reduced `create_record` and `update_record` tool responses to return only essential fields (id, name, display_name) to minimize LLM context usage
+
 ## [0.2.1] - 2025-06-28
 
 ### Changed

mcp_server_odoo/tools.py

@@ -771,14 +771,21 @@ async def _handle_create_record_tool(
                 # Create the record
                 record_id = self.connection.create(model, values)
 
-                # Read the created record to return full details
-                records = self.connection.read(model, [record_id])
+                # Return only essential fields to minimize context usage
+                # Users can use get_record if they need more fields
+                essential_fields = ["id", "name", "display_name"]
+
+                # Read only the essential fields
+                records = self.connection.read(model, [record_id], essential_fields)
                 if not records:
                     raise ToolError(f"Failed to read created record: {model} with ID {record_id}")
 
+                # Process dates in the minimal record
+                record = self._process_record_dates(records[0], model)
+
                 return {
                     "success": True,
-                    "record": records[0],
+                    "record": record,
                     "message": f"Successfully created {model} record with ID {record_id}",
                 }
 
@@ -814,22 +821,29 @@ async def _handle_update_record_tool(
                 if not values:
                     raise ValidationError("No values provided for record update")
 
-                # Check if record exists
-                existing = self.connection.read(model, [record_id])
+                # Check if record exists (only fetch ID to verify existence)
+                existing = self.connection.read(model, [record_id], ["id"])
                 if not existing:
                     raise NotFoundError(f"Record not found: {model} with ID {record_id}")
 
                 # Update the record
                 success = self.connection.write(model, [record_id], values)
 
-                # Read the updated record to return full details
-                records = self.connection.read(model, [record_id])
+                # Return only essential fields to minimize context usage
+                # Users can use get_record if they need more fields
+                essential_fields = ["id", "name", "display_name"]
+
+                # Read only the essential fields
+                records = self.connection.read(model, [record_id], essential_fields)
                 if not records:
                     raise ToolError(f"Failed to read updated record: {model} with ID {record_id}")
 
+                # Process dates in the minimal record
+                record = self._process_record_dates(records[0], model)
+
                 return {
                     "success": success,
-                    "record": records[0],
+                    "record": record,
                     "message": f"Successfully updated {model} record with ID {record_id}",
                 }
 

tests/test_write_tools.py

@@ -1,6 +1,6 @@
 """Tests for write operation tools."""
 
-from unittest.mock import Mock
+from unittest.mock import Mock, call
 
 import pytest
 
@@ -53,20 +53,26 @@ async def test_create_record_success(self, tool_handler, mock_connection):
         model = "res.partner"
         values = {"name": "Test Partner", "email": "[email protected]"}
         created_id = 123
-        created_record = {"id": created_id, "name": "Test Partner", "email": "[email protected]"}
+        essential_record = {
+            "id": created_id,
+            "name": "Test Partner",
+            "display_name": "Test Partner",
+        }
 
         mock_connection.create.return_value = created_id
-        mock_connection.read.return_value = [created_record]
+        mock_connection.read.return_value = [essential_record]
 
         # Execute
         result = await tool_handler._handle_create_record_tool(model, values)
 
         # Verify
         assert result["success"] is True
-        assert result["record"] == created_record
+        assert result["record"] == essential_record
         assert "Successfully created" in result["message"]
         mock_connection.create.assert_called_once_with(model, values)
-        mock_connection.read.assert_called_once_with(model, [created_id])
+        mock_connection.read.assert_called_once_with(
+            model, [created_id], ["id", "name", "display_name"]
+        )
 
     @pytest.mark.asyncio
     async def test_create_record_no_values(self, tool_handler):
@@ -91,8 +97,10 @@ async def test_update_record_success(self, tool_handler, mock_connection):
         model = "res.partner"
         record_id = 123
         values = {"email": "[email protected]"}
-        existing_record = {"id": record_id, "name": "Test Partner", "email": "[email protected]"}
-        updated_record = {"id": record_id, "name": "Test Partner", "email": "[email protected]"}
+        # First read call (existence check) returns just ID
+        existing_record = {"id": record_id}
+        # Second read call returns essential fields
+        updated_record = {"id": record_id, "name": "Test Partner", "display_name": "Test Partner"}
 
         mock_connection.read.side_effect = [[existing_record], [updated_record]]
         mock_connection.write.return_value = True
@@ -105,6 +113,12 @@ async def test_update_record_success(self, tool_handler, mock_connection):
         assert result["record"] == updated_record
         assert "Successfully updated" in result["message"]
         mock_connection.write.assert_called_once_with(model, [record_id], values)
+        # Verify both read calls with correct parameters
+        expected_calls = [
+            call(model, [record_id], ["id"]),  # Existence check
+            call(model, [record_id], ["id", "name", "display_name"]),  # Essential fields
+        ]
+        mock_connection.read.assert_has_calls(expected_calls)
 
     @pytest.mark.asyncio
     async def test_update_record_not_found(self, tool_handler, mock_connection):