opencode-raven 2.1.6 → 2.1.7

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 2.1.7
+
+### Fixed
+- Stats and `/raven` setting updates now patch the latest on-disk configuration instead of rewriting stale in-memory state, preserving externally edited MCP servers and headers.
+- Existing malformed or partially written configuration files are preserved for repair instead of being replaced with defaults.
+
 ## 2.1.6
 
 ### Added

package/index.ts

@@ -469,28 +469,27 @@ export default (async (input: PluginInput) => {
   }
 
   function loadConfig(): RavenConfig {
-    try {
-      if (existsSync(configFile)) {
+    if (existsSync(configFile)) {
+      try {
         const raw = JSON.parse(readFileSync(configFile, "utf-8"))
-        const normalized = normalizeConfig(raw)
-        if (JSON.stringify(raw) !== JSON.stringify(normalized)) {
-          saveConfig(normalized)
-        }
-        return normalized
+        return normalizeConfig(raw)
+      } catch {
+        // Preserve malformed or partially-written config so the user can repair it.
+        return { ...DEFAULT_CONFIG }
       }
-    } catch { /* ignore corruption, use defaults */ }
-    // Auto-create config file with defaults on first run
-    saveConfig(DEFAULT_CONFIG)
-    return { ...DEFAULT_CONFIG }
-  }
-
-  function saveConfig(config: RavenConfig) {
-    try {
-      mkdirSync(ravenConfigDir, { recursive: true })
-      writeFileSync(configFile, JSON.stringify(config, null, 2) + "\n")
-    } catch { /* non-fatal: config won't persist but toggle still works in-session */ }
-  }
-
+    }
+    // Auto-create config file with defaults on first run
+    writeConfig(DEFAULT_CONFIG)
+    return { ...DEFAULT_CONFIG }
+  }
+
+  function writeConfig(config: RavenConfig) {
+    try {
+      mkdirSync(ravenConfigDir, { recursive: true })
+      writeFileSync(configFile, JSON.stringify(config, null, 2) + "\n")
+    } catch { /* non-fatal: config won't persist but toggle still works in-session */ }
+  }
+
   function loadMcpMetadata(): Record<string, OnDemandMcpMetadata> {
     const defaults = cloneOnDemandMcpMetadata(DEFAULT_ON_DEMAND_MCP_METADATA)
     try {
@@ -515,8 +514,23 @@ export default (async (input: PluginInput) => {
       writeFileSync(dynamicGuidanceFile, `## Raven on-demand MCP capabilities\n\n${onDemandMcpGuidance("delegate", dynamicGuidanceMcpOrder())}\n`, "utf-8")
     } catch { /* non-fatal: guidance still exists in tool/subagent paths */ }
   }
-
+
   let config = loadConfig()
+
+  function updateConfig(update: Partial<RavenConfig> | ((latest: RavenConfig) => Partial<RavenConfig>)): RavenConfig {
+    try {
+      const raw = existsSync(configFile) ? JSON.parse(readFileSync(configFile, "utf-8")) : config
+      const latest = normalizeConfig(raw)
+      const patch = typeof update === "function" ? update(latest) : update
+      config = normalizeConfig({ ...latest, ...patch })
+      writeConfig({ ...raw, ...patch })
+    } catch {
+      // Keep malformed or partially-written external edits intact instead of replacing them.
+      const patch = typeof update === "function" ? update(config) : update
+      config = normalizeConfig({ ...config, ...patch })
+    }
+    return config
+  }
   let mcpMetadata = loadMcpMetadata()
   const ravenSessions = new Set<string>()
   const ravenTaskCalls = new Set<string>()
@@ -1274,16 +1288,17 @@ Then restart opencode for the model change to take effect. If the task can be co
     return parsed
   }
 
-  // ── Context saved by Raven delegation ──
-  let sessionBytes = 0
-  let totalBytes = config.stats?.bytes ?? 0
-
-  function addBytes(bytes: number) {
-    sessionBytes += bytes
-    totalBytes += bytes
-    config.stats = { bytes: totalBytes }
-    saveConfig(config)
-  }
+  // ── Context saved by Raven delegation ──
+  let sessionBytes = 0
+  let totalBytes = config.stats?.bytes ?? 0
+
+  function addBytes(bytes: number) {
+    sessionBytes += bytes
+    const updated = updateConfig((latest) => ({
+      stats: { bytes: Math.max(totalBytes, latest.stats?.bytes ?? 0) + bytes },
+    }))
+    totalBytes = updated.stats?.bytes ?? totalBytes + bytes
+  }
 
   function formatBytes(bytes: number): string {
     return bytes >= 1_000_000 ? `${(bytes / 1_000_000).toFixed(1)}MB`
@@ -1814,16 +1829,14 @@ Then restart opencode for the model change to take effect. If the task can be co
       const raw = input.arguments.trim()
       const arg = raw.toLowerCase()
 
-      if (arg === "on") {
-        config.enabled = true
-        saveConfig(config)
-        output.parts.push({ type: "text", text: "Raven tool/MCP routing enabled. Non-Raven agents will be redirected to raven_seek for configured tools." })
+      if (arg === "on") {
+        updateConfig({ enabled: true })
+        output.parts.push({ type: "text", text: "Raven tool/MCP routing enabled. Non-Raven agents will be redirected to raven_seek for configured tools." })
       } else if (arg === "help") {
         output.parts.push({ type: "text", text: helpText() })
-      } else if (arg === "off") {
-        config.enabled = false
-        saveConfig(config)
-        output.parts.push({ type: "text", text: "Raven tool/MCP routing disabled. All agents can use tools directly." })
+      } else if (arg === "off") {
+        updateConfig({ enabled: false })
+        output.parts.push({ type: "text", text: "Raven tool/MCP routing disabled. All agents can use tools directly." })
       } else if (arg === "stats") {
         output.parts.push({ type: "text", text: statsText() })
       } else if (arg === "route") {
@@ -1835,16 +1848,18 @@ Then restart opencode for the model change to take effect. If the task can be co
         const name = parts.slice(3).join(" ").trim()
         const key = kind === "tool" ? "routeTools" : kind === "keyword" ? "routeToolKeywords" : undefined
 
-        if (!key || !["add", "remove", "rm"].includes(action) || !name) {
-          output.parts.push({ type: "text", text: "Usage:\n  /raven route tool add <tool_name>\n  /raven route tool remove <tool_name>\n  /raven route keyword add <keyword>\n  /raven route keyword remove <keyword>\n\nGlobal MCPs are auto-routed by detected OpenCode MCP server name." })
-        } else {
-          const values = uniqueStrings(config[key])
-          const exists = values.some((value) => value.toLowerCase() === name.toLowerCase())
-          config[key] = action === "add"
-            ? exists ? values : [...values, name]
-            : values.filter((value) => value.toLowerCase() !== name.toLowerCase())
-          saveConfig(config)
-          output.parts.push({ type: "text", text: routeSummary() })
+        if (!key || !["add", "remove", "rm"].includes(action) || !name) {
+          output.parts.push({ type: "text", text: "Usage:\n  /raven route tool add <tool_name>\n  /raven route tool remove <tool_name>\n  /raven route keyword add <keyword>\n  /raven route keyword remove <keyword>\n\nGlobal MCPs are auto-routed by detected OpenCode MCP server name." })
+        } else {
+          updateConfig((latest) => {
+            const values = uniqueStrings(latest[key])
+            const exists = values.some((value) => value.toLowerCase() === name.toLowerCase())
+            const next = action === "add"
+              ? exists ? values : [...values, name]
+              : values.filter((value) => value.toLowerCase() !== name.toLowerCase())
+            return key === "routeTools" ? { routeTools: next } : { routeToolKeywords: next }
+          })
+          output.parts.push({ type: "text", text: routeSummary() })
         }
       } else if (arg === "mcp") {
         output.parts.push({ type: "text", text: `${onDemandMcpStatus()}\n\nUsage:\n  /raven mcp refresh [server_name]\n  /raven mcp detail full\n  /raven mcp detail minimized` })
@@ -1861,12 +1876,11 @@ Then restart opencode for the model change to take effect. If the task can be co
           })
         } else if (action === "detail") {
           const detail = parts[2]?.toLowerCase()
-          if (detail !== "full" && detail !== "minimized") {
-            output.parts.push({ type: "text", text: `Usage: /raven mcp detail full|minimized\nCurrent: ${config.onDemandMcpDescriptionDetail ?? "full"}` })
-          } else {
-            config.onDemandMcpDescriptionDetail = detail
-            saveConfig(config)
-            output.parts.push({ type: "text", text: `On-demand MCP guidance detail set to: ${detail}\nRestart opencode for startup instructions/tool descriptions to fully refresh.` })
+          if (detail !== "full" && detail !== "minimized") {
+            output.parts.push({ type: "text", text: `Usage: /raven mcp detail full|minimized\nCurrent: ${config.onDemandMcpDescriptionDetail ?? "full"}` })
+          } else {
+            updateConfig({ onDemandMcpDescriptionDetail: detail })
+            output.parts.push({ type: "text", text: `On-demand MCP guidance detail set to: ${detail}\nRestart opencode for startup instructions/tool descriptions to fully refresh.` })
           }
         } else {
           output.parts.push({ type: "text", text: "Usage:\n  /raven mcp\n  /raven mcp refresh [server_name]\n  /raven mcp detail full\n  /raven mcp detail minimized" })
@@ -1887,31 +1901,28 @@ Then restart opencode for the model change to take effect. If the task can be co
         }
       } else if (arg.startsWith("model ")) {
         const model = raw.slice(6).trim()
-        if (!model) {
-          output.parts.push({ type: "text", text: `Usage: /raven model <name>\nCurrent model: ${config.model || fm.model || "(default)"}` })
-        } else {
-          config.model = model
-          saveConfig(config)
-          ravenProviderIssue = undefined
+        if (!model) {
+          output.parts.push({ type: "text", text: `Usage: /raven model <name>\nCurrent model: ${config.model || fm.model || "(default)"}` })
+        } else {
+          updateConfig({ model })
+          ravenProviderIssue = undefined
           output.parts.push({ type: "text", text: `Raven model set to: ${model}\nRestart opencode for the change to take effect.` })
         }
       } else if (arg.startsWith("effort ")) {
         const effort = raw.slice(7).trim()
-        if (!effort) {
-          output.parts.push({ type: "text", text: `Usage: /raven effort <value>\nCurrent: ${config.reasoning_effort || fm.reasoning_effort || "(default)"}` })
-        } else {
-          config.reasoning_effort = effort
-          saveConfig(config)
-          output.parts.push({ type: "text", text: `Raven reasoning effort set to: ${effort}\nRestart opencode for the change to take effect.` })
+        if (!effort) {
+          output.parts.push({ type: "text", text: `Usage: /raven effort <value>\nCurrent: ${config.reasoning_effort || fm.reasoning_effort || "(default)"}` })
+        } else {
+          updateConfig({ reasoning_effort: effort })
+          output.parts.push({ type: "text", text: `Raven reasoning effort set to: ${effort}\nRestart opencode for the change to take effect.` })
         }
       } else if (arg.startsWith("timeout ")) {
         const secs = Number(raw.slice(8).trim())
         if (!Number.isInteger(secs) || secs < 10 || secs > MAX_TIMEOUT_SECONDS) {
           output.parts.push({ type: "text", text: `Usage: /raven timeout <seconds>\nMust be between 10 and ${MAX_TIMEOUT_SECONDS}. Current: ${config.timeout ?? 600}s` })
-        } else {
-          config.timeout = secs
-          saveConfig(config)
-          output.parts.push({ type: "text", text: `Raven timeout set to ${secs}s. Takes effect immediately.` })
+        } else {
+          updateConfig({ timeout: secs })
+          output.parts.push({ type: "text", text: `Raven timeout set to ${secs}s. Takes effect immediately.` })
         }
       } else {
         const enabled = config.enabled ? "enabled" : "disabled"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-raven",
-  "version": "2.1.6",
+  "version": "2.1.7",
   "description": "Context-saving search and on-demand MCP routing for OpenCode through a focused Raven subagent.",
   "main": "./index.ts",
   "exports": {