gogcli-mcp 2.0.7 → 2.0.8

package/.claude-plugin/marketplace.json

@@ -7,7 +7,7 @@
   },
   "metadata": {
     "description": "Google Sheets (and more) for Claude via gogcli — read, write, and manage spreadsheets",
-    "version": "2.0.7"
+    "version": "2.0.8"
   },
   "plugins": [
     {
@@ -15,7 +15,7 @@
       "displayName": "gogcli",
       "source": "./",
       "description": "Google Sheets (and more) for Claude via gogcli — read, write, and manage spreadsheets",
-      "version": "2.0.7",
+      "version": "2.0.8",
       "author": {
         "name": "Chris Hall"
       },

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "gogcli-mcp",
   "displayName": "gogcli",
-  "version": "2.0.7",
+  "version": "2.0.8",
   "description": "Google Sheets (and more) for Claude via gogcli — read, write, and manage spreadsheets",
   "author": {
     "name": "Chris Hall",

package/dist/index.js

@@ -8415,8 +8415,8 @@ function emoji() {
 }
 var ipv4 = /^(?:(?:25[0-5]|2[0-4][0-9]|1[0-9][0-9]|[1-9][0-9]|[0-9])\.){3}(?:25[0-5]|2[0-4][0-9]|1[0-9][0-9]|[1-9][0-9]|[0-9])$/;
 var ipv6 = /^(([0-9a-fA-F]{1,4}:){7}[0-9a-fA-F]{1,4}|([0-9a-fA-F]{1,4}:){1,7}:|([0-9a-fA-F]{1,4}:){1,6}:[0-9a-fA-F]{1,4}|([0-9a-fA-F]{1,4}:){1,5}(:[0-9a-fA-F]{1,4}){1,2}|([0-9a-fA-F]{1,4}:){1,4}(:[0-9a-fA-F]{1,4}){1,3}|([0-9a-fA-F]{1,4}:){1,3}(:[0-9a-fA-F]{1,4}){1,4}|([0-9a-fA-F]{1,4}:){1,2}(:[0-9a-fA-F]{1,4}){1,5}|[0-9a-fA-F]{1,4}:((:[0-9a-fA-F]{1,4}){1,6})|:((:[0-9a-fA-F]{1,4}){1,7}|:))$/;
-var mac = (delimiter) => {
-  const escapedDelim = escapeRegex(delimiter ?? ":");
+var mac = (delimiter2) => {
+  const escapedDelim = escapeRegex(delimiter2 ?? ":");
   return new RegExp(`^(?:[0-9A-F]{2}${escapedDelim}){5}[0-9A-F]{2}$|^(?:[0-9a-f]{2}${escapedDelim}){5}[0-9a-f]{2}$`);
 };
 var cidrv4 = /^((25[0-5]|2[0-4][0-9]|1[0-9][0-9]|[1-9][0-9]|[0-9])\.){3}(25[0-5]|2[0-4][0-9]|1[0-9][0-9]|[1-9][0-9]|[0-9])\/([0-9]|[1-2][0-9]|3[0-2])$/;
@@ -30863,12 +30863,34 @@ var EMPTY_COMPLETION_RESULT = {
 
 // src/runner.ts
 import { spawn } from "node:child_process";
+import { delimiter } from "node:path";
 var TIMEOUT_MS = 3e4;
 function envOrUndefined(key) {
   const value = process.env[key];
   if (!value || value.startsWith("${")) return void 0;
   return value;
 }
+function augmentedPath() {
+  const home = process.env.HOME;
+  const candidates = [
+    process.env.PATH ?? "",
+    "/opt/homebrew/bin",
+    "/usr/local/bin",
+    home ? `${home}/.local/bin` : "",
+    home ? `${home}/go/bin` : ""
+  ];
+  const seen = /* @__PURE__ */ new Set();
+  const parts = [];
+  for (const c of candidates) {
+    if (!c) continue;
+    for (const dir of c.split(delimiter)) {
+      if (!dir || seen.has(dir)) continue;
+      seen.add(dir);
+      parts.push(dir);
+    }
+  }
+  return parts.join(delimiter);
+}
 function formatTimeout(ms) {
   const seconds = Math.round(ms / 1e3);
   if (seconds >= 60) {
@@ -30891,7 +30913,8 @@ async function run(args, options = {}) {
   const effectiveTimeout = timeout ?? TIMEOUT_MS;
   return new Promise((resolve, reject) => {
     const { GOG_ACCESS_TOKEN: _, ...cleanEnv } = process.env;
-    const child = spawner(envOrUndefined("GOG_PATH") ?? "gog", fullArgs, { env: cleanEnv });
+    const childEnv = { ...cleanEnv, PATH: augmentedPath() };
+    const child = spawner(envOrUndefined("GOG_PATH") ?? "gog", fullArgs, { env: childEnv });
     const stdoutChunks = [];
     const stderrChunks = [];
     let settled = false;
@@ -30926,6 +30949,12 @@ async function run(args, options = {}) {
       clearTimeout(timer);
       if (settled) return;
       settled = true;
+      if (err.code === "ENOENT") {
+        reject(new Error(
+          "gog executable not found. Install gogcli (https://github.com/steipete/gogcli) or set GOG_PATH in your MCP client config to the absolute binary path (run `which gog` in a terminal to find it)."
+        ));
+        return;
+      }
       reject(err);
     });
   });
@@ -32123,7 +32152,7 @@ function registerTasksTools(server2) {
 }
 
 // src/server.ts
-var VERSION = true ? "2.0.7" : "0.0.0";
+var VERSION = true ? "2.0.8" : "0.0.0";
 function createServer(options) {
   return new McpServer({
     name: options?.name ?? "gogcli",

package/dist/lib.js

@@ -12044,8 +12044,8 @@ function emoji() {
 }
 var ipv4 = /^(?:(?:25[0-5]|2[0-4][0-9]|1[0-9][0-9]|[1-9][0-9]|[0-9])\.){3}(?:25[0-5]|2[0-4][0-9]|1[0-9][0-9]|[1-9][0-9]|[0-9])$/;
 var ipv6 = /^(([0-9a-fA-F]{1,4}:){7}[0-9a-fA-F]{1,4}|([0-9a-fA-F]{1,4}:){1,7}:|([0-9a-fA-F]{1,4}:){1,6}:[0-9a-fA-F]{1,4}|([0-9a-fA-F]{1,4}:){1,5}(:[0-9a-fA-F]{1,4}){1,2}|([0-9a-fA-F]{1,4}:){1,4}(:[0-9a-fA-F]{1,4}){1,3}|([0-9a-fA-F]{1,4}:){1,3}(:[0-9a-fA-F]{1,4}){1,4}|([0-9a-fA-F]{1,4}:){1,2}(:[0-9a-fA-F]{1,4}){1,5}|[0-9a-fA-F]{1,4}:((:[0-9a-fA-F]{1,4}){1,6})|:((:[0-9a-fA-F]{1,4}){1,7}|:))$/;
-var mac = (delimiter) => {
-  const escapedDelim = escapeRegex(delimiter ?? ":");
+var mac = (delimiter2) => {
+  const escapedDelim = escapeRegex(delimiter2 ?? ":");
   return new RegExp(`^(?:[0-9A-F]{2}${escapedDelim}){5}[0-9A-F]{2}$|^(?:[0-9a-f]{2}${escapedDelim}){5}[0-9a-f]{2}$`);
 };
 var cidrv4 = /^((25[0-5]|2[0-4][0-9]|1[0-9][0-9]|[1-9][0-9]|[0-9])\.){3}(25[0-5]|2[0-4][0-9]|1[0-9][0-9]|[1-9][0-9]|[0-9])\/([0-9]|[1-2][0-9]|3[0-2])$/;
@@ -30770,12 +30770,34 @@ var EMPTY_COMPLETION_RESULT = {
 
 // src/runner.ts
 import { spawn } from "node:child_process";
+import { delimiter } from "node:path";
 var TIMEOUT_MS = 3e4;
 function envOrUndefined(key) {
   const value = process.env[key];
   if (!value || value.startsWith("${")) return void 0;
   return value;
 }
+function augmentedPath() {
+  const home = process.env.HOME;
+  const candidates = [
+    process.env.PATH ?? "",
+    "/opt/homebrew/bin",
+    "/usr/local/bin",
+    home ? `${home}/.local/bin` : "",
+    home ? `${home}/go/bin` : ""
+  ];
+  const seen = /* @__PURE__ */ new Set();
+  const parts = [];
+  for (const c of candidates) {
+    if (!c) continue;
+    for (const dir of c.split(delimiter)) {
+      if (!dir || seen.has(dir)) continue;
+      seen.add(dir);
+      parts.push(dir);
+    }
+  }
+  return parts.join(delimiter);
+}
 function formatTimeout(ms) {
   const seconds = Math.round(ms / 1e3);
   if (seconds >= 60) {
@@ -30798,7 +30820,8 @@ async function run(args, options = {}) {
   const effectiveTimeout = timeout ?? TIMEOUT_MS;
   return new Promise((resolve, reject) => {
     const { GOG_ACCESS_TOKEN: _, ...cleanEnv } = process.env;
-    const child = spawner(envOrUndefined("GOG_PATH") ?? "gog", fullArgs, { env: cleanEnv });
+    const childEnv = { ...cleanEnv, PATH: augmentedPath() };
+    const child = spawner(envOrUndefined("GOG_PATH") ?? "gog", fullArgs, { env: childEnv });
     const stdoutChunks = [];
     const stderrChunks = [];
     let settled = false;
@@ -30833,6 +30856,12 @@ async function run(args, options = {}) {
       clearTimeout(timer);
       if (settled) return;
       settled = true;
+      if (err.code === "ENOENT") {
+        reject(new Error(
+          "gog executable not found. Install gogcli (https://github.com/steipete/gogcli) or set GOG_PATH in your MCP client config to the absolute binary path (run `which gog` in a terminal to find it)."
+        ));
+        return;
+      }
       reject(err);
     });
   });
@@ -32030,7 +32059,7 @@ function registerTasksTools(server) {
 }
 
 // src/server.ts
-var VERSION = true ? "2.0.7" : "0.0.0";
+var VERSION = true ? "2.0.8" : "0.0.0";
 function createServer(options) {
   return new McpServer({
     name: options?.name ?? "gogcli",

package/manifest.json

@@ -3,7 +3,7 @@
   "manifest_version": "0.3",
   "name": "gogcli-mcp",
   "display_name": "gogcli",
-  "version": "2.0.7",
+  "version": "2.0.8",
   "description": "Google Sheets (and more) for Claude via gogcli — read, write, and manage spreadsheets",
   "author": {
     "name": "Chris Hall",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gogcli-mcp",
-  "version": "2.0.7",
+  "version": "2.0.8",
   "mcpName": "io.github.chrischall/gogcli-mcp",
   "description": "MCP server wrapping gogcli for Google service access",
   "author": "Claude Code (AI) <https://www.anthropic.com/claude>",

package/server.json

@@ -7,12 +7,12 @@
     "source": "github",
     "subfolder": "packages/gogcli-mcp"
   },
-  "version": "2.0.7",
+  "version": "2.0.8",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "gogcli-mcp",
-      "version": "2.0.7",
+      "version": "2.0.8",
       "transport": {
         "type": "stdio"
       },

package/src/runner.ts

@@ -1,5 +1,6 @@
 import { spawn } from 'node:child_process';
 import type { ChildProcess } from 'node:child_process';
+import { delimiter } from 'node:path';
 
 export type Spawner = (
   command: string,
@@ -26,6 +27,32 @@ function envOrUndefined(key: string): string | undefined {
   return value;
 }
 
+// MCP desktop clients often spawn servers with a stripped PATH that excludes
+// Homebrew, user-local, and Go's default install dirs — so even when gog is
+// installed, the spawned server can't find it. Augment the child's PATH with
+// the locations where gogcli is commonly installed.
+function augmentedPath(): string {
+  const home = process.env.HOME;
+  const candidates = [
+    process.env.PATH ?? '',
+    '/opt/homebrew/bin',
+    '/usr/local/bin',
+    home ? `${home}/.local/bin` : '',
+    home ? `${home}/go/bin` : '',
+  ];
+  const seen = new Set<string>();
+  const parts: string[] = [];
+  for (const c of candidates) {
+    if (!c) continue;
+    for (const dir of c.split(delimiter)) {
+      if (!dir || seen.has(dir)) continue;
+      seen.add(dir);
+      parts.push(dir);
+    }
+  }
+  return parts.join(delimiter);
+}
+
 function formatTimeout(ms: number): string {
   const seconds = Math.round(ms / 1000);
   if (seconds >= 60) {
@@ -55,7 +82,8 @@ export async function run(args: string[], options: RunOptions = {}): Promise<str
     // Strip GOG_ACCESS_TOKEN so gogcli uses stored refresh tokens instead of
     // a potentially stale direct access token passed through MCP env config.
     const { GOG_ACCESS_TOKEN: _, ...cleanEnv } = process.env;
-    const child = spawner(envOrUndefined('GOG_PATH') ?? 'gog', fullArgs, { env: cleanEnv });
+    const childEnv = { ...cleanEnv, PATH: augmentedPath() };
+    const child = spawner(envOrUndefined('GOG_PATH') ?? 'gog', fullArgs, { env: childEnv });
     const stdoutChunks: Buffer[] = [];
     const stderrChunks: Buffer[] = [];
     let settled = false;
@@ -90,6 +118,14 @@ export async function run(args: string[], options: RunOptions = {}): Promise<str
       clearTimeout(timer);
       if (settled) return;
       settled = true;
+      if ((err as NodeJS.ErrnoException).code === 'ENOENT') {
+        reject(new Error(
+          'gog executable not found. Install gogcli (https://github.com/steipete/gogcli) ' +
+          'or set GOG_PATH in your MCP client config to the absolute binary path ' +
+          '(run `which gog` in a terminal to find it).',
+        ));
+        return;
+      }
       reject(err);
     });
   });

package/tests/runner.test.ts

@@ -24,7 +24,7 @@ describe('run', () => {
     expect(spawner).toHaveBeenCalledWith(
       'gog',
       ['--json', '--color=never', '--no-input', 'sheets', 'get', 'id1', 'A1'],
-      expect.objectContaining({ env: process.env }),
+      expect.objectContaining({ env: expect.any(Object) }),
     );
   });
 
@@ -206,6 +206,93 @@ describe('run', () => {
       .rejects.toThrow('gog not found');
   });
 
+  it('wraps ENOENT spawn errors with an install-or-set-GOG_PATH hint', async () => {
+    const spawner = vi.fn(() => {
+      const proc = new EventEmitter() as ReturnType<Spawner>;
+      (proc as unknown as { stdout: EventEmitter; stderr: EventEmitter }).stdout = new EventEmitter();
+      (proc as unknown as { stdout: EventEmitter; stderr: EventEmitter }).stderr = new EventEmitter();
+      setTimeout(() => {
+        const err = new Error('spawn gog ENOENT') as NodeJS.ErrnoException;
+        err.code = 'ENOENT';
+        proc.emit('error', err);
+      }, 0);
+      return proc;
+    }) as unknown as Spawner;
+    await expect(run(['sheets', 'get', 'id', 'A1'], { spawner }))
+      .rejects.toThrow(/gog executable not found.*Install gogcli.*GOG_PATH/s);
+  });
+
+  it('augments child PATH with common gogcli install dirs', async () => {
+    const spawner = makeSpawner(0, '{}');
+    const originalHome = process.env.HOME;
+    const originalPath = process.env.PATH;
+    process.env.HOME = '/Users/test';
+    process.env.PATH = '/usr/bin:/bin';
+    try {
+      await run(['sheets', 'metadata', 'id1'], { spawner });
+      const passedEnv = (spawner as ReturnType<typeof vi.fn>).mock.calls[0][2].env as NodeJS.ProcessEnv;
+      const passedPath = passedEnv.PATH!;
+      expect(passedPath).toContain('/usr/bin');
+      expect(passedPath).toContain('/opt/homebrew/bin');
+      expect(passedPath).toContain('/usr/local/bin');
+      expect(passedPath).toContain('/Users/test/.local/bin');
+      expect(passedPath).toContain('/Users/test/go/bin');
+    } finally {
+      if (originalHome === undefined) delete process.env.HOME;
+      else process.env.HOME = originalHome;
+      if (originalPath === undefined) delete process.env.PATH;
+      else process.env.PATH = originalPath;
+    }
+  });
+
+  it('does not duplicate dirs that are already on PATH', async () => {
+    const spawner = makeSpawner(0, '{}');
+    const originalPath = process.env.PATH;
+    process.env.PATH = '/opt/homebrew/bin:/usr/bin';
+    try {
+      await run(['sheets', 'metadata', 'id1'], { spawner });
+      const passedEnv = (spawner as ReturnType<typeof vi.fn>).mock.calls[0][2].env as NodeJS.ProcessEnv;
+      const passedPath = passedEnv.PATH!;
+      const homebrewCount = passedPath.split(':').filter(d => d === '/opt/homebrew/bin').length;
+      expect(homebrewCount).toBe(1);
+    } finally {
+      if (originalPath === undefined) delete process.env.PATH;
+      else process.env.PATH = originalPath;
+    }
+  });
+
+  it('augments PATH even when HOME is unset', async () => {
+    const spawner = makeSpawner(0, '{}');
+    const originalHome = process.env.HOME;
+    const originalPath = process.env.PATH;
+    delete process.env.HOME;
+    process.env.PATH = '/usr/bin';
+    try {
+      await run(['sheets', 'metadata', 'id1'], { spawner });
+      const passedEnv = (spawner as ReturnType<typeof vi.fn>).mock.calls[0][2].env as NodeJS.ProcessEnv;
+      const passedPath = passedEnv.PATH!;
+      expect(passedPath).toContain('/opt/homebrew/bin');
+      expect(passedPath).not.toContain('.local/bin');
+    } finally {
+      if (originalHome !== undefined) process.env.HOME = originalHome;
+      if (originalPath === undefined) delete process.env.PATH;
+      else process.env.PATH = originalPath;
+    }
+  });
+
+  it('handles empty PATH gracefully', async () => {
+    const spawner = makeSpawner(0, '{}');
+    const originalPath = process.env.PATH;
+    delete process.env.PATH;
+    try {
+      await run(['sheets', 'metadata', 'id1'], { spawner });
+      const passedEnv = (spawner as ReturnType<typeof vi.fn>).mock.calls[0][2].env as NodeJS.ProcessEnv;
+      expect(passedEnv.PATH).toContain('/opt/homebrew/bin');
+    } finally {
+      if (originalPath !== undefined) process.env.PATH = originalPath;
+    }
+  });
+
   it('ignores close event if error event already settled the promise', async () => {
     const spawner = vi.fn(() => {
       const proc = new EventEmitter() as ReturnType<Spawner>;