opencode-thinking-fix 1.1.7 → 1.1.9

package/CHANGELOG.md

@@ -2,6 +2,27 @@
 
 All notable changes to `opencode-thinking-fix`.
 
+## [1.1.9] - 2026-06-23
+
+### Added
+- **File-based logging**: JSON-lines log at `~/.local/share/opencode/thinking-fix.log` — captures `plugin_loaded`, `inspect`, `patched`, and `error` events with timestamps.
+- **`inspect` event**: logs field coverage on every request — `isReasoningModel`, `assistantTurns`, `missingContent`, `missingReasoningContent`, `missingReasoning` — proving plugin activity even when zero patches needed.
+- **Console fallback**: `writeLog` catches file write failures and emits `console.error` so failures are visible somewhere.
+
+### Changed
+- `client.app.log()` calls now use `await` for proper async handling.
+- Removal of `hook_fired` debug artifact — replaced by structured `inspect` event.
+
+### Fixed
+- **TUI install flow**: Ctrl+P "install plugin" → type `opencode-thinking-fix` no longer errors. Server-only plugin with bare `export default` properly installs via TUI and CLI without TUI target warnings.
+
+## [1.1.8] - 2026-06-23
+
+### Fixed
+- **package.json aligned with project Documents**: description shortened, `homepage` added, `peerDependenciesMeta.optional` set to `false`, `repository.url` normalized, keywords added (`mimo`, `minimax`), duplicate keywords removed.
+- **Stale `node_modules/` deleted** from repo (was committed accidentally).
+- **Stale npm cache entry** (`npm i opencode-thinking-fix`) removed from `~/.cache/opencode/packages/`.
+
 ## [1.1.7] - 2026-06-23
 
 ### Fixed

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "opencode-thinking-fix",
-  "version": "1.1.7",
-  "description": "Fix DeepSeek/Kimi/GLM/MiMo reasoning_content 400 errors in OpenCode. Plugin + proxy + watchdog.",
+  "version": "1.1.9",
+  "description": "Fix reasoning_content 400 errors for DeepSeek/Kimi/GLM/MiMo/MiniMax in OpenCode",
   "type": "module",
   "license": "MIT",
   "author": "tbosancheros39",
@@ -10,16 +10,17 @@
     "deepseek",
     "kimi",
     "glm",
+    "mimo",
+    "minimax",
     "reasoning",
     "reasoning_content",
-    "thinking",
-    "proxy",
     "plugin"
   ],
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/tbosancheros39/opencode-thinking-fix.git"
+    "url": "https://github.com/tbosancheros39/opencode-thinking-fix"
   },
