Refactor

.gitignore

@@ -20,8 +20,8 @@ target/
 
 # NetBeans files
 nbproject/private/
-build/
-nbbuild/
+/build**
+/nbbuild/**
 dist/
 nbdist/
 .nb-gradle/

api-project-platform/pom.xml

@@ -6,7 +6,7 @@
     <parent>
         <groupId>org.opendevstack.apiservice</groupId>
         <artifactId>devstack-api-service</artifactId>
-        <version>0.0.1-SNAPSHOT</version>
+        <version>0.0.2-SNAPSHOT</version>
     </parent>
 
     <artifactId>api-project-platform</artifactId>

api-project-users/pom.xml

@@ -6,7 +6,7 @@
     <parent>
         <groupId>org.opendevstack.apiservice</groupId>
         <artifactId>devstack-api-service</artifactId>
-        <version>0.0.1-SNAPSHOT</version>
+        <version>0.0.2-SNAPSHOT</version>
     </parent>
 
     <artifactId>api-project-users</artifactId>
@@ -50,17 +50,14 @@
         <dependency>
             <groupId>io.jsonwebtoken</groupId>
             <artifactId>jjwt-api</artifactId>
-            <version>0.12.3</version>
         </dependency>
         <dependency>
             <groupId>io.jsonwebtoken</groupId>
             <artifactId>jjwt-impl</artifactId>
-            <version>0.12.3</version>
         </dependency>
         <dependency>
             <groupId>io.jsonwebtoken</groupId>
             <artifactId>jjwt-jackson</artifactId>
-            <version>0.12.3</version>
         </dependency>
 
         <dependency>

api-project-users/src/main/java/org/opendevstack/apiservice/projectusers/exception/GlobalExceptionHandler.java

@@ -11,14 +11,18 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.http.HttpStatus;
+import org.springframework.http.MediaType;
 import org.springframework.http.ResponseEntity;
 import org.springframework.http.converter.HttpMessageNotReadableException;
+import org.springframework.web.HttpMediaTypeNotSupportedException;
+import org.springframework.web.HttpRequestMethodNotSupportedException;
 import org.springframework.web.bind.MethodArgumentNotValidException;
 import org.springframework.web.bind.MissingPathVariableException;
 import org.springframework.web.bind.MissingServletRequestParameterException;
 import org.springframework.web.bind.annotation.ControllerAdvice;
 import org.springframework.web.bind.annotation.ExceptionHandler;
 import org.springframework.web.method.annotation.MethodArgumentTypeMismatchException;
+import org.springframework.web.servlet.NoHandlerFoundException;
 import org.opendevstack.apiservice.externalservice.aap.exception.AutomationPlatformException;
 
 import java.util.ArrayList;
@@ -164,6 +168,82 @@ public ResponseEntity<BaseApiResponse> handleHttpMessageNotReadableException(
         return ResponseEntity.status(HttpStatus.BAD_REQUEST).body(errorResponse);
     }
 
+    /**
+     * Handles unsupported media type exceptions (wrong Content-Type header).
+     */
+    @ExceptionHandler(HttpMediaTypeNotSupportedException.class)
+    public ResponseEntity<BaseApiResponse> handleHttpMediaTypeNotSupportedException(
+            HttpMediaTypeNotSupportedException ex) {
+
+        logger.warn("Unsupported media type: {}", ex.getMessage());
+
+        String supportedTypes = ex.getSupportedMediaTypes().stream()
+                .map(MediaType::toString)
+                .collect(Collectors.joining(", "));
+
+        String errorMessage = String.format(
+                "Content type '%s' is not supported. Supported media types are: %s",
+                ex.getContentType() != null ? ex.getContentType() : "unknown",
+                supportedTypes);
+
+        BaseApiResponse errorResponse = new BaseApiResponse();
+        errorResponse.setSuccess(false);
+        errorResponse.setMessage(errorMessage);
+        errorResponse.setError(ErrorCodes.PROJECT_USER_ERROR);
+        errorResponse.setTimestamp(java.time.OffsetDateTime.now());
+        return ResponseEntity.status(HttpStatus.UNSUPPORTED_MEDIA_TYPE).body(errorResponse);
+    }
+
+    /**
+     * Handles unsupported HTTP method exceptions (wrong HTTP method used).
+     */
+    @ExceptionHandler(HttpRequestMethodNotSupportedException.class)
+    public ResponseEntity<BaseApiResponse> handleHttpRequestMethodNotSupportedException(
+            HttpRequestMethodNotSupportedException ex) {
+
+        logger.warn("Unsupported HTTP method: {}", ex.getMessage());
+
+        String supportedMethods = ex.getSupportedHttpMethods() != null
+                ? ex.getSupportedHttpMethods().stream()
+                    .map(Object::toString)
+                    .collect(Collectors.joining(", "))
+                : "unknown";
+
+        String errorMessage = String.format(
+                "HTTP method '%s' is not supported for this endpoint. Supported methods are: %s",
+                ex.getMethod(),
+                supportedMethods);
+
+        BaseApiResponse errorResponse = new BaseApiResponse();
+        errorResponse.setSuccess(false);
+        errorResponse.setMessage(errorMessage);
+        errorResponse.setError(ErrorCodes.PROJECT_USER_ERROR);
+        errorResponse.setTimestamp(java.time.OffsetDateTime.now());
+        return ResponseEntity.status(HttpStatus.METHOD_NOT_ALLOWED).body(errorResponse);
+    }
+
+    /**
+     * Handles no handler found exceptions (endpoint does not exist).
+     */
+    @ExceptionHandler(NoHandlerFoundException.class)
+    public ResponseEntity<BaseApiResponse> handleNoHandlerFoundException(
+            NoHandlerFoundException ex) {
+
+        logger.warn("No endpoint found: {}", ex.getMessage());
+
+        String errorMessage = String.format(
+                "No endpoint '%s %s' found",
+                ex.getHttpMethod(),
+                ex.getRequestURL());
+
+        BaseApiResponse errorResponse = new BaseApiResponse();
+        errorResponse.setSuccess(false);
+        errorResponse.setMessage(errorMessage);
+        errorResponse.setError(ErrorCodes.PROJECT_USER_ERROR);
+        errorResponse.setTimestamp(java.time.OffsetDateTime.now());
+        return ResponseEntity.status(HttpStatus.NOT_FOUND).body(errorResponse);
+    }
+
     /**
      * Handles missing path variable exceptions.
      */
