@eclipse-glsp/server-mcp 2.7.0-next.10 → 2.7.0-next.11

package/src/{index.ts → common/index.ts}

@@ -1,5 +1,5 @@
 /********************************************************************************
- * Copyright (c) 2025-2026 EclipseSource and others.
+ * Copyright (c) 2026 EclipseSource and others.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0 which is available at
@@ -13,14 +13,16 @@
  *
  * SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0
  ********************************************************************************/
+
+export * from './modules/abstract-mcp-server-module';
 export * from './modules/mcp-diagram-module';
-export * from './modules/mcp-server-module';
 export * from './prompts/handlers/describe-diagram-mcp-prompt-handler';
 export * from './prompts/handlers/suggest-improvements-mcp-prompt-handler';
 export * from './resources/handlers/diagram-png-mcp-resource-handler';
 export * from './resources/handlers/diagram-svg-mcp-resource-handler';
 export * from './resources/services/element-types-provider';
 export * from './resources/services/mcp-model-serializer';
+export * from './server/abstract-mcp-server-launcher';
 export * from './server/glsp-mcp-server';
 export * from './server/lru-event-store';
 export * from './server/mcp-diagram-handler-dispatcher';
@@ -28,7 +30,6 @@ export * from './server/mcp-diagram-prompt-handler-registry';
 export * from './server/mcp-diagram-resource-handler-registry';
 export * from './server/mcp-diagram-tool-handler-registry';
 export * from './server/mcp-handler-shared';
-export * from './server/mcp-http-transport';
 export * from './server/mcp-id-alias-service';
 export * from './server/mcp-input-schemas';
 export * from './server/mcp-label-provider';
@@ -40,7 +41,6 @@ export * from './server/mcp-progress-reporter';
 export * from './server/mcp-prompt-handler';
 export * from './server/mcp-request-context';
 export * from './server/mcp-resource-handler';
-export * from './server/mcp-server-launcher';
 export * from './server/mcp-session';
 export * from './server/mcp-tool-handler';
 export * from './tools/handlers/count-elements-mcp-tool-handler';

package/src/{modules/mcp-server-module.ts → common/modules/abstract-mcp-server-module.ts}

@@ -29,24 +29,24 @@ import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { interfaces } from 'inversify';
 import { DescribeDiagramMcpPromptHandler } from '../prompts/handlers/describe-diagram-mcp-prompt-handler';
 import { SuggestImprovementsMcpPromptHandler } from '../prompts/handlers/suggest-improvements-mcp-prompt-handler';
-import { ElementTypesMcpToolHandler } from '../tools/handlers/element-types-mcp-tool-handler';
-import { SessionInfoMcpToolHandler } from '../tools/handlers/session-info-mcp-tool-handler';
+import { AbstractMcpServerLauncher } from '../server/abstract-mcp-server-launcher';
 import { DefaultGLSPMcpServer, GLSPMcpServerFactory } from '../server/glsp-mcp-server';
+import { LruEventStore } from '../server/lru-event-store';
 import { DefaultMcpDiagramHandlerDispatcher, McpDiagramHandlerDispatcher } from '../server/mcp-diagram-handler-dispatcher';
 import { DefaultMcpLogLevelRegistry, McpLogLevelRegistry } from '../server/mcp-log-level-registry';
-import { LruEventStore } from '../server/lru-event-store';
-import { McpHttpTransport } from '../server/mcp-http-transport';
 import { McpLogger } from '../server/mcp-logger';
 import { McpServerDefaults, McpServerOptions } from '../server/mcp-options';
 import { McpProgressReporter } from '../server/mcp-progress-reporter';
 import { McpPromptHandler } from '../server/mcp-prompt-handler';
+import { McpRequestContext } from '../server/mcp-request-context';
 import { McpResourceHandler } from '../server/mcp-resource-handler';
