poker-mcp 1.2.5 → 1.2.6

package/README.md

@@ -53,10 +53,10 @@ npx poker-mcp
 **POKER_INSTALL_PATH環境変数**（オプション）:
 ```bash
 # Windowsの場合（コマンドプロンプト）
-set POKER_INSTALL_PATH=C:\Program Files\POKER
+set POKER_INSTALL_PATH=C:/Poker
 
 # Windowsの場合（PowerShell）
-$env:POKER_INSTALL_PATH="C:\Program Files\POKER"
+$env:POKER_INSTALL_PATH="C:/Poker"
 
 # Linux/macOSの場合
 export POKER_INSTALL_PATH="/usr/local/share/poker"
@@ -86,7 +86,7 @@ export POKER_INSTALL_PATH="/usr/local/share/poker"
          "command": "node",
          "args": ["C:\\Users\\yoshi\\Desktop\\poker_mcp\\src\\mcp_server_stdio_v4.js"],
          "env": {
-           "POKER_INSTALL_PATH": "C:/Program Files/POKER"
+           "POKER_INSTALL_PATH": "C:/Poker"
          }
        }
      }
@@ -101,7 +101,7 @@ export POKER_INSTALL_PATH="/usr/local/share/poker"
          "command": "npx",
          "args": ["poker-mcp"],
          "env": {
-           "POKER_INSTALL_PATH": "C:/Program Files/POKER"
+           "POKER_INSTALL_PATH": "C:/Poker"
          }
        }
      }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "poker-mcp",
-  "version": "1.2.5",
+  "version": "1.2.6",
   "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/config/ConfigManager.js

@@ -1,14 +1,21 @@
 // src/config/ConfigManager.js
 import fs from 'fs/promises';
 import path from 'path';
+import { fileURLToPath } from 'url';
 import { logger } from '../utils/logger.js';
 import { DataError } from '../utils/errors.js';
+import { USER_CONFIG } from '../utils/paths.js';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname  = path.dirname(__filename);
 
 export class ConfigManager {
   constructor() {
     this.config = null;
-    this.configPath = 'src/config/default.json';
-    this.userConfigPath = 'config.json';
+    // default.json はパッケージ同梱なので __dirname 基準の絶対パスで解決
+    this.configPath     = path.join(__dirname, 'default.json');
+    // ユーザー設定は POKER_MCP_HOME 配下に置く
+    this.userConfigPath = USER_CONFIG;
   }
 
   async load() {

package/src/mcp/server.js

@@ -7,9 +7,10 @@ import { allTools } from './tools/index.js';
 import { createAllHandlers } from './handlers/index.js';
 import { safeExecute } from './middleware/errorHandler.js';
 import { logger } from '../utils/logger.js';
+import { YAML_FILE, PENDING_FILE } from '../utils/paths.js';
 
 export class PokerMcpServer {
-  constructor(yamlFile = 'tasks/poker.yaml', pendingFile = 'tasks/pending_changes.json') {
+  constructor(yamlFile = YAML_FILE, pendingFile = PENDING_FILE) {
     this.server = new Server(
       {
         name: 'poker-mcp',

package/src/mcp_server_stdio_v4.js

@@ -1,45 +1,47 @@
-#!/usr/bin/env node
-// mcp_server_stdio_v4.js - MCP専用版（stdout汚染防止）
-import { PokerMcpServer } from './mcp/server.js';
-import { logger } from './utils/logger.js';
-
-async function main() {
-  try {
-    const server = new PokerMcpServer();
-    await server.initialize();
-    await server.start();
-  } catch (error) {
-    logger.error('サーバー起動失敗', { error: error.message });
-    // console.errorは削除（stdout汚染防止）
-    process.exit(1);
-  }
-}
-
-// 未処理の例外とプロミス拒否のハンドリング
-process.on('uncaughtException', (error) => {
-  logger.error('Uncaught Exception', { error: error.message, stack: error.stack });
-  // console.errorは削除（stdout汚染防止）
-  process.exit(1);
-});
-
-process.on('unhandledRejection', (reason, promise) => {
-  logger.error('Unhandled Rejection', { reason: reason?.message || reason, promise });
-  // console.errorは削除（stdout汚染防止）
-  process.exit(1);
-});
-
-// グレースフルシャットダウン
-process.on('SIGINT', () => {
-  logger.info('SIGINT受信、サーバーを終了します');
-  process.exit(0);
-});
-
-process.on('SIGTERM', () => {
-  logger.info('SIGTERM受信、サーバーを終了します');
-  process.exit(0);
-});
-
-main().catch((error) => {
-  logger.error('メイン関数エラー', { error: error.message });
-  process.exit(1);
-});
+#!/usr/bin/env node
+// mcp_server_stdio_v4.js - MCP専用版（stdout汚染防止）
+import { PokerMcpServer } from './mcp/server.js';
+import { logger } from './utils/logger.js';
+
+async function main() {
+  try {
+    const server = new PokerMcpServer();
+    await server.initialize();
+    await server.start();
+  } catch (error) {
+    logger.error('サーバー起動失敗', { error: error.message });
+    // stdoutは汚染禁止だが、stderrへの出力は許容（Claude Desktopがログに記録する）
+    process.stderr.write(`[poker-mcp] Fatal: ${error.message}\n`);
+    process.exit(1);
+  }
+}
+
+// 未処理の例外とプロミス拒否のハンドリング
+process.on('uncaughtException', (error) => {
+  logger.error('Uncaught Exception', { error: error.message, stack: error.stack });
+  process.stderr.write(`[poker-mcp] Uncaught Exception: ${error.message}\n`);
+  process.exit(1);
+});
+
+process.on('unhandledRejection', (reason, promise) => {
+  logger.error('Unhandled Rejection', { reason: reason?.message || reason, promise });
+  process.stderr.write(`[poker-mcp] Unhandled Rejection: ${reason?.message || reason}\n`);
+  process.exit(1);
+});
+
+// グレースフルシャットダウン
+process.on('SIGINT', () => {
+  logger.info('SIGINT受信、サーバーを終了します');
+  process.exit(0);
+});
+
+process.on('SIGTERM', () => {
+  logger.info('SIGTERM受信、サーバーを終了します');
+  process.exit(0);
+});
+
+main().catch((error) => {
+  logger.error('メイン関数エラー', { error: error.message });
+  process.stderr.write(`[poker-mcp] Main error: ${error.message}\n`);
+  process.exit(1);
+});

package/src/services/DataManager.js

@@ -7,13 +7,16 @@ import { DataError } from '../utils/errors.js';
 import CollisionDetector from '../utils/CollisionDetector.js';
 import NuclideManager from '../utils/NuclideManager.js';
 import EnhancedValidator from '../utils/EnhancedValidator.js';
+import { BACKUPS_DIR, LOGS_DIR, TASKS_DIR, DATA_DIR, NDX_FILE,
+         YAML_FILE, PENDING_FILE } from '../utils/paths.js';
 
 export class SafeDataManager {
   constructor(yamlFile, pendingFile) {
-    this.yamlFile = yamlFile;
-    this.pendingFile = pendingFile;
-    this.backupDir = 'backups';
-    this.maxBackups = 10;
+    // 引数が省略された場合は paths.js の絶対パスをデフォルトとして使用
+    this.yamlFile    = yamlFile    ?? YAML_FILE;
+    this.pendingFile = pendingFile ?? PENDING_FILE;
+    this.backupDir   = BACKUPS_DIR;
+    this.maxBackups  = 10;
     this.data = null;
     this.pendingChanges = [];
     
@@ -24,11 +27,11 @@ export class SafeDataManager {
       max_auto_corrections: 10
     });
     
-    // 核種管理の初期化
+    // 核種管理の初期化（絶対パスを使用）
     this.nuclideManager = new NuclideManager({
       contribution_threshold: 0.05,
       user_confirmation: true,
-      database_file: 'data/ICRP-07.NDX'
+      database_file: NDX_FILE
     });
     
     // 強化検証の初期化
@@ -41,11 +44,11 @@ export class SafeDataManager {
 
   async initialize() {
     try {
-      // 必要なフォルダをすべて作成
-      await fs.mkdir(this.backupDir, { recursive: true });  // backups
-      await fs.mkdir('logs', { recursive: true });          // logs  
-      await fs.mkdir('tasks', { recursive: true });         // tasks
-      await fs.mkdir('data', { recursive: true });          // data
+      // 必要なフォルダをすべて作成（絶対パスで権限エラーを回避）
+      await fs.mkdir(BACKUPS_DIR, { recursive: true });  // backups
+      await fs.mkdir(LOGS_DIR,    { recursive: true });  // logs
+      await fs.mkdir(TASKS_DIR,   { recursive: true });  // tasks
+      await fs.mkdir(DATA_DIR,    { recursive: true });  // data
 
       // 必要なデータファイルをコピー
       await this.ensureDataFiles();
@@ -70,7 +73,7 @@ export class SafeDataManager {
   // 新規メソッド: 必要なデータファイルの確保
   async ensureDataFiles() {
     try {
-      const targetFile = 'data/ICRP-07.NDX';
+      const targetFile = NDX_FILE;  // 絶対パスを使用
       
       // ターゲットファイルの存在チェック
       try {

package/src/utils/logger.js

@@ -1,24 +1,29 @@
-// utils/logger.js - MCP専用版（stdout汚染防止）
-import winston from 'winston';
-import path from 'path';
-
-const logger = winston.createLogger({
-  level: process.env.LOG_LEVEL || 'info',
-  format: winston.format.combine(
-    winston.format.timestamp(),
-    winston.format.errors({ stack: true }),
-    winston.format.json()
-  ),
-  transports: [
-    new winston.transports.File({ 
-      filename: path.join('logs', 'error.log'), 
-      level: 'error' 
-    }),
-    new winston.transports.File({ 
-      filename: path.join('logs', 'combined.log') 
-    })
-    // Console transportは削除（MCPサーバーではstdout汚染禁止）
-  ]
-});
-
-export { logger };
+// utils/logger.js - MCP専用版（stdout汚染防止）
+import winston from 'winston';
+import path from 'path';
+import fs from 'fs';
+import { LOGS_DIR } from './paths.js';
+
+// ログディレクトリを起動時に確実に作成（相対パス問題を回避）
+fs.mkdirSync(LOGS_DIR, { recursive: true });
+
+const logger = winston.createLogger({
+  level: process.env.LOG_LEVEL || 'info',
+  format: winston.format.combine(
+    winston.format.timestamp(),
+    winston.format.errors({ stack: true }),
+    winston.format.json()
+  ),
+  transports: [
+    new winston.transports.File({ 
+      filename: path.join(LOGS_DIR, 'error.log'), 
+      level: 'error' 
+    }),
+    new winston.transports.File({ 
+      filename: path.join(LOGS_DIR, 'combined.log') 
+    })
+    // Console transportは削除（MCPサーバーではstdout汚染禁止）
+  ]
+});
+
+export { logger };

package/src/utils/paths.js

@@ -0,0 +1,24 @@
+// utils/paths.js - 実行時データの格納先を一元管理
+//
+// 優先順位:
+//   1. 環境変数 POKER_MCP_HOME が設定されていれば、その値を使用
+//   2. 未設定なら os.homedir()/.poker-mcp/ をデフォルトとする
+//
+// claude_desktop_config.json での設定例:
+//   "env": { "POKER_MCP_HOME": "C:\\Users\\yoshi\\poker_mcp_workspace" }
+//
+import path from 'path';
+import os from 'os';
+
+export const BASE_DIR = process.env.POKER_MCP_HOME
+  ?? path.join(os.homedir(), '.poker-mcp');
+
+export const LOGS_DIR   = path.join(BASE_DIR, 'logs');
+export const TASKS_DIR  = path.join(BASE_DIR, 'tasks');
+export const BACKUPS_DIR = path.join(BASE_DIR, 'backups');
+export const DATA_DIR   = path.join(BASE_DIR, 'data');
+
+export const YAML_FILE    = path.join(TASKS_DIR,  'poker.yaml');
+export const PENDING_FILE = path.join(TASKS_DIR,  'pending_changes.json');
+export const NDX_FILE     = path.join(DATA_DIR,   'ICRP-07.NDX');
+export const USER_CONFIG  = path.join(BASE_DIR,   'config.json');