poker-mcp 1.2.6 → 1.2.7

package/README.md

@@ -4,12 +4,33 @@ YAML-based input file management tool for radiation-shielding calculation code P
 
 ## 📋 クイック情報
 
-- **バージョン**: 1.2.5 (Enhanced Release)
+- **バージョン**: 1.2.7
 - **プロトコル**: MCP (Model Context Protocol) 1.0.0 完全準拠
 - **メインサーバー**: `src/mcp_server_stdio_v4.js`
-- **データ保存**: tasks/ディレクトリ（実行時自動作成）
+- **データ保存**: `~/.poker-mcp/`（`POKER_MCP_HOME`環境変数で変更可）
 - **実行方式**: STDIO通信（MCPプロトコル標準）
 
+## 🆕 バージョン1.2.7の修正（バグフィックス）
+
+### 🐛 `poker_executeCalculation` の yaml_file パス解決を修正
+ファイル名のみ（例: `poker.yaml`）を渡すと絶対パス要求でエラーになっていた
+スキーマ・ハンドラー間の矛盾を修正しました。
+
+- **ファイル名のみ指定** → `POKER_MCP_HOME/tasks/` に自動解決
+- **絶対パス指定** → そのまま使用（後方互換）
+
+## 🆕 バージョン1.2.6の修正（バグフィックス）
+
+### 🐛 SERVER DISCONNECTED 問題を修正
+`npx poker-mcp` 実行時にClaude Desktopがカレントディレクトリを
+`C:\Windows\System32` に設定するため、相対パスでのフォルダ作成が
+権限エラー（EPERM）で失敗していた問題を修正しました。
+
+- **`src/utils/paths.js` 新設**: 作業ディレクトリを一元管理
+- **全ファイルパスを絶対パス化**: `logs/`・`backups/`・`tasks/`・`data/`
+- **`POKER_MCP_HOME` 環境変数サポート**: 作業場所を自由に指定可能
+- **エラー出力を `stderr` に追加**: 問題発生時の原因特定が容易に
+
 ## 🆕 バージョン1.2.5の新機能
 
 ### ⚡ 衝突検出システム
@@ -41,31 +62,45 @@ YAML-based input file management tool for radiation-shielding calculation code P
 
 ### 1. インストール
 ```bash
-# 依存関係インストール
+# 依存関係インストール（ローカル開発時のみ）
 npm install
 
-# または NPX で直接使用
+# または NPX で直接使用（インストール不要）
 npx poker-mcp
 ```
 
-### 2. 環境変数設定（オプション）
+### 2. 環境変数設定
+
+#### `POKER_MCP_HOME`（推奨・新設）
+作業ファイル（YAML・バックアップ・ログ・核種DB）の格納先を指定します。
+**未設定時は `~/.poker-mcp/` が自動的に使用されます。**
+
+#### `POKER_INSTALL_PATH`（オプション）
+ICRP-07.NDX 核種データベースのコピー元を指定します。
 
-**POKER_INSTALL_PATH環境変数**（オプション）:
 ```bash
-# Windowsの場合（コマンドプロンプト）
+# Windows（コマンドプロンプト）
+set POKER_MCP_HOME=C:\Users\yoshi\poker_mcp_workspace
 set POKER_INSTALL_PATH=C:/Poker
 
-# Windowsの場合（PowerShell）
+# Windows（PowerShell）
+$env:POKER_MCP_HOME="C:\Users\yoshi\poker_mcp_workspace"
 $env:POKER_INSTALL_PATH="C:/Poker"
 
-# Linux/macOSの場合
+# Linux/macOS
+export POKER_MCP_HOME="$HOME/.poker-mcp"
 export POKER_INSTALL_PATH="/usr/local/share/poker"
 ```
 
-- **デフォルト値**: `C:/Poker`
-- **目的**: `lib/ICRP-07.NDX`核種データベースファイルの取得元ディレクトリ指定
-- **動作**: 初回起動時に`{POKER_INSTALL_PATH}/lib/ICRP-07.NDX`を`data/`ディレクトリにコピー
-- **注意**: `data/ICRP-07.NDX`が既に存在する場合はコピーをスキップ
+**データ格納先の構造:**
+```
+POKER_MCP_HOME/          # デフォルト: ~/.poker-mcp/
+  ├── tasks/             # poker.yaml, pending_changes.json
+  ├── backups/           # 自動バックアップ（最大10世代）
+  ├── data/              # ICRP-07.NDX 核種データベース
+  ├── logs/              # error.log, combined.log
+  └── config.json        # ユーザー設定（任意）
+```
 
 ### 3. Claude Desktop設定
 
