owner-clone-ai-mcp 1.77.0 → 1.78.0

package/domains/skills.js

@@ -190,7 +190,7 @@ is_admin_published / allowed_plan_ids はAdmin権限がある場合のみ反映
         display_name:               { type: 'string', description: '表示名（日本語OK）' },
         description:                { type: 'string', description: 'スキルの用途説明' },
         prompt_template:            { type: 'string', description: 'スキル実行時の指示テキスト。prepare_skill（Claude Code自律実行）で使う場合はMCPツール呼び出し指示を含めてよい。invoke_skill（サーバー実行）で使う場合はテキスト処理の指示のみ。' },
-        parameters:                 { type: 'array', items: { type: 'object' }, description: 'パラメータ定義リスト。配列全体を置き換えるため、既存パラメータも含めて全て渡すこと [{name, description, required, default_value, input_type, options(input_type=select時の選択肢文字列配列), hidden}]' },
+        parameters:                 { type: 'array', items: { type: 'object' }, description: '⚠️ 配列全体を置き換える。1つ追加・変更するだけの場合も、まず get_skill の「パラメータ定義（生JSON）」をそのままコピーし、そこへ追加・変更して全件渡すこと（整形テキストから書き直すと options / input_type / hidden が失われて選択肢が壊れる）。[{name, description, required, default_value, input_type, options(input_type=select時の選択肢文字列配列), hidden}]' },
         knowledge_item_ids:         { type: 'array', items: { type: 'string' }, description: '参照するナレッジのID一覧（UUID文字列）。配列全体を置き換える' },
         knowledge_folder_ids:       { type: 'array', items: { type: 'string' }, description: '参照するナレッジフォルダID一覧（UUID文字列）。配列全体を置き換える' },
         preset_id:                  { type: 'integer', description: 'AIパートナーのID（整数）。null を渡すと解除' },
@@ -488,6 +488,12 @@ const handlers = {
     const paramStr = params.length
       ? params.map(p => `  - ${p.name}${p.required ? '（必須）' : '（任意）'}: ${p.description || ''}${p.default_value ? ` [デフォルト: ${p.default_value}]` : ''}`).join('\n')
       : '  なし';
+    // ⚠️ 2026-10-01: 上の整形は name / 必須任意 / description / default_value だけで、
+    //    options（select の選択肢）・input_type・hidden が落ちていた。
+    //    update_skill の parameters は **配列全体の置き換え** なので、1つ足すだけでも
+    //    取得できない options を推測で書き直すことになり、選択肢が壊れる危険があった。
+    //    そのまま update_skill に渡せる生JSONを併記する。
+    const paramJson = params.length ? JSON.stringify(params, null, 2) : '[]';
     const isGoal = s.skill_type === 'agent_goal';
     return [
       `ID: ${s.id}`,
@@ -504,6 +510,9 @@ const handlers = {
         `プロンプトテンプレート:\n${s.prompt_template || 'なし'}`,
       ]),
       `パラメータ:\n${paramStr}`,
+      // ⚠️ update_skill の parameters は配列全体の置き換え。1つ足すときも**このJSONをそのままコピーして
+      //    末尾に追加**する。整形テキストから書き直すと options / input_type / hidden が失われる
+      `パラメータ定義（生JSON・update_skill にそのまま渡せる）:\n${paramJson}`,
       `ナレッジ指定アイテム数: ${(s.knowledge_item_ids || []).length}件`,
       `ナレッジ指定フォルダ数: ${(s.knowledge_folder_ids || []).length}件`,
       `AIパートナー: ${s.preset_name || 'なし'}`,

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/index.js

@@ -35,6 +35,8 @@ const HANDLERS = Object.assign({}, ...allDomains.map(d => d.handlers));
 // adminOnly ツール（標準AIパートナー操作など）は、接続キーがAdminの時だけ tools/list に出す。
 // 取得失敗時は安全側で「非Admin扱い＝隠す」。ツール返却時は非標準フィールド adminOnly を除去。
 const { callApi } = require('./lib/api');
+// 配布物の版数（serverInfo で返す）。ハードコードするとずれるので package.json から読む
+const PKG_VERSION = require('./package.json').version;
 let _isAdminCache = null;
 async function keyIsAdmin() {
   if (_isAdminCache !== null) return _isAdminCache;
@@ -53,7 +55,12 @@ async function executeTool(name, args) {
 // ── MCPサーバー起動 ────────────────────────────────────────────
 async function main() {
   const server = new Server(
-    { name: 'owner-clone-ai', version: '1.11.0' },
+    // ⚠️ 2026-10-01: ここが '1.11.0' で固定されており、package.json（実際の版）と
+    //    ずれていた。MCPのハンドシェイクで返る版が実物と違うと、
+    //    「どの版が動いているか」を利用者側から確認できず、版数トラブルの切り分けができない
+    //    （1.77.1 の公開直後にCDN伝播待ちで 404 になった件で実際に困った）。
+    //    package.json から読むことで、以後ずれない。
+    { name: 'owner-clone-ai', version: PKG_VERSION },
     { capabilities: { tools: {} } }
   );
 

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.78.0",
   "description": "MCP server for Owner Clone AI — exposes knowledge and agent presets to Claude Code",
   "main": "index.js",
   "bin": {