@codesys/mcp-toolkit 1.4.8 → 1.4.9

package/CHANGELOG.md

@@ -5,6 +5,15 @@ 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.9] - 2025-05-02
+### Fixed
+- Fixed critical issue with create_project not succeeding in persistent sessions
+- Added comprehensive debugging for project creation operations
+- Improved result file handling to prevent file access issues
+- Enhanced success detection for project file creation when opening fails
+- Fixed issue with result files not being properly written during critical operations
+- Added atomic file writing with fallback mechanisms for result files
+
 ## [1.4.8] - 2025-05-02
 ### Fixed
 - Fixed critical bug with project creation in persistent sessions

package/dist/scripts/start_session.py

@@ -287,8 +287,28 @@ def execute_command(command_data):
                     # For project creation/opening, we'll execute the script with some guards
                     print("Executing critical script with guards...")
                     try:
-                        # Execute the script
+                        # Execute the script in a more controlled manner with debugging
+                        print("DEBUG-CRITICAL: About to execute project creation/open script")
+                        # First write project ops directly to the result file to prevent hanging
+                        try:
+                            with open(RESULT_FILE, 'w') as f:
+                                result_data = {
+                                    "status": "in_progress",
+                                    "result": {
+                                        "id": command.get("id", "unknown"),
+                                        "success": False,
+                                        "output": "Project creation/opening operation in progress...",
+                                    },
+                                    "timestamp": time.time()
+                                }
+                                json.dump(result_data, f, indent=2)
+                            print("DEBUG-CRITICAL: Created preliminary result file")
+                        except Exception as rf_err:
+                            print("DEBUG-CRITICAL: Error creating preliminary result: {0}".format(rf_err))
+                        
+                        # Now execute the script
                         exec(script, local_globals)
+                        print("DEBUG-CRITICAL: Script executed successfully")
                     except Exception as e:
                         # If the script fails, capture the error
                         print("ERROR in critical script execution: {0}".format(e))
@@ -315,6 +335,32 @@ def execute_command(command_data):
                 stdout_output = stdout_buffer.getvalue()
                 stderr_output = stderr_buffer.getvalue()
                 
+                # Special handling for project creation - always check if we can find success markers
+                # in partial output even if script appears to have issues
+                if has_project_create:
+                    # Look specifically for the file copy completion
+                    if "File copy VERIFIED - file exists at destination" in stdout_output or "DEBUG: File copy complete" in stdout_output:
+                        print("IMPORTANT: Detected successful file copy in project creation script")
+                        project_path = ""
+                        
+                        # Try to extract the project path from output
+                        project_path_match = None
+                        for line in stdout_output.split('\n'):
+                            if "DEBUG: Target Path =" in line:
+                                project_path_match = line.split('=')[1].strip()
+                                break
+                        
+                        if project_path_match:
+                            project_path = project_path_match
+                            print("IMPORTANT: Found project path: {0}".format(project_path))
+                            
+                            # Even if there was an error later (like opening), consider this a partial success
+                            if "Error creating project" in stdout_output and "currently in use" in stdout_output:
+                                print("IMPORTANT: Project file was created but couldn't be opened due to 'in use' error")
+                                # Combine success and error markers
+                                stdout_output += "\nSCRIPT_SUCCESS: Project file was created at {0} but could not be opened because it is in use by another CODESYS instance.".format(project_path)
+                                result["success"] = True
+                
                 # Check for success/error markers
                 if SCRIPT_SUCCESS_MARKER in stdout_output or SCRIPT_SUCCESS_MARKER in stderr_output:
                     result["success"] = True
@@ -323,9 +369,18 @@ def execute_command(command_data):
                     result["success"] = False
                     print("Found error marker in script output")
                 else:
-                    # No explicit markers, assume success
-                    result["success"] = True
-                    print("No markers found in script output, assuming success")
+                    # No explicit markers, assume success for non-critical operations
+                    # For critical operations like project creation, be more careful
+                    if has_project_create:
+                        if "File copy complete" in stdout_output:
+                            result["success"] = True
+                            print("Project creation seems successful based on file copy completion")
+                        else:
+                            result["success"] = False
+                            print("Project creation script lacks success markers and file copy confirmation")
+                    else:
+                        result["success"] = True
+                        print("No markers found in script output, assuming success")
                 
                 # Tally output length to check if output is reasonable
                 print("Output length: stdout={0} chars, stderr={1} chars".format(
@@ -450,7 +505,7 @@ def check_command():
                 result, stop_requested = execute_command(command)
                 print("Command execution complete, stop_requested: {0}".format(stop_requested))
                 
-                # Write the result
+                # Write the result with extra safety measures
                 result_data = {
                     "status": "complete",
                     "result": result,
@@ -458,8 +513,50 @@ def check_command():
                 }
                 
                 print("Writing result to {0}".format(RESULT_FILE))
-                with open(RESULT_FILE, 'w') as f:
-                    json.dump(result_data, f, indent=2)
+                sys.stdout.flush()
+                
+                # First, write to a temporary file to avoid corruption
+                temp_result_file = RESULT_FILE + ".tmp"
+                try:
+                    with open(temp_result_file, 'w') as f:
+                        json.dump(result_data, f, indent=2)
+                    print("Temporary result file written successfully")
+                    
+                    # If temporary write succeeded, move to the real file
+                    if os.path.exists(temp_result_file):
+                        # On Windows, we may need to remove the destination first
+                        if os.path.exists(RESULT_FILE):
+                            try:
+                                os.remove(RESULT_FILE)
+                            except Exception as rm_err:
+                                print("Warning: Could not remove existing result file: {0}".format(rm_err))
+                        
+                        # Now rename the temp file to the real file
+                        os.rename(temp_result_file, RESULT_FILE)
+                        print("Result file updated successfully")
+                    else:
+                        print("ERROR: Temporary result file not found after writing!")
+                        # Try direct write as fallback
+                        with open(RESULT_FILE, 'w') as f:
+                            json.dump(result_data, f, indent=2)
+                except Exception as result_err:
+                    print("ERROR writing result file: {0}".format(result_err))
+                    traceback.print_exc()
+                    sys.stdout.flush()
+                    
+                    # Desperate attempt - try direct write with minimal content
+                    try:
+                        with open(RESULT_FILE, 'w') as f:
+                            f.write('{"status":"complete","result":{"id":"' + 
+                                   result["id"] + '","success":' + 
+                                   ('true' if result["success"] else 'false') + 
+                                   ',"output":"Result writing failed but operation completed"},"timestamp":' + 
+                                   str(time.time()) + '}')
+                        print("Emergency minimal result file written")
+                    except Exception as emergency_err:
+                        print("CRITICAL ERROR: Even emergency result write failed: {0}".format(emergency_err))
+                
+                sys.stdout.flush()
                 
                 # Reset command file
                 command_data = {

package/package.json

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