dsh-baton-mcp 0.3.0 → 0.3.1

package/WINDOWS.md

@@ -34,7 +34,7 @@ Nothing else to configure: Claude Code waits for a run in the background (Git Ba
 
 Remove: `npx -y dsh-baton-mcp setup --remove`, then Harness → Plugins → dsh-baton-host → Uninstall, then `setup --remove` once more to delete the app folder.
 
-Upgrading from dsh-bridge: `setup` replaces the old MCP entries and reuses an existing `%USERPROFILE%\.dsh-bridge` (the app then goes to `.dsh-bridge\app`). Uninstall `dsh-bridge-host` in Harness, then add the path `setup` copied. With the hoisted profile layout, uninstalling may leave a `node_modules\dsh-bridge-host` link behind in `%USERPROFILE%\.dsh\profiles\desktop`; it is no longer in the profile's dependencies, so delete it with Harness quit.
+Upgrading from dsh-bridge: `setup` replaces the old MCP entries and reuses an existing `%USERPROFILE%\.dsh-bridge` (the app then goes to `.dsh-bridge\app`). Uninstall `dsh-bridge-host` in Harness, then add the path `setup` copied. With the hoisted profile layout, uninstalling may leave a `node_modules\dsh-bridge-host` link behind in `%USERPROFILE%\.dsh\profiles\desktop`; `setup --remove` deletes such a leftover link (old or new name) once the profile no longer lists the plugin.
 
 From source instead: `git clone https://github.com/JialaoLiu/dsh-baton.git C:\dev\dsh-baton` and `node C:\dev\dsh-baton\src\cli.mjs setup`; the plugin path is then `C:\dev\dsh-baton\host`.
 

package/host/lib/index.js

@@ -15,7 +15,7 @@ import { socketEndpoint, batonHome, socketPath as socketPathSetting } from './so
 export const name = 'dsh-baton-host'
 export const inject = ['sessionController', 'workspaceRegistry']
 
-const VERSION = '0.3.0'
+const VERSION = JSON.parse(fs.readFileSync(new URL('../package.json', import.meta.url), 'utf8')).version
 const MAX_EVENTS = 200
 const MAX_RUNS = 100
 const QUIET_CONFIRM_MS = 2000

package/host/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-baton-host",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "DeepSeek Harness plugin for dsh-baton: takes the baton from Claude Code or Codex over a local, owner-only control socket, runs it in a specific session through the host's own session controller, and hands the result back when the turn is done.",
   "type": "module",
   "main": "lib/index.js",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-baton-mcp",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Pass the baton from Claude Code, Codex or any MCP client to a local DeepSeek Harness: an MCP server that hands off tasks, continues a specific session, shows the run live in Harness and hands the result back when it is done.",
   "type": "module",
   "bin": {

package/src/core.mjs

@@ -150,15 +150,28 @@ function hostCallAt(endpoint, method, params = {}, { timeoutMs = 15000 } = {}) {
   })
 }
 
+export const VERSION = JSON.parse(fs.readFileSync(path.join(ROOT, 'package.json'), 'utf8')).version
+
+// The plugin version from the last successful ping.
+let hostVersion = null
+
 export async function hostAvailable() {
   try {
-    await hostCall('ping', {}, { timeoutMs: 2000 })
+    hostVersion = (await hostCall('ping', {}, { timeoutMs: 2000 }))?.version ?? null
     return true
   } catch {
     return false
   }
 }
 
