chore: fix compile issue and remove some unused code (#141)

Author: wolf4ood

core/negotiation-manager/src/main/java/org/eclipse/edc/virtual/controlplane/contract/negotiation/ContractManagerExtension.java

@@ -25,10 +25,8 @@
 import org.eclipse.edc.spi.message.RemoteMessageDispatcherRegistry;
 import org.eclipse.edc.spi.monitor.Monitor;
 import org.eclipse.edc.spi.system.ServiceExtension;
-import org.eclipse.edc.spi.system.ServiceExtensionContext;
 import org.eclipse.edc.transaction.spi.TransactionContext;
 import org.eclipse.edc.virtual.controlplane.contract.negotiation.task.ContractNegotiationTaskExecutorImpl;
-import org.eclipse.edc.virtual.controlplane.contract.spi.negotiation.ContractNegotiationStateMachineService;
 import org.eclipse.edc.virtual.controlplane.contract.spi.negotiation.ContractNegotiationTaskExecutor;
 import org.eclipse.edc.virtual.controlplane.participantcontext.spi.ParticipantWebhookResolver;
 import org.eclipse.edc.virtual.controlplane.tasks.TaskService;
@@ -76,11 +74,6 @@ public ConsumerContractNegotiationManager consumerContractNegotiationManager() {
         return new VirtualConsumerContractNegotiationManager(contractNegotiationStore, observable, monitor);
     }
 
-    @Provider
-    public ContractNegotiationStateMachineService contractNegotiationStateMachineService(ServiceExtensionContext context) {
-        return new ContractNegotiationStateMachineServiceImpl(clock, identityResolver, webhookResolver, dispatcherRegistry, contractNegotiationStore, transactionContext, pendingGuard, observable, monitor);
-    }
-
     @Provider
     public ContractNegotiationTaskExecutor contractNegotiationTaskExecutor() {
         return ContractNegotiationTaskExecutorImpl.Builder.newInstance()

core/negotiation-manager/src/main/java/org/eclipse/edc/virtual/controlplane/contract/negotiation/ContractNegotiationStateMachineServiceImpl.java

@@ -24,7 +24,7 @@
 import org.eclipse.edc.spi.monitor.Monitor;
 import org.eclipse.edc.spi.response.StatusResult;
 import org.eclipse.edc.spi.telemetry.Telemetry;
-import org.eclipse.edc.virtual.controlplane.contract.spi.negotiation.ContractNegotiationStateMachineService;
+import org.eclipse.edc.virtual.controlplane.contract.spi.negotiation.ContractNegotiationTaskExecutor;
 
 import java.util.UUID;
 
@@ -33,7 +33,7 @@
 /**
  * An implementation of the {@link ConsumerContractNegotiationManager} that only handles initial contract negotiation
  * requests by transitioning them to the initial state. The subsequent state transitions are handled by the
- * by {@link ContractNegotiationStateMachineService},
+ * by {@link ContractNegotiationTaskExecutor},
  * which can be invoked using different mechanisms, such as a loopback mechanism, a message bus.
  */
 public class VirtualConsumerContractNegotiationManager implements ConsumerContractNegotiationManager {

core/negotiation-manager/src/main/java/org/eclipse/edc/virtual/controlplane/contract/negotiation/VirtualConsumerContractNegotiationManager.java

@@ -29,7 +29,6 @@
 import org.eclipse.edc.spi.monitor.Monitor;
 import org.eclipse.edc.spi.security.Vault;
 import org.eclipse.edc.spi.system.ServiceExtension;
-import org.eclipse.edc.spi.system.ServiceExtensionContext;
 import org.eclipse.edc.transaction.spi.TransactionContext;
 import org.eclipse.edc.virtual.controlplane.participantcontext.spi.ParticipantWebhookResolver;
 import org.eclipse.edc.virtual.controlplane.tasks.TaskService;
@@ -38,7 +37,6 @@
 import org.eclipse.edc.virtual.controlplane.transfer.process.task.command.ResumeTransferCommandHandler;
 import org.eclipse.edc.virtual.controlplane.transfer.process.task.command.SuspendTransferCommandHandler;
 import org.eclipse.edc.virtual.controlplane.transfer.process.task.command.TerminateTransferCommandHandler;
-import org.eclipse.edc.virtual.controlplane.transfer.spi.TransferProcessStateMachineService;
 import org.eclipse.edc.virtual.controlplane.transfer.spi.TransferProcessTaskExecutor;
 
 import java.time.Clock;
@@ -106,23 +104,6 @@ public TransferProcessManager transferProcessManager() {
         return new VirtualTransferProcessManager(store, observable, policyArchive, clock, monitor);
     }
 
-    @Provider
-    public TransferProcessStateMachineService transferProcessStateMachineService(ServiceExtensionContext context) {
-        return TransferProcessStateMachineServiceImpl.Builder.newInstance()
-                .store(store)
-                .transactionContext(transactionContext)
-                .dataFlowController(dataFlowController)
-                .dispatcherRegistry(dispatcherRegistry)
-                .webhookResolver(webhookResolver)
-                .vault(vault)
-                .addressResolver(dataAddressResolver)
-                .monitor(monitor)
-                .pendingGuard(pendingGuard)
-                .observable(observable)
-                .policyArchive(policyArchive)
-                .build();
-    }
-
     @Provider
     public TransferProcessTaskExecutor transferProcessTaskExecutor() {
         return TransferProcessTaskExecutorImpl.Builder.newInstance()