@monoharada/wcf-mcp 0.8.0 → 0.9.1

package/README.md

@@ -78,7 +78,7 @@ claude mcp add wcf -- npx @monoharada/wcf-mcp
 }
 ```
 
-## 提供機能（19 tools + 1 prompt + 4 resources）
+## 提供機能（16 tools + 1 prompt + 5 resources）
 
 ### ガードレール
 
@@ -92,7 +92,7 @@ claude mcp add wcf -- npx @monoharada/wcf-mcp
 |--------|------|
 | `list_components` | カテゴリ/クエリ/limit/offset でコンポーネントを段階的に取得（デフォルト20件。全件取得は `limit: 200`） |
 | `search_icons` | アイコン名をキーワード検索し、usage example を返す |
-| `get_component_api` | tagName or className で属性・スロット・イベント・CSS Parts・CSS Custom Properties を取得。`components` 配列でバッチ取得可能（最大10件） |
+| `get_component_api` | tagName or className で属性・スロット・イベント・CSS Parts・CSS Custom Properties を取得。`components` 配列でバッチ取得可能（最大10件。レスポンス予算次第で `structuredContent` は省略される） |
 | `generate_usage_snippet` | コンポーネントの最小限 HTML スニペットを生成 |
 | `get_install_recipe` | componentId・依存関係・define関数・インストールコマンドを取得 |
 
@@ -145,17 +145,6 @@ claude mcp add wcf -- npx @monoharada/wcf-mcp
 | `get_accessibility_docs` | component/topic/wcagLevel で A11y チェックリストとガイドライン要点を検索（`topic=all` では両ソースを混在返却） |
 | `search_guidelines` | ガイドライン（topic/query）をスコア付きで検索 |
 
-### スキル管理（リポジトリ内専用）
-
-| ツール | 説明 |
-|--------|------|
-| `list_skills` | 登録済みデザインシステムスキル一覧を取得 |
-| `get_skill_manifest` | スキルの SKILL.md マニフェストを取得 |
-| `check_drift` | スキルレジストリとリポジトリの整合性を検証 |
-
-> **注意**: これらのツールはリポジトリ内でのみ有効です。npx 実行時は `SKILLS_REGISTRY_UNAVAILABLE` エラーを返します。
-> 有効化には `wcf-mcp.config.json` で `design-system-skills` プラグインを設定してください（下記参照）。
-
 #### `get_design_tokens` の使用例
 
 **リクエスト:**
@@ -253,6 +242,7 @@ claude mcp add wcf -- npx @monoharada/wcf-mcp
 | `wcf://tokens` | トークン summary（type/category/themes/sample） |
 | `wcf://guidelines/{topic}` | topic 別ガイドライン要約（`accessibility`,`css`,`patterns`,`all`） |
 | `wcf://llms-full` | `llms-full.txt` の全文 |
+| `wcf://skills` | 登録済み Claude Code / Cursor / Codex スキルカタログ（skills-registry.json ベース） |
 
 ### Figma MCP との併用
 
@@ -295,7 +285,9 @@ npx @monoharada/wcf-mcp --transport=http --port=3100
 ```
 
 - bind: `127.0.0.1`
-- endpoint: `http://127.0.0.1:3100/mcp`
+- endpoint: `http://127.0.0.1:3100/mcp` のみ
+- Host / Origin validation を有効化して DNS rebinding を抑止します
+- `--config` が不正な場合は待受開始前に起動失敗します
 
 ## 設定ファイル
 
@@ -339,7 +331,7 @@ npx @monoharada/wcf-mcp --transport=http --port=3100
 ※ `./plugins/custom-validation-plugin.mjs` は利用側プロジェクトに配置してください。  
 このリポジトリには参照用として `packages/mcp-server/examples/plugins/custom-validation-plugin.mjs` を同梱しています。
 
-### plugin 契約（v1.1）
+### plugin 契約（v1）
 
 詳細仕様: [docs/plugin-contract-v1.md](../../docs/plugin-contract-v1.md)
 
@@ -354,9 +346,11 @@ npx @monoharada/wcf-mcp --transport=http --port=3100
 
 ## structuredContent rollback
 
-`get_component_api` / `get_design_tokens` / `get_design_token_detail` / `get_accessibility_docs` / `search_guidelines` は通常 `structuredContent` を返します。
+`get_component_api` / `get_design_tokens` / `get_design_token_detail` / `get_accessibility_docs` / `search_guidelines` のうち、object payload を返すケースでは通常 `structuredContent` を返します。
 