+// The plugin and the MCP server update separately (Harness and npx), so they can drift.
+// A different major.minor gets a note asking to update the older side; patch releases mix.
+const minorOf = (v) => String(v).split('.').slice(0, 2).join('.')
+export function versionWarning(plugin = hostVersion, server = VERSION) {
+  if (!plugin || minorOf(plugin) === minorOf(server)) return null
+  return `The Harness plugin dsh-baton-host is ${plugin} but this MCP server is ${server}. Update the older one: the server with \`npx -y dsh-baton-mcp@latest setup\`, the plugin in Harness (Plugins → dsh-baton-host → Uninstall, then Add plugin → dsh-baton-host).`
+}
+
 // Who a relayed message is from, shown on it in the Harness chat: DSH_BATON_SENDER if set,
 // else the MCP client's name (claude-code, claude-ai → claude; codex-mcp-client → codex).
 export function senderOf(clientName = setting('SENDER')) {

package/src/server.mjs

@@ -5,7 +5,7 @@ import path from 'node:path'
 import * as core from './core.mjs'
 import { stableNodePath } from './setup.mjs'
 
-const SERVER = { name: 'dsh-baton', version: '0.3.0' }
+const SERVER = { name: 'dsh-baton', version: core.VERSION }
 const PROTOCOLS = ['2025-06-18', '2025-03-26', '2024-11-05']
 
 const str = (description) => ({ type: 'string', description })
@@ -154,6 +154,7 @@ const TOOLS = [
 ]
 
 let clientName = null
+let versionWarned = false
 
 function send(msg) {
   process.stdout.write(JSON.stringify(msg) + '\n')
@@ -184,7 +185,11 @@ async function handle(req) {
         const progress = token === undefined ? undefined : (done, message) =>
           send({ jsonrpc: '2.0', method: 'notifications/progress', params: { progressToken: token, progress: done, message } })
         const result = await tool.run(req.params.arguments || {}, { progress })
-        return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] }
+        const content = [{ type: 'text', text: JSON.stringify(result, null, 2) }]
+        // Said once per server process, the first time a call finds the versions apart.
+        const warning = !versionWarned && core.versionWarning()
+        if (warning) { versionWarned = true; content.push({ type: 'text', text: `Note for the user: ${warning}` }) }
+        return { content }
       } catch (e) {
         return { content: [{ type: 'text', text: e.message }], isError: true }
       }

package/src/setup.mjs

@@ -85,6 +85,31 @@ function installedPlugins() {
   }
 }
 
+// With pnpm's hoisted layout, uninstalling the plugin can leave its node_modules link
+// behind. Remove such a link once the profile no longer lists the plugin; only the
+// link itself goes, never the folder it points to. Returns the links removed and the
+// ones that could not be.
+export function cleanStalePluginLinks(profile = path.join(config.dshHome, 'profiles', 'desktop')) {
+  let deps
+  try { deps = JSON.parse(fs.readFileSync(path.join(profile, 'package.json'), 'utf8')).dependencies ?? {} } catch { return { removed: [], failed: [] } }
+  const removed = [], failed = []
+  for (const name of ['dsh-baton-host', 'dsh-bridge-host']) {
+    const link = path.join(profile, 'node_modules', name)
+    if (deps[name]) continue
+    let st
+    try { st = fs.lstatSync(link) } catch { continue }
+    if (!st.isSymbolicLink()) continue
+    try {
+      // A Windows junction is a directory link: rmdir removes the link, not its target.
+      try { fs.unlinkSync(link) } catch { fs.rmdirSync(link) }
+      removed.push(link)
+    } catch {
+      failed.push(link)
+    }
+  }
+  return { removed, failed }
+}
+
 const pluginFrom = (source, hostDir) => source === `link:${hostDir}` || source === `link:${hostDir.split(path.sep).join('/')}`
 
 // The node to register: the one on PATH when it is this same binary, since that path
@@ -143,13 +168,24 @@ export async function setup({ remove = false } = {}) {
 
   if (remove) {
     const plugins = installedPlugins()
-    lines.push('', `To remove the Harness plugin: Harness → Plugins → ${plugins.old && !plugins.current ? 'dsh-bridge-host' : 'dsh-baton-host'} → Uninstall.`)
+    if (plugins.current || plugins.old) {
+      lines.push('', `To remove the Harness plugin: Harness → Plugins → ${plugins.current ? 'dsh-baton-host' : 'dsh-bridge-host'} → Uninstall.`)
+    } else {
+      lines.push('', 'Harness plugin: not installed.')
+      const links = cleanStalePluginLinks()
+      for (const l of links.removed) lines.push(`Removed the leftover plugin link ${l}.`)
+      for (const l of links.failed) lines.push(`Could not remove the leftover plugin link ${l}: quit Harness and delete it.`)
+    }
     const app = appDir()
     if (fs.existsSync(app)) {
       // Harness loads the plugin from this folder: delete it only once the plugin is gone.
       if (pluginFrom(plugins.current, path.join(app, 'host'))) lines.push(`Then run this again to delete ${app}.`)
       else { fs.rmSync(app, { recursive: true, force: true }); lines.push(`Deleted ${app}.`) }
     }
+    const home = config.batonHome
+    if (['runs', 'host-runs'].some((d) => fs.existsSync(path.join(home, d)))) {
+      lines.push(`Run records are kept in ${home} (runs, host-runs); delete that folder to remove them too.`)
+    }
     return lines.join('\n')
   }
   lines.push('', 'MCP server registered user-wide: new conversations in Claude Code and Codex have the dsh_* tools.')