dsh-mcp-skills 0.2.2 → 0.2.3

package/lib/index.js

@@ -7,6 +7,7 @@
 // @deepseek-ai packages so out-of-tree resolution stays reliable (the modlens
 // precedent); `yaml` and `fflate` are runtime dependencies, and the MCP SDK is
 // an optional peer the host installation already provides (see loadMcpSdk).
+import { existsSync } from 'node:fs'
 import { access, copyFile, mkdir, readdir, readFile, rename, rm, writeFile } from 'node:fs/promises'
 import { homedir } from 'node:os'
 import { dirname, isAbsolute, join, relative, resolve } from 'node:path'
@@ -19,7 +20,7 @@ export const inject = []
 
 const SKILL_NAME = /^[a-z0-9]+(?:-[a-z0-9]+)*$/
 /** Kept in step with package.json (the smoke test reports it to the server). */
-const PLUGIN_VERSION = '0.2.2'
+const PLUGIN_VERSION = '0.2.3'
 const MCP_CONFIG_FILENAME = 'mcp-servers.yaml'
 const MCP_PROFILE_ROW_PREFIX = 'mcp-'
 /** Same grammar as @deepseek-ai/dsh-mcp-client's SERVER_NAME_PATTERN. */
@@ -406,13 +407,32 @@ async function writeMcpServers(ctx, servers) {
   await syncMcpProfilePatch(ctx, servers)
 }
 
+/** Executable directories a GUI-launched harness commonly lacks. */
+const EXTRA_PATH_DIRS = ['/usr/local/bin', '/opt/homebrew/bin', '/usr/bin', '/bin']
+
+/**
+ * The environment one stdio server is spawned with.
+ *
+ * MCP servers are spawned by `@deepseek-ai/dsh-mcp-client` from the harness
+ * process environment, and a harness launched from a desktop icon inherits a
+ * minimal PATH (macOS gives just `/usr/bin:/bin:/usr/sbin:/sbin`). A configured
+ * `npx` / `uvx` then fails with ENOENT even though the user's shell resolves it,
+ * so the well-known executable directories are prepended when they exist.
+ * An explicit `PATH` in the server config still wins.
+ */
+function serverEnv(env) {
+  const current = (process.env.PATH ?? '').split(':').filter((entry) => entry !== '')
+  const missing = EXTRA_PATH_DIRS.filter((dir) => existsSync(dir) && !current.includes(dir))
+  return { ...(missing.length > 0 ? { PATH: [...missing, ...current].join(':') } : {}), ...env }
+}
+
 /**
  * The transport-shaped fields of one managed server, shared by the generated
  * patch row and the connection smoke test so both always agree.
  */
 function serverConnection(server) {
   return server.transport === 'stdio'
-    ? { command: server.command, args: server.args ?? [], env: server.env ?? {}, cwd: server.cwd || process.cwd() }
+    ? { command: server.command, args: server.args ?? [], env: serverEnv(server.env ?? {}), cwd: server.cwd || process.cwd() }
     : { url: server.url, headers: server.headers ?? {} }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-mcp-skills",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "DeepSeek Harness web plugin: settings pages for managing user skills and MCP servers",
   "type": "module",
   "repository": {