-import { McpServerLauncher } from '../server/mcp-server-launcher';
 import { McpToolHandler } from '../server/mcp-tool-handler';
+import { ElementTypesMcpToolHandler } from '../tools/handlers/element-types-mcp-tool-handler';
+import { SessionInfoMcpToolHandler } from '../tools/handlers/session-info-mcp-tool-handler';
 
 /**
  * GLSP-generic default agent persona — adopters typically pass a product-specific persona to
- * the {@link DefaultMcpServerModule} constructor (e.g. workflow-server might say "You are the
+ * the concrete server module's constructor (e.g. workflow-server might say "You are the
  * Workflow Modeling Agent...").
  *
  * **Spec note.** This is wired to the MCP `instructions` field, which the spec describes as
@@ -55,7 +55,7 @@ import { McpToolHandler } from '../server/mcp-tool-handler';
  * via `mcpDefaults.agentPersona`. Don't trim on autopilot if a "concise" interpretation drifts
  * back: the verbose form materially improves LLM tool-use compliance for graphical modelling.
  */
-const DEFAULT_AGENT_PERSONA = `
+export const DEFAULT_AGENT_PERSONA = `
 You are the GLSP Modeling Agent. Your primary goal is to assist in the creation and modification of graphical models using the
 GLSP MCP server. You have to adhere to the following principles:
 - MCP-Interaction: Any modeling related activity has to occur using the MCP server.
@@ -73,6 +73,17 @@ GLSP MCP server. You have to adhere to the following principles:
   \`action: "center-on-elements"\` — to point the user at the elements you reference.
 `;
 
