gogcli-mcp-sheets 2.0.8 → 2.0.9

package/.claude-plugin/marketplace.json

@@ -7,7 +7,7 @@
   },
   "metadata": {
     "description": "Extended Google Sheets for Claude via gogcli — auth + full Sheets support",
-    "version": "2.0.8"
+    "version": "2.0.9"
   },
   "plugins": [
     {
@@ -15,7 +15,7 @@
       "displayName": "gogcli (Sheets)",
       "source": "./",
       "description": "Extended Google Sheets for Claude via gogcli — auth + full Sheets support",
-      "version": "2.0.8",
+      "version": "2.0.9",
       "author": {
         "name": "Chris Hall"
       },

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "gogcli-mcp-sheets",
   "displayName": "gogcli (Sheets)",
-  "version": "2.0.8",
+  "version": "2.0.9",
   "description": "Extended Google Sheets for Claude via gogcli — auth + full Sheets support",
   "author": {
     "name": "Chris Hall",

package/dist/index.js

@@ -3105,6 +3105,9 @@ var require_utils = __commonJS({
     "use strict";
     var isUUID = RegExp.prototype.test.bind(/^[\da-f]{8}-[\da-f]{4}-[\da-f]{4}-[\da-f]{4}-[\da-f]{12}$/iu);
     var isIPv4 = RegExp.prototype.test.bind(/^(?:(?:25[0-5]|2[0-4]\d|1\d{2}|[1-9]\d|\d)\.){3}(?:25[0-5]|2[0-4]\d|1\d{2}|[1-9]\d|\d)$/u);
+    var isHexPair = RegExp.prototype.test.bind(/^[\da-f]{2}$/iu);
+    var isUnreserved = RegExp.prototype.test.bind(/^[\da-z\-._~]$/iu);
+    var isPathCharacter = RegExp.prototype.test.bind(/^[\da-z\-._~!$&'()*+,;=:@/]$/iu);
     function stringArrayToHexStripped(input) {
       let acc = "";
       let code = 0;
@@ -3297,27 +3300,77 @@ var require_utils = __commonJS({
       }
       return output.join("");
     }
-    function normalizeComponentEncoding(component, esc2) {
-      const func = esc2 !== true ? escape : unescape;
-      if (component.scheme !== void 0) {
-        component.scheme = func(component.scheme);
-      }
-      if (component.userinfo !== void 0) {
-        component.userinfo = func(component.userinfo);
-      }
-      if (component.host !== void 0) {
-        component.host = func(component.host);
+    var HOST_DELIMS = { "@": "%40", "/": "%2F", "?": "%3F", "#": "%23", ":": "%3A" };
+    var HOST_DELIM_RE = /[@/?#:]/g;
+    var HOST_DELIM_NO_COLON_RE = /[@/?#]/g;
+    function reescapeHostDelimiters(host, isIP) {
+      const re = isIP ? HOST_DELIM_NO_COLON_RE : HOST_DELIM_RE;
+      re.lastIndex = 0;
+      return host.replace(re, (ch) => HOST_DELIMS[ch]);
+    }
+    function normalizePercentEncoding(input, decodeUnreserved = false) {
+      if (input.indexOf("%") === -1) {
+        return input;
       }
-      if (component.path !== void 0) {
-        component.path = func(component.path);
+      let output = "";
+      for (let i = 0; i < input.length; i++) {
+        if (input[i] === "%" && i + 2 < input.length) {
+          const hex3 = input.slice(i + 1, i + 3);
+          if (isHexPair(hex3)) {
+            const normalizedHex = hex3.toUpperCase();
+            const decoded = String.fromCharCode(parseInt(normalizedHex, 16));
+            if (decodeUnreserved && isUnreserved(decoded)) {
+              output += decoded;
+            } else {
+              output += "%" + normalizedHex;
+            }
+            i += 2;
+            continue;
+          }
+        }
+        output += input[i];
       }
-      if (component.query !== void 0) {
-        component.query = func(component.query);
+      return output;
+    }
+    function normalizePathEncoding(input) {
+      let output = "";
+      for (let i = 0; i < input.length; i++) {
+        if (input[i] === "%" && i + 2 < input.length) {
+          const hex3 = input.slice(i + 1, i + 3);
+          if (isHexPair(hex3)) {
+            const normalizedHex = hex3.toUpperCase();
+            const decoded = String.fromCharCode(parseInt(normalizedHex, 16));
+            if (decoded !== "." && isUnreserved(decoded)) {
+              output += decoded;
+            } else {
+              output += "%" + normalizedHex;
+            }
+            i += 2;
+            continue;
+          }
+        }
+        if (isPathCharacter(input[i])) {
+          output += input[i];
+        } else {
+          output += escape(input[i]);
+        }
       }
-      if (component.fragment !== void 0) {
-        component.fragment = func(component.fragment);
+      return output;
+    }
+    function escapePreservingEscapes(input) {
+      let output = "";
+      for (let i = 0; i < input.length; i++) {
+        if (input[i] === "%" && i + 2 < input.length) {
+          const hex3 = input.slice(i + 1, i + 3);
+          if (isHexPair(hex3)) {
+            output += "%" + hex3.toUpperCase();
+            i += 2;
+            continue;
+          }
+        }
+        output += escape(input[i]);
       }
-      return component;
+      return output;
     }
     function recomposeAuthority(component) {
       const uriTokens = [];
@@ -3332,7 +3385,7 @@ var require_utils = __commonJS({
           if (ipV6res.isIPV6 === true) {
             host = `[${ipV6res.escapedHost}]`;
           } else {
-            host = component.host;
+            host = reescapeHostDelimiters(host, false);
           }
         }
         uriTokens.push(host);
@@ -3346,7 +3399,10 @@ var require_utils = __commonJS({
     module.exports = {
       nonSimpleDomain,
       recomposeAuthority,
-      normalizeComponentEncoding,
+      reescapeHostDelimiters,
+      normalizePercentEncoding,
+      normalizePathEncoding,
+      escapePreservingEscapes,
       removeDotSegments,
       isIPv4,
       isUUID,
@@ -3570,12 +3626,12 @@ var require_schemes = __commonJS({
 var require_fast_uri = __commonJS({
   "../../node_modules/fast-uri/index.js"(exports, module) {
     "use strict";
-    var { normalizeIPv6, removeDotSegments, recomposeAuthority, normalizeComponentEncoding, isIPv4, nonSimpleDomain } = require_utils();
+    var { normalizeIPv6, removeDotSegments, recomposeAuthority, normalizePercentEncoding, normalizePathEncoding, escapePreservingEscapes, reescapeHostDelimiters, isIPv4, nonSimpleDomain } = require_utils();
     var { SCHEMES, getSchemeHandler } = require_schemes();
     function normalize(uri, options) {
       if (typeof uri === "string") {
         uri = /** @type {T} */
-        serialize(parse3(uri, options), options);
+        normalizeString(uri, options);
       } else if (typeof uri === "object") {
         uri = /** @type {T} */
         parse3(serialize(uri, options), options);
@@ -3642,19 +3698,9 @@ var require_fast_uri = __commonJS({
       return target;
     }
     function equal(uriA, uriB, options) {
-      if (typeof uriA === "string") {
-        uriA = unescape(uriA);
-        uriA = serialize(normalizeComponentEncoding(parse3(uriA, options), true), { ...options, skipEscape: true });
-      } else if (typeof uriA === "object") {
-        uriA = serialize(normalizeComponentEncoding(uriA, true), { ...options, skipEscape: true });
-      }
-      if (typeof uriB === "string") {
-        uriB = unescape(uriB);
-        uriB = serialize(normalizeComponentEncoding(parse3(uriB, options), true), { ...options, skipEscape: true });
-      } else if (typeof uriB === "object") {
-        uriB = serialize(normalizeComponentEncoding(uriB, true), { ...options, skipEscape: true });
-      }
-      return uriA.toLowerCase() === uriB.toLowerCase();
+      const normalizedA = normalizeComparableURI(uriA, options);
+      const normalizedB = normalizeComparableURI(uriB, options);
+      return normalizedA !== void 0 && normalizedB !== void 0 && normalizedA.toLowerCase() === normalizedB.toLowerCase();
     }
     function serialize(cmpts, opts) {
       const component = {
@@ -3679,12 +3725,12 @@ var require_fast_uri = __commonJS({
       if (schemeHandler && schemeHandler.serialize) schemeHandler.serialize(component, options);
       if (component.path !== void 0) {
         if (!options.skipEscape) {
-          component.path = escape(component.path);
+          component.path = escapePreservingEscapes(component.path);
           if (component.scheme !== void 0) {
             component.path = component.path.split("%3A").join(":");
           }
         } else {
-          component.path = unescape(component.path);
+          component.path = normalizePercentEncoding(component.path);
         }
       }
       if (options.reference !== "suffix" && component.scheme) {
@@ -3719,7 +3765,16 @@ var require_fast_uri = __commonJS({
       return uriTokens.join("");
     }
     var URI_PARSE = /^(?:([^#/:?]+):)?(?:\/\/((?:([^#/?@]*)@)?(\[[^#/?\]]+\]|[^#/:?]*)(?::(\d*))?))?([^#?]*)(?:\?([^#]*))?(?:#((?:.|[\n\r])*))?/u;
-    function parse3(uri, opts) {
+    function getParseError(parsed, matches) {
+      if (matches[2] !== void 0 && parsed.path && parsed.path[0] !== "/") {
+        return 'URI path must start with "/" when authority is present.';
+      }
+      if (typeof parsed.port === "number" && (parsed.port < 0 || parsed.port > 65535)) {
+        return "URI port is malformed.";
+      }
+      return void 0;
+    }
+    function parseWithStatus(uri, opts) {
       const options = Object.assign({}, opts);
       const parsed = {
         scheme: void 0,
@@ -3730,6 +3785,7 @@ var require_fast_uri = __commonJS({
         query: void 0,
         fragment: void 0
       };
+      let malformedAuthorityOrPort = false;
       let isIP = false;
       if (options.reference === "suffix") {
         if (options.scheme) {
@@ -3750,6 +3806,11 @@ var require_fast_uri = __commonJS({
         if (isNaN(parsed.port)) {
           parsed.port = matches[5];
         }
+        const parseError = getParseError(parsed, matches);
+        if (parseError !== void 0) {
+          parsed.error = parsed.error || parseError;
+          malformedAuthorityOrPort = true;
+        }
         if (parsed.host) {
           const ipv4result = isIPv4(parsed.host);
           if (ipv4result === false) {
@@ -3788,14 +3849,18 @@ var require_fast_uri = __commonJS({
               parsed.scheme = unescape(parsed.scheme);
             }
             if (parsed.host !== void 0) {
-              parsed.host = unescape(parsed.host);
+              parsed.host = reescapeHostDelimiters(unescape(parsed.host), isIP);
             }
           }
           if (parsed.path) {
-            parsed.path = escape(unescape(parsed.path));
+            parsed.path = normalizePathEncoding(parsed.path);
           }
           if (parsed.fragment) {
-            parsed.fragment = encodeURI(decodeURIComponent(parsed.fragment));
+            try {
+              parsed.fragment = encodeURI(decodeURIComponent(parsed.fragment));
+            } catch {
+              parsed.error = parsed.error || "URI malformed";
+            }
           }
         }
         if (schemeHandler && schemeHandler.parse) {
@@ -3804,7 +3869,29 @@ var require_fast_uri = __commonJS({
       } else {
         parsed.error = parsed.error || "URI can not be parsed.";
       }
-      return parsed;
+      return { parsed, malformedAuthorityOrPort };
+    }
+    function parse3(uri, opts) {
+      return parseWithStatus(uri, opts).parsed;
+    }
+    function normalizeString(uri, opts) {
+      return normalizeStringWithStatus(uri, opts).normalized;
+    }
+    function normalizeStringWithStatus(uri, opts) {
+      const { parsed, malformedAuthorityOrPort } = parseWithStatus(uri, opts);
+      return {
+        normalized: malformedAuthorityOrPort ? uri : serialize(parsed, opts),
+        malformedAuthorityOrPort
+      };
+    }
+    function normalizeComparableURI(uri, opts) {
+      if (typeof uri === "string") {
+        const { normalized, malformedAuthorityOrPort } = normalizeStringWithStatus(uri, opts);
+        return malformedAuthorityOrPort ? void 0 : normalized;
+      }
+      if (typeof uri === "object") {
+        return serialize(uri, opts);
+      }
     }
     var fastUri = {
       SCHEMES,
@@ -9886,14 +9973,14 @@ var $ZodObjectJIT = /* @__PURE__ */ $constructor("$ZodObjectJIT", (inst, def) =>
       return `shape[${k}]._zod.run({ value: input[${k}], issues: [] }, ctx)`;
     };
     doc.write(`const input = payload.value;`);
-    const ids = /* @__PURE__ */ Object.create(null);
+    const ids2 = /* @__PURE__ */ Object.create(null);
     let counter = 0;
     for (const key of normalized.keys) {
-      ids[key] = `key_${counter++}`;
+      ids2[key] = `key_${counter++}`;
     }
     doc.write(`const newResult = {};`);
     for (const key of normalized.keys) {
-      const id = ids[key];
+      const id = ids2[key];
       const k = esc(key);
       const schema = shape[key];
       const isOptionalIn = schema?._zod?.optin === "optional";
@@ -30870,6 +30957,32 @@ function envOrUndefined(key) {
   if (!value || value.startsWith("${")) return void 0;
   return value;
 }
+function sanitizedEnv() {
+  const result = {};
+  for (const [key, value] of Object.entries(process.env)) {
+    if (key === "GOG_ACCESS_TOKEN") continue;
+    if (key === "GOOGLE_APPLICATION_CREDENTIALS") continue;
+    if (/(_TOKEN|_SECRET|_API_KEY|_PRIVATE_KEY)$/.test(key)) continue;
+    result[key] = value;
+  }
+  return result;
+}
+var TOKEN_PATTERNS = [
+  /Bearer\s+[A-Za-z0-9._\-+/=]+/gi,
+  /ya29\.[A-Za-z0-9._\-]+/g,
+  // OAuth2 access tokens
+  /1\/\/[A-Za-z0-9._\-]+/g,
+  // OAuth2 refresh tokens
+  /AIza[A-Za-z0-9_\-]{35}/g
+  // Google API keys
+];
+function redactSecrets(text) {
+  let redacted = text;
+  for (const re of TOKEN_PATTERNS) {
+    redacted = redacted.replace(re, "[REDACTED]");
+  }
+  return redacted;
+}
 function augmentedPath() {
   const home = process.env.HOME;
   const candidates = [
@@ -30912,8 +31025,7 @@ async function run(args, options = {}) {
   fullArgs.push(...args);
   const effectiveTimeout = timeout ?? TIMEOUT_MS;
   return new Promise((resolve, reject) => {
-    const { GOG_ACCESS_TOKEN: _, ...cleanEnv } = process.env;
-    const childEnv = { ...cleanEnv, PATH: augmentedPath() };
+    const childEnv = { ...sanitizedEnv(), PATH: augmentedPath() };
     const child = spawner(envOrUndefined("GOG_PATH") ?? "gog", fullArgs, { env: childEnv });
     const stdoutChunks = [];
     const stderrChunks = [];
@@ -30942,7 +31054,7 @@ async function run(args, options = {}) {
           resolve(stdout);
         }
       } else {
-        reject(new Error(stderr || `gog exited with code ${code}`));
+        reject(new Error(redactSecrets(stderr || `gog exited with code ${code}`)));
       }
     });
     child.on("error", (err) => {
@@ -30964,6 +31076,55 @@ async function run(args, options = {}) {
 var accountParam = external_exports.string().optional().describe(
   "Google account email to use (overrides GOG_ACCOUNT env var)"
 );
+var ids = {
+  course: external_exports.string().describe("Course ID"),
+  coursework: external_exports.string().describe("Coursework ID"),
+  submission: external_exports.string().describe("Submission ID"),
+  announcement: external_exports.string().describe("Announcement ID"),
+  topic: external_exports.string().describe("Topic ID"),
+  invitation: external_exports.string().describe("Invitation ID"),
+  spreadsheet: external_exports.string().describe("Spreadsheet ID (from the URL)"),
+  doc: external_exports.string().describe("Doc ID (from the URL)"),
+  presentation: external_exports.string().describe("Presentation ID"),
+  slide: external_exports.string().describe("Slide ID"),
+  file: external_exports.string().describe("File ID"),
+  message: external_exports.string().describe("Message ID"),
+  thread: external_exports.string().describe("Thread ID"),
+  draft: external_exports.string().describe("Draft ID"),
+  label: external_exports.string().describe("Label ID or name"),
+  attachment: external_exports.string().describe("Attachment ID"),
+  comment: external_exports.string().describe("Comment ID"),
+  meetingCode: external_exports.string().describe("Meeting code (e.g. abc-defg-hij)"),
+  permission: external_exports.string().describe("Permission ID"),
+  user: external_exports.string().describe("User ID"),
+  // People API uses fully-qualified resource names ("people/c123") not bare IDs.
+  person: external_exports.string().describe("Person resource name (people/...) or email")
+};
+var paginationParams = {
+  max: external_exports.number().int().optional().describe("Max results"),
+  page: external_exports.string().optional().describe("Page token"),
+  all: external_exports.boolean().optional().describe("Fetch all pages")
+};
+function registerRunTool(server2, options) {
+  const { service, examples, omitAccount = false, note } = options;
+  const baseDescription = `Run any gog ${service} subcommand not covered by the other tools. Run \`gog ${service} --help\` for the full list of subcommands, or \`gog ${service} <subcommand> --help\` for flags on a specific subcommand.`;
+  const description = note ? `${baseDescription} ${note}` : baseDescription;
+  const inputSchema = {
+    subcommand: external_exports.string().describe(`The gog ${service} subcommand to run, e.g. ${examples}`),
+    args: external_exports.array(external_exports.string()).describe("Additional positional args and flags")
+  };
+  if (!omitAccount) {
+    inputSchema.account = accountParam;
+  }
+  server2.registerTool(`gog_${service}_run`, {
+    description,
+    annotations: { destructiveHint: true },
+    inputSchema
+  }, async (rawArgs) => {
+    const { subcommand, args, account } = rawArgs;
+    return runOrDiagnose([service, subcommand, ...args], { account });
+  });
+}
 function toText(output) {
   return { content: [{ type: "text", text: output }] };
 }
@@ -31049,15 +31210,11 @@ function registerAuthTools(server2) {
       return toError(err);
     }
   });
-  server2.registerTool("gog_auth_run", {
-    description: "Run any gog auth subcommand. Run `gog auth --help` to see all available subcommands and flags. Note: for browser-based authorization, use gog_auth_add instead.",
-    annotations: { destructiveHint: true },
-    inputSchema: {
-      subcommand: external_exports.string().describe('The gog auth subcommand, e.g. "remove", "alias", "tokens"'),
-      args: external_exports.array(external_exports.string()).describe("Additional positional args and flags")
-    }
-  }, async ({ subcommand, args }) => {
-    return runOrDiagnose(["auth", subcommand, ...args], {});
+  registerRunTool(server2, {
+    service: "auth",
+    examples: '"remove", "alias", "tokens"',
+    omitAccount: true,
+    note: "For browser-based authorization, use gog_auth_add instead."
   });
 }
 
@@ -31147,21 +31304,11 @@ function registerSheetsTools(server2) {
   }, async ({ spreadsheetId, find, replace, account }) => {
     return runOrDiagnose(["sheets", "find-replace", spreadsheetId, find, replace], { account });
   });
-  server2.registerTool("gog_sheets_run", {
-    description: "Run any gog sheets subcommand not covered by the other tools. Run `gog sheets --help` for the full list of subcommands, or `gog sheets <subcommand> --help` for flags on a specific subcommand.",
-    annotations: { destructiveHint: true },
-    inputSchema: {
-      subcommand: external_exports.string().describe('The gog sheets subcommand to run, e.g. "freeze", "add-tab", "rename-tab"'),
-      args: external_exports.array(external_exports.string()).describe('Additional positional args and flags, e.g. ["<spreadsheetId>", "--rows=1"]'),
-      account: accountParam
-    }
-  }, async ({ subcommand, args, account }) => {
-    return runOrDiagnose(["sheets", subcommand, ...args], { account });
-  });
+  registerRunTool(server2, { service: "sheets", examples: '"freeze", "add-tab", "rename-tab"' });
 }
 
 // ../gogcli-mcp/src/server.ts
-var VERSION = true ? "2.0.8" : "0.0.0";
+var VERSION = true ? "2.0.9" : "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-sheets",
   "display_name": "gogcli (Sheets)",
-  "version": "2.0.8",
+  "version": "2.0.9",
   "description": "Extended Google Sheets for Claude via gogcli — auth + full Sheets support",
   "author": {
     "name": "Chris Hall",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gogcli-mcp-sheets",
-  "version": "2.0.8",
+  "version": "2.0.9",
   "mcpName": "io.github.chrischall/gogcli-mcp-sheets",
   "description": "Extended Google Sheets MCP server via gogcli — all base tools plus full Sheets support",
   "author": "Claude Code (AI) <https://www.anthropic.com/claude>",

package/server.json

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

package/src/tools/sheets-extra.ts

@@ -3,7 +3,7 @@ import { z } from 'zod';
 import { accountParam, runOrDiagnose } from '../../../gogcli-mcp/src/lib.js';
 
 export function registerExtraSheetsTools(server: McpServer): void {
-  // 1. Add tab
+
   server.registerTool('gog_sheets_add_tab', {
     description: 'Add a new sheet tab to a spreadsheet.',
     inputSchema: {
@@ -15,7 +15,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(['sheets', 'add-tab', spreadsheetId, tabName], { account });
   });
 
-  // 2. Delete tab
   server.registerTool('gog_sheets_delete_tab', {
     description: 'Delete a sheet tab from a spreadsheet.',
     annotations: { destructiveHint: true },
@@ -28,7 +27,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(['sheets', 'delete-tab', spreadsheetId, tabName], { account });
   });
 
-  // 3. Rename tab
   server.registerTool('gog_sheets_rename_tab', {
     description: 'Rename a sheet tab.',
     annotations: { destructiveHint: true },
@@ -42,7 +40,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(['sheets', 'rename-tab', spreadsheetId, oldName, newName], { account });
   });
 
-  // 4. Copy spreadsheet
   server.registerTool('gog_sheets_copy', {
     description: 'Copy a spreadsheet to a new spreadsheet with the given title.',
     inputSchema: {
@@ -57,7 +54,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 5. Export spreadsheet
   server.registerTool('gog_sheets_export', {
     description: 'Export a spreadsheet as CSV, TSV, or PDF.',
     annotations: { readOnlyHint: true },
@@ -74,7 +70,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 6. Freeze rows/columns
   server.registerTool('gog_sheets_freeze', {
     description: 'Freeze rows and/or columns in a sheet.',
     annotations: { destructiveHint: true },
@@ -93,7 +88,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 7. Insert rows/columns
   server.registerTool('gog_sheets_insert', {
     description: 'Insert rows or columns into a sheet.',
     annotations: { destructiveHint: true },
@@ -113,7 +107,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 8. Merge cells
   server.registerTool('gog_sheets_merge', {
     description: 'Merge cells in a range.',
     annotations: { destructiveHint: true },
@@ -129,7 +122,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 9. Unmerge cells
   server.registerTool('gog_sheets_unmerge', {
     description: 'Unmerge previously merged cells in a range.',
     annotations: { destructiveHint: true },
@@ -142,7 +134,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(['sheets', 'unmerge', spreadsheetId, range], { account });
   });
 
-  // 10. Format cells
   server.registerTool('gog_sheets_format', {
     description: 'Apply cell formatting (bold, colors, alignment, etc.) to a range. Pass format as a JSON CellFormat object.',
     annotations: { destructiveHint: true },
@@ -159,7 +150,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 11. Number format
   server.registerTool('gog_sheets_number_format', {
     description: 'Set number format on a range (currency, percentage, date, etc.).',
     annotations: { destructiveHint: true },
@@ -177,7 +167,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 12. Read format
   server.registerTool('gog_sheets_read_format', {
     description: 'Read cell formatting for a range.',
     annotations: { readOnlyHint: true },
@@ -193,7 +182,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 13. Resize columns
   server.registerTool('gog_sheets_resize_columns', {
     description: 'Resize column widths. Use --auto for auto-fit or --width for a specific pixel width.',
     annotations: { destructiveHint: true },
@@ -211,7 +199,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 14. Resize rows
   server.registerTool('gog_sheets_resize_rows', {
     description: 'Resize row heights. Use --auto for auto-fit or --height for a specific pixel height.',
     annotations: { destructiveHint: true },
@@ -229,7 +216,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 15. Notes (read)
   server.registerTool('gog_sheets_notes', {
     description: 'Read cell notes in a range.',
     annotations: { readOnlyHint: true },
@@ -242,7 +228,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(['sheets', 'notes', spreadsheetId, range], { account });
   });
 
-  // 16. Update note
   server.registerTool('gog_sheets_update_note', {
     description: 'Add, update, or clear a cell note. Pass an empty string to clear the note.',
     annotations: { destructiveHint: true },
@@ -256,7 +241,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(['sheets', 'update-note', spreadsheetId, range, `--note=${note}`], { account });
   });
 
-  // 17. Links
   server.registerTool('gog_sheets_links', {
     description: 'List hyperlinks in a range.',
     annotations: { readOnlyHint: true },
@@ -269,7 +253,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(['sheets', 'links', spreadsheetId, range], { account });
   });
 
-  // 18. Named ranges list
   server.registerTool('gog_sheets_named_ranges_list', {
     description: 'List all named ranges in a spreadsheet.',
     annotations: { readOnlyHint: true },
@@ -281,7 +264,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(['sheets', 'named-ranges', 'list', spreadsheetId], { account });
   });
 
-  // 19. Named ranges get
   server.registerTool('gog_sheets_named_ranges_get', {
     description: 'Get a named range by name or ID.',
     annotations: { readOnlyHint: true },
@@ -294,7 +276,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(['sheets', 'named-ranges', 'get', spreadsheetId, nameOrId], { account });
   });
 
-  // 20. Named ranges add
   server.registerTool('gog_sheets_named_ranges_add', {
     description: 'Create a new named range.',
     inputSchema: {
@@ -307,7 +288,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(['sheets', 'named-ranges', 'add', spreadsheetId, name, range], { account });
   });
 
-  // 21. Named ranges update
   server.registerTool('gog_sheets_named_ranges_update', {
     description: 'Update a named range (change its name, range, or both).',
     annotations: { destructiveHint: true },
@@ -325,7 +305,6 @@ export function registerExtraSheetsTools(server: McpServer): void {
     return runOrDiagnose(args, { account });
   });
 
-  // 22. Named ranges delete
   server.registerTool('gog_sheets_named_ranges_delete', {
     description: 'Delete a named range.',
     annotations: { destructiveHint: true },

package/tests/tools/sheets-extra.test.ts

@@ -1,7 +1,7 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { registerExtraSheetsTools } from '../../src/tools/sheets-extra.js';
 import * as lib from '../../../gogcli-mcp/src/lib.js';
-import { setupExtrasHandlers, toText } from '../../../gogcli-mcp/tests/helpers/extras-harness.js';
+import { setupHandlers as setupHandlersBase, toText } from '../../../gogcli-mcp/tests/helpers/test-harness.js';
 
 vi.mock('../../../gogcli-mcp/src/lib.js', async (importOriginal) => {
   const actual = await importOriginal<typeof lib>();
@@ -11,7 +11,7 @@ vi.mock('../../../gogcli-mcp/src/lib.js', async (importOriginal) => {
   };
 });
 
-const setupHandlers = () => setupExtrasHandlers(registerExtraSheetsTools);
+const setupHandlers = () => setupHandlersBase(registerExtraSheetsTools);
 
 beforeEach(() => vi.clearAllMocks());