npm - gogcli-mcp-docs - Versions diffs - 2.8.0 → 2.18.1 - Mend

gogcli-mcp-docs 2.8.0 → 2.18.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (10) hide show

package/.claude-plugin/marketplace.json +2 -2
package/.claude-plugin/plugin.json +1 -1
package/dist/index.js +20340 -19648
package/manifest.json +70 -2
package/package.json +3 -2
package/server.json +2 -2
package/src/index.ts +7 -9
package/src/tools/docs-extra.ts +413 -16
package/tests/tools/docs-extra.test.ts +952 -438
package/tsconfig.json +1 -1

package/tests/tools/docs-extra.test.ts CHANGED Viewed

@@ -1,7 +1,8 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { registerExtraDocsTools } from '../../src/tools/docs-extra.js';
 import * as lib from '../../../gogcli-mcp/src/lib.js';
-import { setupHandlers as setupHandlersBase, toText } from '../../../gogcli-mcp/tests/helpers/test-harness.js';
+import { createTestHarness } from '@chrischall/mcp-utils/test';
+import { rawTextResult } from '@chrischall/mcp-utils';
 vi.mock('../../../gogcli-mcp/src/lib.js', async (importOriginal) => {
   const actual = await importOriginal<typeof lib>();
@@ -11,7 +12,7 @@ vi.mock('../../../gogcli-mcp/src/lib.js', async (importOriginal) => {
   };
 });
-const setupHandlers = () => setupHandlersBase(registerExtraDocsTools);
+const setupHandlers = () => createTestHarness(registerExtraDocsTools);
 beforeEach(() => vi.clearAllMocks());
@@ -19,16 +20,16 @@ beforeEach(() => vi.clearAllMocks());
 describe('gog_docs_copy', () => {
   it('calls runOrDiagnose with correct args', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_copy')!({ docId: 'abc', title: 'My Copy' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_copy', { docId: 'abc', title: 'My Copy' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'copy', 'abc', 'My Copy'], { account: undefined });
   });
   it('includes --parent when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_copy')!({ docId: 'abc', title: 'Copy', parent: 'folder123' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_copy', { docId: 'abc', title: 'Copy', parent: 'folder123' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'copy', 'abc', 'Copy', '--parent=folder123'],
       { account: undefined },
@@ -36,16 +37,16 @@ describe('gog_docs_copy', () => {
   });
   it('omits --parent when not provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_copy')!({ docId: 'abc', title: 'Copy' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_copy', { docId: 'abc', title: 'Copy' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'copy', 'abc', 'Copy'], { account: undefined });
   });
   it('forwards account override', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_copy')!({ docId: 'abc', title: 'Copy', account: 'other@gmail.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_copy', { docId: 'abc', title: 'Copy', account: 'other@gmail.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'copy', 'abc', 'Copy'], { account: 'other@gmail.com' });
   });
 });