+/**
+ * Target-neutral default option values shared by every {@link AbstractMcpServerModule} subclass.
+ * Node and Browser concrete modules spread this and add their own target-specific fields
+ * (e.g. `host`/`allowedHosts` on Node).
+ */
+export const DEFAULT_MCP_OPTIONS: McpServerDefaults = {
+    dataMode: 'tools',
+    agentPersona: DEFAULT_AGENT_PERSONA,
+    eventStoreLimit: LruEventStore.DEFAULT_LIMIT
+};
+
 /**
  * Multi-binding helper for MCP handler classes. Singleton-scoped sibling of core's
  * `MultiBinding<T>` — same `binding.add(...)` / `binding.rebind(old, new)` adopter shape, but
@@ -91,9 +102,11 @@ export class McpHandlerMultiBinding<T> extends AbstractMultiBinding<interfaces.N
 }
 
 /**
- * Server-scope DI module for the MCP server. Adopters subclass {@link DefaultMcpServerModule}
- * and override the `bind*` hooks to swap single-instance services or the `configure*` hooks to
- * add/replace handlers in the multi-bindings. Mirrors the `DiagramModule` pattern from core
+ * Server-scope DI module for the MCP server. Target-neutral: concrete subclasses (Node, Browser)
+ * supply the launcher + request-context bindings via {@link bindMcpServerLauncher} and
+ * {@link bindMcpRequestContext}. Adopters subclass the concrete module and override the `bind*`
+ * hooks to swap single-instance services or the `configure*` hooks to add/replace handlers in
+ * the multi-bindings. Mirrors the `DiagramModule` pattern from core
  * (`bindGModelSerializer()`, `configureActionHandlers(binding)`, etc.).
  *
  * Adopter-provided default option values flow through the constructor — pass a
@@ -103,13 +116,13 @@ export class McpHandlerMultiBinding<T> extends AbstractMultiBinding<interfaces.N
  * @example
  * ```ts
  * // Drop-in: GLSP defaults.
- * launcher.configure(serverModule, new DefaultMcpServerModule());
+ * launcher.configure(serverModule, new NodeMcpServerModule());
  *
  * // Drop-in with a product-specific override.
- * launcher.configure(serverModule, new DefaultMcpServerModule({ dataMode: 'resources' }));
+ * launcher.configure(serverModule, new NodeMcpServerModule({ dataMode: 'resources' }));
  *
  * // Subclass when handler customization is needed.
- * class WorkflowMcpServerModule extends DefaultMcpServerModule {
+ * class WorkflowMcpServerModule extends NodeMcpServerModule {
  *     constructor() { super({ agentPersona: WORKFLOW_PERSONA }); }
  *     protected override configureToolHandlers(binding) {
  *         super.configureToolHandlers(binding);
@@ -119,9 +132,6 @@ export class McpHandlerMultiBinding<T> extends AbstractMultiBinding<interfaces.N
  * ```
  */
 export abstract class AbstractMcpServerModule extends GLSPModule {
-    protected bind!: interfaces.Bind;
-    protected rebind!: interfaces.Rebind;
-
     constructor(protected readonly defaultOptions: McpServerDefaults = {}) {
         super();
     }
@@ -130,23 +140,26 @@ export abstract class AbstractMcpServerModule extends GLSPModule {
         bind: interfaces.Bind,
         _unbind: interfaces.Unbind,
         isBound: interfaces.IsBound,
-        rebind: interfaces.Rebind
+        _rebind: interfaces.Rebind
     ): void {
-        this.bind = bind;
-        this.rebind = rebind;
         const context = { bind, isBound };
-        applyBindingTarget(context, McpServerLauncher, this.bindMcpServerLauncher()).inSingletonScope();
-        // The launcher is bound under two additional service identifiers so core's existing
+        applyBindingTarget(context, AbstractMcpServerLauncher, this.bindMcpServerLauncher())
+            .inSingletonScope()
+            .onActivation((_, launcher) => {
+                this.onMcpServerLauncherActivated(launcher);
+                return launcher;
+            });
+        // The launcher is aliased under two additional service identifiers so core's existing
         // multi-bindings pick it up alongside the rest of the server's contributions/listeners.
-        bind(GLSPServerInitializer).toService(McpServerLauncher);
-        bind(GLSPServerListener).toService(McpServerLauncher);
-        applyBindingTarget(context, McpHttpTransport, this.bindMcpHttpTransport()).inSingletonScope();
+        bind(GLSPServerInitializer).toService(AbstractMcpServerLauncher);
+        bind(GLSPServerListener).toService(AbstractMcpServerLauncher);
         applyBindingTarget(context, McpDiagramHandlerDispatcher, this.bindMcpDiagramHandlerDispatcher()).inSingletonScope();
         applyBindingTarget(context, McpServerOptions, this.bindMcpServerOptions()).inSingletonScope();
         applyBindingTarget(context, McpServerDefaults, this.bindMcpServerDefaults());
         applyBindingTarget(context, McpLogger, this.bindMcpLogger()).inSingletonScope();
         applyBindingTarget(context, McpLogLevelRegistry, this.bindMcpLogLevelRegistry()).inSingletonScope();
         applyBindingTarget(context, McpProgressReporter, this.bindMcpProgressReporter()).inSingletonScope();
+        applyBindingTarget(context, McpRequestContext, this.bindMcpRequestContext()).inSingletonScope();
         applyBindingTarget(context, GLSPMcpServerFactory, this.bindGLSPMcpServerFactory());
         this.configureMultiBinding(new McpHandlerMultiBinding<McpToolHandler>(McpToolHandler), binding =>
             this.configureToolHandlers(binding as McpHandlerMultiBinding<McpToolHandler>)
@@ -160,18 +173,23 @@ export abstract class AbstractMcpServerModule extends GLSPModule {
     }
 
     /**
-     * {@link McpServerLauncher} binding. Bound as a singleton AND aliased to
-     * `GLSPServerInitializer` + `GLSPServerListener` (the launcher implements both).
-     * Override to swap in a custom launcher impl.
+     * {@link AbstractMcpServerLauncher} binding. Bound as a singleton AND aliased to
+     * `GLSPServerInitializer` + `GLSPServerListener` (the launcher implements both). Concrete
+     * modules return the target-specific launcher class (Node, Web, ...).
      */
-    protected bindMcpServerLauncher(): BindingTarget<McpServerLauncher> {
-        return McpServerLauncher;
-    }
+    protected abstract bindMcpServerLauncher(): BindingTarget<AbstractMcpServerLauncher>;
 
-    /** {@link McpHttpTransport} binding. Override to swap to a different transport implementation. */
-    protected bindMcpHttpTransport(): BindingTarget<McpHttpTransport> {
-        return McpHttpTransport;
-    }
+    /**
+     * Hook fired once when the {@link AbstractMcpServerLauncher} singleton is first resolved by
+     * the DI container — lets adopters obtain the instance without subclassing it. No-op by default.
+     */
+    protected onMcpServerLauncherActivated(_launcher: AbstractMcpServerLauncher): void {}
+
+    /**
+     * {@link McpRequestContext} binding. Concrete modules return the target-specific async-context
+     * implementation (e.g. Node's `AsyncLocalStorage`-backed variant or a browser-friendly one).
+     */
+    protected abstract bindMcpRequestContext(): BindingTarget<McpRequestContext>;
 
     /**
      * {@link McpDiagramHandlerDispatcher} binding. Owns diagram-scope handler discovery,
@@ -196,16 +214,12 @@ export abstract class AbstractMcpServerModule extends GLSPModule {
         return { constantValue: this.defaultOptions };
     }
 
-    /**
-     * {@link McpLogger} binding. Bound on the server container; per-session containers inherit
-     * it, so handlers at any scope can inject it; routes through the active MCP request via
-     * {@link mcpRequestContext}.
-     */
+    /** {@link McpLogger} binding — routes through the active MCP request via {@link McpRequestContext}. */
     protected bindMcpLogger(): BindingTarget<McpLogger> {
         return McpLogger;
     }
 
-    /** {@link McpProgressReporter} binding. Same scope/lifecycle story as {@link bindMcpLogger}. */
+    /** {@link McpProgressReporter} binding. */
     protected bindMcpProgressReporter(): BindingTarget<McpProgressReporter> {
         return McpProgressReporter;
     }
@@ -259,31 +273,3 @@ export abstract class AbstractMcpServerModule extends GLSPModule {
         binding.add(SuggestImprovementsMcpPromptHandler);
     }
 }
-
-/**
- * Default {@link AbstractMcpServerModule} entry point. Ships GLSP-default option values (see
- * {@link DEFAULT_OPTIONS}) on top of the abstract module's hook defaults. Adopter-provided
- * overrides via the constructor merge on top.
- *
- * @experimental The MCP integration is under active development. Option names, schema shapes,
- * and handler contracts MAY change in minor releases until the feature graduates from
- * experimental status.
- */
-export class DefaultMcpServerModule extends AbstractMcpServerModule {
-    static readonly DEFAULT_OPTIONS: McpServerDefaults = {
-        host: '127.0.0.1',
-        allowedHosts: ['127.0.0.1', 'localhost'],
-        // `allowedOrigins` deliberately undefined: accept absent Origin (typical for desktop-IDE
-        // MCP clients) and rely on Host validation to gate DNS-rebinding. Adopters whose
-        // deployment is browser-fronted set this explicitly to their frontend's origin.
-        dataMode: 'tools',
-        agentPersona: DEFAULT_AGENT_PERSONA,
-        // 10K events per session is generous for typical workloads (a few MB) and large enough
-        // that disconnects within seconds recover via `Last-Event-ID` resumability.
-        eventStoreLimit: LruEventStore.DEFAULT_LIMIT
-    };
-
-    constructor(overrides: McpServerDefaults = {}) {
-        super({ ...DefaultMcpServerModule.DEFAULT_OPTIONS, ...overrides });
-    }
-}