@@ -78,22 +113,7 @@ export POKER_INSTALL_PATH="/usr/local/share/poker"
    Linux: ~/.config/claude/claude_desktop_config.json
    ```
 
-2. **MCP設定を追加**
-   ```json
-   {
-     "mcpServers": {
-       "poker-mcp": {
-         "command": "node",
-         "args": ["C:\\Users\\yoshi\\Desktop\\poker_mcp\\src\\mcp_server_stdio_v4.js"],
-         "env": {
-           "POKER_INSTALL_PATH": "C:/Poker"
-         }
-       }
-     }
-   }
-   ```
-
-3. **NPXを使用する場合**
+2. **推奨設定（v1.2.6）**
    ```json
    {
      "mcpServers": {
@@ -101,18 +121,20 @@ export POKER_INSTALL_PATH="/usr/local/share/poker"
          "command": "npx",
          "args": ["poker-mcp"],
          "env": {
+           "POKER_MCP_HOME": "C:\\Users\\<username>\\poker_mcp_workspace",
            "POKER_INSTALL_PATH": "C:/Poker"
          }
        }
      }
    }
    ```
+   `<username>` はご自身のWindowsユーザー名に置き換えてください。  
+   `POKER_INSTALL_PATH` は省略可能です（デフォルト: `C:/Poker`）。
 
-4. **Claude Desktopを再起動** してMCPサーバーを有効化
+   > **⚠️ 注意:** `cwd`（作業ディレクトリ）の指定は不要です。`POKER_MCP_HOME`
+   > 環境変数で管理するため、`cwd` を設定すると v1.2.5 以前の問題が再発します。
 
-**注意**: 
-- `env`セクションの`POKER_INSTALL_PATH`はオプションです（省略時は`C:/Poker`を使用）
-- 環境変数で指定したパスの`lib/ICRP-07.NDX`ファイルが初回起動時に`data/`にコピーされます
+3. **Claude Desktopを再起動** してMCPサーバーを有効化
 
 ### 4. 動作確認
 Claude Desktopで以下のようにテストできます：
@@ -188,28 +210,28 @@ poker_mcp/
 │   ├── 📁 services/                 # ビジネスロジック
 │   ├── 📁 validators/               # データ検証（物理・単位・衝突）
 │   ├── 📁 utils/                    # ユーティリティ
+│   │   ├── paths.js                 # ★ パス管理（POKER_MCP_HOME起点）
+│   │   ├── logger.js                # ログ出力（絶対パス）
+│   │   └── ...
 │   └── 📁 config/                   # 設定管理
-├── 📁 data/                         # 📊 核種データベース
-│   └── ICRP-07.NDX                  # ICRP-107核種データ（1,254核種）
-├── 📁 config/                       # ⚙️ 設定ファイル
-│   ├── mcp-manifest.json            # MCPマニフェスト (2,276行)
-│   └── mcp-example.json             # 設定例
 ├── 📁 docs/                         # 📚 完全ドキュメント
-│   ├── README.md                    # 詳細ドキュメント
-│   ├── 📁 manuals/                  # マニュアル集
-│   └── 📁 interactive_guides/       # インタラクティブ学習ガイド
-├── 📁 node_modules/                 # 📦 依存パッケージ
 ├── .mcp.json                        # MCPクライアント接続設定
-├── package.json                     # パッケージ定義
-├── package-lock.json                # 依存関係ロック
+├── package.json                     # パッケージ定義（v1.2.6）
 └── README.md                        # このファイル
 
-# 実行時に自動作成されるディレクトリ
-├── 📁 tasks/                        # 📊 作業ディレクトリ（自動作成）
+# 実行時に自動作成されるディレクトリ（POKER_MCP_HOME配下）
+# デフォルト: C:\Users\<username>\.poker-mcp\  または  ~/.poker-mcp/
+POKER_MCP_HOME/
+├── 📁 tasks/                        # 📊 作業ディレクトリ
 │   ├── poker.yaml                   # メインYAMLファイル
 │   └── pending_changes.json         # 保留中の変更
-├── 📁 backups/                      # 💾 自動バックアップ（自動作成）
-└── 📁 logs/                         # 📝 ログファイル（自動作成）
+├── 📁 backups/                      # 💾 自動バックアップ（最大10世代）
+├── 📁 data/                         # 🧪 核種データベース
+│   └── ICRP-07.NDX                  # ICRP-07核種データ（1,254核種）
+├── 📁 logs/                         # 📝 ログファイル
+│   ├── error.log
+│   └── combined.log
+└── config.json                      # ユーザー設定（任意）
 ```
 
 ## 🔧 Claude経由での使用例
