html2pptx-local-mcp 1.1.26 → 1.1.28

package/lib/pptx-studio-mcp-core.js

@@ -734,7 +734,56 @@ function buildServerInstructions(client = {}, { localOnly = false } = {}) {
   return lines.join('\n');
 }
 
-export async function executeTool(name, args, client, { sendNotification, progressToken } = {}) {
+/**
+ * Pull a sanitized `{ name, version }` snapshot out of the raw
+ * `params.clientInfo` that MCP clients send during `initialize`. Returns
+ * `null` if the payload is unusable so callers can treat "no info" uniformly.
+ *
+ * @param {unknown} raw
+ * @returns {{ name: string, version?: string } | null}
+ */
+export function extractClientInfo(raw) {
+  if (!raw || typeof raw !== 'object') return null;
+  const name = typeof raw.name === 'string' ? raw.name.trim() : '';
+  if (!name) return null;
+  const version = typeof raw.version === 'string' ? raw.version.trim() : '';
+  return version ? { name, version } : { name };
+}
+
+/**
+ * Decide whether html2pptx_open_local_slide_editor should auto-open the OS
+ * default browser when the MCP caller did not explicitly set `openBrowser`.
+ *
+ * Strategy: conservative allowlist. Only known terminal-style CLI clients
+ * (Claude Code, Codex CLI, anything whose advertised clientInfo.name ends in
+ * `-cli`) get the auto-open default. Desktop/IDE clients (Claude Desktop,
+ * Cursor, Cline, unknown) keep the existing "return URL only, let the host
+ * render it inline" behavior — that's the right default for future MCP
+ * side-pane UI surfaces.
+ *
+ * @param {{ name?: string } | null | undefined} clientInfo
+ * @returns {boolean}
+ */
+export function inferOpenBrowserDefault(clientInfo) {
+  const rawName = String(clientInfo?.name ?? '').trim().toLowerCase();
+  if (!rawName) return false;
+  if (/^claude[\s\-_]?code/.test(rawName)) return true;
+  if (/^codex(?:[\s\-_]|$)/.test(rawName)) return true;
+  if (/[\-_]cli$/.test(rawName)) return true;
+  return false;
+}
+
+/**
+ * Resolve the effective `openBrowser` flag for the local slide editor tool.
+ * Explicit boolean from args always wins; otherwise fall back to the
+ * clientInfo heuristic above.
+ */
+export function resolveOpenBrowserFlag(argsOpenBrowser, clientInfo) {
+  if (typeof argsOpenBrowser === 'boolean') return argsOpenBrowser;
+  return inferOpenBrowserDefault(clientInfo);
+}
+
+export async function executeTool(name, args, client, { sendNotification, progressToken, clientInfo } = {}) {
   switch (name) {
     case 'html2pptx_list_export_plans': {
       const data = await client.listExportPlans();
@@ -918,7 +967,7 @@ export async function executeTool(name, args, client, { sendNotification, progre
         baseUrl: typeof args.baseUrl === 'string' ? args.baseUrl : undefined,
         editorPort: Number.isFinite(args.editorPort) ? args.editorPort : undefined,
         port: Number.isFinite(args.port) ? args.port : undefined,
-        openBrowser: args.openBrowser === true,
+        openBrowser: resolveOpenBrowserFlag(args.openBrowser, clientInfo),
         reuseExisting: args.reuseExisting !== false,
       });
       return buildToolResponse(renderLocalSlideEditorText(data), data);
@@ -936,7 +985,7 @@ export async function executeTool(name, args, client, { sendNotification, progre
   }
 }
 
-export async function handleMcpMessage(message, { protocolVersion = DEFAULT_PROTOCOL, client, sendNotification, serverInfo = SERVER_INFO, localOnly = false }) {
+export async function handleMcpMessage(message, { protocolVersion = DEFAULT_PROTOCOL, client, sendNotification, serverInfo = SERVER_INFO, localOnly = false, clientInfo = null }) {
   if (!message || typeof message !== 'object') {
     return { protocolVersion, response: null };
   }
@@ -955,6 +1004,7 @@ export async function handleMcpMessage(message, { protocolVersion = DEFAULT_PROT
     const negotiatedProtocol = negotiateProtocol(params?.protocolVersion);
     return {
       protocolVersion: negotiatedProtocol,
+      clientInfo: extractClientInfo(params?.clientInfo),
       response: {
         jsonrpc: '2.0',
         id,
@@ -1168,7 +1218,7 @@ export async function handleMcpMessage(message, { protocolVersion = DEFAULT_PROT
     try {
       const toolArgs = params?.arguments ?? {};
       const progressToken = toolArgs._meta?.progressToken ?? params?._meta?.progressToken;
-      const payload = await executeTool(params?.name, toolArgs, client, { sendNotification, progressToken });
+      const payload = await executeTool(params?.name, toolArgs, client, { sendNotification, progressToken, clientInfo });
       return {
         protocolVersion,
         response: {

package/local-editor-app/app/edit-slide/page.jsx

@@ -1,4 +1,4 @@
-import EditSlideClient from './edit-slide-client';
+import EditSlideClient from './edit-slide-client.jsx';
 import { isLocalEditSlideLaunch, isLoopbackRequest } from '../../lib/edit-slide-url-security.js';
 import { notFound } from 'next/navigation';
 import { headers } from 'next/headers';

package/mcp/pptx-studio-mcp-server.mjs

@@ -14,6 +14,7 @@ import {
 let inputBuffer = Buffer.alloc(0);
 let negotiatedProtocol = DEFAULT_PROTOCOL;
 let wireMode = 'content-length';
+let mcpClientInfo = null;
 
 process.stdin.on('data', (chunk) => {
   inputBuffer = Buffer.concat([inputBuffer, chunk]);
@@ -126,6 +127,7 @@ async function handleMessage(message) {
   const handled = await handleMcpMessage(message, {
     protocolVersion: negotiatedProtocol,
     client,
+    clientInfo: mcpClientInfo,
     serverInfo: {
       ...SERVER_INFO,
       name: 'html2pptx-local',
@@ -136,6 +138,9 @@ async function handleMessage(message) {
     },
   });
   negotiatedProtocol = handled.protocolVersion;
+  if (handled.clientInfo !== undefined) {
+    mcpClientInfo = handled.clientInfo;
+  }
 
   if (handled.response) {
     sendMessage(handled.response);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "html2pptx-local-mcp",
-  "version": "1.1.26",
+  "version": "1.1.28",
   "type": "module",
   "description": "Local stdio MCP server for opening html2pptx slide HTML in the local edit-slide editor.",
   "bin": {