gogcli-mcp 2.19.0 → 2.19.1

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.19.0"
+    "version": "2.19.1"
   },
   "plugins": [
     {
@@ -15,7 +15,7 @@
       "displayName": "gogcli",
       "source": "./",
       "description": "Google Sheets (and more) for Claude via gogcli — read, write, and manage spreadsheets",
-      "version": "2.19.0",
+      "version": "2.19.1",
       "author": {
         "name": "Chris Hall"
       },

package/.claude-plugin/plugin.json

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

package/dist/index.js

@@ -33072,7 +33072,7 @@ function registerTasksTools(server) {
 }
 
 // src/server.ts
-var VERSION = true ? "2.19.0" : "0.0.0";
+var VERSION = true ? "2.19.1" : "0.0.0";
 var BASE_TOOL_REGISTRARS = [
   registerApiTools,
   registerAuthTools,

package/dist/lib.js

@@ -24978,7 +24978,7 @@ function registerTasksTools(server) {
 }
 
 // src/server.ts
-var VERSION = true ? "2.19.0" : "0.0.0";
+var VERSION = true ? "2.19.1" : "0.0.0";
 var BASE_TOOL_REGISTRARS = [
   registerApiTools,
   registerAuthTools,

package/manifest.json

@@ -3,7 +3,7 @@
   "manifest_version": "0.3",
   "name": "gogcli-mcp",
   "display_name": "gogcli",
-  "version": "2.19.0",
+  "version": "2.19.1",
   "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.19.0",
+  "version": "2.19.1",
   "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.19.0",
+  "version": "2.19.1",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "gogcli-mcp",
-      "version": "2.19.0",
+      "version": "2.19.1",
       "transport": {
         "type": "stdio"
       },

package/src/worker.ts

@@ -38,7 +38,7 @@ import { gogAuth, type GogProps } from './connector-auth.js';
 // connector with all ~360 tools at once. Add whichever paths you want as separate
 // connectors in claude.ai (each authorizes with the same connector key).
 
-const VERSION = '2.19.0'; // x-release-please-version
+const VERSION = '2.19.1'; // x-release-please-version
 
 // Build an McpAgent subclass whose init() registers `registrars` onto its server,
 // each handler wrapped in the ALS scope carrying the per-session Fly executor.

package/tests/remote-runner.test.ts

@@ -1,3 +1,6 @@
+import { readdirSync, readFileSync } from 'node:fs';
+import { join } from 'node:path';
+import { fileURLToPath } from 'node:url';
 import { describe, it, expect, vi, afterEach } from 'vitest';
 import { useRemoteGogRunner } from '../src/remote-runner.js';
 import { runExecutor } from '../src/runner.js';
@@ -51,3 +54,37 @@ describe('useRemoteGogRunner', () => {
     expect((init.headers as Record<string, string>).Authorization).toBe('Bearer secret');
   });
 });
+
+/**
+ * The seam is opt-in PER BIN — `useRemoteGogRunner()` is a call in each
+ * package's `index.ts`, and a package that omits it spawns the binary no
+ * matter what the host sets. That shipped: 2.19.0 wired base, sheets, docs,
+ * drive and gmail, and left calendar, classroom, contacts and slides behind,
+ * so those four answered every tool call on mcp-host with "gog executable not
+ * found" — a host with no binary and no way to ask for the backend.
+ *
+ * Nothing caught it because each package's suite covers its TOOLS, and
+ * `src/index.ts` is excluded from the coverage gate in every package (it is
+ * the bin: it boots a server and cannot be imported under test). So this
+ * asserts on the source text instead, across the whole workspace — the one
+ * check that scales to the next sub-package, which will otherwise be added by
+ * copying an index.ts that predates the seam.
+ */
+describe('every package bin', () => {
+  const packagesDir = fileURLToPath(new URL('../../', import.meta.url));
+  const bins = readdirSync(packagesDir)
+    .filter((name) => name.startsWith('gogcli-mcp'))
+    .map((name) => [name, join(packagesDir, name, 'src', 'index.ts')] as const);
+
+  it('finds every package (guards the glob itself against silently matching nothing)', () => {
+    expect(bins.length).toBeGreaterThanOrEqual(9);
+  });
+
+  it.each(bins)('%s installs the remote executor before starting the server', (_name, path) => {
+    const source = readFileSync(path, 'utf8');
+    expect(source).toContain('useRemoteGogRunner()');
+    // Order matters as much as presence: `runMcp` starts serving, so a call
+    // placed after it could be beaten by a tool call and fall back to spawning.
+    expect(source.indexOf('useRemoteGogRunner()')).toBeLessThan(source.indexOf('runMcp({'));
+  });
+});