@@ -323,6 +345,18 @@ poker_mcp/
 
 ## 📝 更新履歴
 
+### v1.2.7 (2026-05-16)
+- 🐛 `poker_executeCalculation` の `yaml_file` パス解決バグを修正（スキーマ・ハンドラー間の矛盾）
+- ✨ ファイル名のみの指定で `POKER_MCP_HOME/tasks/` 配下を自動参照
+- 📝 `API_COMPLETE.md`・`INTEGRATION_GUIDE.md`・`RESEARCH_WORKFLOWS.md` 更新
+
+### v1.2.6 (2026-05-16)
+- 🐛 `npx` 実行時のSERVER DISCONNECTED問題を修正（EPERM: C:\Windows\System32\logs）
+- ✨ `src/utils/paths.js` 新設（`POKER_MCP_HOME`環境変数によるパス一元管理）
+- ✨ `POKER_MCP_HOME`環境変数サポート（未設定時は`~/.poker-mcp/`をデフォルト使用）
+- 🐛 全ファイルパスを絶対パスに変更（logger, DataManager, ConfigManager, server）
+- ✨ 致命的エラーを`stderr`にも出力（Claude Desktopログから原因確認可能に）
+
 ### v1.2.5 (2025-01-24)
 - ✨ 衝突検出システム実装
 - ✨ 子孫核種自動補完機能追加（ICRP-07統合）
@@ -330,9 +364,7 @@ poker_mcp/
 - ✨ YAMLリセット機能実装（3段階レベル）
 - ✨ 検出器分析機能追加
 - 🐛 NuclideManagerデフォルトパス統一
-- 📝 .mcp.json完全化
 - 📝 材料数13→14（VOID追加明記）
-- 🗑️ archiveディレクトリ削除（未使用ファイル整理）
 
 ### v1.1.0 (Previous)
 - 基本24メソッド実装
@@ -348,11 +380,11 @@ poker_mcp/
 - **📖 詳細README**: [docs/README.md](docs/README.md)
 - **📚 完全マニュアル**: [docs/manuals/](docs/manuals/)
 - **🎓 インタラクティブガイド**: [docs/interactive_guides/](docs/interactive_guides/)
