From d6ce5db3f2225a6f88c729203880834afa261404 Mon Sep 17 00:00:00 2001
From: bee4come <bee4come@gmail.com>
Date: Fri, 30 Jan 2026 09:00:47 +0000
Subject: [PATCH] fix(providers): remove MiniMax from reasoning tag providers

MiniMax was incorrectly included in isReasoningTagProvider(), causing
enforceFinalTag=true for all MiniMax runs. Since MiniMax models don't
output <think>/<final> tags, this caused all streaming content to be
discarded, resulting in '(no output)' in the TUI.

The fix removes MiniMax from the reasoning tag provider list, allowing
content to stream normally without requiring <final> tags.

Fixes #4499
---
 src/utils/provider-utils.ts | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/utils/provider-utils.ts b/src/utils/provider-utils.ts
index e29ffe1ab..f46489082 100644
--- a/src/utils/provider-utils.ts
+++ b/src/utils/provider-utils.ts
@@ -25,10 +25,11 @@ export function isReasoningTagProvider(provider: string | undefined | null): boo
     return true;
   }
 
-  // Handle Minimax (M2.1 is chatty/reasoning-like)
-  if (normalized.includes("minimax")) {
-    return true;
-  }
+  // NOTE: MiniMax was previously included here but removed because:
+  // 1. MiniMax models don't reliably output <think>/<final> tags
+  // 2. enforceFinalTag=true causes all content to be stripped → "(no output)"
+  // 3. MiniMax's reasoning is handled differently (not via text stream tags)
+  // See: https://github.com/openclaw/openclaw/issues/4499
 
   return false;
 }