+  "homepage": "https://github.com/tbosancheros39/opencode-thinking-fix#readme",
   "main": "./plugins/opencode-thinking-fix-universal.ts",
   "exports": {
     ".": "./plugins/opencode-thinking-fix-universal.ts",
@@ -46,7 +47,7 @@
   },
   "peerDependenciesMeta": {
     "@opencode-ai/plugin": {
-      "optional": true
+      "optional": false
     }
   },
   "devDependencies": {

package/plugins/opencode-thinking-fix-universal.ts

@@ -1,4 +1,7 @@
 import type { Plugin } from '@opencode-ai/plugin'
+import { appendFileSync, mkdirSync } from 'fs'
+import { homedir } from 'os'
+import { join } from 'path'
 
 // ─────────────────────────────────────────────────────────────────────────────
 // CONFIRMED FACTS (cross-validated across DeepSeek V4 Pro, Claude MAX, Kimi
@@ -27,57 +30,88 @@ import type { Plugin } from '@opencode-ai/plugin'
 //           OR: "plugin": ["opencode-thinking-fix-universal"] (array, not object)
 //   Build: none — OpenCode compiles .ts plugins at load time
 // ─────────────────────────────────────────────────────────────────────────────
+// LOGGING
+//   File: ~/.local/share/opencode/thinking-fix.log
+//   Format: JSON lines — one entry per event
+//   View:  tail -f ~/.local/share/opencode/thinking-fix.log | jq
+// ─────────────────────────────────────────────────────────────────────────────
+
+const LOG_DIR  = join(homedir(), '.local', 'share', 'opencode')
+const LOG_FILE = join(LOG_DIR, 'thinking-fix.log')
+
+function writeLog(entry: Record<string, unknown>): void {
+  try {
+    mkdirSync(LOG_DIR, { recursive: true })
+    appendFileSync(LOG_FILE, JSON.stringify({ ts: new Date().toISOString(), ...entry }) + '\n', 'utf8')
+  } catch (err: any) {
+    try { console.error('[thinking-fix] log write failed:', err?.message) } catch {}
+  }
+}
 
-function patchMessages(messages: any[]): number {
-  let count = 0
+// ─────────────────────────────────────────────────────────────────────────────
+// Message patching — self-detection, in-place mutation
+// ─────────────────────────────────────────────────────────────────────────────
+interface PatchResult {
+  patched:       number
+  totalMessages: number
+  turns:         Array<{ index: number; fields: string[] }>
+}
+
+function patchMessages(messages: any[]): PatchResult {
+  const result: PatchResult = { patched: 0, totalMessages: messages.length, turns: [] }
 
   // ── Step 1: Self-detect reasoning model ──────────────────────────────────
-  // Scan for any assistant message that already has reasoning_content.
-  // OpenCode sets this field on the most recent response turn.
-  // Non-reasoning providers (Qwen/GPT/Claude/Mistral/Llama) never have it.
   const isReasoningModel = messages.some((wrapper) => {
     const msg = wrapper?.info ?? wrapper
     return msg?.role === 'assistant' && ('reasoning_content' in msg || 'reasoning' in msg)
   })
 
   // ── Step 2: Non-reasoning model — exit immediately, zero modification ─────
-  // Qwen/GPT/Claude reject unknown fields with HTTP 400.
-  // This guard is what prevents multi-session errors on non-reasoning models.
-  if (!isReasoningModel) return 0
+  if (!isReasoningModel) return result
 
   // ── Step 3: Patch all assistant turns missing required fields ─────────────
-  for (const wrapper of messages) {
+  messages.forEach((wrapper, index) => {
     const msg = wrapper?.info ?? wrapper
-    if (!msg || msg.role !== 'assistant') continue
+    if (!msg || msg.role !== 'assistant') return
+
+    const fields: string[] = []
 
-    // Fix 1: content must be non-null string on every assistant turn.
-    // OpenAI-compatible SDK omits it when only tool_calls are present.
+    // Fix 1: content must be non-null string on every assistant turn
     if (!msg.content && msg.content !== '') {
       msg.content = msg.tool_calls?.length ? 'call tool' : ''
-      count++
+      fields.push('content')
     }
 
-    // Fix 2: reasoning_content must be present on every assistant turn in history.
-    // Absence causes HTTP 400 from DeepSeek/Kimi/GLM/MiMo on turn 2+.
-    // Use "" (empty string) — confirmed working cross-provider.
-    // "" and null both work per DeepSeek docs; "" is safer cross-provider.
+    // Fix 2: reasoning_content — absent causes HTTP 400 from DeepSeek/Kimi/GLM/MiMo on turn 2+
     if (!('reasoning_content' in msg)) {
       msg.reasoning_content = ''
-      count++
+      fields.push('reasoning_content')
     }
 
-    // Fix 3: reasoning must be present for OpenCode Go provider on every turn.
+    // Fix 3: reasoning — required for OpenCode Go provider on every turn
     if (!('reasoning' in msg)) {
       msg.reasoning = ''
-      count++
+      fields.push('reasoning')
     }
-  }
 
-  return count
+    if (fields.length > 0) {
+      result.patched += fields.length
+      result.turns.push({ index, fields })
+    }
+  })
+
+  return result
 }
 
+// ─────────────────────────────────────────────────────────────────────────────
+// Plugin
+// ─────────────────────────────────────────────────────────────────────────────
 export const ThinkingFixPlugin: Plugin = async ({ client }) => {
-  client.app.log({ body: { service: 'thinking-fix', level: 'info', message: 'Plugin loaded — universal reasoning_content fix active' } })
+  mkdirSync(LOG_DIR, { recursive: true })
+
+  writeLog({ event: 'plugin_loaded', logFile: LOG_FILE })
+
+  await client.app.log({ body: { service: 'thinking-fix', level: 'info', message: `Plugin loaded — logging to ${LOG_FILE}` } })
 
   return {
     'chat.headers': async (input: any, output: any) => {
@@ -91,13 +125,48 @@ export const ThinkingFixPlugin: Plugin = async ({ client }) => {
         const messages = output?.messages
         if (!Array.isArray(messages) || messages.length === 0) return
 
-        const patched = patchMessages(messages)
+        const assistantTurns = messages.filter((w: any) => {
+          const m = w?.info ?? w
+          return m?.role === 'assistant'
+        })
+        const isReasoningModel = assistantTurns.some((w: any) => {
+          const m = w?.info ?? w
+          return 'reasoning_content' in m || 'reasoning' in m
+        })
+        writeLog({
+          event:                  'inspect',
+          isReasoningModel,
+          totalMessages:          messages.length,
+          assistantTurns:         assistantTurns.length,
+          missingContent:         assistantTurns.filter((w: any) => {
+            const m = w?.info ?? w
+            return !m.content && m.content !== ''
+          }).length,
+          missingReasoningContent: assistantTurns.filter((w: any) => {
+            const m = w?.info ?? w
+            return !('reasoning_content' in m)
+          }).length,
+          missingReasoning:        assistantTurns.filter((w: any) => {
+            const m = w?.info ?? w
+            return !('reasoning' in m)
+          }).length,
+        })
+
+        const result = patchMessages(messages)
+
+        if (result.patched > 0) {
+          writeLog({
+            event:         'patched',
+            patchedFields: result.patched,
+            totalMessages: result.totalMessages,
+            turns:         result.turns,
+          })
 
-        if (patched > 0) {
-          client.app.log({ body: { service: 'thinking-fix', level: 'info', message: `patched ${patched} field(s) across ${messages.length} message(s)` } })
+          await client.app.log({ body: { service: 'thinking-fix', level: 'info', message: `patched ${result.patched} field(s) across ${result.totalMessages} message(s)` } })
         }
-      } catch (err) {
-        client.app.log({ body: { service: 'thinking-fix', level: 'error', message: `Error — passing through unmodified: ${err}` } })
+      } catch (err: any) {
+        writeLog({ event: 'error', message: err?.message ?? String(err) })
+        await client.app.log({ body: { service: 'thinking-fix', level: 'error', message: `Error — passing through unmodified: ${err?.message ?? err}` } })
       }
     },
   }