-- 100KB 制限を超える場合は自動的に `structuredContent` を省略し、`content` のみ返します
+- `structuredContent` は MCP 仕様どおり JSON object を直接返します
+- 100KB 制限を超える場合は自動的に `structuredContent` を省略し、必要に応じて `content` を compact JSON に切り替え、それでも収まらなければ `TOOL_RESULT_TOO_LARGE` warning payload にフォールバックします
+- plugin handler が raw MCP result を返す場合も、最終返却サイズには同じ上限が適用されます
 - 緊急切り戻し時は環境変数 `WCF_MCP_DISABLE_STRUCTURED_CONTENT=1` を設定してください
 
 例:
@@ -392,21 +386,6 @@ Claude Desktop 設定例:
 prefix: "myui" → dads-button → myui-button
 ```
 
-## v0.8.0 新機能
-
-### 新ツール（スキル管理）
-- **`list_skills`** — 登録済みデザインシステムスキルの一覧を取得
-- **`get_skill_manifest`** — スキルの SKILL.md マニフェスト内容を取得
-- **`check_drift`** — スキルレジストリとリポジトリの整合性を検証（ドリフト検出）
-
-### 改善
-- **Plugin 契約 v1.1** — module plugin の `dataSources` パス解決をモジュールディレクトリ基準に統一
-- **Skills Registry plugin** — `plugins/design-system-skills/` として同梱。`wcf-mcp.config.json` で有効化
-
-> スキル管理ツールはリポジトリ内でのみ動作します。npx 実行時は graceful に `SKILLS_REGISTRY_UNAVAILABLE` を返します。
-
----
-
 ## v0.4.0 新機能
 
 ### 新ツール
@@ -668,8 +647,6 @@ packages/mcp-server/
 ├── server.mjs       # MCP サーバー本体
 ├── validator.mjs    # HTML バリデーター
 ├── package.json     # npm パッケージ定義
-├── plugins/         # 同梱プラグイン
-│   └── design-system-skills/  # スキル管理ツール (list_skills, get_skill_manifest, check_drift)
 └── data/            # バンドルデータ (npm run mcp:build で生成)
     ├── custom-elements.json
     ├── install-registry.json

package/bin.mjs

@@ -9,9 +9,12 @@
  */
 
 import { createServer } from './server.mjs';
+import fs from 'node:fs';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
 
-const USAGE = [
+export const USAGE = [
   'Usage:',
   '  wcf-mcp',
   '  wcf-mcp --transport=stdio',
@@ -19,8 +22,9 @@ const USAGE = [
   '  wcf-mcp --config=./wcf-mcp.config.json',
   '  wcf-mcp --help',
 ].join('\n');
+export const HTTP_ENDPOINT_PATH = '/mcp';
 
-function parseArgs(argv) {
+export function parseArgs(argv) {
   let transport = 'stdio';
   let port = 3100;
   let configPath;
@@ -90,6 +94,144 @@ function parseArgs(argv) {
   return { help: false, transport, port, configPath };
 }
 
+export function buildHttpTransportOptions({ host = '127.0.0.1', port }) {
+  const allowedHostnames = new Set([host]);
+  if (host === '127.0.0.1') {
+    allowedHostnames.add('localhost');
+  }
+  if (host === 'localhost') {
+    allowedHostnames.add('127.0.0.1');
+  }
+
+  const defaultHttpPort = port === 80;
+  const allowedHosts = new Set();
+  const allowedOrigins = new Set();
+  for (const allowedHostname of allowedHostnames) {
+    allowedHosts.add(`${allowedHostname}:${port}`);
+    allowedOrigins.add(`http://${allowedHostname}:${port}`);
+    if (defaultHttpPort) {
+      allowedHosts.add(allowedHostname);
+      allowedOrigins.add(`http://${allowedHostname}`);
+    }
+  }
+
+  return {
+    sessionIdGenerator: undefined,
+    allowedHosts: [...allowedHosts],
+    allowedOrigins: [...allowedOrigins],
+    enableDnsRebindingProtection: true,
+  };
+}
+
+function sendJsonRpcError(res, status, message) {
+  if (res.headersSent) return;
+  res.statusCode = status;
+  res.setHeader('content-type', 'application/json');
+  res.end(JSON.stringify({
+    jsonrpc: '2.0',
+    error: {
+      code: -32603,
+      message,
+    },
+    id: null,
+  }));
+}
+
+function sendNotFound(res) {
+  if (res.headersSent) return;
+  res.statusCode = 404;
+  res.setHeader('content-type', 'text/plain; charset=utf-8');
+  res.end('Not Found');
+}
+
+function getRequestPath(req) {
+  const rawUrl = req.url ?? '/';
+  try {
+    return new URL(rawUrl, 'http://localhost').pathname;
+  } catch {
+    return rawUrl;
+  }
+}
+
+export async function validateHttpStartup({
+  configPath,
+  createServerImpl = createServer,
+} = {}) {
+  const bootstrap = await createServerImpl({ configPath });
+  await Promise.allSettled([
+    bootstrap?.server?.close?.(),
+  ]);
+}
+
+export function createHttpRequestHandler({
+  port,
+  host = '127.0.0.1',
+  configPath,
+  createServerImpl = createServer,
+} = {}) {
+  const transportOptions = buildHttpTransportOptions({ host, port });
+
+  return async function handleHttpRequest(req, res) {
+    if (getRequestPath(req) !== HTTP_ENDPOINT_PATH) {
+      sendNotFound(res);
+      return;
+    }
+
+    let server;
+    let transport;
+    let cleanedUp = false;
+    const cleanup = async () => {
+      if (cleanedUp) return;
+      cleanedUp = true;
+      await Promise.allSettled([
+        transport?.close?.(),
+        server?.close?.(),
+      ]);
+    };
+
+    res.on('close', () => {
+      void cleanup();
+    });
+
+    try {
+      ({ server } = await createServerImpl({ configPath }));
+      const { StreamableHTTPServerTransport } = await import(
+        '@modelcontextprotocol/sdk/server/streamableHttp.js'
+      );
+      transport = new StreamableHTTPServerTransport(transportOptions);
+      await server.connect(transport);
+      await transport.handleRequest(req, res);
+      if (res.writableEnded) {
+        await cleanup();
+      }
+    } catch (error) {
+      await cleanup();
+      const message = error instanceof Error ? error.message : String(error);
+      if (res.headersSent) {
+        res.destroy(error instanceof Error ? error : new Error(message));
+        return;
+      }
+      sendJsonRpcError(res, 500, message);
+    }
+  };
+}
+
+function isDirectRun(metaUrl = import.meta.url, argv = process.argv) {
+  const entryPath = argv[1];
+  if (!entryPath) return false;
+
+  const resolveFilePath = (value) => {
+    const resolvedPath = path.resolve(value);
+    try {
+      return fs.realpathSync.native?.(resolvedPath) ?? fs.realpathSync(resolvedPath);
+    } catch {
+      return resolvedPath;
+    }
+  };
+
+  return resolveFilePath(entryPath) === resolveFilePath(fileURLToPath(metaUrl));
+}
+
 async function main() {
   let parsed;
   try {
@@ -106,25 +248,28 @@ async function main() {
     return;
   }
 
-  const { server } = await createServer({ configPath: parsed.configPath });
-
   if (parsed.transport === 'http') {
-    const { StreamableHTTPServerTransport } = await import(
-      '@modelcontextprotocol/sdk/server/streamableHttp.js'
-    );
+    await validateHttpStartup({ configPath: parsed.configPath });
     const { createServer: createHttpServer } = await import('node:http');
-    const httpTransport = new StreamableHTTPServerTransport({ sessionIdGenerator: undefined });
-    await server.connect(httpTransport);
-    const httpServer = createHttpServer((req, res) => httpTransport.handleRequest(req, res));
+    const handleHttpRequest = createHttpRequestHandler({
+      port: parsed.port,
+      configPath: parsed.configPath,
+    });
+    const httpServer = createHttpServer((req, res) => {
+      void handleHttpRequest(req, res);
+    });
     httpServer.listen(parsed.port, '127.0.0.1', () => {
-      console.error(`MCP HTTP server listening on http://127.0.0.1:${parsed.port}/mcp`);
+      console.error(`MCP HTTP server listening on http://127.0.0.1:${parsed.port}${HTTP_ENDPOINT_PATH}`);
     });
   } else {
+    const { server } = await createServer({ configPath: parsed.configPath });
     await server.connect(new StdioServerTransport());
   }
 }
 
-main().catch((err) => {
-  console.error(err);
-  process.exit(1);
-});
+if (isDirectRun()) {
+  main().catch((err) => {
+    console.error(err);
+    process.exit(1);
+  });
+}