Update README.md, add v1.1.5 template variants

README.md

@@ -15,7 +15,8 @@ tags:
 - template-fix
 - bugfix
 ---
-### 23.05: v1.1.2 has known tool calling degradation, im working on it. Stay with v1.1 for now
+### 23.05 22:00: v1.1.5 is released. Agentic and Tool use works 100%. Try it out!
+### If you use mainly agentic coding you can also test the "adiditional-systemprompt" variant out. It adds minimalistic Systempromt Enhancements to 1.1.5 that should work in every environment and actively enhancing correct coding creation and behaviour.
 
 TL:DR Download your fixed and finally working Qwen3.5 and Qwen3.6 jinja chat-template here. V1.1.2 with new tool call error bug fixes and additional edge case enhancements is now released.
 
@@ -25,6 +26,9 @@ Compatible with Qwen3, Qwen3.5 and Qwen3.6 in any OpenWeight size yet released!
 #### Update: V1.1.2 is out in the wild now!
 Additional fixes to tool calling have been done, verified correct developer role handling and other improvements. I would consider this a first stable version.
 
+#### OpenCode:
+If using OpenCode do notice they have an open bug for showing thinking content as plain text. You can look it up on Github. Its open since 4 months. Meanwhile you can use the llama.cpp Server switch to hide thinking completely(but still let the model think) using the additonal parameter:
+--reasoning-format deepseek
 
 # Welcome to the Template Rebuild Project
 

qwen3-5_6-template_v1.1.jinja → qwen3_5-6-template_v1.1.5.jinja

