@votadev/tooncode 2.2.9 → 2.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@votadev/tooncode",
-  "version": "2.2.9",
+  "version": "2.3.1",
   "description": "🇹🇭 Thai Coding Agent CLI — Claude Code alternative powered by free models. 20 tools, multi-agent team, browser automation, MCP servers.",
   "author": "VotaLab",
   "license": "MIT",

package/tooncode.py

@@ -8,7 +8,7 @@ Usage:
     python tooncode.py
 """
 
-VERSION = "2.2.9"
+VERSION = "2.3.1"
 
 import httpx
 import json
@@ -2917,27 +2917,53 @@ def _get_current_api_url() -> str:
 
 def _validate_messages(messages: list):
     """Fix message ordering issues before sending to API."""
-    # Rule: tool_result must follow assistant with tool_use
-    # Rule: alternating user/assistant (with tool_result counting as user)
+    # Rules:
+    # 1. Must alternate user/assistant
+    # 2. tool_result (in user msg) must come right after assistant with tool_use
+    # 3. First message must be user
     fixed = []
-    for i, msg in enumerate(messages):
+    for msg in messages:
         role = msg.get("role", "")
-        # Skip empty messages
         content = msg.get("content", [])
         if not content:
             continue
-        # Prevent consecutive same-role messages (except tool_result after user)
-        if fixed:
-            prev_role = fixed[-1].get("role", "")
-            if role == prev_role and role == "user":
-                # Check if this is tool_result (allowed after tool_use response)
-                is_tool_result = isinstance(content, list) and any(
-                    b.get("type") == "tool_result" for b in content if isinstance(b, dict))
-                if not is_tool_result:
-                    fixed.append({"role": "assistant", "content": [{"type": "text", "text": "(continuing)"}]})
-            elif role == prev_role and role == "assistant":
+
+        # Detect message types
+        is_tool_result = isinstance(content, list) and any(
+            isinstance(b, dict) and b.get("type") == "tool_result" for b in content)
+        has_tool_use = isinstance(content, list) and any(
+            isinstance(b, dict) and b.get("type") == "tool_use" for b in content)
+
+        if not fixed:
+            # First message must be user
+            if role != "user":
+                fixed.append({"role": "user", "content": [{"type": "text", "text": "(start)"}]})
+            fixed.append(msg)
+            continue
+
+        prev = fixed[-1]
+        prev_role = prev.get("role", "")
+        prev_content = prev.get("content", [])
+        prev_has_tool_use = isinstance(prev_content, list) and any(
+            isinstance(b, dict) and b.get("type") == "tool_use" for b in prev_content)
+
+        # tool_result must follow assistant with tool_use
+        if is_tool_result and role == "user":
+            if prev_role != "assistant" or not prev_has_tool_use:
+                # Remove this tool_result — it's orphaned
+                continue
+            fixed.append(msg)
+            continue
+
+        # Fix consecutive same role
+        if role == prev_role:
+            if role == "user":
+                fixed.append({"role": "assistant", "content": [{"type": "text", "text": "(continuing)"}]})
+            else:
                 fixed.append({"role": "user", "content": [{"type": "text", "text": "(continue)"}]})
+
         fixed.append(msg)
+
     messages.clear()
     messages.extend(fixed)
 
@@ -5359,51 +5385,6 @@ def main(_initial_prompt=None):
                     console.print()
                     tool_results = execute_tools(content)
 
-                    # -- Stuck detection: if same error repeats, auto-call bosshelp --
-                    for tr in tool_results:
-                        tr_content = tr.get("content", "")
-                        if isinstance(tr_content, str) and ("[error" in tr_content or "exit code" in tr_content):
-                            _last_errors.append(tr_content[:200])
-                            # If 3+ errors in recent tools, we're stuck
-                            if len(_last_errors) >= 3:
-                                recent = _last_errors[-3:]
-                                # Check if errors are similar (stuck in loop)
-                                if len(set(recent)) <= 2:
-                                    console.print("\n[bold magenta][auto-bosshelp] Stuck detected! Asking Claude Code for help...[/bold magenta]")
-                                    # Gather context
-                                    recent_text = "\n".join(
-                                        block.get("text", "")
-                                        for msg in messages[-4:]
-                                        if msg.get("role") == "assistant"
-                                        for block in (msg.get("content") if isinstance(msg.get("content"), list) else [])
-                                        if block.get("type") == "text"
-                                    )
-                                    boss_answer = _boss_help(
-                                        problem=f"I keep getting the same error and can't fix it",
-                                        context=recent_text[:1000],
-                                        error="\n".join(recent),
-                                    )
-                                    _last_errors.clear()
-                                    _post_bosshelp = True
-                                    # MUST append tool_results first (API requires it after tool_use)
-                                    messages.append({"role": "user", "content": tool_results})
-                                    # Then add bosshelp as assistant+user pair
-                                    fix_instructions = f"[BOSSHELP]\n{boss_answer}\n\n"
-                                    fix_instructions += "APPLY THIS FIX NOW:\n"
-                                    fix_instructions += "1. Read each file mentioned\n"
-                                    fix_instructions += "2. Use edit tool to apply changes\n"
-                                    fix_instructions += "3. Run commands if needed\n"
-                                    fix_instructions += "4. Verify the fix works\n"
-                                    fix_instructions += "USE TOOLS. Do NOT explain."
-                                    messages.append({"role": "assistant", "content": [{"type": "text", "text": "Let me apply the fix."}]})
-                                    messages.append({"role": "user", "content": [{"type": "text", "text":
-                                        fix_instructions,
-                                        "cache_control": {"type": "ephemeral"}}]})
-                                    continue
-                        else:
-                            # Successful tool = reset error tracking
-                            _last_errors.clear()
-
                     messages.append({"role": "user", "content": tool_results})
                     continue  # Always continue after tool use