-- **📋 マニフェスト**: [config/mcp-manifest.json](config/mcp-manifest.json)
+- **📋 変更履歴**: [CHANGELOG.md](CHANGELOG.md)
 - **🐛 Issues**: [GitHub Issues](https://github.com/Hirao-Y/poker_mcp/issues)
 
 ---
 
-**🎯 Poker MCP Server v1.2.5**  
+**🎯 Poker MCP Server v1.2.7**  
 **プロトコル**: MCP 1.0.0 完全準拠  
 **作者**: Yoshihiro Hirao | **ライセンス**: ISC

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "poker-mcp",
-  "version": "1.2.6",
+  "version": "1.2.7",
   "description": "YAML-based input file management tool for radiation shielding calculation code POKER with MCP support - Enhanced with collision detection, daughter nuclide auto-completion, and environment variable support",
   "main": "src/mcp_server_stdio_v4.js",
   "type": "module",

package/src/mcp/handlers/calculationHandlers.js

@@ -3,6 +3,7 @@ import { CalculationService } from '../../services/CalculationService.js';
 import { ValidationError, CalculationError } from '../../utils/errors.js';
 import { logger } from '../../utils/logger.js';
 import path from 'path';
+import { TASKS_DIR } from '../../utils/paths.js';
 
 export function createCalculationHandlers(taskManager) {
   const calculationService = new CalculationService();
@@ -15,18 +16,22 @@ export function createCalculationHandlers(taskManager) {
           throw new ValidationError('YAML file is required', 'yaml_file', args.yaml_file);
         }
 
-        // 絶対パス形式の事前チェック（詳細検証はサービス層で実行）
+        // パス解決:
+        //   絶対パスが渡された場合 → そのまま使用（後方互換）
+        //   ファイル名のみの場合  → TASKS_DIR と結合して絶対パスに解決
         const absolutePathPattern = /^([a-zA-Z]:[\\\/]|\/)/;
-        if (!absolutePathPattern.test(args.yaml_file)) {
-          throw new ValidationError(
-            'YAML file must be specified with absolute path. Examples: C:\\path\\to\\file.yaml or /path/to/file.yaml',
-            'yaml_file',
-            args.yaml_file
-          );
-        }
+        const resolvedYamlFile = absolutePathPattern.test(args.yaml_file)
+          ? args.yaml_file
+          : path.join(TASKS_DIR, args.yaml_file);
+
+        logger.info('yaml_file path resolved', {
+          input: args.yaml_file,
+          resolved: resolvedYamlFile,
+          tasksDir: TASKS_DIR
+        });
 
         // YAML拡張子チェック
-        if (!args.yaml_file.match(/\.(yaml|yml)$/i)) {
+        if (!resolvedYamlFile.match(/\.(yaml|yml)$/i)) {
           throw new ValidationError(
             'YAML file must have .yaml or .yml extension',
             'yaml_file',
@@ -46,13 +51,12 @@ export function createCalculationHandlers(taskManager) {
         const resolvedOutputFiles = {};
         
         if (Object.keys(outputFiles).length > 0) {
-          // 入力YAMLファイルのディレクトリを取得
-          const inputDir = path.dirname(args.yaml_file);
+          // 解決済みの入力YAMLファイルのディレクトリを取得
+          const inputDir = path.dirname(resolvedYamlFile);
           
           // 各出力ファイルの絶対パス化
           for (const [key, fileName] of Object.entries(outputFiles)) {
             if (fileName) {
-              // ファイル名の拡張子チェック
               if (!fileName.match(/\.(yaml|yml)$/i)) {
                 throw new ValidationError(
                   `Output file ${key} must have .yaml or .yml extension`,
@@ -60,42 +64,35 @@ export function createCalculationHandlers(taskManager) {
                   fileName
                 );
               }
-              
-              // 絶対パスに解決
               resolvedOutputFiles[key] = path.join(inputDir, fileName);
-              
               logger.debug('Output file path resolved', {
-                key,
-                fileName,
-                inputDir,
+                key, fileName, inputDir,
                 resolvedPath: resolvedOutputFiles[key]
               });
             }
           }
         }
 
-        // 出力ファイル検証（従来の検証は不要、上記で実施済み）
-
         logger.info('Starting radiation shielding calculation', {
-          yamlFile: args.yaml_file,
+          yamlFile: resolvedYamlFile,
           summaryOptions,
-          outputFiles: outputFiles,
+          outputFiles,
           resolvedOutputFiles
         });
 
-        // 計算実行（統合検証付き、絶対パス化済みの出力ファイルパスを使用）
+        // 計算実行（統合検証付き）
         const result = await calculationService.executeWithValidation(
-          args.yaml_file,
+          resolvedYamlFile,
           summaryOptions,
-          resolvedOutputFiles,  // 絶対パス化済み
-          undefined,           // timeout（デフォルト値を使用）
-          taskManager.dataManager  // 事前検証用DataManager
+          resolvedOutputFiles,
+          undefined,
+          taskManager.dataManager
         );
 
         // 事前検証で重大エラーが検出された場合の処理
         if (result.success === false && result.stage === 'pre_validation') {
           logger.error('Pre-calculation validation failed', {
-            yamlFile: args.yaml_file,
+            yamlFile: resolvedYamlFile,
             criticalErrors: result.criticalErrors
           });
           
@@ -128,7 +125,7 @@ export function createCalculationHandlers(taskManager) {
           success: true,
           message: 'Radiation shielding calculation completed successfully',
           calculation: {
-            input_file: args.yaml_file,
+            input_file: resolvedYamlFile,
             execution_time_ms: result.executionTime,
             timestamp: result.summary.timestamp,
             options: result.summary.options
@@ -161,7 +158,7 @@ export function createCalculationHandlers(taskManager) {
         }
 
         logger.info('Calculation completed successfully', {
-          yamlFile: args.yaml_file,
+          yamlFile: resolvedYamlFile,
           executionTime: result.executionTime,
           outputFiles: Object.keys(fileVerification).filter(k => k !== 'allFilesGenerated')
         });

package/src/mcp/tools/calculationTools.js

@@ -8,8 +8,8 @@ export const calculationTools = [
       properties: {
         yaml_file: {
           type: 'string',
-          description: '計算に使用するYAMLファイル名（拡張子.yamlを含む）',
-          pattern: '^[a-zA-Z0-9_\\-\\.]+\\.(yaml|yml)$'
+          description: '計算に使用するYAMLファイル名（拡張子.yamlを含む）。ファイル名のみ指定（例: poker.yaml）で、POKER_MCP_HOME/tasks/ 配下のファイルが自動的に参照されます。絶対パスも指定可能です。',
+          pattern: '^([a-zA-Z0-9_\\-\\.]+\\.(yaml|yml)|([a-zA-Z]:[\\\\\/]|\\/)[^\\\\]+\\.(yaml|yml))$'
         },
         summary_options: {
           type: 'object',