From 6458385cb9d190afa668cd7d4d647ada94ac3c55 Mon Sep 17 00:00:00 2001
From: latinogino <154024559+latinogino@users.noreply.github.com>
Date: Tue, 23 Sep 2025 16:13:48 +0200
Subject: [PATCH] Fix product operations: Add required 'ref' field for Dolibarr
 products

---
 src/dolibarr_mcp/dolibarr_client.py | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/src/dolibarr_mcp/dolibarr_client.py b/src/dolibarr_mcp/dolibarr_client.py
index f631746..9eb8575 100644
--- a/src/dolibarr_mcp/dolibarr_client.py
+++ b/src/dolibarr_mcp/dolibarr_client.py
@@ -304,14 +304,32 @@ class DolibarrClient:
         self,
         label: str,
         price: float,
+        ref: Optional[str] = None,  # Product reference/SKU
         description: Optional[str] = None,
         stock: Optional[int] = None,
         **kwargs
     ) -> Dict[str, Any]:
-        """Create a new product."""
+        """Create a new product.
+        
+        Args:
+            label: Product name/label
+            price: Product price
+            ref: Product reference/SKU (required by Dolibarr, auto-generated if not provided)
+            description: Product description
+            stock: Initial stock quantity
+            **kwargs: Additional product fields
+        """
+        import time
+        
+        # Generate ref if not provided (required field in Dolibarr)
+        if ref is None:
+            ref = f"PROD-{int(time.time())}"
+        
         data = {
+            "ref": ref,  # Required field
             "label": label,
             "price": price,
+            "price_ttc": price,  # Price including tax (using same as price for simplicity)
             **kwargs
         }