gogcli-mcp 2.27.0 → 2.28.0

package/manifest.json

@@ -3,7 +3,7 @@
   "manifest_version": "0.3",
   "name": "gogcli-mcp",
   "display_name": "gogcli",
-  "version": "2.27.0",
+  "version": "2.28.0",
   "description": "Google Sheets (and more) for Claude via gogcli — read, write, and manage spreadsheets",
   "author": {
     "name": "Chris Hall",
@@ -452,6 +452,10 @@
       "name": "gog_chat_messages_list",
       "description": "Read messages in a space, with mentions and reaction summaries"
     },
+    {
+      "name": "gog_chat_messages_search",
+      "description": "Search Chat messages across every space and DM the account can see"
+    },
     {
       "name": "gog_chat_messages_send",
       "description": "Post a message to a Chat space (immediately visible; supports attachments)"

package/mint.yaml

@@ -86,7 +86,7 @@ dependencies:
   # nine pins too.
   - kind: github-release
     repo: openclaw/gogcli
-    tag: v0.38.1
+    tag: v0.39.0
     asset: "gogcli_*_linux_amd64.tar.gz"
     bin: [gog]
 state:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gogcli-mcp",
-  "version": "2.27.0",
+  "version": "2.28.0",
   "mcpName": "io.github.chrischall/gogcli-mcp",
   "description": "MCP server wrapping gogcli for Google service access",
   "author": "Claude Code (AI) <https://www.anthropic.com/claude>",
@@ -41,12 +41,12 @@
     "test:coverage": "vitest run --coverage"
   },
   "dependencies": {
-    "@chrischall/mcp-utils": "^0.15.0",
+    "@chrischall/mcp-utils": "^0.17.1",
     "@modelcontextprotocol/sdk": "^1.30.0",
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@types/node": "^26.2.0",
+    "@types/node": "^26.4.0",
     "@vitest/coverage-v8": "^4.1.8",
     "esbuild": "^0.28.2",
     "typescript": "^7.0.2",

package/server.json

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

package/src/runner.ts

@@ -217,7 +217,7 @@ const TIMEOUT_MS = 30_000;
 // so the requirement change is surfaced in the release notes (see
 // .github/release.yml). This is the single source of truth for the required
 // version; keep the README/CLAUDE.md mention in sync.
-export const MIN_GOG_VERSION = '0.38.1';
+export const MIN_GOG_VERSION = '0.39.0';
 
 // Interpret the GOG_READONLY kill-switch. `readEnvVar` already treats blank
 // values, 'undefined'/'null' sentinels, and unresolved .mcpb placeholders

package/src/tools/chat.ts

@@ -11,7 +11,7 @@ import type { GogArg } from '../runner.js';
 import { attachInlineParam, inlineAttachmentArgs } from '../attachments.js';
 
 // Google Chat (gog >= 0.38.0 for the mention/reaction metadata in
-// `messages list`; the rest of the surface is older).
+// `messages list`, >= 0.39.0 for `messages search`; the rest is older).
 //
 // TWO NAMING SYSTEMS MEET HERE, and mixing them is the mistake this module's
 // descriptions exist to prevent. Chat identifies everything by RESOURCE NAME —
@@ -135,6 +135,41 @@ export function registerChatTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
+  server.registerTool('gog_chat_messages_search', {
+    description:
+      'Search Chat messages ACROSS every space and DM the account can see (gog >= 0.39.0) — the tool to reach for when the '
+      + 'user asks "where did we discuss X" without naming a room, since gog_chat_messages_list needs a space up front. '
+      + 'The query is Google Chat filter syntax, so it takes plain keywords or filters like sender, space, date, mention, '
+      + 'unread, link and attachment. It is a search, NOT an export: Chat excludes some conversations (muted spaces among '
+      + 'them), so an empty result does not prove a message never existed. view="full" adds each hit\'s read state and space '
+      + 'mute setting; read state works on an ordinary chat grant, but the mute setting needs chat.users.spacesettings, '
+      + 'which gog\'s chat scope set does NOT request (re-auth with extraScopes to get it). Missing metadata is OMITTED '
+      + 'rather than defaulted, so an absent `read` means unknown while an explicit false means unread.' + workspaceOnlyNote,
+    annotations: { readOnlyHint: true },
+    inputSchema: {
+      query: z.string().describe('Google Chat filter-syntax query — keywords, or filters such as "from:alice@example.com budget"'),
+      order: z.enum(['create_time desc', 'relevance desc']).optional().describe(
+        'Sort order. NOTE the snake_case, which differs from gog_chat_messages_list\'s camelCase. "relevance desc" needs '
+        + 'Google Developer Preview access and errors without it',
+      ),
+      view: z.enum(['basic', 'full']).optional().describe(
+        'Result view (default "basic"). "full" also requests read state (covered by an ordinary chat grant) and space '
+        + 'mute setting (needs chat.users.spacesettings, which that grant does not include)',
+      ),
+      markup: z.enum(['chat', 'markdown']).optional().describe('Syntax to render each hit\'s formatted text in'),
+      ...paginationParams,
+      max: z.number().int().min(1).max(100).optional().describe('Max results per page (1-100; Chat search caps a page at 100)'),
+      account: accountParam,
+    },
+  }, async ({ query, order, view, markup, max, pageToken, page, all, account }) => {
+    const args = ['chat', 'messages', 'search', query];
+    if (order) args.push(`--order=${order}`);
+    if (view) args.push(`--view=${view}`);
+    if (markup) args.push(`--markup=${markup}`);
+    pushPaginationFlags(args, { max, pageToken, page, all });
+    return runOrDiagnose(args, { account });
+  });
+
   server.registerTool('gog_chat_messages_send', {
     description:
       'Post a message to a Chat space. THIS IS IMMEDIATELY VISIBLE TO EVERYONE IN THE SPACE and cannot be unsent through '

package/src/worker.ts

@@ -38,7 +38,7 @@ import { gogAuth, CONNECTOR_INSTRUCTIONS, type GogProps } from './connector-auth
 // 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.27.0'; // x-release-please-version
+const VERSION = '2.28.0'; // 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/server.test.ts

@@ -90,8 +90,14 @@ describe('published tool counts match the registrars', () => {
     expect(rows.length).toBeGreaterThan(0);
     const service = (row: string) => row.toLowerCase().replace(/[^a-z]/g, '');
     // Map a table label onto the gog_<service>_ prefix its tools carry.
+    // Only labels whose normalised form differs from their tool prefix belong
+    // here. "Apps Script" normalises to `appsscript` but the tools are
+    // `gog_appscript_`, so that mapping is load-bearing. A `discoveryapi` entry
+    // was not: both tables label the row "API", which normalises to `api` and
+    // already resolves — so it never matched and only implied a label that
+    // does not exist.
     const prefixFor: Record<string, string> = {
-      appsscript: 'appscript', discoveryapi: 'api',
+      appsscript: 'appscript',
     };
     for (const row of rows) {
       const stated = Number(row[2] ?? row[3]);

package/tests/tools/chat.test.ts

@@ -46,7 +46,7 @@ describe('gog_chat_spaces_list', () => {
       return undefined as never;
     });
     registerChatTools(server);
-    expect(configs.size).toBe(12);
+    expect(configs.size).toBe(13);
     for (const [name, config] of configs) {
       expect(config.description, name).toMatch(/consumer accounts|Workspace/i);
     }
@@ -132,6 +132,66 @@ describe('gog_chat_messages_list', () => {
   });
 });
 
+describe('gog_chat_messages_search', () => {
+  it('searches across every space the account can see', async () => {
+    vi.mocked(runner.run).mockResolvedValue('{"results":[]}');
+    const harness = await setupHandlers();
+    await harness.callTool('gog_chat_messages_search', { query: 'project decision' });
+    expect(runner.run).toHaveBeenCalledWith(
+      ['chat', 'messages', 'search', 'project decision'],
+      { account: undefined },
+    );
+  });
+
+  it('passes view, markup, order and pagination flags', async () => {
+    vi.mocked(runner.run).mockResolvedValue('{"results":[]}');
+    const harness = await setupHandlers();
+    await harness.callTool('gog_chat_messages_search', {
+      query: 'from:alice@example.com budget',
+      view: 'full',
+      markup: 'markdown',
+      order: 'create_time desc',
+      max: 100,
+      pageToken: 'tok',
+      all: true,
+    });
+    expect(runner.run).toHaveBeenCalledWith(
+      ['chat', 'messages', 'search', 'from:alice@example.com budget', '--order=create_time desc',
+        '--view=full', '--markup=markdown', '--max=100', '--page=tok', '--all'],
+      { account: undefined },
+    );
+  });
+
+  // Chat's own ordering vocabulary is snake_case here and camelCase in
+  // `messages list`; a model that copies the list tool's "createTime desc"
+  // would get a gog error at runtime, so the enum has to reject it here.
+  it('rejects an order value search does not accept', async () => {
+    const harness = await setupHandlers();
+    const result = await harness.callTool('gog_chat_messages_search', {
+      query: 'x', order: 'createTime desc',
+    });
+    expect(result.isError).toBe(true);
+    expect(runner.run).not.toHaveBeenCalled();
+  });
+
+  // Chat caps a search page at 100; `messages list` allows more. Letting the
+  // larger number through would surface as a gog error on a call the tool
+  // already had enough information to refuse.
+  it('rejects a page size above Chat search\'s cap of 100', async () => {
+    const harness = await setupHandlers();
+    const result = await harness.callTool('gog_chat_messages_search', { query: 'x', max: 250 });
+    expect(result.isError).toBe(true);
+    expect(runner.run).not.toHaveBeenCalled();
+  });
+
+  it('returns error text on failure', async () => {
+    vi.mocked(runner.run).mockRejectedValue(new Error('Search failed'));
+    const harness = await setupHandlers();
+    const result = await harness.callTool('gog_chat_messages_search', { query: 'x' });
+    expect(result.content[0].text).toBe('Error: Search failed');
+  });
+});
+
 describe('gog_chat_messages_send', () => {
   it('sends text to a space', async () => {
     vi.mocked(runner.run).mockResolvedValue('{}');