@gmickel/gno 1.16.0 → 1.18.0

package/src/mcp/resources/index.ts

@@ -92,6 +92,9 @@ function formatResourceContent(
  * Register gno:// resources with the MCP server.
  */
 export function registerResources(server: McpServer, ctx: ToolContext): void {
+  const withSnapshot = <T>(operation: () => Promise<T>): Promise<T> =>
+    ctx.runWithSnapshot?.(operation) ?? operation();
+
   // Resource template for gno://{collection}/{path} URIs
   const template = new ResourceTemplate(`${URI_PREFIX}{collection}/{+path}`, {
     list: async () => {
@@ -113,101 +116,103 @@ export function registerResources(server: McpServer, ctx: ToolContext): void {
   });
 
   // Register the template-based resource handler
-  server.resource("gno-document", template, {}, async (uri, _variables) => {
-    // Check shutdown before acquiring mutex
-    if (ctx.isShuttingDown()) {
-      throw new Error("Server is shutting down");
-    }
-
-    // Serialize resource reads same as tools (prevent concurrent DB access + shutdown race)
-    const release = await ctx.toolMutex.acquire();
-    try {
-      // Use parseUri for proper URL decoding (handles %20, etc.)
-      const parsed = parseUri(uri.href);
-      if (!parsed) {
-        throw new Error(`Invalid gno:// URI: ${uri.href}`);
-      }
-
-      const resolution = resolveEffectiveIndex([uri.href], ctx.indexName);
-      if (!resolution.ok) {
-        throw new Error(resolution.error);
+  server.resource("gno-document", template, {}, (uri, _variables) =>
+    withSnapshot(async () => {
+      // Check shutdown before acquiring mutex
+      if (ctx.isShuttingDown()) {
+        throw new Error("Server is shutting down");
       }
-      const scoped = await openScopedIndexStore({
-        activeStore: ctx.store,
-        activeIndexName: ctx.indexName,
-        requestedIndexName: resolution.value.indexName,
-        config: ctx.config,
-        configPath: ctx.actualConfigPath,
-      });
 
+      // Serialize resource reads same as tools (prevent concurrent DB access + shutdown race)
+      const release = await ctx.toolMutex.acquire();
       try {
-        const { collection, path } = parsed;
-
-        // Validate collection exists
-        const collectionExists = ctx.collections.some(
-          (c) => c.name === collection
-        );
-        if (!collectionExists) {
-          throw new Error(`Collection not found: ${collection}`);
+        // Use parseUri for proper URL decoding (handles %20, etc.)
+        const parsed = parseUri(uri.href);
+        if (!parsed) {
+          throw new Error(`Invalid gno:// URI: ${uri.href}`);
         }
 
-        // Look up document (path is properly decoded by parseUri)
-        const docResult = await scoped.store.getDocument(collection, path);
-        if (!docResult.ok) {
-          throw new Error(
-            `Failed to lookup document: ${docResult.error.message}`
-          );
+        const resolution = resolveEffectiveIndex([uri.href], ctx.indexName);
+        if (!resolution.ok) {
+          throw new Error(resolution.error);
         }
+        const scoped = await openScopedIndexStore({
+          activeStore: ctx.store,
+          activeIndexName: ctx.indexName,
+          requestedIndexName: resolution.value.indexName,
+          config: ctx.config,
+          configPath: ctx.actualConfigPath,
+        });
+
+        try {
+          const { collection, path } = parsed;
+
+          // Validate collection exists
+          const collectionExists = ctx.collections.some(
+            (c) => c.name === collection
+          );
+          if (!collectionExists) {
+            throw new Error(`Collection not found: ${collection}`);
+          }
 
-        const doc = docResult.value;
-        if (!doc) {
-          throw new Error(`Document not found: ${uri.href}`);
-        }
+          // Look up document (path is properly decoded by parseUri)
+          const docResult = await scoped.store.getDocument(collection, path);
+          if (!docResult.ok) {
+            throw new Error(
+              `Failed to lookup document: ${docResult.error.message}`
+            );
+          }
 
-        // Get content
-        if (!doc.mirrorHash) {
-          throw new Error(`Document has no indexed content: ${uri.href}`);
-        }
+          const doc = docResult.value;
+          if (!doc) {
+            throw new Error(`Document not found: ${uri.href}`);
+          }
+
+          // Get content
+          if (!doc.mirrorHash) {
+            throw new Error(`Document has no indexed content: ${uri.href}`);
+          }
+
+          const contentResult = await scoped.store.getContent(doc.mirrorHash);
+          if (!contentResult.ok) {
+            throw new Error(
+              `Failed to read content: ${contentResult.error.message}`
+            );
+          }
+
+          const content = contentResult.value ?? "";
 
-        const contentResult = await scoped.store.getContent(doc.mirrorHash);
-        if (!contentResult.ok) {
-          throw new Error(
-            `Failed to read content: ${contentResult.error.message}`
+          // Format with header and line numbers
+          const formattedContent = formatResourceContent(
+            doc,
+            content,
+            ctx,
+            scoped.indexName
+          );
+
+          // Build canonical URI
+          const canonicalUri = decorateUriForIndex(
+            buildUri(collection, path),
+            scoped.indexName
           );
-        }
 
-        const content = contentResult.value ?? "";
-
-        // Format with header and line numbers
-        const formattedContent = formatResourceContent(
-          doc,
-          content,
-          ctx,
-          scoped.indexName
-        );
-
-        // Build canonical URI
-        const canonicalUri = decorateUriForIndex(
-          buildUri(collection, path),
-          scoped.indexName
-        );
-
-        return {
-          contents: [
-            {
-              uri: canonicalUri,
-              mimeType: "text/markdown",
-              text: formattedContent,
-            },
-          ],
-        };
+          return {
+            contents: [
+              {
+                uri: canonicalUri,
+                mimeType: "text/markdown",
+                text: formattedContent,
+              },
+            ],
+          };
+        } finally {
+          await scoped.close();
+        }
       } finally {
-        await scoped.close();
+        release();
       }
-    } finally {
-      release();
-    }
-  });
+    })
+  );
 
   // Register gno://tags resource for listing tags
   // Use ResourceTemplate with RFC6570 query expansion for proper routing
@@ -231,65 +236,67 @@ export function registerResources(server: McpServer, ctx: ToolContext): void {
     "gno-tags",
     tagsTemplate,
     { mimeType: "application/json" },
-    async (uri) => {
-      // Check shutdown before acquiring mutex
-      if (ctx.isShuttingDown()) {
-        throw new Error("Server is shutting down");
-      }
-
-      const release = await ctx.toolMutex.acquire();
-      try {
-        // Parse query params from URI
-        const url = new URL(uri.href);
-        const collectionParam = url.searchParams.get("collection") || undefined;
-        const prefixParam = url.searchParams.get("prefix") || undefined;
-
-        // Normalize and validate collection (case-insensitive)
-        let collection: string | undefined;
-        if (collectionParam) {
-          collection = normalizeCollectionName(collectionParam);
-          const exists = ctx.collections.some(
-            (c) => c.name.toLowerCase() === collection
-          );
-          if (!exists) {
-            throw new Error(
-              `${MCP_ERRORS.NOT_FOUND.code}: Collection not found: ${collectionParam}`
-            );
-          }
+    (uri) =>
+      withSnapshot(async () => {
+        // Check shutdown before acquiring mutex
+        if (ctx.isShuttingDown()) {
+          throw new Error("Server is shutting down");
         }
 
-        // Normalize and validate prefix
-        let prefix: string | undefined;
-        if (prefixParam) {
-          const trimmed = prefixParam.trim().replace(/\/+$/, "");
-          if (trimmed.length > 0) {
-            prefix = normalizeTag(trimmed);
-            if (!validateTag(prefix)) {
+        const release = await ctx.toolMutex.acquire();
+        try {
+          // Parse query params from URI
+          const url = new URL(uri.href);
+          const collectionParam =
+            url.searchParams.get("collection") || undefined;
+          const prefixParam = url.searchParams.get("prefix") || undefined;
+
+          // Normalize and validate collection (case-insensitive)
+          let collection: string | undefined;
+          if (collectionParam) {
+            collection = normalizeCollectionName(collectionParam);
+            const exists = ctx.collections.some(
+              (c) => c.name.toLowerCase() === collection
+            );
+            if (!exists) {
               throw new Error(
-                `${MCP_ERRORS.INVALID_INPUT.code}: Invalid tag prefix "${prefixParam}"`
+                `${MCP_ERRORS.NOT_FOUND.code}: Collection not found: ${collectionParam}`
               );
             }
           }
-        }
 
-        // Get tag counts
-        const result = await ctx.store.getTagCounts({ collection, prefix });
-        if (!result.ok) {
-          throw new Error(`Failed to get tags: ${result.error.message}`);
-        }
+          // Normalize and validate prefix
+          let prefix: string | undefined;
+          if (prefixParam) {
+            const trimmed = prefixParam.trim().replace(/\/+$/, "");
+            if (trimmed.length > 0) {
+              prefix = normalizeTag(trimmed);
+              if (!validateTag(prefix)) {
+                throw new Error(
+                  `${MCP_ERRORS.INVALID_INPUT.code}: Invalid tag prefix "${prefixParam}"`
+                );
+              }
+            }
+          }
 
-        return {
-          contents: [
-            {
-              uri: uri.href,
-              mimeType: "application/json",
-              text: formatTagsContent(result.value, collection, prefix),
-            },
-          ],
-        };
-      } finally {
-        release();
-      }
-    }
+          // Get tag counts
+          const result = await ctx.store.getTagCounts({ collection, prefix });
+          if (!result.ok) {
+            throw new Error(`Failed to get tags: ${result.error.message}`);
+          }
+
+          return {
+            contents: [
+              {
+                uri: uri.href,
+                mimeType: "application/json",
+                text: formatTagsContent(result.value, collection, prefix),
+              },
+            ],
+          };
+        } finally {
+          release();
+        }
+      })
   );
 }

package/src/mcp/server.ts

@@ -5,67 +5,28 @@
  * @module src/mcp/server
  */
 
-import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 // node:path for join/dirname (no Bun path utils)
 import { dirname, join } from "node:path";
 
-import type { Collection, Config } from "../config/types";
-import type { SqliteAdapter } from "../store/sqlite/adapter";
-
-import { MCP_SERVER_NAME, VERSION, getIndexDbPath } from "../app/constants";
+import {
+  DEFAULT_INDEX_NAME,
+  MCP_SERVER_NAME,
+  VERSION,
+  getIndexDbPath,
+} from "../app/constants";
+import { canonicalizeIndexName } from "../app/index-name";
 import { JobManager } from "../core/job-manager";
 import { envIsSet } from "../llm/policy";
 import { MCP_ACTIVATION_VERIFICATION_ENV } from "./activation-verification-mode";
+import {
+  createMcpServerSurface,
+  createToolContext,
+  Mutex,
+  type ToolContext,
+} from "./context";
 
-// ─────────────────────────────────────────────────────────────────────────────
-// Simple Promise Mutex (avoids async-mutex dependency)
-// ─────────────────────────────────────────────────────────────────────────────
-
-class Mutex {
-  #locked = false;
-  readonly #queue: Array<() => void> = [];
-
-  acquire(): Promise<() => void> {
-    return new Promise((resolve) => {
-      const tryAcquire = () => {
-        if (this.#locked) {
-          this.#queue.push(tryAcquire);
-        } else {
-          this.#locked = true;
-          resolve(() => this.#release());
-        }
-      };
-      tryAcquire();
-    });
-  }
-
-  #release(): void {
-    this.#locked = false;
-    const next = this.#queue.shift();
-    if (next) {
-      next();
-    }
-  }
-}
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Tool Context
-// ─────────────────────────────────────────────────────────────────────────────
-
-export interface ToolContext {
-  store: SqliteAdapter;
-  config: Config;
-  collections: Collection[];
-  actualConfigPath: string;
-  indexName?: string;
-  toolMutex: Mutex;
-  jobManager: JobManager;
-  serverInstanceId: string;
-  writeLockPath: string;
-  enableWrite: boolean;
-  isShuttingDown: () => boolean;
-}
+export type { ToolContext } from "./context";
 
 // ─────────────────────────────────────────────────────────────────────────────
 // Server Options
@@ -120,7 +81,7 @@ export async function startMcpServer(options: McpServerOptions): Promise<void> {
     MCP_ACTIVATION_VERIFICATION_ENV
   );
   const init = await initStore({
-    indexName: options.indexName,
+    indexName: canonicalizeIndexName(options.indexName ?? DEFAULT_INDEX_NAME),
     configPath: options.configPath,
     syncConfig: !activationVerification,
   });
@@ -130,21 +91,7 @@ export async function startMcpServer(options: McpServerOptions): Promise<void> {
     process.exit(1);
   }
 
-  const { store, config, collections, actualConfigPath } = init;
-
-  // Create MCP server
-  const server = new McpServer(
-    {
-      name: MCP_SERVER_NAME,
-      version: VERSION,
-    },
-    {
-      capabilities: {
-        tools: { listChanged: false },
-        resources: { subscribe: false, listChanged: false },
-      },
-    }
-  );
+  const { store, config, actualConfigPath } = init;
 
   // Sequential execution mutex
   const toolMutex = new Mutex();
@@ -167,27 +114,26 @@ export async function startMcpServer(options: McpServerOptions): Promise<void> {
   let shuttingDown = false;
 
   // Tool context (passed to all handlers)
-  const ctx: ToolContext = {
+  let currentConfig = config;
+  const ctx: ToolContext = createToolContext({
     store,
-    config,
-    collections,
+    getConfig: () => currentConfig,
+    setConfig: (nextConfig) => {
+      currentConfig = nextConfig;
+    },
     actualConfigPath,
-    indexName: options.indexName,
+    indexName: canonicalizeIndexName(options.indexName ?? DEFAULT_INDEX_NAME),
     toolMutex,
     jobManager,
     serverInstanceId,
     writeLockPath,
     enableWrite,
     isShuttingDown: () => shuttingDown,
-  };
-
-  // Register tools (T10.2)
-  const { registerTools } = await import("./tools/index.js");
-  registerTools(server, ctx);
-
-  // Register resources (T10.3)
-  const { registerResources } = await import("./resources/index.js");
-  registerResources(server, ctx);
+  });
+  const server = createMcpServerSurface(ctx, {
+    name: MCP_SERVER_NAME,
+    version: VERSION,
+  });
 
   if (options.verbose) {
     console.error(

package/src/mcp/tools/add-collection.ts

@@ -14,6 +14,7 @@ import { applyConfigChange } from "../../core/config-mutation";
 import { MCP_ERRORS } from "../../core/errors";
 import { acquireWriteLock, type WriteLockHandle } from "../../core/file-lock";
 import { JobError } from "../../core/job-manager";
+import { recordContentMutation } from "../../core/mutation-generations";
 import {
   normalizeCollectionName,
   validateCollectionRoot,
@@ -152,9 +153,10 @@ export function handleAddCollection(
                   gitPull: args.gitPull ?? false,
                   runUpdateCmd: false,
                 },
-                ctx.config
+                mutationResult.config
               )
             );
+            recordContentMutation(result, ctx.markContentMutation);
 
             return {
               collections: [result],

package/src/mcp/tools/capture.ts

@@ -210,6 +210,9 @@ export function handleCapture(
             serverInstanceId: ctx.serverInstanceId,
           }) as McpCaptureResult;
         }
+        if (syncResult.status === "added" || syncResult.status === "updated") {
+          ctx.markContentMutation?.();
+        }
 
         let docid = syncResult.docid;
         let documentId: number | undefined;

package/src/mcp/tools/clear-collection-embeddings.ts

@@ -8,6 +8,7 @@ import type { ToolContext } from "../server";
 
 import { MCP_ERRORS } from "../../core/errors";
 import { withWriteLock } from "../../core/file-lock";
+import { recordIndexMutation } from "../../core/mutation-generations";
 import { resolveModelUri } from "../../llm/registry";
 import { runTool, type ToolResult } from "./index";
 
@@ -81,6 +82,7 @@ export function handleClearCollectionEmbeddings(
         if (!result.ok) {
           throw new Error(`${result.error.code}: ${result.error.message}`);
         }
+        recordIndexMutation(result.value.deletedVectors, ctx.markIndexMutation);
 
         return {
           ...result.value,