gogcli-mcp 2.0.6 → 2.0.7

package/.claude-plugin/marketplace.json

@@ -6,16 +6,16 @@
     "email": "chris.c.hall@gmail.com"
   },
   "metadata": {
-    "description": "Google Sheets (and more) for Claude via gogcli \u2014 read, write, and manage spreadsheets",
-    "version": "2.0.3"
+    "description": "Google Sheets (and more) for Claude via gogcli — read, write, and manage spreadsheets",
+    "version": "2.0.7"
   },
   "plugins": [
     {
       "name": "gogcli-mcp",
       "displayName": "gogcli",
       "source": "./",
-      "description": "Google Sheets (and more) for Claude via gogcli \u2014 read, write, and manage spreadsheets",
-      "version": "2.0.3",
+      "description": "Google Sheets (and more) for Claude via gogcli — read, write, and manage spreadsheets",
+      "version": "2.0.7",
       "author": {
         "name": "Chris Hall"
       },

package/.claude-plugin/plugin.json

@@ -1,8 +1,8 @@
 {
   "name": "gogcli-mcp",
   "displayName": "gogcli",
-  "version": "2.0.3",
-  "description": "Google Sheets (and more) for Claude via gogcli \u2014 read, write, and manage spreadsheets",
+  "version": "2.0.7",
+  "description": "Google Sheets (and more) for Claude via gogcli — read, write, and manage spreadsheets",
   "author": {
     "name": "Chris Hall",
     "email": "chris.c.hall@gmail.com"

package/dist/index.js

@@ -30864,6 +30864,11 @@ var EMPTY_COMPLETION_RESULT = {
 // src/runner.ts
 import { spawn } from "node:child_process";
 var TIMEOUT_MS = 3e4;
+function envOrUndefined(key) {
+  const value = process.env[key];
+  if (!value || value.startsWith("${")) return void 0;
+  return value;
+}
 function formatTimeout(ms) {
   const seconds = Math.round(ms / 1e3);
   if (seconds >= 60) {
@@ -30874,7 +30879,7 @@ function formatTimeout(ms) {
 }
 async function run(args, options = {}) {
   const { account, spawner = spawn, interactive = false, timeout } = options;
-  const effectiveAccount = account ?? process.env.GOG_ACCOUNT;
+  const effectiveAccount = account ?? envOrUndefined("GOG_ACCOUNT");
   const fullArgs = ["--json", "--color=never"];
   if (!interactive) {
     fullArgs.push("--no-input");
@@ -30886,7 +30891,7 @@ 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(process.env.GOG_PATH ?? "gog", fullArgs, { env: cleanEnv });
+    const child = spawner(envOrUndefined("GOG_PATH") ?? "gog", fullArgs, { env: cleanEnv });
     const stdoutChunks = [];
     const stderrChunks = [];
     let settled = false;
@@ -32118,7 +32123,7 @@ function registerTasksTools(server2) {
 }
 
 // src/server.ts
-var VERSION = true ? "2.0.5" : "0.0.0";
+var VERSION = true ? "2.0.7" : "0.0.0";
 function createServer(options) {
   return new McpServer({
     name: options?.name ?? "gogcli",

package/dist/lib.js

@@ -30771,6 +30771,11 @@ var EMPTY_COMPLETION_RESULT = {
 // src/runner.ts
 import { spawn } from "node:child_process";
 var TIMEOUT_MS = 3e4;
+function envOrUndefined(key) {
+  const value = process.env[key];
+  if (!value || value.startsWith("${")) return void 0;
+  return value;
+}
 function formatTimeout(ms) {
   const seconds = Math.round(ms / 1e3);
   if (seconds >= 60) {
@@ -30781,7 +30786,7 @@ function formatTimeout(ms) {
 }
 async function run(args, options = {}) {
   const { account, spawner = spawn, interactive = false, timeout } = options;
-  const effectiveAccount = account ?? process.env.GOG_ACCOUNT;
+  const effectiveAccount = account ?? envOrUndefined("GOG_ACCOUNT");
   const fullArgs = ["--json", "--color=never"];
   if (!interactive) {
     fullArgs.push("--no-input");
@@ -30793,7 +30798,7 @@ 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(process.env.GOG_PATH ?? "gog", fullArgs, { env: cleanEnv });
+    const child = spawner(envOrUndefined("GOG_PATH") ?? "gog", fullArgs, { env: cleanEnv });
     const stdoutChunks = [];
     const stderrChunks = [];
     let settled = false;
@@ -32025,7 +32030,7 @@ function registerTasksTools(server) {
 }
 
 // src/server.ts
-var VERSION = true ? "2.0.5" : "0.0.0";
+var VERSION = true ? "2.0.7" : "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.6",
+  "version": "2.0.7",
   "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.6",
+  "version": "2.0.7",
   "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

@@ -1,18 +1,18 @@
 {
   "$schema": "https://static.modelcontextprotocol.io/schemas/2025-12-11/server.schema.json",
   "name": "io.github.chrischall/gogcli-mcp",
-  "description": "Google Sheets (and more) for Claude via gogcli \u2014 read, write, and manage spreadsheets",
+  "description": "Google Sheets (and more) for Claude via gogcli — read, write, and manage spreadsheets",
   "repository": {
     "url": "https://github.com/chrischall/gogcli-mcp",
     "source": "github",
     "subfolder": "packages/gogcli-mcp"
   },
-  "version": "2.0.3",
+  "version": "2.0.7",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "gogcli-mcp",
-      "version": "2.0.3",
+      "version": "2.0.7",
       "transport": {
         "type": "stdio"
       },

package/src/runner.ts

@@ -16,6 +16,16 @@ export interface RunOptions {
 
 const TIMEOUT_MS = 30_000;
 
+// Treat unresolved .mcpb placeholders ("${user_config.gog_path}") and empty
+// strings the same as an unset env var. When an optional .mcpb user_config
+// field is left blank, some clients pass the literal placeholder text through
+// to the spawned server instead of substituting "" or omitting the key.
+function envOrUndefined(key: string): string | undefined {
+  const value = process.env[key];
+  if (!value || value.startsWith('${')) return undefined;
+  return value;
+}
+
 function formatTimeout(ms: number): string {
   const seconds = Math.round(ms / 1000);
   if (seconds >= 60) {
@@ -28,7 +38,7 @@ function formatTimeout(ms: number): string {
 export async function run(args: string[], options: RunOptions = {}): Promise<string> {
   const { account, spawner = spawn as unknown as Spawner, interactive = false, timeout } = options;
 
-  const effectiveAccount = account ?? process.env.GOG_ACCOUNT;
+  const effectiveAccount = account ?? envOrUndefined('GOG_ACCOUNT');
 
   const fullArgs = ['--json', '--color=never'];
   if (!interactive) {
@@ -45,10 +55,7 @@ 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;
-    // Use `||` (not `??`) so an empty-string GOG_PATH — common when the .mcpb
-    // user_config "gog_path" is left blank and substituted as "" — falls back
-    // to PATH lookup instead of trying to spawn an empty executable name.
-    const child = spawner(process.env.GOG_PATH || 'gog', fullArgs, { env: cleanEnv });
+    const child = spawner(envOrUndefined('GOG_PATH') ?? 'gog', fullArgs, { env: cleanEnv });
     const stdoutChunks: Buffer[] = [];
     const stderrChunks: Buffer[] = [];
     let settled = false;

package/tests/runner.test.ts

@@ -143,6 +143,39 @@ describe('run', () => {
     }
   });
 
+  it('falls back to "gog" on PATH when GOG_PATH is an unresolved .mcpb placeholder', async () => {
+    const spawner = makeSpawner(0, '{}');
+    const originalEnv = process.env.GOG_PATH;
+    process.env.GOG_PATH = '${user_config.gog_path}';
+    try {
+      await run(['sheets', 'metadata', 'id1'], { spawner });
+      expect(spawner).toHaveBeenCalledWith('gog', expect.any(Array), expect.any(Object));
+    } finally {
+      if (originalEnv === undefined) {
+        delete process.env.GOG_PATH;
+      } else {
+        process.env.GOG_PATH = originalEnv;
+      }
+    }
+  });
+
+  it('omits --account when GOG_ACCOUNT is an unresolved .mcpb placeholder', async () => {
+    const spawner = makeSpawner(0, '{}');
+    const originalEnv = process.env.GOG_ACCOUNT;
+    process.env.GOG_ACCOUNT = '${user_config.gog_account}';
+    try {
+      await run(['sheets', 'metadata', 'id1'], { spawner });
+      const callArgs = (spawner as ReturnType<typeof vi.fn>).mock.calls[0][1] as string[];
+      expect(callArgs).not.toContain('--account');
+    } finally {
+      if (originalEnv === undefined) {
+        delete process.env.GOG_ACCOUNT;
+      } else {
+        process.env.GOG_ACCOUNT = originalEnv;
+      }
+    }
+  });
+
   it('returns stdout on exit code 0', async () => {
     const spawner = makeSpawner(0, '{"values":[["hello"]]}');
     const result = await run(['sheets', 'get', 'id1', 'A1'], { spawner });

package/dist/index.d.ts

@@ -1,3 +0,0 @@
-#!/usr/bin/env node
-export {};
-//# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":""}

package/dist/lib.d.ts

@@ -1,6 +0,0 @@
-export { createBaseServer, VERSION } from './server.js';
-export { run } from './runner.js';
-export type { RunOptions, Spawner } from './runner.js';
-export { accountParam, runOrDiagnose, toText, toError } from './tools/utils.js';
-export type { ToolResult } from './tools/utils.js';
-//# sourceMappingURL=lib.d.ts.map

package/dist/lib.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"lib.d.ts","sourceRoot":"","sources":["../src/lib.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,gBAAgB,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AACxD,OAAO,EAAE,GAAG,EAAE,MAAM,aAAa,CAAC;AAClC,YAAY,EAAE,UAAU,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AACvD,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,kBAAkB,CAAC;AAChF,YAAY,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAC"}

package/dist/runner.d.ts

@@ -1,12 +0,0 @@
-import type { ChildProcess } from 'node:child_process';
-export type Spawner = (command: string, args: string[], options: {
-    env: NodeJS.ProcessEnv;
-}) => ChildProcess;
-export interface RunOptions {
-    account?: string;
-    spawner?: Spawner;
-    interactive?: boolean;
-    timeout?: number;
-}
-export declare function run(args: string[], options?: RunOptions): Promise<string>;
-//# sourceMappingURL=runner.d.ts.map

package/dist/runner.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../src/runner.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAEvD,MAAM,MAAM,OAAO,GAAG,CACpB,OAAO,EAAE,MAAM,EACf,IAAI,EAAE,MAAM,EAAE,EACd,OAAO,EAAE;IAAE,GAAG,EAAE,MAAM,CAAC,UAAU,CAAA;CAAE,KAChC,YAAY,CAAC;AAElB,MAAM,WAAW,UAAU;IACzB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAaD,wBAAsB,GAAG,CAAC,IAAI,EAAE,MAAM,EAAE,EAAE,OAAO,GAAE,UAAe,GAAG,OAAO,CAAC,MAAM,CAAC,CA0DnF"}

package/dist/server.d.ts

@@ -1,7 +0,0 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-export declare const VERSION: string;
-export declare function createBaseServer(options?: {
-    name?: string;
-    version?: string;
-}): McpServer;
-//# sourceMappingURL=server.d.ts.map

package/dist/server.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"server.d.ts","sourceRoot":"","sources":["../src/server.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAYpE,eAAO,MAAM,OAAO,QAAmE,CAAC;AAExF,wBAAgB,gBAAgB,CAAC,OAAO,CAAC,EAAE;IAAE,IAAI,CAAC,EAAE,MAAM,CAAC;IAAC,OAAO,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,SAAS,CAgBzF"}

package/dist/tools/auth.d.ts

@@ -1,3 +0,0 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-export declare function registerAuthTools(server: McpServer): void;
-//# sourceMappingURL=auth.d.ts.map

package/dist/tools/auth.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"auth.d.ts","sourceRoot":"","sources":["../../src/tools/auth.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAKpE,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,SAAS,GAAG,IAAI,CAwEzD"}

package/dist/tools/calendar.d.ts

@@ -1,3 +0,0 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-export declare function registerCalendarTools(server: McpServer): void;
-//# sourceMappingURL=calendar.d.ts.map

package/dist/tools/calendar.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"calendar.d.ts","sourceRoot":"","sources":["../../src/tools/calendar.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAIpE,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,SAAS,GAAG,IAAI,CA2H7D"}

package/dist/tools/contacts.d.ts

@@ -1,3 +0,0 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-export declare function registerContactsTools(server: McpServer): void;
-//# sourceMappingURL=contacts.d.ts.map

package/dist/tools/contacts.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"contacts.d.ts","sourceRoot":"","sources":["../../src/tools/contacts.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAIpE,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,SAAS,GAAG,IAAI,CAkE7D"}

package/dist/tools/docs.d.ts

@@ -1,3 +0,0 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-export declare function registerDocsTools(server: McpServer): void;
-//# sourceMappingURL=docs.d.ts.map

package/dist/tools/docs.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"docs.d.ts","sourceRoot":"","sources":["../../src/tools/docs.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAIpE,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,SAAS,GAAG,IAAI,CAoGzD"}

package/dist/tools/drive.d.ts

@@ -1,3 +0,0 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-export declare function registerDriveTools(server: McpServer): void;
-//# sourceMappingURL=drive.d.ts.map

package/dist/tools/drive.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"drive.d.ts","sourceRoot":"","sources":["../../src/tools/drive.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAIpE,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,SAAS,GAAG,IAAI,CAgH1D"}

package/dist/tools/gmail.d.ts

@@ -1,3 +0,0 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-export declare function registerGmailTools(server: McpServer): void;
-//# sourceMappingURL=gmail.d.ts.map

package/dist/tools/gmail.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"gmail.d.ts","sourceRoot":"","sources":["../../src/tools/gmail.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAIpE,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,SAAS,GAAG,IAAI,CA8D1D"}

package/dist/tools/sheets.d.ts

@@ -1,3 +0,0 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-export declare function registerSheetsTools(server: McpServer): void;
-//# sourceMappingURL=sheets.d.ts.map

package/dist/tools/sheets.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"sheets.d.ts","sourceRoot":"","sources":["../../src/tools/sheets.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAIpE,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,SAAS,GAAG,IAAI,CAuG3D"}

package/dist/tools/tasks.d.ts

@@ -1,3 +0,0 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-export declare function registerTasksTools(server: McpServer): void;
-//# sourceMappingURL=tasks.d.ts.map

package/dist/tools/tasks.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"tasks.d.ts","sourceRoot":"","sources":["../../src/tools/tasks.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAIpE,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,SAAS,GAAG,IAAI,CAsF1D"}

package/dist/tools/utils.d.ts

@@ -1,14 +0,0 @@
-import { z } from 'zod';
-export type ToolResult = {
-    content: [{
-        type: 'text';
-        text: string;
-    }];
-};
-export declare const accountParam: z.ZodOptional<z.ZodString>;
-export declare function toText(output: string): ToolResult;
-export declare function toError(err: unknown): ToolResult;
-export declare function runOrDiagnose(args: string[], options: {
-    account?: string;
-}): Promise<ToolResult>;
-//# sourceMappingURL=utils.d.ts.map

package/dist/tools/utils.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../src/tools/utils.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAGxB,MAAM,MAAM,UAAU,GAAG;IAAE,OAAO,EAAE,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC,CAAA;CAAE,CAAC;AAEvE,eAAO,MAAM,YAAY,4BAExB,CAAC;AAEF,wBAAgB,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,UAAU,CAEjD;AAED,wBAAgB,OAAO,CAAC,GAAG,EAAE,OAAO,GAAG,UAAU,CAEhD;AAQD,wBAAsB,aAAa,CACjC,IAAI,EAAE,MAAM,EAAE,EACd,OAAO,EAAE;IAAE,OAAO,CAAC,EAAE,MAAM,CAAA;CAAE,GAC5B,OAAO,CAAC,UAAU,CAAC,CAerB"}