@codesys/mcp-toolkit 1.1.7 → 1.1.8

package/CHANGELOG.md

@@ -5,6 +5,12 @@ 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.1.8] - 2025-05-06
+
+### Fixed
+- Enhanced create_pou script to automatically find the Application object when only "Application" is specified as the parent path
+- Added better error messages with suggestions when Application isn't found
+
 ## [1.1.7] - 2025-05-06
 
 ### Fixed

package/dist/scripts/create_pou.py

@@ -35,9 +35,52 @@ try:
     pou_type_enum = pou_type_map.get(POU_TYPE_STR)
     if not pou_type_enum: raise ValueError("Invalid POU type string: %s. Use Program, FunctionBlock, or Function." % POU_TYPE_STR)
 
-    # Find parent object using the robust function
-    parent_object = find_object_by_path_robust(primary_project, PARENT_PATH_REL, "parent container")
-    if not parent_object: raise ValueError("Parent object not found for path: %s" % PARENT_PATH_REL)
+    # For common case where user just specified "Application", automatically try to find it
+    if PARENT_PATH_REL == "Application":
+        # Get project name from file path to build the likely full path
+        project_name = os.path.splitext(os.path.basename(PROJECT_FILE_PATH))[0]
+        potential_paths = [
+            PARENT_PATH_REL,                      # Original "Application"
+            f"{project_name}.{PARENT_PATH_REL}",  # "projectName.Application"
+            f"{project_name}/{PARENT_PATH_REL}",  # "projectName/Application"
+            f"{project_name}"                     # Just the project name itself might work
+        ]
+        
+        print(f"DEBUG: Parent path is simply 'Application', trying several variants to find it")
+        
+        # Try each potential path until one works
+        parent_object = None
+        for path in potential_paths:
+            print(f"DEBUG: Attempting to find parent with path: '{path}'")
+            parent_candidate = find_object_by_path_robust(primary_project, path, "parent container")
+            if parent_candidate:
+                parent_object = parent_candidate
+                print(f"DEBUG: Successfully found parent using path: '{path}'")
+                break
+                
+        if not parent_object:
+            # For diagnostics, try to get the application object directly as a fallback
+            print("DEBUG: All path attempts failed. Trying to access application directly...")
+            try:
+                if hasattr(primary_project, 'active_application'):
+                    app = primary_project.active_application
+                    if app:
+                        parent_object = app
+                        print(f"DEBUG: Found application object directly: {app.get_name()}")
+                if not parent_object and hasattr(primary_project, 'find'):
+                    apps = primary_project.find("Application", True)
+                    if apps:
+                        parent_object = apps[0]
+                        print(f"DEBUG: Found application via search: {parent_object.get_name()}")
+            except Exception as e:
+                print(f"ERROR: Direct application access also failed: {e}")
+    else:
+        # Use the provided path normally
+        parent_object = find_object_by_path_robust(primary_project, PARENT_PATH_REL, "parent container")
+    
+    # Final check if parent was found
+    if not parent_object: 
+        raise ValueError("Parent object not found for path: %s. Try using the full path like 'ProjectName.Application' or run get_project_structure first to see the correct structure." % PARENT_PATH_REL)
 
     parent_name = getattr(parent_object, 'get_name', lambda: str(parent_object))()
     print("DEBUG: Using parent object: %s (Type: %s)" % (parent_name, type(parent_object).__name__))

package/package.json

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

package/src/scripts/create_pou.py

@@ -35,9 +35,52 @@ try:
     pou_type_enum = pou_type_map.get(POU_TYPE_STR)
     if not pou_type_enum: raise ValueError("Invalid POU type string: %s. Use Program, FunctionBlock, or Function." % POU_TYPE_STR)
 
-    # Find parent object using the robust function
-    parent_object = find_object_by_path_robust(primary_project, PARENT_PATH_REL, "parent container")
-    if not parent_object: raise ValueError("Parent object not found for path: %s" % PARENT_PATH_REL)
+    # For common case where user just specified "Application", automatically try to find it
+    if PARENT_PATH_REL == "Application":
+        # Get project name from file path to build the likely full path
+        project_name = os.path.splitext(os.path.basename(PROJECT_FILE_PATH))[0]
+        potential_paths = [
+            PARENT_PATH_REL,                      # Original "Application"
+            f"{project_name}.{PARENT_PATH_REL}",  # "projectName.Application"
+            f"{project_name}/{PARENT_PATH_REL}",  # "projectName/Application"
+            f"{project_name}"                     # Just the project name itself might work
+        ]
+        
+        print(f"DEBUG: Parent path is simply 'Application', trying several variants to find it")
+        
+        # Try each potential path until one works
+        parent_object = None
+        for path in potential_paths:
+            print(f"DEBUG: Attempting to find parent with path: '{path}'")
+            parent_candidate = find_object_by_path_robust(primary_project, path, "parent container")
+            if parent_candidate:
+                parent_object = parent_candidate
+                print(f"DEBUG: Successfully found parent using path: '{path}'")
+                break
+                
+        if not parent_object:
+            # For diagnostics, try to get the application object directly as a fallback
+            print("DEBUG: All path attempts failed. Trying to access application directly...")
+            try:
+                if hasattr(primary_project, 'active_application'):
+                    app = primary_project.active_application
+                    if app:
+                        parent_object = app
+                        print(f"DEBUG: Found application object directly: {app.get_name()}")
+                if not parent_object and hasattr(primary_project, 'find'):
+                    apps = primary_project.find("Application", True)
+                    if apps:
+                        parent_object = apps[0]
+                        print(f"DEBUG: Found application via search: {parent_object.get_name()}")
+            except Exception as e:
+                print(f"ERROR: Direct application access also failed: {e}")
+    else:
+        # Use the provided path normally
+        parent_object = find_object_by_path_robust(primary_project, PARENT_PATH_REL, "parent container")
+    
+    # Final check if parent was found
+    if not parent_object: 
+        raise ValueError("Parent object not found for path: %s. Try using the full path like 'ProjectName.Application' or run get_project_structure first to see the correct structure." % PARENT_PATH_REL)
 
     parent_name = getattr(parent_object, 'get_name', lambda: str(parent_object))()
     print("DEBUG: Using parent object: %s (Type: %s)" % (parent_name, type(parent_object).__name__))