@@ -54,9 +55,9 @@ describe('gog_docs_copy', () => {
 describe('gog_docs_delete', () => {
   it('calls runOrDiagnose with correct args', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_delete')!({ docId: 'abc', start: 5, end: 10 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_delete', { docId: 'abc', start: 5, end: 10 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'delete', '--start=5', '--end=10', 'abc'],
       { account: undefined },
@@ -65,9 +66,9 @@ describe('gog_docs_delete', () => {
   // gog 0.23.0
   it('anchors by text with --at/--occurrence/--match-case', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_delete')!({ docId: 'abc', at: 'TODO', occurrence: 2, matchCase: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_delete', { docId: 'abc', at: 'TODO', occurrence: 2, matchCase: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'delete', 'abc', '--at=TODO', '--occurrence=2', '--match-case'],
       { account: undefined },
@@ -75,9 +76,9 @@ describe('gog_docs_delete', () => {
   });
   it('includes --tab-id when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_delete')!({ docId: 'abc', start: 1, end: 5, tabId: 'tab1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_delete', { docId: 'abc', start: 1, end: 5, tabId: 'tab1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'delete', '--start=1', '--end=5', 'abc', '--tab-id=tab1'],
       { account: undefined },
@@ -85,9 +86,9 @@ describe('gog_docs_delete', () => {
   });
   it('omits --tab-id when not provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_delete')!({ docId: 'abc', start: 1, end: 5 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_delete', { docId: 'abc', start: 1, end: 5 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'delete', '--start=1', '--end=5', 'abc'],
       { account: undefined },
@@ -99,9 +100,9 @@ describe('gog_docs_delete', () => {
 describe('gog_docs_edit', () => {
   it('calls runOrDiagnose with correct args', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_edit')!({ docId: 'abc', find: 'old', replace: 'new' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_edit', { docId: 'abc', find: 'old', replace: 'new' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'edit', 'abc', 'old', 'new'],
       { account: undefined },
@@ -109,9 +110,9 @@ describe('gog_docs_edit', () => {
   });
   it('includes --match-case when true', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_edit')!({ docId: 'abc', find: 'old', replace: 'new', matchCase: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_edit', { docId: 'abc', find: 'old', replace: 'new', matchCase: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'edit', 'abc', 'old', 'new', '--match-case'],
       { account: undefined },
@@ -119,9 +120,9 @@ describe('gog_docs_edit', () => {
   });
   it('omits --match-case when false', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_edit')!({ docId: 'abc', find: 'old', replace: 'new', matchCase: false });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_edit', { docId: 'abc', find: 'old', replace: 'new', matchCase: false });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'edit', 'abc', 'old', 'new'],
       { account: undefined },
@@ -133,16 +134,16 @@ describe('gog_docs_edit', () => {
 describe('gog_docs_export', () => {
   it('calls runOrDiagnose with correct args', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_export')!({ docId: 'abc' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_export', { docId: 'abc' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'export', 'abc'], { account: undefined });
   });
   it('includes --format when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_export')!({ docId: 'abc', format: 'html' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_export', { docId: 'abc', format: 'html' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'export', 'abc', '--format=html'],
       { account: undefined },
@@ -150,9 +151,9 @@ describe('gog_docs_export', () => {
   });
   it('includes --out when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_export')!({ docId: 'abc', out: '/tmp/doc.pdf' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_export', { docId: 'abc', out: '/tmp/doc.pdf' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'export', 'abc', '--out=/tmp/doc.pdf'],
       { account: undefined },
@@ -160,9 +161,9 @@ describe('gog_docs_export', () => {
   });
   it('includes both --format and --out when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_export')!({ docId: 'abc', format: 'txt', out: '/tmp/doc.txt' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_export', { docId: 'abc', format: 'txt', out: '/tmp/doc.txt' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'export', 'abc', '--format=txt', '--out=/tmp/doc.txt'],
       { account: undefined },
@@ -170,20 +171,30 @@ describe('gog_docs_export', () => {
   });
   it('omits optional flags when not provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_export')!({ docId: 'abc' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_export', { docId: 'abc' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'export', 'abc'], { account: undefined });
   });
+  it('includes --overwrite when requested', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_export', { docId: 'abc', out: '/tmp/doc.pdf', overwrite: true });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'export', 'abc', '--out=/tmp/doc.pdf', '--overwrite'],
+      { account: undefined },
+    );
+  });
 });
 // --- gog_docs_insert ---
 describe('gog_docs_insert', () => {
   it('calls runOrDiagnose with content', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert')!({ docId: 'abc', content: 'Hello world' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert', { docId: 'abc', content: 'Hello world' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert', 'abc', 'Hello world'],
       { account: undefined },
@@ -192,9 +203,9 @@ describe('gog_docs_insert', () => {
   // gog 0.23.0
   it('anchors by text with --at/--occurrence/--match-case', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert')!({ docId: 'abc', content: 'X', at: 'HERE', occurrence: 3, matchCase: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert', { docId: 'abc', content: 'X', at: 'HERE', occurrence: 3, matchCase: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert', 'abc', 'X', '--at=HERE', '--occurrence=3', '--match-case'],
       { account: undefined },
@@ -202,9 +213,9 @@ describe('gog_docs_insert', () => {
   });
   it('includes --index when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert')!({ docId: 'abc', content: 'text', index: 5 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert', { docId: 'abc', content: 'text', index: 5 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert', 'abc', 'text', '--index=5'],
       { account: undefined },
@@ -212,9 +223,9 @@ describe('gog_docs_insert', () => {
   });
   it('includes --index=0 when index is zero', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert')!({ docId: 'abc', content: 'text', index: 0 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert', { docId: 'abc', content: 'text', index: 0 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert', 'abc', 'text', '--index=0'],
       { account: undefined },
@@ -222,9 +233,9 @@ describe('gog_docs_insert', () => {
   });
   it('includes --file when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert')!({ docId: 'abc', file: '/tmp/text.txt' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert', { docId: 'abc', file: '/tmp/text.txt' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert', 'abc', '--file=/tmp/text.txt'],
       { account: undefined },
@@ -232,9 +243,9 @@ describe('gog_docs_insert', () => {
   });
   it('includes --tab-id when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert')!({ docId: 'abc', content: 'hi', tabId: 'tab1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert', { docId: 'abc', content: 'hi', tabId: 'tab1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert', 'abc', 'hi', '--tab-id=tab1'],
       { account: undefined },
@@ -242,9 +253,9 @@ describe('gog_docs_insert', () => {
   });
   it('omits optional flags when not provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert')!({ docId: 'abc' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert', { docId: 'abc' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'insert', 'abc'], { account: undefined });
   });
 });
@@ -253,16 +264,16 @@ describe('gog_docs_insert', () => {
 describe('gog_docs_list_tabs', () => {
   it('calls runOrDiagnose with correct args', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_list_tabs')!({ docId: 'abc' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_list_tabs', { docId: 'abc' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'list-tabs', 'abc'], { account: undefined });
   });
   it('forwards account override', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_list_tabs')!({ docId: 'abc', account: 'other@gmail.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_list_tabs', { docId: 'abc', account: 'other@gmail.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'list-tabs', 'abc'], { account: 'other@gmail.com' });
   });
 });
@@ -271,9 +282,9 @@ describe('gog_docs_list_tabs', () => {
 describe('gog_docs_sed', () => {
   it('calls runOrDiagnose with single expression', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_sed')!({ docId: 'abc', expression: 's/old/new/g' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_sed', { docId: 'abc', expression: 's/old/new/g' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'sed', 'abc', 's/old/new/g'],
       { account: undefined },
@@ -281,9 +292,9 @@ describe('gog_docs_sed', () => {
   });
   it('calls runOrDiagnose with multiple expressions', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_sed')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_sed', {
       docId: 'abc',
       expressions: ['s/foo/bar/g', 's/baz/qux/g'],
     });
@@ -294,9 +305,9 @@ describe('gog_docs_sed', () => {
   });
   it('includes --file when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_sed')!({ docId: 'abc', file: '/tmp/sed.txt' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_sed', { docId: 'abc', file: '/tmp/sed.txt' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'sed', 'abc', '--file=/tmp/sed.txt'],
       { account: undefined },
@@ -304,9 +315,9 @@ describe('gog_docs_sed', () => {
   });
   it('includes --tab when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_sed')!({ docId: 'abc', expression: 's/a/b/', tab: 'Notes' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_sed', { docId: 'abc', expression: 's/a/b/', tab: 'Notes' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'sed', 'abc', 's/a/b/', '--tab=Notes'],
       { account: undefined },
@@ -314,9 +325,9 @@ describe('gog_docs_sed', () => {
   });
   it('omits optional flags when not provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_sed')!({ docId: 'abc' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_sed', { docId: 'abc' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'sed', 'abc'], { account: undefined });
   });
 });
@@ -325,9 +336,9 @@ describe('gog_docs_sed', () => {
 describe('gog_docs_update', () => {
   it('calls runOrDiagnose with --text flag', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', text: 'Hello' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', text: 'Hello' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc', '--text=Hello'],
       { account: undefined },
@@ -336,9 +347,9 @@ describe('gog_docs_update', () => {
   // gog 0.23.0
   it('anchors by text with --at/--occurrence/--match-case', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', text: 'New', at: 'OLD', occurrence: 1, matchCase: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', text: 'New', at: 'OLD', occurrence: 1, matchCase: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc', '--text=New', '--at=OLD', '--occurrence=1', '--match-case'],
       { account: undefined },
@@ -346,9 +357,9 @@ describe('gog_docs_update', () => {
   });
   it('includes --file when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', file: '/tmp/content.txt' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', file: '/tmp/content.txt' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc', '--file=/tmp/content.txt'],
       { account: undefined },
@@ -356,9 +367,9 @@ describe('gog_docs_update', () => {
   });
   it('includes --index when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', text: 'hi', index: 10 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', text: 'hi', index: 10 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc', '--text=hi', '--index=10'],
       { account: undefined },
@@ -366,9 +377,9 @@ describe('gog_docs_update', () => {
   });
   it('includes --index=0 when index is zero', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', text: 'hi', index: 0 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', text: 'hi', index: 0 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc', '--text=hi', '--index=0'],
       { account: undefined },
@@ -376,9 +387,9 @@ describe('gog_docs_update', () => {
   });
   it('includes --tab-id when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', text: 'hi', tabId: 'tab1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', text: 'hi', tabId: 'tab1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc', '--text=hi', '--tab-id=tab1'],
       { account: undefined },
@@ -386,9 +397,9 @@ describe('gog_docs_update', () => {
   });
   it('includes --pageless when true', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', pageless: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', pageless: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc', '--pageless'],
       { account: undefined },
@@ -396,9 +407,9 @@ describe('gog_docs_update', () => {
   });
   it('omits --pageless when false', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', pageless: false });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', pageless: false });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc'],
       { account: undefined },
@@ -406,16 +417,16 @@ describe('gog_docs_update', () => {
   });
   it('omits optional flags when not provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'update', 'abc'], { account: undefined });
   });
   it('includes --replace-range when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', text: 'new', replaceRange: '25:40' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', text: 'new', replaceRange: '25:40' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc', '--text=new', '--replace-range=25:40'],
       { account: undefined },
@@ -423,9 +434,9 @@ describe('gog_docs_update', () => {
   });
   it('includes --markdown when true', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', text: '# Heading', markdown: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', text: '# Heading', markdown: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc', '--text=# Heading', '--markdown'],
       { account: undefined },
@@ -433,9 +444,9 @@ describe('gog_docs_update', () => {
   });
   it('omits --markdown when false', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_update')!({ docId: 'abc', text: 'hi', markdown: false });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'abc', text: 'hi', markdown: false });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'update', 'abc', '--text=hi'],
       { account: undefined },
@@ -447,16 +458,16 @@ describe('gog_docs_update', () => {
 describe('gog_docs_comments_list', () => {
   it('calls runOrDiagnose with correct args', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_list')!({ docId: 'abc' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_list', { docId: 'abc' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'comments', 'list', 'abc'], { account: undefined });
   });
   it('includes --include-resolved when set', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_list')!({ docId: 'abc', includeResolved: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_list', { docId: 'abc', includeResolved: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'comments', 'list', 'abc', '--include-resolved'],
       { account: undefined },
@@ -464,16 +475,16 @@ describe('gog_docs_comments_list', () => {
   });
   it('omits --include-resolved when false', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_list')!({ docId: 'abc', includeResolved: false });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_list', { docId: 'abc', includeResolved: false });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'comments', 'list', 'abc'], { account: undefined });
   });
   it('forwards account override', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_list')!({ docId: 'abc', account: 'other@gmail.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_list', { docId: 'abc', account: 'other@gmail.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'comments', 'list', 'abc'],
       { account: 'other@gmail.com' },
@@ -482,9 +493,9 @@ describe('gog_docs_comments_list', () => {
   // gog 0.22.0 adds --since; pagination flags were already supported by gog.
   it('includes --since and pagination flags when set', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_list')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_list', {
       docId: 'abc', includeResolved: true, since: '2026-06-01T00:00:00Z', max: 10, page: 'tok', all: true,
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
@@ -496,9 +507,9 @@ describe('gog_docs_comments_list', () => {
 describe('gog_docs_table_column_width', () => {
   it('sets a fixed column width', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_table_column_width')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_column_width', {
       docId: 'd1', col: 2, width: 120, tableIndex: 1, tab: 'Body',
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
@@ -508,9 +519,9 @@ describe('gog_docs_table_column_width', () => {
   });
   it('resets all columns to even distribution', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_table_column_width')!({ docId: 'd1', evenlyDistributed: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_column_width', { docId: 'd1', evenlyDistributed: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'table-column-width', 'd1', '--evenly-distributed'],
       { account: undefined },
@@ -518,9 +529,9 @@ describe('gog_docs_table_column_width', () => {
   });
   it('forwards account override', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_table_column_width')!({ docId: 'd1', col: 1, width: 80, account: 'other@gmail.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_column_width', { docId: 'd1', col: 1, width: 80, account: 'other@gmail.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'table-column-width', 'd1', '--col=1', '--width=80'],
       { account: 'other@gmail.com' },
@@ -531,16 +542,16 @@ describe('gog_docs_table_column_width', () => {
 // gog 0.23.0
 describe('gog_docs_find_range', () => {
   it('maps matched text to index ranges', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_find_range')!({ docId: 'd1', text: 'hello' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_find_range', { docId: 'd1', text: 'hello' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'find-range', 'd1', 'hello'], { account: undefined });
   });
   it('passes all match flags', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_find_range')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_find_range', {
       docId: 'd1', text: 'hello', occurrence: 2, matchCase: true, normalizeWhitespace: true, all: true, failEmpty: true, tab: 'T',
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
@@ -552,16 +563,16 @@ describe('gog_docs_find_range', () => {
 describe('gog_docs_comments_locate', () => {
   it('locates a comment by id', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_locate')!({ docId: 'd1', commentId: 'c1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_locate', { docId: 'd1', commentId: 'c1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'comments', 'locate', 'd1', 'c1'], { account: undefined });
   });
   it('passes match flags', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_locate')!({ docId: 'd1', commentId: 'c1', matchCase: true, normalizeWhitespace: true, tab: 'T' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_locate', { docId: 'd1', commentId: 'c1', matchCase: true, normalizeWhitespace: true, tab: 'T' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'comments', 'locate', 'd1', 'c1', '--match-case', '--normalize-whitespace', '--tab=T'],
       { account: undefined },
@@ -571,18 +582,18 @@ describe('gog_docs_comments_locate', () => {
 describe('gog_docs_comments_get', () => {
   it('calls runOrDiagnose with correct args', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_get')!({ docId: 'abc', commentId: 'c1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_get', { docId: 'abc', commentId: 'c1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'comments', 'get', 'abc', 'c1'], { account: undefined });
   });
 });
 describe('gog_docs_comments_add', () => {
   it('calls runOrDiagnose with content', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_add')!({ docId: 'abc', content: 'Please review' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_add', { docId: 'abc', content: 'Please review' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'comments', 'add', 'abc', 'Please review'],
       { account: undefined },
@@ -590,9 +601,9 @@ describe('gog_docs_comments_add', () => {
   });
   it('includes --quoted when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_add')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_add', {
       docId: 'abc', content: 'Typo here', quoted: 'teh',
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
@@ -602,9 +613,9 @@ describe('gog_docs_comments_add', () => {
   });
   it('omits --quoted when not provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_add')!({ docId: 'abc', content: 'Nice' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_add', { docId: 'abc', content: 'Nice' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'comments', 'add', 'abc', 'Nice'],
       { account: undefined },
@@ -614,9 +625,9 @@ describe('gog_docs_comments_add', () => {
 describe('gog_docs_comments_reply', () => {
   it('calls runOrDiagnose with correct args', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_reply')!({ docId: 'abc', commentId: 'c1', content: 'Done' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_reply', { docId: 'abc', commentId: 'c1', content: 'Done' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'comments', 'reply', 'abc', 'c1', 'Done'],
       { account: undefined },
@@ -626,9 +637,9 @@ describe('gog_docs_comments_reply', () => {
 describe('gog_docs_comments_resolve', () => {
   it('calls runOrDiagnose with correct args', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_resolve')!({ docId: 'abc', commentId: 'c1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_resolve', { docId: 'abc', commentId: 'c1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'comments', 'resolve', 'abc', 'c1'],
       { account: undefined },
@@ -636,9 +647,9 @@ describe('gog_docs_comments_resolve', () => {
   });
   it('includes --message when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_resolve')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_resolve', {
       docId: 'abc', commentId: 'c1', message: 'Fixed in v2',
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
@@ -648,9 +659,9 @@ describe('gog_docs_comments_resolve', () => {
   });
   it('omits --message when not provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_resolve')!({ docId: 'abc', commentId: 'c1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_resolve', { docId: 'abc', commentId: 'c1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'comments', 'resolve', 'abc', 'c1'],
       { account: undefined },
@@ -660,11 +671,11 @@ describe('gog_docs_comments_resolve', () => {
 describe('gog_docs_comments_delete', () => {
   it('calls runOrDiagnose with correct args', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_delete')!({ docId: 'abc', commentId: 'c1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_delete', { docId: 'abc', commentId: 'c1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
-      ['docs', 'comments', 'delete', 'abc', 'c1'],
+      ['docs', 'comments', 'delete', 'abc', 'c1', '--force'],
       { account: undefined },
     );
   });
@@ -672,18 +683,18 @@ describe('gog_docs_comments_delete', () => {
 describe('gog_docs_trash', () => {
   it('routes to gog drive delete <docId>', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_trash')!({ docId: 'd1' });
-    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['drive', 'delete', 'd1'], { account: undefined });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_trash', { docId: 'd1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['drive', 'delete', 'd1', '--force'], { account: undefined });
   });
 });
 describe('gog_docs_append', () => {
   it('uses gog docs write --append', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_append')!({ docId: 'd1', text: 'Hello' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_append', { docId: 'd1', text: 'Hello' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'write', 'd1', '--append', '--text=Hello'],
       { account: undefined },
@@ -691,9 +702,9 @@ describe('gog_docs_append', () => {
   });
   it('passes file, markdown, and tab flags', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_append')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_append', {
       docId: 'd1', file: '/tmp/section.md', markdown: true, tab: 'Notes',
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
@@ -727,35 +738,49 @@ describe('gog_docs_append', () => {
 describe('gog_docs_read', () => {
   it('defaults to plain text via gog docs cat', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('hello'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_read')!({ docId: 'd1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('hello'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_read', { docId: 'd1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'cat', 'd1'], { account: undefined });
   });
   it('routes json format to gog docs raw --pretty', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_read')!({ docId: 'd1', format: 'json' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_read', { docId: 'd1', format: 'json' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'raw', 'd1', '--pretty'], { account: undefined });
   });
   it('passes tab, allTabs, maxBytes in text mode', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText(''));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_read')!({ docId: 'd1', tab: 'Section A', allTabs: true, maxBytes: 0 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult(''));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_read', { docId: 'd1', tab: 'Section A', allTabs: true, maxBytes: 0 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'cat', 'd1', '--tab=Section A', '--all-tabs', '--max-bytes=0'],
       { account: undefined },
     );
   });
+  it('appends --chips in text mode', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult(''));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_read', { docId: 'd1', chips: true });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'cat', 'd1', '--chips'], { account: undefined });
+  });
+  it('ignores chips in json mode', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_read', { docId: 'd1', format: 'json', chips: true });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'raw', 'd1', '--pretty'], { account: undefined });
+  });
 });
 describe('gog_docs_format', () => {
   it('passes all text/paragraph attribute flags', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_format')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', {
       docId: 'd1',
       match: 'Title',
       matchAll: true,
@@ -795,9 +820,9 @@ describe('gog_docs_format', () => {
   });
   it('emits clear-style flags (noBold/noItalic/...) without the set variants', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_format')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', {
       docId: 'd1',
       noBold: true,
       noItalic: true,
@@ -811,17 +836,17 @@ describe('gog_docs_format', () => {
   });
   it('omits all flags when not provided (whole-doc no-op call passes through)', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_format')!({ docId: 'd1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', { docId: 'd1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'format', 'd1'], { account: undefined });
   });
   // gog 0.18.0
   it('passes --heading-level and --named-style when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_format')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', {
       docId: 'd1', match: 'Intro', headingLevel: 1, namedStyle: 'HEADING_1',
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
@@ -831,9 +856,9 @@ describe('gog_docs_format', () => {
   });
   it('passes --heading-level=0 when explicitly 0 (HEADING_0 reserved but argument should reach gog)', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_format')!({ docId: 'd1', headingLevel: 0 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', { docId: 'd1', headingLevel: 0 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'format', 'd1', '--heading-level=0'],
       { account: undefined },
@@ -842,9 +867,9 @@ describe('gog_docs_format', () => {
   // gog 0.22.0
   it('passes --code when code is true', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_format')!({ docId: 'd1', match: 'snippet', code: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', { docId: 'd1', match: 'snippet', code: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'format', 'd1', '--match=snippet', '--code'],
       { account: undefined },
@@ -853,9 +878,9 @@ describe('gog_docs_format', () => {
   // gog 0.23.0
   it('sets a hyperlink with --link', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_format')!({ docId: 'd1', match: 'see docs', link: 'https://example.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', { docId: 'd1', match: 'see docs', link: 'https://example.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'format', 'd1', '--match=see docs', '--link=https://example.com'],
       { account: undefined },
@@ -863,9 +888,9 @@ describe('gog_docs_format', () => {
   });
   it('clears a hyperlink with --no-link', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_format')!({ docId: 'd1', match: 'linked', noLink: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', { docId: 'd1', match: 'linked', noLink: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'format', 'd1', '--match=linked', '--no-link'],
       { account: undefined },
@@ -877,9 +902,9 @@ describe('gog_docs_format', () => {
 describe('gog_docs_insert_page_break', () => {
   it('inserts at index when --index provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_page_break')!({ docId: 'd1', index: 42 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_page_break', { docId: 'd1', index: 42 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-page-break', 'd1', '--index=42'],
       { account: undefined },
@@ -888,9 +913,9 @@ describe('gog_docs_insert_page_break', () => {
   // gog 0.23.0
   it('anchors by text with --at/--occurrence/--match-case', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_page_break')!({ docId: 'd1', at: 'Chapter 2', occurrence: 1, matchCase: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_page_break', { docId: 'd1', at: 'Chapter 2', occurrence: 1, matchCase: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-page-break', 'd1', '--at=Chapter 2', '--occurrence=1', '--match-case'],
       { account: undefined },
@@ -898,9 +923,9 @@ describe('gog_docs_insert_page_break', () => {
   });
   it('uses --at-end when atEnd is true', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_page_break')!({ docId: 'd1', atEnd: true, tab: 'Body' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_page_break', { docId: 'd1', atEnd: true, tab: 'Body' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-page-break', 'd1', '--at-end', '--tab=Body'],
       { account: undefined },
@@ -908,9 +933,9 @@ describe('gog_docs_insert_page_break', () => {
   });
   it('passes --index=1 when explicitly 1', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_page_break')!({ docId: 'd1', index: 1 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_page_break', { docId: 'd1', index: 1 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-page-break', 'd1', '--index=1'],
       { account: undefined },
@@ -920,9 +945,9 @@ describe('gog_docs_insert_page_break', () => {
 describe('gog_docs_page_layout', () => {
   it('passes --layout when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_page_layout')!({ docId: 'd1', layout: 'pages' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_page_layout', { docId: 'd1', layout: 'pages' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'page-layout', 'd1', '--layout=pages'],
       { account: undefined },
@@ -930,16 +955,16 @@ describe('gog_docs_page_layout', () => {
   });
   it('omits --layout when not provided (defaults to pageless on gog side)', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_page_layout')!({ docId: 'd1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_page_layout', { docId: 'd1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'page-layout', 'd1'], { account: undefined });
   });
   it('passes --page-size, page dimensions and all margins', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_page_layout')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_page_layout', {
       docId: 'd1', layout: 'pages', pageSize: 'A4', pageWidth: '8.5in', pageHeight: '11in',
       marginTop: '1in', marginBottom: '1in', marginLeft: '0.75in', marginRight: '0.75in',
     });
@@ -953,33 +978,82 @@ describe('gog_docs_page_layout', () => {
 describe('gog_docs_cell_update', () => {
   it('passes required --row and --col', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_cell_update')!({ docId: 'd1', row: 2, col: 3, content: 'hi' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_cell_update', { docId: 'd1', row: 2, col: 3, content: 'hi' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'cell-update', 'd1', '--row=2', '--col=3', '--content=hi'],
       { account: undefined },
     );
   });
-  it('supports empty-string content, --content-file, --append, --format, --table-index and --tab', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_cell_update')!({
-      docId: 'd1', row: 1, col: 1, content: '', contentFile: '/tmp/c.md', append: true,
+  it('supports empty-string content plus --append, --format, --table-index and --tab', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_cell_update', {
+      docId: 'd1', row: 1, col: 1, content: '', append: true,
       format: 'plain', tableIndex: -1, tab: 'Tab2',
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
-      ['docs', 'cell-update', 'd1', '--row=1', '--col=1', '--content=', '--content-file=/tmp/c.md',
+      ['docs', 'cell-update', 'd1', '--row=1', '--col=1', '--content=',
         '--append', '--format=plain', '--table-index=-1', '--tab=Tab2'],
       { account: undefined },
     );
   });
+  it('passes --content-file on its own', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_cell_update', {
+      docId: 'd1', row: 1, col: 1, contentFile: '/tmp/c.md',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'cell-update', 'd1', '--row=1', '--col=1', '--content-file=/tmp/c.md'],
+      { account: undefined },
+    );
+  });
+  // gog itself refuses --content together with --content-file, so the wrapper
+  // rejects the combination up front rather than shipping a doomed argv.
+  it('rejects content and contentFile together without invoking gog', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    const res = await harness.callTool('gog_docs_cell_update', {
+      docId: 'd1', row: 1, col: 1, content: 'hi', contentFile: '/tmp/c.md',
+    });
+    expect(res.isError).toBe(true);
+    expect(JSON.stringify(res.content)).toContain('not both');
+    expect(lib.runOrDiagnose).not.toHaveBeenCalled();
+  });
+  // An empty string clears the cell, so it must NOT be treated as "no content"
+  // and must NOT collide with contentFile's truthiness guard.
+  it('treats empty-string content as a conflict with contentFile', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    const res = await harness.callTool('gog_docs_cell_update', {
+      docId: 'd1', row: 1, col: 1, content: '', contentFile: '/tmp/c.md',
+    });
+    expect(res.isError).toBe(true);
+    expect(lib.runOrDiagnose).not.toHaveBeenCalled();
+  });
+  it('routes a large content value to a --content-file payload arg', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    const big = 'c'.repeat(lib.PAYLOAD_INLINE_MAX + 1);
+    await harness.callTool('gog_docs_cell_update', { docId: 'd1', row: 2, col: 3, content: big });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'cell-update', 'd1', '--row=2', '--col=3',
+        { kind: 'file', flag: 'content-file', contents: big, ext: 'md' }],
+      { account: undefined },
+    );
+  });
   it('omits content when not provided and forwards account', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_cell_update')!({ docId: 'd1', row: 1, col: 2, account: 'a@b.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_cell_update', { docId: 'd1', row: 1, col: 2, account: 'a@b.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'cell-update', 'd1', '--row=1', '--col=2'],
       { account: 'a@b.com' },
@@ -989,9 +1063,9 @@ describe('gog_docs_cell_update', () => {
 describe('gog_docs_cell_style', () => {
   it('passes required --row and --col', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_cell_style')!({ docId: 'd1', row: 0, col: 0 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_cell_style', { docId: 'd1', row: 0, col: 0 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'cell-style', 'd1', '--row=0', '--col=0'],
       { account: undefined },
@@ -999,9 +1073,9 @@ describe('gog_docs_cell_style', () => {
   });
   it('passes spans, colors and text styling', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_cell_style')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_cell_style', {
       docId: 'd1', row: 1, col: 2, rowSpan: 2, colSpan: 3, backgroundColor: '#eee',
       textColor: '#111', bold: true, italic: true, underline: true, tableIndex: 1, tab: 'T',
     });
@@ -1014,37 +1088,67 @@ describe('gog_docs_cell_style', () => {
   });
   it('omits text styling flags when false', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_cell_style')!({ docId: 'd1', row: 0, col: 0, bold: false, italic: false, underline: false });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_cell_style', { docId: 'd1', row: 0, col: 0, bold: false, italic: false, underline: false });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'cell-style', 'd1', '--row=0', '--col=0'],
       { account: undefined },
     );
   });
+  it('passes border, padding and content-alignment flags', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_cell_style', {
+      docId: 'd1', row: 0, col: 0,
+      borderAll: '1pt,#000,DASH', borderTop: '2pt', borderBottom: '2pt', borderLeft: '1pt', borderRight: '1pt',
+      paddingAll: '5', paddingTop: '6pt', paddingBottom: '6pt', paddingLeft: '4mm', paddingRight: '4mm',
+      contentAlign: 'middle',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'cell-style', 'd1', '--row=0', '--col=0',
+        '--border-all=1pt,#000,DASH', '--border-top=2pt', '--border-bottom=2pt', '--border-left=1pt', '--border-right=1pt',
+        '--padding-all=5', '--padding-top=6pt', '--padding-bottom=6pt', '--padding-left=4mm', '--padding-right=4mm',
+        '--content-align=middle'],
+      { account: undefined },
+    );
+  });
 });
 describe('gog_docs_insert_image', () => {
   it('passes required --file and defaults', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_image')!({ docId: 'd1', file: '/tmp/pic.png' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_image', { docId: 'd1', file: '/tmp/pic.png' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
-      ['docs', 'insert-image', 'd1', '--file=/tmp/pic.png'],
+      ['docs', 'insert-image', 'd1', '--file=/tmp/pic.png', '--force'],
       { account: undefined },
     );
   });
   it('passes placement, sizing, naming and restriction handling', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_image')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_image', {
       docId: 'd1', file: '/tmp/pic.png', at: '{{logo}}', width: 200, height: 100,
       name: 'logo.png', parent: 'folder1', onRestricted: 'link', tab: 'T',
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-image', 'd1', '--file=/tmp/pic.png', '--at={{logo}}', '--width=200',
-        '--height=100', '--name=logo.png', '--parent=folder1', '--on-restricted=link', '--tab=T'],
+        '--height=100', '--name=logo.png', '--parent=folder1', '--on-restricted=link', '--tab=T', '--force'],
+      { account: undefined },
+    );
+  });
+  it('passes --before and --after anchors', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_image', {
+      docId: 'd1', url: 'https://x.test/i.png', before: 'Intro', after: 'Outro',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'insert-image', 'd1', '--url=https://x.test/i.png', '--before=Intro', '--after=Outro'],
       { account: undefined },
     );
   });
@@ -1052,9 +1156,9 @@ describe('gog_docs_insert_image', () => {
 describe('gog_docs_insert_person', () => {
   it('passes required --email', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_person')!({ docId: 'd1', email: 'a@b.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_person', { docId: 'd1', email: 'a@b.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-person', 'd1', '--email=a@b.com'],
       { account: undefined },
@@ -1063,9 +1167,9 @@ describe('gog_docs_insert_person', () => {
   // gog 0.23.0
   it('anchors by text with --at/--occurrence/--match-case', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_person')!({ docId: 'd1', email: 'a@b.com', at: '@alice', occurrence: 1, matchCase: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_person', { docId: 'd1', email: 'a@b.com', at: '@alice', occurrence: 1, matchCase: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-person', 'd1', '--email=a@b.com', '--at=@alice', '--occurrence=1', '--match-case'],
       { account: undefined },
@@ -1073,9 +1177,9 @@ describe('gog_docs_insert_person', () => {
   });
   it('passes --index, --at-end and --tab', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_person')!({ docId: 'd1', email: 'a@b.com', index: 0, atEnd: true, tab: 'T' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_person', { docId: 'd1', email: 'a@b.com', index: 0, atEnd: true, tab: 'T' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-person', 'd1', '--email=a@b.com', '--index=0', '--at-end', '--tab=T'],
       { account: undefined },
@@ -1085,16 +1189,16 @@ describe('gog_docs_insert_person', () => {
 describe('gog_docs_insert_date_chip', () => {
   it('omits all optional flags by default', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_date_chip')!({ docId: 'd1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_date_chip', { docId: 'd1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'insert-date-chip', 'd1'], { account: undefined });
   });
   it('passes --date, --format, --index and --at-end', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_date_chip')!({ docId: 'd1', date: '2026-06-01', format: 'iso', index: 5, atEnd: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_date_chip', { docId: 'd1', date: '2026-06-01', format: 'iso', index: 5, atEnd: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-date-chip', 'd1', '--date=2026-06-01', '--format=iso', '--index=5', '--at-end'],
       { account: undefined },
@@ -1104,9 +1208,9 @@ describe('gog_docs_insert_date_chip', () => {
 describe('gog_docs_insert_table', () => {
   it('passes required --rows and --cols', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_table')!({ docId: 'd1', rows: 3, cols: 2 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_table', { docId: 'd1', rows: 3, cols: 2 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-table', 'd1', '--rows=3', '--cols=2'],
       { account: undefined },
@@ -1114,10 +1218,10 @@ describe('gog_docs_insert_table', () => {
   });
   it('passes --values-json, --index, --tab when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
     const vj = '[["a","b"],["c","d"]]';
-    await handlers.get('gog_docs_insert_table')!({
+    await harness.callTool('gog_docs_insert_table', {
       docId: 'd1', rows: 2, cols: 2, index: 10, valuesJson: vj, tab: 'Body',
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
@@ -1127,9 +1231,9 @@ describe('gog_docs_insert_table', () => {
   });
   it('uses --at-end when atEnd is true', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_table')!({ docId: 'd1', rows: 1, cols: 1, atEnd: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_table', { docId: 'd1', rows: 1, cols: 1, atEnd: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-table', 'd1', '--rows=1', '--cols=1', '--at-end'],
       { account: undefined },
@@ -1139,9 +1243,9 @@ describe('gog_docs_insert_table', () => {
 describe('gog_docs_comments_reopen', () => {
   it('calls runOrDiagnose with docId and commentId', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_reopen')!({ docId: 'd1', commentId: 'c1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_reopen', { docId: 'd1', commentId: 'c1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'comments', 'reopen', 'd1', 'c1'],
       { account: undefined },
@@ -1149,9 +1253,9 @@ describe('gog_docs_comments_reopen', () => {
   });
   it('forwards account', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_comments_reopen')!({ docId: 'd1', commentId: 'c1', account: 'a@b.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_comments_reopen', { docId: 'd1', commentId: 'c1', account: 'a@b.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'comments', 'reopen', 'd1', 'c1'],
       { account: 'a@b.com' },
@@ -1163,16 +1267,16 @@ describe('gog_docs_comments_reopen', () => {
 describe('gog_docs_add_tab', () => {
   it('calls runOrDiagnose with only docId when no flags', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_add_tab')!({ docId: 'abc' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_add_tab', { docId: 'abc' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'add-tab', 'abc'], { account: undefined });
   });
   it('includes all flags when provided', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_add_tab')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_add_tab', {
       docId: 'abc',
       title: 'Notes',
       index: 2,
@@ -1187,9 +1291,9 @@ describe('gog_docs_add_tab', () => {
   });
   it('includes --index when zero', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_add_tab')!({ docId: 'abc', index: 0 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_add_tab', { docId: 'abc', index: 0 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'add-tab', 'abc', '--index=0'],
       { account: undefined },
@@ -1201,9 +1305,9 @@ describe('gog_docs_add_tab', () => {
 describe('gog_docs_rename_tab', () => {
   it('calls runOrDiagnose with tab and title', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_rename_tab')!({ docId: 'abc', tab: 'Old', title: 'New' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_rename_tab', { docId: 'abc', tab: 'Old', title: 'New' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'rename-tab', 'abc', '--tab=Old', '--title=New'],
       { account: undefined },
@@ -1211,9 +1315,9 @@ describe('gog_docs_rename_tab', () => {
   });
   it('forwards account override', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_rename_tab')!({ docId: 'abc', tab: 't1', title: 'New', account: 'a@b.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_rename_tab', { docId: 'abc', tab: 't1', title: 'New', account: 'a@b.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'rename-tab', 'abc', '--tab=t1', '--title=New'],
       { account: 'a@b.com' },
@@ -1225,21 +1329,21 @@ describe('gog_docs_rename_tab', () => {
 describe('gog_docs_delete_tab', () => {
   it('calls runOrDiagnose with tab', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_delete_tab')!({ docId: 'abc', tab: 'Old' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_delete_tab', { docId: 'abc', tab: 'Old' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
-      ['docs', 'delete-tab', 'abc', '--tab=Old'],
+      ['docs', 'delete-tab', 'abc', '--tab=Old', '--force'],
       { account: undefined },
     );
   });
   it('forwards account override', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_delete_tab')!({ docId: 'abc', tab: 't1', account: 'a@b.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_delete_tab', { docId: 'abc', tab: 't1', account: 'a@b.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
-      ['docs', 'delete-tab', 'abc', '--tab=t1'],
+      ['docs', 'delete-tab', 'abc', '--tab=t1', '--force'],
       { account: 'a@b.com' },
     );
   });
@@ -1249,16 +1353,16 @@ describe('gog_docs_delete_tab', () => {
 describe('gog_docs_clear', () => {
   it('calls runOrDiagnose with docId', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_clear')!({ docId: 'abc' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_clear', { docId: 'abc' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'clear', 'abc'], { account: undefined });
   });
   it('forwards account override', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_clear')!({ docId: 'abc', account: 'a@b.com' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_clear', { docId: 'abc', account: 'a@b.com' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'clear', 'abc'], { account: 'a@b.com' });
   });
 });
@@ -1267,9 +1371,9 @@ describe('gog_docs_clear', () => {
 describe('gog_docs_table_row_insert', () => {
   it('appends a row with values to a selected table', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_table_row_insert')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_row_insert', {
       docId: 'd1', table: '2', at: 'end', valuesJson: '["A","B"]', tab: 'T',
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
@@ -1279,18 +1383,18 @@ describe('gog_docs_table_row_insert', () => {
   });
   it('works with no optional flags', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_table_row_insert')!({ docId: 'd1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_row_insert', { docId: 'd1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'table-row', 'insert', 'd1'], { account: undefined });
   });
 });
 describe('gog_docs_table_row_delete', () => {
   it('deletes a row by 1-based number', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_table_row_delete')!({ docId: 'd1', row: -1, table: 'Budget' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_row_delete', { docId: 'd1', row: -1, table: 'Budget' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'table-row', 'delete', 'd1', '--row=-1', '--table=Budget'],
       { account: undefined },
@@ -1300,9 +1404,9 @@ describe('gog_docs_table_row_delete', () => {
 describe('gog_docs_table_column_insert', () => {
   it('inserts a column before a 1-based index', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_table_column_insert')!({ docId: 'd1', at: '3', table: '*', tab: 'T' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_column_insert', { docId: 'd1', at: '3', table: '*', tab: 'T' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'table-column', 'insert', 'd1', '--at=3', '--table=*', '--tab=T'],
       { account: undefined },
@@ -1312,9 +1416,9 @@ describe('gog_docs_table_column_insert', () => {
 describe('gog_docs_table_column_delete', () => {
   it('deletes a column by 1-based number', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_table_column_delete')!({ docId: 'd1', col: 2 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_column_delete', { docId: 'd1', col: 2 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'table-column', 'delete', 'd1', '--col=2'],
       { account: undefined },
@@ -1324,9 +1428,9 @@ describe('gog_docs_table_column_delete', () => {
 describe('gog_docs_table_merge', () => {
   it('merges a 1-based cell range', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_table_merge')!({ docId: 'd1', range: '1,1:2,3', table: '1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_merge', { docId: 'd1', range: '1,1:2,3', table: '1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'table-merge', 'd1', '--range=1,1:2,3', '--table=1'],
       { account: undefined },
@@ -1336,9 +1440,9 @@ describe('gog_docs_table_merge', () => {
 describe('gog_docs_table_unmerge', () => {
   it('unmerges the region containing a cell', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_table_unmerge')!({ docId: 'd1', cell: '1,1', tab: 'T' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_unmerge', { docId: 'd1', cell: '1,1', tab: 'T' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'table-unmerge', 'd1', '--cell=1,1', '--tab=T'],
       { account: undefined },
@@ -1348,9 +1452,9 @@ describe('gog_docs_table_unmerge', () => {
 describe('gog_docs_named_range_create', () => {
   it('creates a range around matched text', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_named_range_create')!({
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_named_range_create', {
       docId: 'd1', name: 'intro', at: 'Introduction', occurrence: 1, matchCase: true,
     });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
@@ -1360,9 +1464,9 @@ describe('gog_docs_named_range_create', () => {
   });
   it('creates a range from explicit indices', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_named_range_create')!({ docId: 'd1', name: 'intro', start: 5, end: 20, tab: 'T' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_named_range_create', { docId: 'd1', name: 'intro', start: 5, end: 20, tab: 'T' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'named-range', 'create', 'd1', '--name=intro', '--start=5', '--end=20', '--tab=T'],
       { account: undefined },
@@ -1372,9 +1476,9 @@ describe('gog_docs_named_range_create', () => {
 describe('gog_docs_named_range_list', () => {
   it('lists named ranges, optionally filtered by name', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_named_range_list')!({ docId: 'd1', name: 'intro' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_named_range_list', { docId: 'd1', name: 'intro' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'named-range', 'list', 'd1', '--name=intro'],
       { account: undefined },
@@ -1384,9 +1488,9 @@ describe('gog_docs_named_range_list', () => {
 describe('gog_docs_named_range_delete', () => {
   it('deletes by name or id', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_named_range_delete')!({ docId: 'd1', nameOrId: 'intro' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_named_range_delete', { docId: 'd1', nameOrId: 'intro' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'named-range', 'delete', 'd1', 'intro'],
       { account: undefined },
@@ -1396,9 +1500,9 @@ describe('gog_docs_named_range_delete', () => {
 describe('gog_docs_named_range_replace', () => {
   it('replaces range content with text', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_named_range_replace')!({ docId: 'd1', nameOrId: 'intro', text: 'New intro', tab: 'T' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_named_range_replace', { docId: 'd1', nameOrId: 'intro', text: 'New intro', tab: 'T' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'named-range', 'replace', 'd1', 'intro', '--text=New intro', '--tab=T'],
       { account: undefined },
@@ -1406,9 +1510,9 @@ describe('gog_docs_named_range_replace', () => {
   });
   it('replaces range content from a file', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_named_range_replace')!({ docId: 'd1', nameOrId: 'kix.abc', file: '/tmp/intro.txt' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_named_range_replace', { docId: 'd1', nameOrId: 'kix.abc', file: '/tmp/intro.txt' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'named-range', 'replace', 'd1', 'kix.abc', '--file=/tmp/intro.txt'],
       { account: undefined },
@@ -1418,36 +1522,36 @@ describe('gog_docs_named_range_replace', () => {
 describe('gog_docs_tables_list', () => {
   it('enumerates tables, tab-aware', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_tables_list')!({ docId: 'd1', tab: 'T' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_tables_list', { docId: 'd1', tab: 'T' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'tables', 'list', 'd1', '--tab=T'], { account: undefined });
   });
 });
 describe('gog_docs_images_list', () => {
   it('enumerates images', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_images_list')!({ docId: 'd1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_images_list', { docId: 'd1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'images', 'list', 'd1'], { account: undefined });
   });
 });
 describe('gog_docs_headings_list', () => {
   it('enumerates headings with a level filter', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_headings_list')!({ docId: 'd1', level: 2 });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_headings_list', { docId: 'd1', level: 2 });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'headings', 'list', 'd1', '--level=2'], { account: undefined });
   });
 });
 describe('gog_docs_paragraphs_list', () => {
   it('enumerates paragraphs with a style filter', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_paragraphs_list')!({ docId: 'd1', style: 'HEADING_2', tab: 'T' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_paragraphs_list', { docId: 'd1', style: 'HEADING_2', tab: 'T' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'paragraphs', 'list', 'd1', '--style=HEADING_2', '--tab=T'],
       { account: undefined },
@@ -1457,9 +1561,9 @@ describe('gog_docs_paragraphs_list', () => {
 describe('gog_docs_insert_image url mode', () => {
   it('inserts a public HTTPS image by url without file', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_insert_image')!({ docId: 'd1', url: 'https://x.test/i.png', at: '{{logo}}' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_image', { docId: 'd1', url: 'https://x.test/i.png', at: '{{logo}}' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['docs', 'insert-image', 'd1', '--url=https://x.test/i.png', '--at={{logo}}'],
       { account: undefined },
@@ -1469,16 +1573,16 @@ describe('gog_docs_insert_image url mode', () => {
 describe('gog_docs_read json tab targeting', () => {
   it('passes --tab through to docs raw in json mode', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_read')!({ docId: 'd1', format: 'json', tab: 'T' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_read', { docId: 'd1', format: 'json', tab: 'T' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'raw', 'd1', '--pretty', '--tab=T'], { account: undefined });
   });
   it('passes --all-tabs through to docs raw in json mode', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_docs_read')!({ docId: 'd1', format: 'json', allTabs: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_read', { docId: 'd1', format: 'json', allTabs: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'raw', 'd1', '--pretty', '--all-tabs'], { account: undefined });
   });
 });
@@ -1487,9 +1591,9 @@ describe('gog_docs_read json tab targeting', () => {
 describe('batch lifecycle tools', () => {
   it('gog_batch_begin opens a batch for a doc', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_batch_begin')!({ docId: 'd1', name: 'big-edit' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_batch_begin', { docId: 'd1', name: 'big-edit' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['batch', 'begin', '--doc=d1', '--name=big-edit'],
       { account: undefined },
@@ -1497,9 +1601,9 @@ describe('batch lifecycle tools', () => {
   });
   it('gog_batch_begin opens a batch without a name', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_batch_begin')!({ docId: 'd1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_batch_begin', { docId: 'd1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['batch', 'begin', '--doc=d1'],
       { account: undefined },
@@ -1507,9 +1611,9 @@ describe('batch lifecycle tools', () => {
   });
   it('gog_batch_end submits with split/recovery modes', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_batch_end')!({ batchId: 'b1', autoSplit: true, continueOnError: true });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_batch_end', { batchId: 'b1', autoSplit: true, continueOnError: true });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['batch', 'end', 'b1', '--auto-split', '--continue-on-error'],
       { account: undefined },
@@ -1517,9 +1621,9 @@ describe('batch lifecycle tools', () => {
   });
   it('gog_batch_end submits atomically with no options', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_batch_end')!({ batchId: 'b1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_batch_end', { batchId: 'b1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(
       ['batch', 'end', 'b1'],
       { account: undefined },
@@ -1527,32 +1631,32 @@ describe('batch lifecycle tools', () => {
   });
   it('gog_batch_abort discards a batch', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_batch_abort')!({ batchId: 'b1' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_batch_abort', { batchId: 'b1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['batch', 'abort', 'b1'], { account: undefined });
   });
   it('gog_batch_list / gog_batch_show read batch state', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_batch_list')!({});
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_batch_list', {});
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['batch', 'list'], { account: undefined });
-    await handlers.get('gog_batch_show')!({ batchId: 'b1' });
+    await harness.callTool('gog_batch_show', { batchId: 'b1' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['batch', 'show', 'b1'], { account: undefined });
   });
   it('gog_batch_prune deletes stale batches', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_batch_prune')!({ olderThan: '72h' });
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_batch_prune', { olderThan: '72h' });
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['batch', 'prune', '--older-than=72h'], { account: undefined });
   });
   it('gog_batch_prune without a duration prunes with no filter', async () => {
-    vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-    const handlers = setupHandlers();
-    await handlers.get('gog_batch_prune')!({});
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_batch_prune', {});
     expect(lib.runOrDiagnose).toHaveBeenCalledWith(['batch', 'prune'], { account: undefined });
   });
 });
@@ -1583,9 +1687,9 @@ describe('batch param on docs mutation tools (gog 0.25.0)', () => {
   for (const { tool, input, expected } of cases) {
     it(`${tool} appends to a batch via --batch`, async () => {
-      vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-      const handlers = setupHandlers();
-      await handlers.get(tool)!(input);
+      vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+      const harness = await setupHandlers();
+      await harness.callTool(tool, input);
       expect(lib.runOrDiagnose).toHaveBeenCalledWith(expected, { account: undefined });
     });
   }
@@ -1616,9 +1720,9 @@ describe('tab/table selector branch coverage', () => {
   ];
   for (const { tool, input, expected } of cases) {
     it(`${tool} passes its tab/table selectors`, async () => {
-      vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-      const handlers = setupHandlers();
-      await handlers.get(tool)!(input);
+      vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+      const harness = await setupHandlers();
+      await harness.callTool(tool, input);
       expect(lib.runOrDiagnose).toHaveBeenCalledWith(expected, { account: undefined });
     });
   }
@@ -1635,10 +1739,420 @@ describe('bare-call branch coverage (no optional flags)', () => {
   ];
   for (const { tool, input, expected } of cases) {
     it(`${tool} with no optional flags`, async () => {
-      vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
-      const handlers = setupHandlers();
-      await handlers.get(tool)!(input);
+      vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+      const harness = await setupHandlers();
+      await harness.callTool(tool, input);
       expect(lib.runOrDiagnose).toHaveBeenCalledWith(expected, { account: undefined });
     });
   }
 });
+// ===========================================================================
+// gog 0.30 docs authoring (PR2)
+// ===========================================================================
+describe('segment targeting (gog 0.30)', () => {
+  it('gog_docs_insert forwards --segment', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert', { docId: 'd1', content: 'hi', segment: 'kix.h1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'insert', 'd1', 'hi', '--segment=kix.h1'],
+      { account: undefined },
+    );
+  });
+  it('gog_docs_update forwards --segment', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_update', { docId: 'd1', text: 'hi', segment: 'kix.f1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'update', 'd1', '--text=hi', '--segment=kix.f1'],
+      { account: undefined },
+    );
+  });
+  it('gog_docs_delete forwards --segment', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_delete', { docId: 'd1', start: 1, end: 5, segment: 'kix.h1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'delete', '--start=1', '--end=5', 'd1', '--segment=kix.h1'],
+      { account: undefined },
+    );
+  });
+  it('gog_docs_find_range forwards --segment', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_find_range', { docId: 'd1', text: 'Heading', segment: 'kix.h1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'find-range', 'd1', 'Heading', '--segment=kix.h1'],
+      { account: undefined },
+    );
+  });
+});
+describe('gog_docs_format paragraph list / indent / spacing / keep (gog 0.30)', () => {
+  it('adds bullets, indentation, spacing, keep (true) and segment', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', {
+      docId: 'd1', bullets: true, bulletPreset: 'BULLET_DISC_CIRCLE_SQUARE',
+      indentStart: 18, indentEnd: 6, indentFirstLine: 36, spaceAbove: 4, spaceBelow: 8,
+      keepLinesTogether: true, keepWithNext: true, segment: 'kix.h1',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'format', 'd1', '--bullets', '--bullet-preset=BULLET_DISC_CIRCLE_SQUARE',
+        '--indent-start=18', '--indent-end=6', '--indent-first-line=36', '--space-above=4', '--space-below=8',
+        '--keep-lines-together', '--keep-with-next', '--segment=kix.h1'],
+      { account: undefined },
+    );
+  });
+  it('adds numbering, removal and negated keep (false)', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', {
+      docId: 'd1', ordered: true, noBullets: true, keepLinesTogether: false, keepWithNext: false,
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'format', 'd1', '--ordered', '--no-bullets', '--no-keep-lines-together', '--no-keep-with-next'],
+      { account: undefined },
+    );
+  });
+});
+describe('gog_docs_insert_footnote', () => {
+  it('passes text and anchor flags', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_footnote', {
+      docId: 'd1', text: 'note', file: '/tmp/n.txt', index: 5, atEnd: true,
+      at: 'word', occurrence: 2, matchCase: true, tab: 'T',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'insert-footnote', 'd1', '--text=note', '--file=/tmp/n.txt', '--index=5', '--at-end',
+        '--at=word', '--occurrence=2', '--match-case', '--tab=T'],
+      { account: undefined },
+    );
+  });
+  it('works with just docId', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_footnote', { docId: 'd1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'insert-footnote', 'd1'], { account: undefined });
+  });
+});
+describe('gog_docs_insert_section_break', () => {
+  it('passes type and anchor flags', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_section_break', {
+      docId: 'd1', type: 'continuous', index: 3, atEnd: true, at: 'x',
+      occurrence: 1, matchCase: true, tab: 'T', batch: 'b1',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'insert-section-break', 'd1', '--type=continuous', '--index=3', '--at-end',
+        '--at=x', '--occurrence=1', '--match-case', '--tab=T', '--batch=b1'],
+      { account: undefined },
+    );
+  });
+  it('works with just docId', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_section_break', { docId: 'd1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'insert-section-break', 'd1'], { account: undefined });
+  });
+});
+describe('gog_docs_insert_horizontal_rule', () => {
+  it('passes anchor flags', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_horizontal_rule', {
+      docId: 'd1', index: 2, atEnd: true, at: 'x', occurrence: 1, matchCase: true, tab: 'T', batch: 'b1',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'insert-horizontal-rule', 'd1', '--index=2', '--at-end', '--at=x',
+        '--occurrence=1', '--match-case', '--tab=T', '--batch=b1'],
+      { account: undefined },
+    );
+  });
+  it('works with just docId', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_insert_horizontal_rule', { docId: 'd1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'insert-horizontal-rule', 'd1'], { account: undefined });
+  });
+});
+describe('gog_docs_section_columns', () => {
+  it('passes count, separator and anchor flags', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_section_columns', {
+      docId: 'd1', count: 2, separator: 'between', index: 4, atEnd: true,
+      at: 'x', occurrence: 1, matchCase: true, tab: 'T', batch: 'b1',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'section-columns', 'd1', '--count=2', '--separator=between', '--index=4', '--at-end',
+        '--at=x', '--occurrence=1', '--match-case', '--tab=T', '--batch=b1'],
+      { account: undefined },
+    );
+  });
+  it('passes only the required --count', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_section_columns', { docId: 'd1', count: 1 });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'section-columns', 'd1', '--count=1'], { account: undefined });
+  });
+});
+describe('gog_docs header lifecycle', () => {
+  it('header_list passes --tab', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_header_list', { docId: 'd1', tab: 'T' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'header', 'list', 'd1', '--tab=T'], { account: undefined });
+  });
+  it('header_list bare', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_header_list', { docId: 'd1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'header', 'list', 'd1'], { account: undefined });
+  });
+  it('header_create passes text and anchor flags', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_header_create', {
+      docId: 'd1', text: 'Title', file: '/tmp/h.txt', index: 1, atEnd: true,
+      at: 'x', occurrence: 1, matchCase: true, tab: 'T',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'header', 'create', 'd1', '--text=Title', '--file=/tmp/h.txt', '--index=1', '--at-end',
+        '--at=x', '--occurrence=1', '--match-case', '--tab=T'],
+      { account: undefined },
+    );
+  });
+  it('header_create bare', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_header_create', { docId: 'd1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'header', 'create', 'd1'], { account: undefined });
+  });
+  it('header_delete passes id and --tab', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_header_delete', { docId: 'd1', headerId: 'kix.h1', tab: 'T' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'header', 'delete', 'd1', 'kix.h1', '--tab=T', '--force'], { account: undefined });
+  });
+  it('header_delete bare', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_header_delete', { docId: 'd1', headerId: 'kix.h1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'header', 'delete', 'd1', 'kix.h1', '--force'], { account: undefined });
+  });
+});
+describe('gog_docs footer lifecycle', () => {
+  it('footer_list passes --tab', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_footer_list', { docId: 'd1', tab: 'T' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'footer', 'list', 'd1', '--tab=T'], { account: undefined });
+  });
+  it('footer_list bare', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_footer_list', { docId: 'd1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'footer', 'list', 'd1'], { account: undefined });
+  });
+  it('footer_create passes text and anchor flags', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_footer_create', {
+      docId: 'd1', text: 'Foot', file: '/tmp/f.txt', index: 1, atEnd: true,
+      at: 'x', occurrence: 1, matchCase: true, tab: 'T',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'footer', 'create', 'd1', '--text=Foot', '--file=/tmp/f.txt', '--index=1', '--at-end',
+        '--at=x', '--occurrence=1', '--match-case', '--tab=T'],
+      { account: undefined },
+    );
+  });
+  it('footer_create bare', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_footer_create', { docId: 'd1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'footer', 'create', 'd1'], { account: undefined });
+  });
+  it('footer_delete passes id and --tab', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_footer_delete', { docId: 'd1', footerId: 'kix.f1', tab: 'T' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'footer', 'delete', 'd1', 'kix.f1', '--tab=T', '--force'], { account: undefined });
+  });
+  it('footer_delete bare', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_footer_delete', { docId: 'd1', footerId: 'kix.f1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'footer', 'delete', 'd1', 'kix.f1', '--force'], { account: undefined });
+  });
+});
+describe('gog_docs_replace_image', () => {
+  it('passes url + object-id targeting', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_replace_image', {
+      docId: 'd1', url: 'https://x.test/i.png', objectId: 'kix.img1', tab: 'T',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'replace-image', 'd1', '--url=https://x.test/i.png', '--object-id=kix.img1', '--tab=T'],
+      { account: undefined },
+    );
+  });
+  it('passes file + alt-text + name + parent targeting', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_replace_image', {
+      docId: 'd1', file: '/tmp/i.png', matchAlt: 'logo', name: 'new.png', parent: 'folder1',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'replace-image', 'd1', '--file=/tmp/i.png', '--match-alt=logo', '--name=new.png', '--parent=folder1', '--force'],
+      { account: undefined },
+    );
+  });
+});
+describe('gog_docs_table_row_pin_header', () => {
+  it('passes --rows and table selector', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_row_pin_header', { docId: 'd1', rows: 2, table: '1', tab: 'T' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'table-row', 'pin-header', 'd1', '--rows=2', '--table=1', '--tab=T'],
+      { account: undefined },
+    );
+  });
+  it('unpins with rows=0 and no selector', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_row_pin_header', { docId: 'd1', rows: 0 });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'table-row', 'pin-header', 'd1', '--rows=0'],
+      { account: undefined },
+    );
+  });
+});
+describe('gog_docs_table_row_style', () => {
+  it('passes row, min-height and --prevent-overflow (true)', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_row_style', {
+      docId: 'd1', row: 1, minHeight: '20pt', preventOverflow: true, table: '1', tab: 'T',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'table-row', 'style', 'd1', '--row=1', '--min-height=20pt', '--prevent-overflow', '--table=1', '--tab=T'],
+      { account: undefined },
+    );
+  });
+  it('uses --no-prevent-overflow (false) and styles all rows', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_row_style', { docId: 'd1', preventOverflow: false });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'table-row', 'style', 'd1', '--no-prevent-overflow'],
+      { account: undefined },
+    );
+  });
+});
+describe('gog_docs_table_row_style overflow-unset branch', () => {
+  it('omits the overflow flag when preventOverflow is unset', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_table_row_style', { docId: 'd1', row: 2, minHeight: '30pt' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'table-row', 'style', 'd1', '--row=2', '--min-height=30pt'],
+      { account: undefined },
+    );
+  });
+});
+// --- gog_docs_suggestions_list ---
+describe('gog_docs_suggestions_list', () => {
+  it('calls runOrDiagnose with correct args', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_suggestions_list', { docId: 'd1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(['docs', 'suggestions', 'list', 'd1'], { account: undefined });
+  });
+  it('includes --tab when provided', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_suggestions_list', { docId: 'd1', tab: 'Notes' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'suggestions', 'list', 'd1', '--tab=Notes'],
+      { account: undefined },
+    );
+  });
+  it('forwards account override', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_suggestions_list', { docId: 'd1', account: 'other@gmail.com' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'suggestions', 'list', 'd1'],
+      { account: 'other@gmail.com' },
+    );
+  });
+});
+// --- gog_docs_format spacingMode ---
+describe('gog_docs_format spacingMode', () => {
+  it('passes --spacing-mode alongside space-above/space-below', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', {
+      docId: 'd1', spaceAbove: 6, spaceBelow: 12, spacingMode: 'COLLAPSE_LISTS',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'format', 'd1', '--space-above=6', '--space-below=12', '--spacing-mode=COLLAPSE_LISTS'],
+      { account: undefined },
+    );
+  });
+  it('omits --spacing-mode when unset', async () => {
+    vi.mocked(lib.runOrDiagnose).mockResolvedValue(rawTextResult('{}'));
+    const harness = await setupHandlers();
+    await harness.callTool('gog_docs_format', { docId: 'd1', spaceAbove: 6 });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['docs', 'format', 'd1', '--space-above=6'],
+      { account: undefined },
+    );
+  });
+});