@codesys/mcp-toolkit 1.4.7 → 1.4.8

package/CHANGELOG.md

@@ -5,6 +5,14 @@ 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.4.8] - 2025-05-02
+### Fixed
+- Fixed critical bug with project creation in persistent sessions
+- Added SKIP_OPEN option to create_project script for session compatibility
+- Improved handling of project creation in sessions to avoid "project in use" errors
+- Added better error recovery when project file exists but can't be opened in a session
+- Added helpful user messages explaining session limitations with project operations
+
 ## [1.4.7] - 2025-05-02
 ### Fixed
 - Fixed create_project freezing in persistent sessions

package/dist/scripts/create_project.py

@@ -4,6 +4,8 @@ import sys, scriptengine as script_engine, os, shutil, time, traceback
 # Paths will be correctly escaped in the template substitution
 TEMPLATE_PROJECT_PATH = '{TEMPLATE_PROJECT_PATH}' 
 PROJECT_FILE_PATH = '{PROJECT_FILE_PATH}'
+# Optional flag to skip opening the project after creation (useful for sessions)
+SKIP_OPEN = '{SKIP_OPEN}' == 'true'
 
 # Print raw paths for debugging
 print("DEBUG: Template path (raw): %s" % repr(TEMPLATE_PROJECT_PATH))
@@ -61,6 +63,13 @@ try:
         
     print("DEBUG: File copy complete.")
 
+    # For sessions, we may want to skip opening the project to avoid conflicts
+    if SKIP_OPEN:
+        print("DEBUG: SKIP_OPEN is set to true, skipping project open step.")
+        print("Project Created from Template Copy at: %s" % PROJECT_FILE_PATH)
+        print("SCRIPT_SUCCESS: Project copied from template (open step skipped for session compatibility).")
+        sys.exit(0)
+    
     # 2. Open the newly copied project file
     print("DEBUG: Opening the copied project: %s" % PROJECT_FILE_PATH)
     # Set flags for silent opening
@@ -71,24 +80,31 @@ try:
     #     print("WARN: VersionUpdateFlags not found, using integer flags for open (1 | 2 = 3).")
     #     update_mode = 3
 
-    project = script_engine.projects.open(PROJECT_FILE_PATH, update_flags=update_mode)
-    print("DEBUG: script_engine.projects.open returned: %s" % project)
-    if project:
-        print("DEBUG: Pausing briefly after open...")
-        time.sleep(1.0) # Allow CODESYS to potentially initialize things
-        try:
-            print("DEBUG: Explicitly saving project after opening copy...")
-            project.save();
-            print("DEBUG: Project save after opening copy succeeded.")
-        except Exception as save_err:
-             print("WARN: Explicit save after opening copy failed: %s" % save_err)
-             # Decide if this is critical - maybe not, but good to know.
-        print("Project Created from Template Copy at: %s" % PROJECT_FILE_PATH)
-        print("SCRIPT_SUCCESS: Project copied from template and opened successfully.")
+    try:
+        project = script_engine.projects.open(PROJECT_FILE_PATH, update_flags=update_mode)
+        print("DEBUG: script_engine.projects.open returned: %s" % project)
+        if project:
+            print("DEBUG: Pausing briefly after open...")
+            time.sleep(1.0) # Allow CODESYS to potentially initialize things
+            try:
+                print("DEBUG: Explicitly saving project after opening copy...")
+                project.save();
+                print("DEBUG: Project save after opening copy succeeded.")
+            except Exception as save_err:
+                print("WARN: Explicit save after opening copy failed: %s" % save_err)
+                # Decide if this is critical - maybe not, but good to know.
+            print("Project Created from Template Copy at: %s" % PROJECT_FILE_PATH)
+            print("SCRIPT_SUCCESS: Project copied from template and opened successfully.")
+            sys.exit(0)
+        else:
+            error_message = "Failed to open project copy %s after copying template %s. projects.open returned None." % (PROJECT_FILE_PATH, TEMPLATE_PROJECT_PATH)
+            print(error_message); print("SCRIPT_ERROR: %s" % error_message); sys.exit(1)
+    except Exception as open_err:
+        print("WARN: Error opening project after creation: %s" % open_err)
+        # Even though we couldn't open it, the project was still created successfully
+        print("Project Created from Template Copy at: %s (but could not be opened automatically)" % PROJECT_FILE_PATH)
+        print("SCRIPT_SUCCESS: Project copied from template (but open operation failed: %s)." % open_err)
         sys.exit(0)
-    else:
-        error_message = "Failed to open project copy %s after copying template %s. projects.open returned None." % (PROJECT_FILE_PATH, TEMPLATE_PROJECT_PATH)
-        print(error_message); print("SCRIPT_ERROR: %s" % error_message); sys.exit(1)
 except Exception as e:
     detailed_error = traceback.format_exc()
     error_message = "Error creating project '%s' from template '%s': %s\\n%s" % (PROJECT_FILE_PATH, TEMPLATE_PROJECT_PATH, e, detailed_error)

package/dist/server.js

@@ -377,9 +377,14 @@ function startMcpServer(config) {
                 const normalizedTargetPath = absPath.replace(/\\/g, '/');
                 const normalizedTemplatePath = templatePath.replace(/\\/g, '/');
                 console.error(`Using normalized paths for script: Target=${normalizedTargetPath}, Template=${normalizedTemplatePath}`);
+                // When running in a session, skip the open step to avoid conflicts
+                const inSession = sessionManager &&
+                    (sessionManager.getState() === sessionManager_1.SessionState.RUNNING ||
+                        sessionManager.getState() === sessionManager_1.SessionState.BUSY);
                 const scriptContent = (0, scriptLoader_1.interpolateScript)(scriptTemplate, {
                     'PROJECT_FILE_PATH': normalizedTargetPath,
-                    'TEMPLATE_PROJECT_PATH': normalizedTemplatePath
+                    'TEMPLATE_PROJECT_PATH': normalizedTemplatePath,
+                    'SKIP_OPEN': inSession ? 'true' : 'false'
                 });
                 console.error(">>> create_project (copy-then-open): PREPARED SCRIPT:", scriptContent.substring(0, 500) + "...");
                 // Use a longer timeout for project creation operations in sessions (120 seconds)
@@ -427,6 +432,30 @@ function startMcpServer(config) {
                     }
                 }
                 const success = result.success && result.output.includes("SCRIPT_SUCCESS");
+                // Special handling for cases where project was created but couldn't be opened automatically
+                if (success && inSession && result.output.includes("open operation failed")) {
+                    return {
+                        content: [{
+                                type: "text",
+                                text: `Project created from template: ${absPath}\n` +
+                                    `Note: The project was created successfully, but couldn't be opened automatically because you're in a session.\n` +
+                                    `You can use open_project to open it now, or create a POU directly with create_pou.`
+                            }],
+                        isError: false
+                    };
+                }
+                // Special handling for skipped open operations in sessions
+                if (success && inSession && result.output.includes("open step skipped for session compatibility")) {
+                    return {
+                        content: [{
+                                type: "text",
+                                text: `Project created from template: ${absPath}\n` +
+                                    `Note: The project opening step was skipped because you're in a session.\n` +
+                                    `You can use open_project to open it now, or create a POU directly with create_pou.`
+                            }],
+                        isError: false
+                    };
+                }
                 return {
                     content: [{
                             type: "text",

package/package.json

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