@codesys/mcp-toolkit 1.5.3 → 1.5.6

package/CHANGELOG.md

@@ -5,6 +5,25 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.5.6] - 2025-05-02
+### Fixed
+- Fixed session manager null reference error in interop.ts
+- Added additional null checks for session manager
+- Improved error handling when session manager is undefined
+
+## [1.5.5] - 2025-05-02
+### Fixed
+- Added enhanced command.json file verification to debug session communication
+- Improved logging for command file operations to diagnose IPC issues
+- Added command data verification to detect potential file corruption
+
+## [1.5.4] - 2025-05-02
+### Fixed
+- Added `--no-session` flag to disable session management when CODESYS UI hangs
+- Improved session handling with better error detection and user feedback
+- Removed hardcoded session disable flag in favor of command-line option
+- Added safeguards to prevent UI blocking during session operations
+
 ## [1.5.3] - 2025-05-02
 ### Fixed
 - Added automatic session startup when server starts

package/dist/bin.js

@@ -38,6 +38,8 @@ commander_1.program
     // Add workspace option if needed, defaults to cwd
     .option('-w, --workspace <dir>', 'Workspace directory for relative project paths', process.cwd() // Default to current working directory
 )
+    // Add option to disable session management
+    .option('--no-session', 'Disable session management (use one-time script execution only)', false)
     .parse(process.argv); // Parse the arguments
 const options = commander_1.program.opts();
 // --- Log the options Commander parsed ---
@@ -54,8 +56,10 @@ console.error(`========================================`);
 const serverConfig = {
     codesysPath: options.codesysPath.trim(),
     profileName: options.codesysProfile.trim(),
-    workspaceDir: options.workspace.trim() // Pass workspace dir
+    workspaceDir: options.workspace.trim(), // Pass workspace dir
+    enableSessionManagement: options.session // Determined by --no-session flag (default: true)
 };
+console.error(`BIN.TS: Session management is ${serverConfig.enableSessionManagement ? 'enabled' : 'disabled'}`);
 // --- End Prepare Config ---
 // --- Call the server setup function ---
 console.error("BIN.TS: Calling startMcpServer...");

package/dist/codesys_interop.js

