owner-clone-ai-mcp 1.77.0 → 1.77.1

package/domains/slides.js

@@ -213,12 +213,34 @@ const handlers = {
       return `- ${b.name}（テーマ:${b.theme || '未指定'}${parts ? ' / ' + parts : ''}）\n  更新: ${b.updated_at}`;
     }).join('\n');
   },
+  // ⚠️ 2026-09-30: この2つは `callApi(path, 'POST', body)` という別流儀で書かれており、
+  //    文字列が options として展開されて {0:'P',1:'O',2:'S',3:'T'} になり、
+  //    method も body も付かず **GET が飛んでいた**（153箇所中この2箇所だけの誤用）。
+  //    ・apply は GET のルートが無く 404 のHTMLを res.json() が食って
+  //      「Unexpected token '<'」になっていた
+  //    ・save は GET /slides/brand-themes（一覧）が 200 を返すためエラーにならず、
+  //      r.name が undefined のまま「保存しました」と**嘘の成功**を返していた（より危険）
+  //    正しい形は callApi(path, { method, body })。
   save_brand_theme: async ({ name, theme, deck }) => {
-    const r = await callApi('/slides/brand-themes', 'POST', { name, theme, deck: deck || {} });
+    const r = await callApi('/slides/brand-themes', {
+      method: 'POST',
+      body: JSON.stringify({ name, theme, deck: deck || {} }),
+    });
+    // ⚠️ 応答に name が無いのに成功を名乗らない（サイレント失敗の再発防止）
+    if (!r || !r.name) {
+      throw new Error('テーマテンプレートの保存結果を確認できませんでした（応答に name がありません）。list_brand_themes で保存されたか確認してください。');
+    }
     return `テーマテンプレート「${r.name}」を保存しました。apply_brand_theme で他のスライドにも当てられます。`;
   },
   apply_brand_theme: async ({ name, deck_id }) => {
-    const r = await callApi(`/slides/brand-themes/${encodeURIComponent(name)}/apply`, 'POST', { deck_id });
+    const r = await callApi(`/slides/brand-themes/${encodeURIComponent(name)}/apply`, {
+      method: 'POST',
+      body: JSON.stringify({ deck_id }),
+    });
+    // ⚠️ 同上。適用できていないのに「適用しました」と言わない
+    if (!r || !r.applied) {
+      throw new Error('テーマテンプレートの適用結果を確認できませんでした（応答に applied がありません）。get_slide_deck でデッキの見た目を確認してください。');
+    }
     return `「${r.applied}」の見た目を「${r.deck?.title || deck_id}」に適用しました（本文は変更していません）。`;
   },
   get_slide_deck: async ({ id, with_narration }) => {

package/lib/api.js

@@ -7,6 +7,18 @@ const API_KEY  = process.env.OCR_API_KEY;
 const BASE_URL = (process.env.OCR_BASE_URL || 'https://api.ownercloneai.com').replace(/\/$/, '');
 
 async function callApi(path, options = {}) {
+  // ⚠️ 2026-09-30: `callApi(path, 'POST', body)` という別流儀で書かれた2箇所が静かに壊れていた
+  //    （domains/slides.js の save_brand_theme / apply_brand_theme）。文字列を options として
+  //    展開すると {0:'P',1:'O',2:'S',3:'T'} になり method も body も付かず GET が飛ぶ。
+  //    GET が 200 を返すエンドポイントでは**嘘の成功**になるため、型で弾いて気づけるようにする。
+  //    ⚠️ 3引数形を受け付ける案は採らない（2つの流儀が併存すると同じ事故が再発する）。
+  if (options === null || typeof options !== 'object' || Array.isArray(options)) {
+    const got = options === null ? 'null' : (Array.isArray(options) ? 'array' : typeof options);
+    throw new Error(
+      `callApi の第2引数は fetch のオプションオブジェクトです（受け取った型: ${got}）。`
+      + `callApi(path, { method: 'POST', body: JSON.stringify(...) }) の形で呼んでください。`
+    );
+  }
   const url = `${BASE_URL}/api/mcp/v1${path}`;
   const res = await fetch(url, {
     ...options,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "owner-clone-ai-mcp",
-  "version": "1.77.0",
+  "version": "1.77.1",
   "description": "MCP server for Owner Clone AI — exposes knowledge and agent presets to Claude Code",
   "main": "index.js",
   "bin": {