Update tools.py

tools.py

@@ -67,7 +67,6 @@ def search_featured_articles_by_date_range(start_date: str, end_date: str) -> li
         return sorted(set(result_titles))
     except Exception as e:
         return [f"Error: {str(e)}"]
-
 @tool
 def detect_non_commutative_subset(table_text: str) -> str:
     """
@@ -82,24 +81,23 @@ def detect_non_commutative_subset(table_text: str) -> str:
     """
     print("🛠️ detect_non_commutative_subset called")
     try:
-        lines = [line.strip() for line in table_text.strip().splitlines() if "|" in line]
+        lines = [line for line in table_text.strip().splitlines() if "|" in line]
         if len(lines) < 3:
             return "Error: Table format invalid or incomplete."
 
-        # Parse headers (set elements)
-        headers = [h.strip() for h in lines[0].split('|') if h.strip()]
+        # Parse headers
+        headers = [h.strip() for h in lines[0].split('|')[1:-1]]
 
         operation = {}
-        for line in lines[2:]:  # skip header and separator
-            cells = [c.strip() for c in line.split('|') if c.strip()]
-            if len(cells) != len(headers) + 1:
-                continue  # skip malformed rows
-            row_label = cells[0]
-            for col_idx, result in enumerate(cells[1:]):
-                col_label = headers[col_idx]
+        for line in lines[2:]:
+            parts = [c.strip() for c in line.split('|')[1:-1]]  # keep strict column alignment
+            if len(parts) != len(headers) + 1:
+                continue
+            row_label = parts[0]
+            for i, result in enumerate(parts[1:]):
+                col_label = headers[i]
                 operation[(row_label, col_label)] = result
 
-        # Find mismatches: x*y ≠ y*x
         involved = set()
         for a in headers:
             for b in headers:
@@ -107,7 +105,6 @@ def detect_non_commutative_subset(table_text: str) -> str:
                     involved.update([a, b])
 
         return ", ".join(sorted(involved)) if involved else "The operation is commutative."
-
     except Exception as e:
         return f"Error analyzing table: {str(e)}"