@@ -306,6 +386,22 @@ public ResponseEntity<BaseApiResponse> handleInvalidRoleException(
         return ResponseEntity.status(HttpStatus.BAD_REQUEST).body(errorResponse);
     }
 
+    /**
+     * Handles invalid token exceptions.
+     */
+    @ExceptionHandler(InvalidTokenException.class)
+    public ResponseEntity<BaseApiResponse> handleInvalidTokenException(
+            InvalidTokenException ex) {
+
+        logger.warn("Invalid token: {}", ex.getMessage());
+        BaseApiResponse errorResponse = new BaseApiResponse();
+        errorResponse.setSuccess(false);
+        errorResponse.setMessage(ex.getMessage());
+        errorResponse.setError(ex.getErrorCode());
+        errorResponse.setTimestamp(java.time.OffsetDateTime.now());
+        return ResponseEntity.status(HttpStatus.BAD_REQUEST).body(errorResponse);
+    }
+
     /**
      * Handles automation platform exceptions.
      */

api-project-users/src/main/java/org/opendevstack/apiservice/projectusers/exception/InvalidTokenException.java

@@ -1,8 +1,12 @@
 package org.opendevstack.apiservice.projectusers.exception;
 
+import org.springframework.http.HttpStatus;
+import org.springframework.web.bind.annotation.ResponseStatus;
+
 /**
  * Exception thrown when an invalid token is provided.
  */
