google-tools-mcp 1.2.10 → 1.2.11

package/dist/googleDocsApiHelpers.js

@@ -291,6 +291,18 @@ function normalizeForSearch(text) {
     }
     return result;
 }
+/**
+ * readDocument(format='markdown') includes markdown list markers ("- ",
+ * "1. ", etc.), but Google Docs text runs do not include list glyphs. When a
+ * caller copies a bullet line from markdown into textToFind, retry without
+ * those line-start markers.
+ */
+export function stripMarkdownListMarkersForSearch(text) {
+    return text
+        .split('\n')
+        .map((line) => line.replace(/^(\s*)(?:[-*+]\s+|\d+[.)]\s+)/, '$1'))
+        .join('\n');
+}
 /**
  * Finds all occurrences of textToFind in the document and returns them with
  * surrounding context and mapped document indices.
@@ -332,6 +344,15 @@ export async function findTextRange(docs, documentId, textToFind, instance, tabI
         const { fullText, segments } = result;
         logger.debug(`Document ${documentId} contains ${segments.length} text segments and ${fullText.length} characters in total.`);
         let allOccurrences = findAllOccurrences(fullText, segments, textToFind);
+        // Fallback: markdown exports include list markers that are absent from
+        // Docs API text runs. Retry after stripping line-start markdown markers.
+        if (allOccurrences.length === 0) {
+            const listMarkerStrippedSearch = stripMarkdownListMarkersForSearch(textToFind);
+            if (listMarkerStrippedSearch !== textToFind) {
+                logger.debug(`Exact match failed, trying match without markdown list markers`);
+                allOccurrences = findAllOccurrences(fullText, segments, listMarkerStrippedSearch);
+            }
+        }
         // Fallback: try normalized matching if exact match fails (issue #11)
         if (allOccurrences.length === 0) {
             const normalizedSearch = normalizeForSearch(textToFind);
@@ -365,6 +386,39 @@ export async function findTextRange(docs, documentId, textToFind, instance, tabI
                 }
             }
         }
+        // Fallback: combine both approaches for copied markdown bullets that
+        // also contain smart quotes, non-breaking spaces, or typographic dashes.
+        if (allOccurrences.length === 0) {
+            const listMarkerStrippedSearch = stripMarkdownListMarkersForSearch(textToFind);
+            const normalizedSearch = normalizeForSearch(listMarkerStrippedSearch);
+            const { normalized: normalizedFull, posMap } = normalizeWithPositionMap(fullText);
+            if (normalizedSearch !== textToFind || normalizedFull !== fullText) {
+                logger.debug(`Exact match failed, trying normalized match without markdown list markers`);
+                const CONTEXT_CHARS = 30;
+                let searchFrom = 0;
+                while (true) {
+                    const idx = normalizedFull.indexOf(normalizedSearch, searchFrom);
+                    if (idx === -1) break;
+                    const origStart = posMap[idx];
+                    const origEnd = posMap[idx + normalizedSearch.length];
+                    const docStart = mapFullTextPositionToDocIndex(origStart, segments);
+                    const docEnd = mapFullTextPositionToDocIndex(origEnd, segments);
+                    const contextStart = Math.max(0, origStart - CONTEXT_CHARS);
+                    const contextEnd = Math.min(fullText.length, origEnd + CONTEXT_CHARS);
+                    const before = fullText.slice(contextStart, origStart).replace(/\n/g, '\\n');
+                    const match = fullText.slice(origStart, origEnd).replace(/\n/g, '\\n');
+                    const after = fullText.slice(origEnd, contextEnd).replace(/\n/g, '\\n');
+                    const context = `${contextStart > 0 ? '...' : ''}${before}[${match}]${after}${contextEnd < fullText.length ? '...' : ''}`;
+                    allOccurrences.push({
+                        instance: allOccurrences.length + 1,
+                        startIndex: docStart,
+                        endIndex: docEnd,
+                        context,
+                    });
+                    searchFrom = idx + 1;
+                }
+            }
+        }
         if (allOccurrences.length === 0) {
             logger.warn(`Text "${textToFind}" not found in document ${documentId}`);
             return null;

package/dist/tools/docs/modifyText.js

@@ -101,6 +101,7 @@ export function register(server) {
             "Use readDocument with format='json' to determine indices. " +
             'Supports \\n for line breaks and \\t for tabs in replacement text. ' +
             'When using textToFind, if multiple matches exist the tool returns all instances with context so you can specify matchInstance. ' +
+            "textToFind tolerates markdown list markers copied from readDocument(format='markdown'), because Google Docs stores bullets outside text runs. " +
             'For multi-line or section-level rewrites, use replaceDocumentWithMarkdown instead. ' +
             'To add content to the end of a doc, use appendMarkdown or appendText.',
         parameters: ModifyTextParameters,

package/dist/tools/docs/modifyText.test.js

@@ -1,6 +1,7 @@
-import { describe, it } from 'node:test';
-import assert from 'node:assert/strict';
-import { buildModifyTextRequests } from './modifyText.js';
+import { describe, it } from 'node:test';
+import assert from 'node:assert/strict';
+import { buildModifyTextRequests } from './modifyText.js';
+import { stripMarkdownListMarkersForSearch } from '../../googleDocsApiHelpers.js';
 
 describe('buildModifyTextRequests', () => {
   it('generates deleteContentRange + insertText for a replacement', () => {
@@ -50,7 +51,7 @@ describe('buildModifyTextRequests', () => {
   });
 });
 
-describe('escape sequence normalization (issue #9)', () => {
+describe('escape sequence normalization (issue #9)', () => {
   // Simulate the normalization that happens in the execute handler
   function normalize(text) {
     return text?.replace(/\\n/g, '\n').replace(/\\t/g, '\t');
@@ -86,5 +87,22 @@ describe('escape sequence normalization (issue #9)', () => {
 
   it('returns undefined for undefined input', () => {
     assert.equal(normalize(undefined), undefined);
-  });
-});
+  });
+});
+
+describe('markdown list marker search normalization', () => {
+  it('strips unordered markdown list markers copied from readDocument markdown', () => {
+    const input = '- Existing Google Docs bullet text';
+    assert.equal(stripMarkdownListMarkersForSearch(input), 'Existing Google Docs bullet text');
+  });
+
+  it('strips ordered markdown list markers copied from readDocument markdown', () => {
+    const input = '1. Existing Google Docs numbered item';
+    assert.equal(stripMarkdownListMarkersForSearch(input), 'Existing Google Docs numbered item');
+  });
+
+  it('preserves indentation when stripping nested list markers', () => {
+    const input = '  - Nested item';
+    assert.equal(stripMarkdownListMarkersForSearch(input), '  Nested item');
+  });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "google-tools-mcp",
-  "version": "1.2.10",
+  "version": "1.2.11",
   "description": "The easiest MCP server for Google Workspace — Drive, Docs, Sheets, Gmail, Calendar, and Forms. 153 tools with one-click browser auth. Read Word docs, PDFs, and spreadsheets straight from Drive.",
   "type": "module",
   "bin": {