cleanup

agent/src/main/java/com/walmartlabs/concord/agent/cfg/GitConfiguration.java

@@ -23,8 +23,6 @@
 import com.typesafe.config.Config;
 
 import javax.inject.Inject;
-import javax.inject.Named;
-import javax.inject.Singleton;
 import java.time.Duration;
 
 import static com.walmartlabs.concord.agent.cfg.Utils.getStringOrDefault;

client/src/main/java/com/walmartlabs/concord/client/CreateSecretRequestV2.java

@@ -24,7 +24,6 @@
 import org.immutables.value.Value;
 
 import javax.annotation.Nullable;
-import java.nio.file.Path;
 import java.util.List;
 import java.util.UUID;
 

client2/src/main/java/com/walmartlabs/concord/client2/impl/HttpEntity.java

@@ -22,7 +22,6 @@
 
 import java.io.IOException;
 import java.io.InputStream;
-import java.io.OutputStream;
 
 public interface HttpEntity {
 

it/server/src/test/java/com/walmartlabs/concord/it/server/AnsibleRetryIT.java

@@ -25,7 +25,6 @@
 import com.walmartlabs.concord.client2.StartProcessResponse;
 import org.junit.jupiter.api.Test;
 
-import java.io.File;
 import java.io.InputStream;
 import java.net.URI;
 

it/server/src/test/java/com/walmartlabs/concord/it/server/ProjectTaskIT.java

@@ -20,7 +20,6 @@
  * =====
  */
 
-import com.walmartlabs.concord.client2.ProcessApi;
 import com.walmartlabs.concord.client2.ProcessEntry;
 import com.walmartlabs.concord.client2.StartProcessResponse;
 import com.walmartlabs.concord.common.IOUtils;

it/tasks/client1-test/src/main/java/com/walmartlabs/concord/it/tasks/suspendtest/Client1TestTask.java

@@ -20,7 +20,6 @@
  * =====
  */
 
-import com.walmartlabs.concord.ApiClient;
 import com.walmartlabs.concord.client.ApiClientConfiguration;
 import com.walmartlabs.concord.client.ApiClientFactory;
 import com.walmartlabs.concord.client.ProcessApi;

plugins/tasks/concord/src/main/java/com/walmartlabs/concord/client/AbstractConcordTask.java

@@ -20,7 +20,6 @@
  * =====
  */
 
-import com.fasterxml.jackson.databind.ObjectMapper;
 import com.walmartlabs.concord.client2.*;
 import com.walmartlabs.concord.sdk.ApiConfiguration;
 import com.walmartlabs.concord.sdk.Context;
@@ -36,8 +35,6 @@
 
 public abstract class AbstractConcordTask implements Task {
 
-    private final ObjectMapper objectMapper = new ObjectMapper();
-
     protected static final String API_KEY = "apiKey";
 
     @Inject
@@ -64,56 +61,6 @@ protected <T> T withClient(Context ctx, boolean withApiKey, CheckedFunction<ApiC
         return f.apply(apiClientFactory.create(cfg));
     }
 
-//    protected <T> T request(Context ctx, String uri, String method, Map<String, Object> input, Class<T> entityType) throws Exception {
-//        return withClient(ctx, client -> {
-//            RequestBody request = input != null ? ClientUtils.createMultipartBody(input).build() : null;
-//
-//            Request.Builder b = new Request.Builder()
-//                    .url(client.getBasePath() + uri)
-//                    .header("Accept", "*/*")
-//                    .method(method, request);
-//
-//            // we're going to use the "raw" OkHttpClient, so we need to set up the auth manually
-//            String apiKey = getApiKey(ctx);
-//            if (apiKey != null) {
-//                b.header("Authorization", apiKey);
-//            } else {
-//                b.header(Constants.Headers.SESSION_TOKEN, apiCfg.getSessionToken(ctx));
-//            }
-//
-//            OkHttpClient ok = client.getHttpClient();
-//            Response resp = ok.newCall(b.build()).execute();
-//            assertResponse(resp);
-//
-//            if (resp.code() == 204) { // HTTP "No Content"
-//                return null;
-//            }
-//
-//            try (ResponseBody body = resp.body()) {
-//                return objectMapper.readValue(body.byteStream(), entityType);
-//            }
-//        });
-//    }
-//
-//    protected void assertResponse(Response resp) throws IOException {
-//        int code = resp.code();
-//        if (code < 200 || code >= 400) {
-//            try (ResponseBody body = resp.body()) {
-//                if (isJson(resp)) {
-//                    Object details = objectMapper.readValue(body.byteStream(), Object.class);
-//                    String msg = extractMessage(details);
-//                    throw new IOException(msg);
-//                } else {
-//                    if (code == 401) {
-//                        throw new IOException("Request error: " + code + ", please verify the credentials used");
-//                    } else {
-//                        throw new IOException("Request error: " + code);
-//                    }
-//                }
-//            }
-//        }
-//    }
-
     private String getApiKey(Context ctx) {
         return (String) ctx.getVariable(API_KEY);
     }

plugins/tasks/concord/src/main/java/com/walmartlabs/concord/client/ConcordTaskCommon.java

@@ -31,7 +31,6 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
 import java.nio.file.Files;

plugins/tasks/files/src/main/java/com/walmartlabs/concord/plugins/file/v2/FilesTaskV2.java

@@ -28,7 +28,6 @@
 import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;
-import java.nio.file.Paths;
 import java.nio.file.StandardCopyOption;
 
 @Named("files")

plugins/tasks/log/src/main/java/com/walmartlabs/concord/plugins/log/LoggingTaskV2.java

@@ -25,7 +25,6 @@
 import com.walmartlabs.concord.runtime.v2.sdk.Variables;
 
 import javax.inject.Named;
-import java.io.Serializable;
 
 @Named("log")
 public class LoggingTaskV2 implements Task {

policy-engine/src/main/java/com/walmartlabs/concord/policyengine/RuntimePolicy.java

@@ -23,7 +23,6 @@
 import java.time.LocalTime;
 import java.time.OffsetDateTime;
 import java.time.ZoneOffset;
-import java.util.Objects;
 import java.util.function.Supplier;
 
 public class RuntimePolicy {

policy-engine/src/main/java/com/walmartlabs/concord/policyengine/RuntimeRule.java

@@ -30,7 +30,6 @@
 import javax.annotation.Nullable;
 import java.io.Serializable;
 import java.time.LocalDate;
-import java.time.OffsetDateTime;
 import java.util.Set;
 
 @Value.Immutable

repository/src/test/java/com/walmartlabs/concord/repository/GitClientFetchTest.java

@@ -25,12 +25,10 @@
 import com.walmartlabs.concord.sdk.Secret;
 import org.eclipse.jgit.api.Git;
 import org.eclipse.jgit.api.errors.GitAPIException;
-import org.eclipse.jgit.api.errors.InvalidRefNameException;
 import org.eclipse.jgit.revwalk.RevCommit;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
-import java.io.File;
 import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;

runtime/common/src/main/java/com/walmartlabs/concord/runtime/common/cfg/LoggingConfiguration.java

@@ -25,8 +25,6 @@
 import com.fasterxml.jackson.databind.annotation.JsonSerialize;
 import org.immutables.value.Value;
 
-import javax.annotation.Nullable;
-
 @Value.Immutable
 @Value.Style(jdkOnly = true)
 @JsonInclude(JsonInclude.Include.NON_EMPTY)

runtime/loader/src/main/java/com/walmartlabs/concord/process/loader/model/SourceMap.java

@@ -20,7 +20,6 @@
  * =====
  */
 
-import com.fasterxml.jackson.core.JsonLocation;
 import com.walmartlabs.concord.runtime.v2.model.Location;
 import org.immutables.serial.Serial;
 import org.immutables.value.Value;

runtime/v2/runner/src/main/java/com/walmartlabs/concord/runtime/v2/runner/StackTraceCollector.java

@@ -23,7 +23,6 @@
 import com.walmartlabs.concord.runtime.v2.model.FlowCall;
 import com.walmartlabs.concord.runtime.v2.model.Location;
 import com.walmartlabs.concord.runtime.v2.runner.vm.FlowCallCommand;
-import com.walmartlabs.concord.runtime.v2.runner.vm.VMUtils;
 import com.walmartlabs.concord.svm.Runtime;
 import com.walmartlabs.concord.svm.*;
 

runtime/v2/runner/src/main/java/com/walmartlabs/concord/runtime/v2/runner/vm/CopyVariablesCommand.java

@@ -32,7 +32,7 @@
  */
 public class CopyVariablesCommand implements Command {
 
-    public interface FrameProducer extends BiFunction<State, ThreadId, Frame>, Serializable {};
+    public interface FrameProducer extends BiFunction<State, ThreadId, Frame>, Serializable {}
 
     private static final long serialVersionUID = 1L;
 

runtime/v2/runner/src/test/java/com/walmartlabs/concord/runtime/v2/runner/tasks/TaskCallInterceptorTest.java

@@ -1,5 +1,25 @@
 package com.walmartlabs.concord.runtime.v2.runner.tasks;
 
+/*-
+ * *****
+ * Concord
+ * -----
+ * Copyright (C) 2017 - 2023 Walmart Inc.
+ * -----
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * =====
+ */
+
 import org.junit.Test;
 
 import java.util.Arrays;

server/impl/src/main/java/com/walmartlabs/concord/server/boot/filters/ConcordAuthenticationHandler.java

@@ -27,7 +27,6 @@
 import com.walmartlabs.concord.server.security.apikey.ApiKeyDao;
 import com.walmartlabs.concord.server.security.apikey.ApiKeyEntry;
 import com.walmartlabs.concord.server.security.sessionkey.SessionKey;
-import org.apache.shiro.authc.AuthenticationException;
 import org.apache.shiro.authc.AuthenticationToken;
 import org.apache.shiro.authc.UsernamePasswordToken;
 import org.apache.shiro.subject.support.DefaultSubjectContext;

server/impl/src/main/java/com/walmartlabs/concord/server/org/secret/SecretResource.java

@@ -38,7 +38,6 @@
 import io.swagger.v3.oas.annotations.headers.Header;
 import io.swagger.v3.oas.annotations.media.Content;
 import io.swagger.v3.oas.annotations.media.Schema;
-import io.swagger.v3.oas.annotations.parameters.RequestBody;
 import io.swagger.v3.oas.annotations.responses.ApiResponse;
 import io.swagger.v3.oas.annotations.tags.Tag;
 import org.jboss.resteasy.plugins.providers.multipart.MultipartInput;

server/impl/src/main/java/com/walmartlabs/concord/server/org/secret/SecretResourceUtils.java

@@ -33,7 +33,6 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.util.*;
-import java.util.stream.Collectors;
 
 public final class SecretResourceUtils {
 

server/impl/src/main/java/com/walmartlabs/concord/server/org/secret/store/concord/ConcordSecretStore.java

@@ -26,8 +26,6 @@
 import org.jooq.DSLContext;
 
 import javax.inject.Inject;
-import javax.inject.Named;
-import javax.inject.Singleton;
 import java.util.UUID;
 
 public class ConcordSecretStore implements SecretStore {

server/impl/src/main/java/com/walmartlabs/concord/server/process/checkpoint/ProcessCheckpointResource.java

@@ -33,9 +33,7 @@
 import com.walmartlabs.concord.server.sdk.metrics.WithTimer;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.Parameter;
-import io.swagger.v3.oas.annotations.media.Content;
 import io.swagger.v3.oas.annotations.media.Schema;
-import io.swagger.v3.oas.annotations.parameters.RequestBody;
 import io.swagger.v3.oas.annotations.tags.Tag;
 import org.jboss.resteasy.plugins.providers.multipart.MultipartInput;
 import org.slf4j.Logger;

server/impl/src/main/java/com/walmartlabs/concord/server/process/checkpoint/ProcessCheckpointV2Resource.java

@@ -33,8 +33,6 @@
 import org.sonatype.siesta.Resource;
 
 import javax.inject.Inject;
-import javax.inject.Named;
-import javax.inject.Singleton;
 import javax.ws.rs.*;
 import javax.ws.rs.core.MediaType;
 import java.util.UUID;

server/impl/src/main/java/com/walmartlabs/concord/server/process/event/ProcessEventResource.java

@@ -33,14 +33,11 @@
 import com.walmartlabs.concord.server.security.Roles;
 import com.walmartlabs.concord.server.security.UserPrincipal;
 import io.swagger.v3.oas.annotations.Operation;
-import io.swagger.v3.oas.annotations.Parameter;
 import io.swagger.v3.oas.annotations.tags.Tag;
 import org.apache.shiro.authz.UnauthorizedException;
 import org.sonatype.siesta.Resource;
 
 import javax.inject.Inject;
-import javax.inject.Named;
-import javax.inject.Singleton;
 import javax.ws.rs.*;
 import javax.ws.rs.core.MediaType;
 import javax.ws.rs.core.Response;

server/impl/src/main/java/com/walmartlabs/concord/server/process/form/FormResourceV1.java

@@ -28,16 +28,13 @@
 import com.walmartlabs.concord.server.sdk.ConcordApplicationException;
 import com.walmartlabs.concord.server.sdk.PartialProcessKey;
 import io.swagger.v3.oas.annotations.Operation;
-import io.swagger.v3.oas.annotations.Parameter;
 import io.swagger.v3.oas.annotations.tags.Tag;
 import io.takari.bpm.form.Form;
 import io.takari.bpm.model.form.FormDefinition;
 import io.takari.bpm.model.form.FormField;
 import org.jboss.resteasy.plugins.providers.multipart.MultipartInput;
 
 import javax.inject.Inject;
-import javax.inject.Named;
-import javax.inject.Singleton;
 import javax.ws.rs.*;
 import javax.ws.rs.core.MediaType;
 import javax.ws.rs.core.Response.Status;

server/impl/src/main/java/com/walmartlabs/concord/server/process/form/FormResourceV2.java

@@ -29,8 +29,6 @@
 import org.jboss.resteasy.plugins.providers.multipart.MultipartInput;
 
 import javax.inject.Inject;
-import javax.inject.Named;
-import javax.inject.Singleton;
 import javax.ws.rs.core.Response.Status;
 import java.io.Serializable;
 import java.util.*;

server/impl/src/main/java/com/walmartlabs/concord/server/process/locks/LockResult.java

@@ -22,7 +22,6 @@
 
 import com.fasterxml.jackson.databind.annotation.JsonDeserialize;
 import com.fasterxml.jackson.databind.annotation.JsonSerialize;
-import io.swagger.v3.oas.annotations.media.Schema;
 import org.immutables.value.Value;
 
 @Value.Immutable
@@ -32,7 +31,6 @@ public interface LockResult {
 
     LockEntry info();
 
-//    @Schema(description = "is lock acquired?", required = true)
     boolean acquired();
 
     static ImmutableLockResult.Builder builder() {

server/impl/src/main/java/com/walmartlabs/concord/server/process/locks/ProcessLocksDao.java

@@ -31,7 +31,6 @@
 import org.jooq.SelectConditionStep;
 
 import javax.inject.Inject;
-import javax.inject.Named;
 import java.util.UUID;
 
 import static com.walmartlabs.concord.server.jooq.Tables.PROCESS_LOCKS;

server/impl/src/main/java/com/walmartlabs/concord/server/process/locks/ProcessLocksResource.java

@@ -32,8 +32,6 @@
 import org.sonatype.siesta.Resource;
 
 import javax.inject.Inject;
-import javax.inject.Named;
-import javax.inject.Singleton;
 import javax.ws.rs.*;
 import javax.ws.rs.core.MediaType;
 import javax.ws.rs.core.Response;

server/impl/src/main/java/com/walmartlabs/concord/server/process/logs/ProcessLogsDao.java

@@ -31,7 +31,6 @@
 import org.jooq.*;
 
 import javax.inject.Inject;
-import javax.inject.Named;
 import java.io.Serializable;
 import java.time.OffsetDateTime;
 import java.util.List;

server/impl/src/main/java/com/walmartlabs/concord/server/process/waits/ProcessWaitManager.java

@@ -30,8 +30,6 @@
 import org.jooq.DSLContext;
 
 import javax.inject.Inject;
-import javax.inject.Named;
-import javax.inject.Singleton;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;

server/impl/src/main/java/com/walmartlabs/concord/server/process/waits/WaitProcessStatusListener.java

@@ -25,8 +25,6 @@
 import com.walmartlabs.concord.server.sdk.ProcessStatus;
 import org.jooq.DSLContext;
 
-import javax.inject.Named;
-
 import static com.walmartlabs.concord.server.jooq.Tables.PROCESS_WAIT_CONDITIONS;
 
 public class WaitProcessStatusListener implements ProcessStatusListener {

server/impl/src/main/java/com/walmartlabs/concord/server/security/ldap/UserLdapGroup.java

@@ -28,7 +28,6 @@
 import com.walmartlabs.concord.server.user.UserManager;
 import com.walmartlabs.concord.server.user.UserType;
 import io.swagger.v3.oas.annotations.Operation;
-import io.swagger.v3.oas.annotations.Parameter;
 import io.swagger.v3.oas.annotations.tags.Tag;
 import org.apache.shiro.authz.UnauthorizedException;
 import org.sonatype.siesta.Resource;