+@ResponseStatus(HttpStatus.BAD_REQUEST)
 public class InvalidTokenException extends ProjectUserException {
     public InvalidTokenException(String message) {
         super(message, ErrorCodes.INVALID_TOKEN);

api-project-users/src/main/java/org/opendevstack/apiservice/projectusers/service/impl/ProjectUserServiceImpl.java

@@ -1,7 +1,9 @@
 package org.opendevstack.apiservice.projectusers.service.impl;
 
+import org.opendevstack.apiservice.externalservice.aap.command.ExecuteWorkflowRequest;
 import org.opendevstack.apiservice.externalservice.aap.model.AutomationExecutionResult;
-import org.opendevstack.apiservice.externalservice.aap.service.AutomationPlatformService;
+import org.opendevstack.apiservice.externalservice.commons.command.ExternalServiceCommand;
+import org.opendevstack.apiservice.externalservice.commons.registry.ExternalServiceRegistry;
 import org.opendevstack.apiservice.projectusers.exception.AutomationPlatformException;
 import org.opendevstack.apiservice.projectusers.exception.ProjectNotFoundException;
 import org.opendevstack.apiservice.projectusers.model.AddUserToProjectRequest;
@@ -17,7 +19,7 @@
 
 /**
  * Implementation of ProjectUserService that manages project users and integrates with automation platform.
- * This is a stateless implementation that uses the automation platform for persistence.
+ * This is a stateless implementation that uses the automation platform for persistence via command pattern.
  */
 @Service("projectUserService")
 public class ProjectUserServiceImpl implements ProjectUserService {
@@ -27,12 +29,12 @@ public class ProjectUserServiceImpl implements ProjectUserService {
     @Value("${apis.project-users.ansible-workflow-name}")
     private String addUserWorkflow;
 
-    private final AutomationPlatformService automationPlatformService;
+    private final ExternalServiceRegistry registry;
     private final MembershipRequestTokenService tokenService;
 
-    public ProjectUserServiceImpl(AutomationPlatformService automationPlatformService,
+    public ProjectUserServiceImpl(ExternalServiceRegistry registry,
                                  MembershipRequestTokenService membershipRequestTokenService) {
-        this.automationPlatformService = automationPlatformService;
+        this.registry = registry;
         this.tokenService = membershipRequestTokenService;
     }
 
@@ -59,8 +61,23 @@ public MembershipRequestResponse addUserToProject(String projectKey, AddUserToPr
             parameters.put("comments", request.getComments());
             parameters.put("reference", uipathReference);
 
-            // Execute workflow on automation platform
-            AutomationExecutionResult result = automationPlatformService.executeWorkflow(addUserWorkflow, parameters);
+            // Execute workflow via command pattern
+            ExternalServiceCommand<ExecuteWorkflowRequest, AutomationExecutionResult> command = 
+                registry.getCommand("aap", "execute-workflow");
+
+            if (command == null) {
+                throw new AutomationPlatformException(
+                    "Execute workflow command not found for automation platform service", null);
+            }
+
+            // Create the proper request object
+            ExecuteWorkflowRequest workflowRequest = ExecuteWorkflowRequest.builder()
+                .workflowName(addUserWorkflow)
+                .parameters(parameters)
+                .async(false)
+                .build();
+
+            AutomationExecutionResult result = command.execute(workflowRequest);
 
             if (result.isSuccessful()) {               
                 // Create request token for status tracking
@@ -95,7 +112,7 @@ public MembershipRequestResponse addUserToProject(String projectKey, AddUserToPr
                     "Failed to add user through automation platform: " + result.getMessage(), null);
             }
 
-        } catch (org.opendevstack.apiservice.externalservice.aap.exception.AutomationPlatformException e) {
+        } catch (Exception e) {
             logger.error("Failed to add user '{}' to project '{}': {}", request.getUser(), projectKey, e.getMessage(), e);
             throw new AutomationPlatformException(
                 "Automation platform execution failed", e);