@@ -91,10 +91,10 @@ function executeCodesysScript(scriptContent, codesysExePath, codesysProfileName,
         if (!fs.existsSync(codesysExePath))
             throw new Error(`CODESYS executable not found at provided path: ${codesysExePath}`);
         // --- End Pre-checks ---
-        // Session management feature control
-        const SESSION_FEATURE_DISABLED = false; // Session feature enabled
+        // Session management feature is now fully controlled by the presence of sessionManager
+        // No need for a separate flag as server.ts will set sessionManager to undefined if disabled
         // --- Check if we should use the persistent session ---
-        if (!SESSION_FEATURE_DISABLED && sessionManager) {
+        if (sessionManager) {
             const sessionState = sessionManager.getState();
             // Check if the session is active
             if (sessionState === sessionManager_1.SessionState.RUNNING || sessionState === sessionManager_1.SessionState.BUSY) {
@@ -141,9 +141,6 @@ function executeCodesysScript(scriptContent, codesysExePath, codesysProfileName,
                 process.stderr.write(`INTEROP: Session not in RUNNING/BUSY state (current: ${sessionState}). Using one-time execution.\n`);
             }
         }
-        else if (SESSION_FEATURE_DISABLED) {
-            process.stderr.write(`INTEROP: Session feature is disabled. Using one-time execution mode.\n`);
-        }
         else {
             process.stderr.write(`INTEROP: No session manager provided. Using one-time execution mode.\n`);
         }

package/dist/direct_session_command.js

@@ -143,9 +143,24 @@ function executeScriptDirect(scriptContent_1) {
             timestamp: Date.now()
         };
         console.error(`DIRECT_SESSION: Writing command to ${COMMAND_FILE}`);
-        // Write command to file
-        fs.writeFileSync(COMMAND_FILE, JSON.stringify(commandData, null, 2));
-        console.error(`DIRECT_SESSION: Command written to file`);
+        console.error(`DIRECT_SESSION: Command data: ${JSON.stringify(commandData)}`);
+        try {
+            // Write command to file
+            fs.writeFileSync(COMMAND_FILE, JSON.stringify(commandData, null, 2));
+            console.error(`DIRECT_SESSION: Command written to file`);
+            // Verify the file was written correctly
+            try {
+                const verifyContent = fs.readFileSync(COMMAND_FILE, 'utf8');
+                console.error(`DIRECT_SESSION: Verification - Command file now contains: ${verifyContent.substring(0, 100)}...`);
+            }
+            catch (verifyErr) {
+                console.error(`DIRECT_SESSION: Verification failed - could not read command file: ${verifyErr}`);
+            }
+        }
+        catch (writeErr) {
+            console.error(`DIRECT_SESSION: ERROR writing command file: ${writeErr}`);
+            throw writeErr; // Re-throw to propagate the error
+        }
         // Poll for result
         return new Promise((resolve, reject) => {
             let pollCount = 0;

package/dist/server.js

@@ -117,25 +117,52 @@ function startMcpServer(config) {
         }
         // --- End Helper Function ---
         // --- Initialize Session Manager ---
-        const sessionManager = new sessionManager_1.CodesysSessionManager(codesysExePath, codesysProfileName);
-        console.error('SERVER.TS: Session manager initialized');
-        // Explicitly start the session when server starts
-        try {
-            // Check if session is already active
-            console.error('SERVER.TS: Checking if session is already active...');
-            const isActive = (0, direct_session_command_1.isSessionActive)();
-            if (isActive) {
-                console.error('SERVER.TS: Session is already active');
+        // Default to true if not specified
+        const enableSessionManagement = config.enableSessionManagement !== false;
+        console.error(`SERVER.TS: Session management is ${enableSessionManagement ? 'enabled' : 'disabled'}`);
+        // Create session manager instance (will remain undefined if session management is disabled)
+        let sessionManager = undefined;
+        if (enableSessionManagement) {
+            try {
+                sessionManager = new sessionManager_1.CodesysSessionManager(codesysExePath, codesysProfileName);
+                console.error('SERVER.TS: Session manager initialized');
+                // Only start a session if explicitly enabled and manager was created successfully
+                if (sessionManager) {
+                    try {
+                        // Check if session is already active
+                        console.error('SERVER.TS: Checking if session is already active...');
+                        const isActive = (0, direct_session_command_1.isSessionActive)();
+                        if (isActive) {
+                            console.error('SERVER.TS: Session is already active');
+                        }
+                        else {
+                            console.error('SERVER.TS: Session not active, starting new session...');
+                            console.error('SERVER.TS: IMPORTANT: If CODESYS hangs during session startup, use --no-session flag');
+                            yield sessionManager.startSession();
+                            console.error('SERVER.TS: Session start initiated');
+                        }
+                    }
+                    catch (error) {
+                        console.error(`SERVER.TS: Error starting session: ${error.message}`);
+                        console.error('SERVER.TS: Will fallback to one-time execution for operations');
+                        // Continue without session, will fallback to one-time execution
+                    }
+                }
+                else {
+                    console.error('SERVER.TS: Session manager creation failed');
+                }
             }
-            else {
-                console.error('SERVER.TS: Session not active, starting new session...');
-                yield sessionManager.startSession();
-                console.error('SERVER.TS: Session start initiated');
+            catch (error) {
+                console.error(`SERVER.TS: Failed to create session manager: ${error.message}`);
+                console.error('SERVER.TS: Will fallback to one-time execution for operations');
+                sessionManager = undefined;
             }
         }
-        catch (error) {
-            console.error(`SERVER.TS: Error starting session: ${error.message}`);
-            // Continue without session, will fallback to one-time execution
+        else {
+            console.error('SERVER.TS: Session management is disabled, will use one-time script execution');
+        }
+        if (!sessionManager) {
+            console.error('SERVER.TS: Session manager is not available - one-time script execution will be used');
         }
         // --- End Session Manager Initialization ---
         // --- Preload script templates ---

package/dist/sessionManager.js

@@ -639,8 +639,24 @@ class CodesysSessionManager extends events_1.EventEmitter {
                     };
                     // Send the command by writing to the command file
                     console.error(`SESSION_MANAGER: Writing command to file ${this.commandFilePath}`);
-                    fs.writeFileSync(this.commandFilePath, JSON.stringify(commandData, null, 2));
-                    console.error(`SESSION_MANAGER: Command written to file`);
+                    console.error(`SESSION_MANAGER: Command data: ${JSON.stringify(commandData)}`);
+                    try {
+                        // Write command to file
+                        fs.writeFileSync(this.commandFilePath, JSON.stringify(commandData, null, 2));
+                        console.error(`SESSION_MANAGER: Command written to file`);
+                        // Verify the file was written correctly
+                        try {
+                            const verifyContent = fs.readFileSync(this.commandFilePath, 'utf8');
+                            console.error(`SESSION_MANAGER: Verification - Command file now contains: ${verifyContent.substring(0, 100)}...`);
+                        }
+                        catch (verifyErr) {
+                            console.error(`SESSION_MANAGER: Verification failed - could not read command file: ${verifyErr}`);
+                        }
+                    }
+                    catch (writeErr) {
+                        console.error(`SESSION_MANAGER: ERROR writing command file: ${writeErr}`);
+                        throw writeErr; // Re-throw to propagate the error
+                    }
                     // Start reading the result file
                     this.readResultFile(scriptId);
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codesys/mcp-toolkit",
-  "version": "1.5.3",
+  "version": "1.5.6",
   "description": "Model Context Protocol (MCP) server for CODESYS automation platform",
   "main": "dist/server.js",
   "bin": {