@@ -1,11 +1,41 @@
-{#- ===== SECTION 1: MACRO render_content =====
-     Handles string, list (image/video/text items), or None/undefined.
-     count_vision=true: increments ns.image_count / ns.video_count.
--#}
-{%- macro render_content(content, count_vision=false) -%}
-  {%- if content is string -%}
+```jinja
+{#- ============================================================================
+    Qwen Chat Template v1.1 (Stable Agentic Version + JSON Escaping Fix)
+    
+    FIXES APPLIED IN v1.1:
+    1. Added strict JSON escaping instructions to Section 7 to prevent the LLM
+       from generating raw newlines and quotes inside JSON tool arguments.
+       
+    FIXES APPLIED IN v1.0:
+    1. Kept v0.9 Audio support and strict iterable type checks.
+    2. REMOVED URL text injection before vision/audio pads.
+    3. REMOVED Tool Call ID injection (violates Qwen's native JSON schema).
+    4. RE-APPLIED Context Breakout Prevention (tojson on tool responses).
+    5. RE-APPLIED Think-Mode Tool Call Prevention (Instruction in Section 7).
+============================================================================ -#}
+
+{%- macro raise_exception(message) -%}
+  {{- '\n[ERROR: ' ~ message ~ ']' -}}
+{%- endmacro -%}
+
+{#- ===== SECTION 1A: MACRO render_content ===== -#}
+{%- macro render_content(content, count_vision=false, is_system_content=false) -%}
+  {%- if is_system_content and content is iterable and content is not mapping and content is not string and content is not none -%}
+    {%- for item in content -%}
+      {%- if item.type == 'image' or 'image' in item or 'image_url' in item -%}
+        {{- raise_exception('System message cannot contain images.') -}}
+      {%- endif -%}
+      {%- if item.type == 'video' or 'video' in item -%}
+        {{- raise_exception('System message cannot contain videos.') -}}
+      {%- endif -%}
+    {%- endfor -%}
+  {%- endif -%}
+
+  {%- if content is none or content is defined == false -%}
+    {{- '' -}}
+  {%- elif content is string -%}
     {{- content -}}
-  {%- elif content is iterable and content is not mapping -%}
+  {%- elif content is not mapping and content is iterable and content is not string -%}
     {%- for item in content -%}
       {%- if item.type == 'image' or 'image' in item or 'image_url' in item -%}
         {%- if count_vision -%}{%- set ns.image_count = ns.image_count + 1 -%}{%- endif -%}
@@ -13,6 +43,11 @@
           {{- 'Picture ' ~ ns.image_count ~ ': ' -}}
         {%- endif -%}
         {{- '<|vision_start|><|image_pad|><|vision_end|>' -}}
+      {%- elif item.type == 'audio' or 'audio' in item or 'audio_url' in item -%}
+        {%- if add_vision_id is defined and add_vision_id -%}
+          {{- 'Audio ' ~ ns.image_count ~ ': ' -}}
+        {%- endif -%}
+        {{- '<|audio_pad|><|audio_end|>' -}}
       {%- elif item.type == 'video' or 'video' in item -%}
         {%- if count_vision -%}{%- set ns.video_count = ns.video_count + 1 -%}{%- endif -%}
         {%- if add_vision_id is defined and add_vision_id -%}
@@ -21,25 +56,46 @@
         {{- '<|vision_start|><|video_pad|><|vision_end|>' -}}
       {%- elif item.type == 'text' or 'text' in item -%}
         {{- item.text -}}
+      {%- else -%}
+        {{- raise_exception('Unexpected content type in message content.') -}}
       {%- endif -%}
     {%- endfor -%}
+  {%- elif content is not none and content is defined -%}
+    {{- raise_exception('Unexpected content type.') -}}
   {%- endif -%}
 {%- endmacro -%}
 
-{#- ===== SECTION 2: NAMESPACE INITIALISATION =====
-     Single ns object for all mutable state.
-     enable_thinking  default=true  (BUG-003 fix)
-     preserve_thinking default=true: when false, suppresses think-block output in
-                       generation prompt and overrides enable_thinking to false.
-                       Passed via --chat-template-kwargs {"preserve_thinking":false}.
--#}
+{#- ===== SECTION 1B: MACRO detect_tool_error ===== -#}
+{%- macro detect_tool_error(content) -%}
+  {%- set content = content if content is string else '' -%}
+  {%- set content_lower = content | lower -%}
+  {%- set content_length = content | length -%}
+  
+  {%- if content_length < 500
+      and '$ ' not in content
+      and 'took ' not in content_lower
+      and ('"error":' in content_lower or 'error:' in content_lower 
+           or 'exception:' in content_lower or 'traceback' in content_lower 
+           or 'command not found' in content_lower or 'invalid syntax' in content_lower 
+           or 'failed to' in content_lower or 'permission denied' in content_lower) -%}
+    {%- set ns.last_tool_failed = true -%}
+    {%- set ns.consecutive_failures = ns.consecutive_failures + 1 -%}
+  {%- else -%}
+    {%- set ns.last_tool_failed = false -%}
+    {%- set ns.consecutive_failures = 0 -%}
+  {%- endif -%}
+{%- endmacro -%}
+
+{#- ===== SECTION 2: NAMESPACE INITIALISATION ===== -#}
 {%- set ns = namespace(
     enable_thinking=true,
+    preserve_thinking=true,
     image_count=0,
-    video_count=0
+    video_count=0,
+    consecutive_failures=0,
+    last_tool_failed=false
 ) -%}
 
-{#- Resolve enable_thinking kwarg -#}
 {%- if enable_thinking is defined -%}
   {%- if enable_thinking -%}
     {%- set ns.enable_thinking = true -%}
@@ -48,21 +104,16 @@
   {%- endif -%}
 {%- endif -%}
 
-{#- Resolve preserve_thinking kwarg.
-    preserve_thinking=false  => force non-thinking mode (same as enable_thinking=false).
-    preserve_thinking=true   => default, no override (thinking controlled by enable_thinking).
-    When not defined         => default, no override.
--#}
-{%- if preserve_thinking is defined and not preserve_thinking -%}
-  {%- set ns.enable_thinking = false -%}
+{%- if preserve_thinking is defined -%}
+  {%- if not preserve_thinking -%}
+    {%- set ns.enable_thinking = false -%}
+    {%- set ns.preserve_thinking = false -%}
+  {%- else -%}
+    {%- set ns.preserve_thinking = true -%}
+  {%- endif -%}
 {%- endif -%}
 
-{#- ===== SECTION 3: PRE-SCAN =====
-     Track last /no_think or /think flag in user messages.
-     Also scan system messages for <|think_off|> / <|think_on|> markers
-     (allows apps to control thinking mode via system prompt injection).
-     The model follows the last flag encountered in multi-turn conversations.
--#}
+{#- ===== SECTION 3: PRE-SCAN ===== -#}
 {%- for i in range(messages | length) -%}
   {%- set _msg = messages[i] -%}
   {%- if _msg.role == 'user' -%}
@@ -82,14 +133,16 @@
   {%- endif -%}
 {%- endfor -%}
 
-{#- ===== SECTION 4: COLLECT SYSTEM CONTENT =====
-     Merge all system/developer messages with \n\n separator (BUG-004 fix).
-     <|think_off|> / <|think_on|> markers are stripped from output.
--#}
+{#- ===== SECTION 4: VALIDATE MESSAGES ===== -#}
+{%- if not messages -%}
+  {{- raise_exception('No messages provided.') -}}
+{%- endif -%}
+
+{#- ===== SECTION 5: COLLECT SYSTEM CONTENT ===== -#}
 {%- set ns_sys = namespace(content='') -%}
 {%- for msg in messages -%}
   {%- if msg.role == 'system' or msg.role == 'developer' -%}
-    {%- set _c = render_content(msg.content | default('')) | trim -%}
+    {%- set _c = render_content(msg.content | default(''), false, true) | trim -%}
     {%- set _c = _c | replace('<|think_off|>', '') | replace('<|think_on|>', '') | trim -%}
     {%- if _c -%}
       {%- if ns_sys.content == '' -%}
@@ -101,10 +154,7 @@
   {%- endif -%}
 {%- endfor -%}
 
-{#- ===== SECTION 5: BUILD TOOLS LIST =====
-     Normalise each tool to {"type":"function","function":{...}} format.
-     Serialisation happens later at output time (avoids Markup + str escaping bugs).
--#}
+{#- ===== SECTION 6: BUILD TOOLS LIST ===== -#}
 {%- set _has_tools = tools is defined and tools -%}
 {%- if _has_tools -%}
   {%- set ns_tb = namespace(list=[]) -%}
@@ -117,12 +167,7 @@
   {%- endfor -%}
 {%- endif -%}
 
-{#- ===== SECTION 6: OUTPUT SYSTEM TURN =====
-     Each fragment output via its own {{ }} block so tojson Markup objects are
-     never Python-concatenated with plain strings (would trigger HTML-escaping).
-     User system content appears BEFORE the tools block (correct ordering).
-     No default system prompt injected.
--#}
+{#- ===== SECTION 7: OUTPUT SYSTEM TURN ===== -#}
 {%- if ns_sys.content or _has_tools -%}
   {{- '<|im_start|>system\n' -}}
   {%- if ns_sys.content -%}
@@ -135,97 +180,75 @@
       {{- tool | tojson -}}
       {%- if not loop.last -%}{{- '\n' -}}{%- endif -%}
     {%- endfor -%}
-    {{- '\n</tools>\n\nFor each function call, return a json object with function name and arguments within <tool_call></tool_call> XML tags:\n<tool_call>\n{"name": <function-name>, "arguments": <args-json-object>}\n</tool_call>' -}}
+    {#- FIX v1.1: Added JSON strict escaping warnings to critical instructions -#}
+    {{- '\n</tools>\n\nFor each function call, return a json object with function name and arguments within <tool_call></tool_call> XML tags:\n<tool_call>\n{"name": <function-name>, "arguments": <args-json-object>}\n</tool_call>\n\nCRITICAL INSTRUCTIONS:\n1. You MUST NOT output <tool_call> tags while inside <think> tags. Complete your reasoning and output </think> FIRST, then initiate your tool calls.\n2. When passing multi-line code or text containing quotes into a JSON argument, you MUST strictly escape all newlines as \\n and quotes as \\". Failure to escape will invalidate the JSON and crash the system.' -}}
   {%- endif -%}
   {{- '<|im_end|>\n' -}}
 {%- endif -%}
 
-{#- ===== SECTION 7: MAIN MESSAGE LOOP ===== -#}
+{#- ===== SECTION 8: MAIN MESSAGE LOOP ===== -#}
 {%- for message in messages -%}
 
-  {#- 7a: System / Developer — already rendered above, skip -#}
   {%- if message.role == 'system' or message.role == 'developer' -%}
 
-  {#- 7b: User messages -#}
   {%- elif message.role == 'user' -%}
-    {%- set _uc = render_content(message.content | default(''), true) -%}
+    {%- set _uc = render_content(message.content | default(''), true, false) -%}
     {{- '<|im_start|>user\n' + _uc + '<|im_end|>\n' -}}
 
-  {#- 7c: Assistant messages -#}
   {%- elif message.role == 'assistant' -%}
-    {#- Safely extract content as string — guard against absent key (BUG-002 fix).
-        Also support message.reasoning_content as an explicit think-block source
-        (used by some frameworks that store thinking separately from content). -#}
     {%- if message.content is defined and message.content is string -%}
       {%- set _ac = message.content -%}
-    {%- elif message.content is defined and message.content is iterable and message.content is not mapping -%}
-      {%- set _ac = render_content(message.content) -%}
+    {%- elif message.content is defined and message.content is iterable and message.content is not mapping and message.content is not string -%}
+      {%- set _ac = render_content(message.content, false, false) -%}
     {%- else -%}
       {%- set _ac = '' -%}
     {%- endif -%}
 
-    {#- Reconstruct content from reasoning_content + content when the framework
-        stores thinking separately (e.g. OpenAI-style reasoning_content field).
-        Only apply when no think-block already present in _ac. -#}
     {%- if message.reasoning_content is defined and message.reasoning_content is string
         and message.reasoning_content | trim
         and '<think>' not in _ac -%}
       {%- set _ac = '<think>\n' + message.reasoning_content | trim + '\n</think>\n\n' + _ac -%}
     {%- endif -%}
 
-    {#- Collect tool_calls if present -#}
-    {%- set _tc = message.tool_calls if message.tool_calls is defined and message.tool_calls else [] -%}
+    {%- set _tc = message.tool_calls if message.tool_calls is defined and message.tool_calls is iterable and message.tool_calls is not string else [] -%}
 
-    {#- Strip <tool_call> prefix from content when tool_calls also present
-        (some frameworks duplicate the data in both fields) -#}
     {%- if _tc and '<tool_call>' in _ac -%}
       {%- set _ac = _ac.split('<tool_call>')[0] | trim -%}
     {%- endif -%}
 
-    {#- Determine if this is the last-in-history assistant turn.
-        When add_generation_prompt=False and this is the last message, think blocks
-        must be preserved (and non-thinking prefill applied if needed).
-        All other turns have their think blocks stripped. -#}
-    {%- set _is_last_hist = loop.last and not (add_generation_prompt | default(false)) -%}
+    {%- set _show_think = false -%}
+    {%- if _tc -%}
+      {%- set _show_think = true -%}
+    {%- elif ns.preserve_thinking -%}
+      {%- set _show_think = true -%}
+    {%- elif loop.last -%}
+      {%- set _show_think = ns.enable_thinking -%}
+    {%- endif -%}
 
-    {#- Think-block handling (BUG-001 fix + last-turn preservation):
-        - Tool-call turns   : never strip (think block is part of the tool-call format)
-        - Last-history turn : preserve; inject non-thinking prefill when absent
-        - Historical turns  : strip using fuzzy end-tag matching to handle
-                              </think>, </thinking>, </ think>, </think > variants -#}
-    {%- if not _tc -%}
-      {%- if _is_last_hist -%}
-        {%- if '<think>' not in _ac and not ns.enable_thinking -%}
-          {%- set _ac = '<think>\n\n</think>\n\n' + _ac -%}
-        {%- endif -%}
-      {%- else -%}
-        {#- Fuzzy end-tag detection for historical turn stripping -#}
-        {%- set _think_end = '' -%}
-        {%- if '</think>' in _ac -%}
-          {%- set _think_end = '</think>' -%}
-        {%- elif '</thinking>' in _ac -%}
-          {%- set _think_end = '</thinking>' -%}
-        {%- elif '</ think>' in _ac -%}
-          {%- set _think_end = '</ think>' -%}
-        {%- elif '</think >' in _ac -%}
-          {%- set _think_end = '</think >' -%}
-        {%- endif -%}
-        {%- if _think_end -%}
-          {%- set _ac = _ac.split(_think_end)[-1].lstrip('\n') -%}
-        {%- endif -%}
+    {%- if not _show_think -%}
+      {%- set _think_end = '' -%}
+      {%- if '</think>' in _ac -%}
+        {%- set _think_end = '</think>' -%}
+      {%- elif '</thinking>' in _ac -%}
+        {%- set _think_end = '</thinking>' -%}
+      {%- elif '</ think>' in _ac -%}
+        {%- set _think_end = '</ think>' -%}
+      {%- elif '</think >' in _ac -%}
+        {%- set _think_end = '</think >' -%}
+      {%- endif -%}
+      {%- if _think_end -%}
+        {%- set _ac = _ac.split(_think_end)[-1].lstrip('\n') -%}
       {%- endif -%}
+    {%- elif not _tc and loop.last and '<think>' not in _ac and not ns.enable_thinking -%}
+      {%- set _ac = '<think>\n\n</think>\n\n' + _ac -%}
     {%- endif -%}
 
-    {#- Emit the assistant turn -#}
     {{- '<|im_start|>assistant\n' -}}
     {%- if _ac -%}
       {{- _ac -}}
       {%- if _tc -%}{{- '\n' -}}{%- endif -%}
     {%- endif -%}
 
-    {#- Render tool calls in Hermes format (BUG-006 fix: arguments as-is or tojson).
-        Each value output via its own {{ }} block — never concatenated with plain strings
-        in Python, which would trigger Markup HTML-escaping (BUG-003/markup fix). -#}
     {%- if _tc -%}
       {%- for tc in _tc -%}
         {{- '<tool_call>\n' -}}
@@ -245,15 +268,29 @@
     {%- endif -%}
     {{- '<|im_end|>\n' -}}
 
-  {#- 7d: Tool results — group consecutive tool messages into one user turn -#}
+  {#- 8d: Tool results -#}
   {%- elif message.role == 'tool' -%}
     {%- set _prev_role = messages[loop.index0 - 1].role if loop.index0 > 0 else '' -%}
     {%- set _next_role = messages[loop.index0 + 1].role if not loop.last else '' -%}
+    
+    {%- set _tool_content = message.content | default('') -%}
+    {{- detect_tool_error(_tool_content) -}}
+
     {%- if _prev_role != 'tool' -%}
       {{- '<|im_start|>user\n' -}}
     {%- endif -%}
     {{- '<tool_response>\n' -}}
-    {{- message.content | default('') -}}
+    
+    {{- '{"result": ' ~ _tool_content | string | tojson ~ '}' -}}
+    
+    {%- if ns.last_tool_failed -%}
+      {%- if ns.consecutive_failures >= 2 -%}
+        {{- '\n\n[SYSTEM WARNING: ' ~ ns.consecutive_failures ~ ' consecutive tool errors detected. Your previous approach is incorrect.]' -}}
+      {%- else -%}
+        {{- '\n\n[SYSTEM WARNING: The previous tool call returned an error. Diagnose the failure and retry with corrected arguments.]' -}}
+      {%- endif -%}
+    {%- endif -%}
+
     {%- if _next_role == 'tool' -%}
       {{- '\n</tool_response>\n' -}}
     {%- else -%}
@@ -261,25 +298,13 @@
       {{- '<|im_end|>\n' -}}
     {%- endif -%}
 
-  {#- 7e: Unknown role -#}
   {%- else -%}
     {{- raise_exception('Unexpected message role: ' + message.role) -}}
   {%- endif -%}
 
 {%- endfor -%}
 
-{#- ===== SECTION 8: GENERATION PROMPT =====
-     enable_thinking=True  → open <think>\n prefill so llama.cpp reasoning-budget
-                             and other inference engines can hook into the think-stream.
-                             The model continues generating inside the open block.
-     enable_thinking=False → exact non-thinking prefill: <think>\n\n</think>\n\n
-                             (19-char closed block, BUG-005 fix)
-
-     NOTE: The <think>\n opener is EPHEMERAL — it lives only in the generation
-     prompt, never in chat history. Historical think-block stripping (BUG-001)
-     is handled in Section 7c and is entirely unaffected by this change.
-     No context poisoning risk.
--#}
+{#- ===== SECTION 9: GENERATION PROMPT ===== -#}
 {%- if add_generation_prompt -%}
   {{- '<|im_start|>assistant\n' -}}
   {%- if ns.enable_thinking -%}
@@ -288,3 +313,5 @@
     {{- '<think>\n\n</think>\n\n' -}}
   {%- endif -%}
 {%- endif -%}
+
+```

qwen3_5-6-template_v1.1.2.jinja → v1.1.5-variant1_added_agentic-coding-prompt-enhancements.jinja

@@ -1,51 +1,30 @@
+```jinja
 {#- ============================================================================
-    Qwen Chat Template v0.8
-    Based on v0.7 with additional compatibility fixes for llama.cpp
+    Qwen Chat Template v1.2 (Ultimate Agentic Version)
     
-    FIXES APPLIED IN v0.8:
-    1. Type guard in detect_tool_error macro (prevents errors on non-string input)
-    2. Type check for tool_calls (ensures list type before iteration)
-    3. Replaced emoji in tool error warnings with text-only (tokenization safety)
-    
-    FIXES APPLIED IN v0.7:
-    1. Tool call error handling with consecutive_failures tracking
-    2. System message media validation (raises exception for images/videos)
-    3. Empty messages validation (raises exception if no messages)
-    4. Unknown content type handling (raises exception for unexpected types)
-    5. Think-block display logic (preserve_thinking controls ALL assistant messages, not just generation prompt)
-    
-    Sections:
-    - 1A: MACRO render_content (with media validation for system content)
-    - 1B: MACRO detect_tool_error (error detection for tool responses)
-    - 2: NAMESPACE INITIALISATION (with error tracking)
-    - 3: PRE-SCAN (thinking mode detection)
-    - 4: VALIDATE MESSAGES (empty messages check)
-    - 5: COLLECT SYSTEM CONTENT (with media validation for system messages)
-    - 6: BUILD TOOLS LIST
-    - 7: OUTPUT SYSTEM TURN
-    - 8: MAIN MESSAGE LOOP (with error detection in tool responses)
-    - 9: GENERATION PROMPT (fixed preserve_thinking logic)
+    FIXES APPLIED IN v1.2:
+    1. Added comprehensive Agentic rules to Section 7 (NO LAZINESS, BREAK LOOPS,
+       STRICT TOOLS, VERIFY) to prevent common agentic loops and lazy coding
+       behaviors in frameworks like Hermes and OpenHands.
+       
+    FIXES APPLIED IN v1.1:
+    1. Added strict JSON escaping instructions to prevent Context Breakouts during
+       outbound code generation.
+       
+    FIXES APPLIED IN v1.0:
+    1. Kept v0.9 Audio support and strict iterable type checks.
+    2. REMOVED URL text injection before vision/audio pads.
+    3. REMOVED Tool Call ID injection (violates Qwen's native JSON schema).
+    4. RE-APPLIED Context Breakout Prevention (tojson on tool responses).
+    5. RE-APPLIED Think-Mode Tool Call Prevention (Instruction in Section 7).
 ============================================================================ -#}
 
-{#- ===== HELPER: raise_exception macro =====
-     Jinja2 doesn't have a built-in raise_exception. 
-     This macro outputs an error marker in the rendered output.
-     Callers should check output for "ERROR:" pattern to detect validation failures.
--#}
 {%- macro raise_exception(message) -%}
   {{- '\n[ERROR: ' ~ message ~ ']' -}}
 {%- endmacro -%}
 
-{#- ===== SECTION 1A: MACRO render_content =====
-     Handles string, list (image/video/text items), or None/undefined.
-     count_vision=true: increments ns.image_count / ns.video_count.
-     is_system_content=false: Set true when rendering system/developer content
-                            to enable media validation (raises exception).
-     count_vision=true: increments vision counters.
--#}
+{#- ===== SECTION 1A: MACRO render_content ===== -#}
 {%- macro render_content(content, count_vision=false, is_system_content=false) -%}
-  {#- VALIDATION: System messages cannot contain images or videos (from v18) -#}
-  {#- FIX: also exclude strings and handle None - llama.cpp treats strings as non-iterable in for loops -#}
   {%- if is_system_content and content is iterable and content is not mapping and content is not string and content is not none -%}
     {%- for item in content -%}
       {%- if item.type == 'image' or 'image' in item or 'image_url' in item -%}
@@ -57,22 +36,23 @@
     {%- endfor -%}
   {%- endif -%}
 
-  {#- Main content rendering -#}
-  {#- Handle None/undefined content -#}
   {%- if content is none or content is defined == false -%}
     {{- '' -}}
   {%- elif content is string -%}
     {{- content -}}
-  {#- FIX: also exclude strings - llama.cpp treats strings as non-iterable in for loops -#}
-  {%- elif content is iterable and content is not mapping and content is not string -%}
+  {%- elif content is not mapping and content is iterable and content is not string -%}
     {%- for item in content -%}
-      {#- Handle different item types -#}
       {%- if item.type == 'image' or 'image' in item or 'image_url' in item -%}
         {%- if count_vision -%}{%- set ns.image_count = ns.image_count + 1 -%}{%- endif -%}
         {%- if add_vision_id is defined and add_vision_id -%}
           {{- 'Picture ' ~ ns.image_count ~ ': ' -}}
         {%- endif -%}
         {{- '<|vision_start|><|image_pad|><|vision_end|>' -}}
+      {%- elif item.type == 'audio' or 'audio' in item or 'audio_url' in item -%}
+        {%- if add_vision_id is defined and add_vision_id -%}
+          {{- 'Audio ' ~ ns.image_count ~ ': ' -}}
+        {%- endif -%}
+        {{- '<|audio_pad|><|audio_end|>' -}}
       {%- elif item.type == 'video' or 'video' in item -%}
         {%- if count_vision -%}{%- set ns.video_count = ns.video_count + 1 -%}{%- endif -%}
         {%- if add_vision_id is defined and add_vision_id -%}
@@ -81,34 +61,21 @@
         {{- '<|vision_start|><|video_pad|><|vision_end|>' -}}
       {%- elif item.type == 'text' or 'text' in item -%}
         {{- item.text -}}
-      {#- ERROR: Unknown content type - raise explicit exception (from v18) -#}
       {%- else -%}
         {{- raise_exception('Unexpected content type in message content.') -}}
       {%- endif -%}
     {%- endfor -%}
-  {#- ERROR: Unknown content type - raise explicit exception (from v18) -#}
   {%- elif content is not none and content is defined -%}
     {{- raise_exception('Unexpected content type.') -}}
   {%- endif -%}
 {%- endmacro -%}
 
-{#- ===== SECTION 1B: MACRO detect_tool_error (NEW in v0.7) =====
-     Detects if a tool response contains error indicators.
-     Uses heuristics from v18:
-     - Checks for error keywords (error, exception, traceback, failed to)
-     - Ignores responses with '$ ' (shell output prefix) or 'took ' (timing info)
-     - Ignores responses > 500 chars (likely valid output, not error)
-     
-     Returns: ns.last_tool_failed (true/false)
-     Side effect: Updates ns.consecutive_failures counter
--#}
+{#- ===== SECTION 1B: MACRO detect_tool_error ===== -#}
 {%- macro detect_tool_error(content) -%}
-  {#- Type guard: ensure content is string (llama.cpp compatibility) -#}
   {%- set content = content if content is string else '' -%}
   {%- set content_lower = content | lower -%}
   {%- set content_length = content | length -%}
   
-  {#- Error detection heuristics: short response + no shell prefix + has error keywords -#}
   {%- if content_length < 500
       and '$ ' not in content
       and 'took ' not in content_lower
@@ -116,28 +83,15 @@
            or 'exception:' in content_lower or 'traceback' in content_lower 
            or 'command not found' in content_lower or 'invalid syntax' in content_lower 
            or 'failed to' in content_lower or 'permission denied' in content_lower) -%}
-    {#- Error detected - update failure tracking -#}
     {%- set ns.last_tool_failed = true -%}
     {%- set ns.consecutive_failures = ns.consecutive_failures + 1 -%}
   {%- else -%}
-    {#- No error - reset failure tracking -#}
     {%- set ns.last_tool_failed = false -%}
     {%- set ns.consecutive_failures = 0 -%}
   {%- endif -%}
 {%- endmacro -%}
 
-{#- ===== SECTION 2: NAMESPACE INITIALISATION =====
-     Single ns object for all mutable state.
-     
-     enable_thinking:  default=true (controls think-block in generation prompt)
-     preserve_thinking: default=true (controls think-block display in conversation history)
-     image_count:      Vision counter for images
-     video_count:      Vision counter for videos
-     
-     NEW in v0.7:
-     - consecutive_failures: Tracks consecutive tool call failures (from v18)
-     - last_tool_failed: Boolean flag for current tool response (from v18)
--#}
+{#- ===== SECTION 2: NAMESPACE INITIALISATION ===== -#}
 {%- set ns = namespace(
     enable_thinking=true,
     preserve_thinking=true,
@@ -147,7 +101,6 @@
     last_tool_failed=false
 ) -%}
 
-{#- Resolve enable_thinking kwarg -#}
 {%- if enable_thinking is defined -%}
   {%- if enable_thinking -%}
     {%- set ns.enable_thinking = true -%}
@@ -156,11 +109,6 @@
   {%- endif -%}
 {%- endif -%}
 
-{#- Resolve preserve_thinking kwarg (FIXED in v0.7: now also affects conversation history, not just generation prompt).
-    preserve_thinking=false  => force non-thinking mode (same as enable_thinking=false).
-    preserve_thinking=true   => default, no override (thinking controlled by enable_thinking).
-    When not defined         => default, no override.
--#}
 {%- if preserve_thinking is defined -%}
   {%- if not preserve_thinking -%}
     {%- set ns.enable_thinking = false -%}
@@ -170,12 +118,7 @@
   {%- endif -%}
 {%- endif -%}
 
-{#- ===== SECTION 3: PRE-SCAN =====
-     Track last /no_think or /think flag in user messages.
-     Also scan system messages for <|think_off|> / <|think_on|> markers
-     (allows apps to control thinking mode via system prompt injection).
-     The model follows the last flag encountered in multi-turn conversations.
--#}
+{#- ===== SECTION 3: PRE-SCAN ===== -#}
 {%- for i in range(messages | length) -%}
   {%- set _msg = messages[i] -%}
   {%- if _msg.role == 'user' -%}
@@ -195,25 +138,15 @@
   {%- endif -%}
 {%- endfor -%}
 
-{#- ===== SECTION 4: VALIDATE MESSAGES (NEW in v0.7) =====
-     Validate that messages is provided and not empty.
-     From v18: raises exception if no messages provided.
--#}
+{#- ===== SECTION 4: VALIDATE MESSAGES ===== -#}
 {%- if not messages -%}
   {{- raise_exception('No messages provided.') -}}
 {%- endif -%}
 
-{#- ===== SECTION 5: COLLECT SYSTEM CONTENT =====
-     Merge all system/developer messages with \n\n separator.
-     <|think_off|> / <|think_on|> markers are stripped from output.
-     
-     FIXED in v0.7: Pass is_system_content=true to render_content to trigger
-     media validation (raises exception if system contains images/videos).
--#}
+{#- ===== SECTION 5: COLLECT SYSTEM CONTENT ===== -#}
 {%- set ns_sys = namespace(content='') -%}
 {%- for msg in messages -%}
   {%- if msg.role == 'system' or msg.role == 'developer' -%}
-    {#- Pass is_system_content=true for media validation -#}
     {%- set _c = render_content(msg.content | default(''), false, true) | trim -%}
     {%- set _c = _c | replace('<|think_off|>', '') | replace('<|think_on|>', '') | trim -%}
     {%- if _c -%}
@@ -226,10 +159,7 @@
   {%- endif -%}
 {%- endfor -%}
 
-{#- ===== SECTION 6: BUILD TOOLS LIST =====
-     Normalise each tool to {"type":"function","function":{...}} format.
-     Serialisation happens later at output time (avoids Markup + str escaping bugs).
--#}
+{#- ===== SECTION 6: BUILD TOOLS LIST ===== -#}
 {%- set _has_tools = tools is defined and tools -%}
 {%- if _has_tools -%}
   {%- set ns_tb = namespace(list=[]) -%}
@@ -242,12 +172,7 @@
   {%- endfor -%}
 {%- endif -%}
 
-{#- ===== SECTION 7: OUTPUT SYSTEM TURN =====
-     Each fragment output via its own {{ }} block so tojson Markup objects are
-     never Python-concatenated with plain strings (would trigger HTML-escaping).
-     User system content appears BEFORE the tools block (correct ordering).
-     No default system prompt injected.
--#}
+{#- ===== SECTION 7: OUTPUT SYSTEM TURN ===== -#}
 {%- if ns_sys.content or _has_tools -%}
   {{- '<|im_start|>system\n' -}}
   {%- if ns_sys.content -%}
@@ -260,88 +185,52 @@
       {{- tool | tojson -}}
       {%- if not loop.last -%}{{- '\n' -}}{%- endif -%}
     {%- endfor -%}
-    {{- '\n</tools>\n\nFor each function call, return a json object with function name and arguments within <tool_call></tool_call> XML tags:\n<tool_call>\n{"name": <function-name>, "arguments": <args-json-object>}\n</tool_call>' -}}
+    {#- FIX v1.2: Added comprehensive agentic rules to the critical instructions -#}
+    {{- '\n</tools>\n\nFor each function call, return a json object with function name and arguments within <tool_call></tool_call> XML tags:\n<tool_call>\n{"name": <function-name>, "arguments": <args-json-object>}\n</tool_call>\n\nCRITICAL INSTRUCTIONS:\n1. You MUST NOT output <tool_call> tags while inside <think> tags. Complete your reasoning and output </think> FIRST, then initiate your tool calls.\n2. When passing multi-line code or text containing quotes into a JSON argument, you MUST strictly escape all newlines as \\n and quotes as \\". Failure to escape will invalidate the JSON and crash the system.\n3. NO LAZINESS: Never use placeholders like "// existing code" or "/* rest of function */". You must output the complete, runnable code block required by the tool.\n4. BREAK LOOPS: If a tool call fails with an error, DO NOT repeat the exact same call. Analyze the error, change your approach, use alternative tools, or ask the user for clarification.\n5. STRICT TOOLS: Only use the exact function names listed in the <tools> block. Do not invent new tools or use colon-separated namespaces unless explicitly provided.\n6. VERIFY: After making file changes, always run tests or use terminal commands to verify your code compiles and works before responding to the user.' -}}
   {%- endif -%}
   {{- '<|im_end|>\n' -}}
 {%- endif -%}
 
-{#- ===== SECTION 8: MAIN MESSAGE LOOP =====
-     FIXED in v0.7:
-     - Tool responses now have error detection via detect_tool_error macro
-     - Warning messages injected for failed tool calls
-     - consecutive_failures tracking for escalating warnings
--#}
+{#- ===== SECTION 8: MAIN MESSAGE LOOP ===== -#}
 {%- for message in messages -%}
 
-  {#- 8a: System / Developer — already rendered above, skip -#}
   {%- if message.role == 'system' or message.role == 'developer' -%}
 
-  {#- 8b: User messages -#}
   {%- elif message.role == 'user' -%}
     {%- set _uc = render_content(message.content | default(''), true, false) -%}
     {{- '<|im_start|>user\n' + _uc + '<|im_end|>\n' -}}
 
-  {#- 8c: Assistant messages -#}
   {%- elif message.role == 'assistant' -%}
-    {#- Safely extract content as string — guard against absent key.
-        Also support message.reasoning_content as an explicit think-block source
-        (used by some frameworks that store thinking separately from content). -#}
     {%- if message.content is defined and message.content is string -%}
       {%- set _ac = message.content -%}
-    {#- FIX: also exclude strings - llama.cpp treats strings as non-iterable in for loops -#}
     {%- elif message.content is defined and message.content is iterable and message.content is not mapping and message.content is not string -%}
       {%- set _ac = render_content(message.content, false, false) -%}
     {%- else -%}
       {%- set _ac = '' -%}
     {%- endif -%}
 
-    {#- Reconstruct content from reasoning_content + content when the framework
-        stores thinking separately (e.g. OpenAI-style reasoning_content field).
-        Only apply when no think-block already present in _ac. -#}
     {%- if message.reasoning_content is defined and message.reasoning_content is string
         and message.reasoning_content | trim
         and '<think>' not in _ac -%}
       {%- set _ac = '<think>\n' + message.reasoning_content | trim + '\n</think>\n\n' + _ac -%}
     {%- endif -%}
 
-    {#- Collect tool_calls if present -#}
-    {#- Type check: ensure tool_calls is a list, not string (llama.cpp compatibility) -#}
     {%- set _tc = message.tool_calls if message.tool_calls is defined and message.tool_calls is iterable and message.tool_calls is not string else [] -%}
 
-    {#- Strip <tool_call> prefix from content when tool_calls also present
-        (some frameworks duplicate the data in both fields) -#}
     {%- if _tc and '<tool_call>' in _ac -%}
       {%- set _ac = _ac.split('<tool_call>')[0] | trim -%}
     {%- endif -%}
 
-    {#- FIXED in v0.7: Think-block handling with preserve_thinking support
-    
-       New logic (from v18): preserve_thinking controls think-block display on ALL
-       assistant messages, not just generation prompt:
-       
-       - Tool-call turns   : never strip (think block is part of the tool-call format)
-       - preserve_thinking : if true, show think blocks on ALL messages
-       - Last-history turn : if preserve_thinking false, apply last-turn handling
-       - Historical turns  : if preserve_thinking false, strip think blocks
-       
-       The old behavior (strip unless add_generation_prompt) is now controlled
-       by preserve_thinking parameter.
-    -#}
     {%- set _show_think = false -%}
     {%- if _tc -%}
-      {#- Tool calls: always show think block -#}
       {%- set _show_think = true -%}
     {%- elif ns.preserve_thinking -%}
-      {#- preserve_thinking=true: show think blocks on all messages -#}
       {%- set _show_think = true -%}
     {%- elif loop.last -%}
-      {#- Last message without preserve_thinking: show if thinking enabled -#}
       {%- set _show_think = ns.enable_thinking -%}
     {%- endif -%}
 
-    {#- Apply think-block stripping based on _show_think flag -#}
     {%- if not _show_think -%}
-      {#- Fuzzy end-tag detection for stripping -#}
       {%- set _think_end = '' -%}
       {%- if '</think>' in _ac -%}
         {%- set _think_end = '</think>' -%}
@@ -356,20 +245,15 @@
         {%- set _ac = _ac.split(_think_end)[-1].lstrip('\n') -%}
       {%- endif -%}
     {%- elif not _tc and loop.last and '<think>' not in _ac and not ns.enable_thinking -%}
-      {#- Last turn, non-thinking: inject empty think block if missing -#}
       {%- set _ac = '<think>\n\n</think>\n\n' + _ac -%}
     {%- endif -%}
 
-    {#- Emit the assistant turn -#}
     {{- '<|im_start|>assistant\n' -}}
     {%- if _ac -%}
       {{- _ac -}}
       {%- if _tc -%}{{- '\n' -}}{%- endif -%}
     {%- endif -%}
 
-    {#- Render tool calls in Hermes format.
-        Each value output via its own {{ }} block — never concatenated with plain strings
-        in Python, which would trigger Markup HTML-escaping. -#}
     {%- if _tc -%}
       {%- for tc in _tc -%}
         {{- '<tool_call>\n' -}}
@@ -389,12 +273,11 @@
     {%- endif -%}
     {{- '<|im_end|>\n' -}}
 
-  {#- 8d: Tool results — with error detection (NEW in v0.7) -#}
+  {#- 8d: Tool results -#}
   {%- elif message.role == 'tool' -%}
     {%- set _prev_role = messages[loop.index0 - 1].role if loop.index0 > 0 else '' -%}
     {%- set _next_role = messages[loop.index0 + 1].role if not loop.last else '' -%}
     
-    {#- NEW in v0.7: Detect errors in tool response -#}
     {%- set _tool_content = message.content | default('') -%}
     {{- detect_tool_error(_tool_content) -}}
 
@@ -402,10 +285,9 @@
       {{- '<|im_start|>user\n' -}}
     {%- endif -%}
     {{- '<tool_response>\n' -}}
-    {{- _tool_content -}}
     
-    {#- NEW in v0.7: Inject warning if tool error detected -#}
-    {#- v0.8: Replaced emoji with text-only for tokenization safety -#}
+    {{- '{"result": ' ~ _tool_content | string | tojson ~ '}' -}}
+    
     {%- if ns.last_tool_failed -%}
       {%- if ns.consecutive_failures >= 2 -%}
         {{- '\n\n[SYSTEM WARNING: ' ~ ns.consecutive_failures ~ ' consecutive tool errors detected. Your previous approach is incorrect.]' -}}
@@ -421,26 +303,13 @@
       {{- '<|im_end|>\n' -}}
     {%- endif -%}
 
-  {#- 8e: Unknown role - explicit error (from v18) -#}
   {%- else -%}
     {{- raise_exception('Unexpected message role: ' + message.role) -}}
   {%- endif -%}
 
 {%- endfor -%}
 
-{#- ===== SECTION 9: GENERATION PROMPT =====
-     FIXED in v0.7: preserve_thinking now affects conversation history (Section 8),
-     so generation prompt logic is simplified.
-     
-     enable_thinking=True  → open <think>\n prefill so llama.cpp reasoning-budget
-                             and other inference engines can hook into the think-stream.
-                             The model continues generating inside the open block.
-     enable_thinking=False → exact non-thinking prefill: </think>\n\n
-     
-     NOTE: The <think>\n opener is EPHEMERAL — it lives only in the generation
-     prompt, never in chat history. Historical think-block stripping is handled 
-     in Section 8 based on preserve_thinking setting.
--#}
+{#- ===== SECTION 9: GENERATION PROMPT ===== -#}
 {%- if add_generation_prompt -%}
   {{- '<|im_start|>assistant\n' -}}
   {%- if ns.enable_thinking -%}
@@ -448,4 +317,6 @@
   {%- else -%}
     {{- '<think>\n\n</think>\n\n' -}}
   {%- endif -%}
-{%- endif -%}
+{%- endif -%}
+
+```