From 54d6ce873e7ac731409188fb318e21b176b0a54b Mon Sep 17 00:00:00 2001
From: a0k04u1 <amith.k.b@walmart.com>
Date: Thu, 2 Jan 2025 23:20:31 +0530
Subject: [PATCH 1/8] add MavenRepositoryProvider

---
 .../concord/agent/RepositoryManager.java      |  8 +-
 .../cli/runner/CliRepositoryExporter.java     |  7 +-
 .../molecules/RepositoryForm/index.tsx        |  4 +-
 console2/src/validation.ts                    |  4 +-
 repository/pom.xml                            |  4 +
 .../repository/MavenRepositoryProvider.java   | 89 +++++++++++++++++++
 .../server/repository/RepositoryManager.java  |  6 +-
 7 files changed, 110 insertions(+), 12 deletions(-)
 create mode 100644 repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java

diff --git a/agent/src/main/java/com/walmartlabs/concord/agent/RepositoryManager.java b/agent/src/main/java/com/walmartlabs/concord/agent/RepositoryManager.java
index 43415a71bb..e500c41d2c 100644
--- a/agent/src/main/java/com/walmartlabs/concord/agent/RepositoryManager.java
+++ b/agent/src/main/java/com/walmartlabs/concord/agent/RepositoryManager.java
@@ -27,11 +27,12 @@
 import com.walmartlabs.concord.imports.Import.SecretDefinition;
 import com.walmartlabs.concord.repository.*;
 import com.walmartlabs.concord.sdk.Secret;
+import com.walmartlabs.concord.dependencymanager.DependencyManager;
 
 import javax.inject.Inject;
 import java.io.IOException;
 import java.nio.file.Path;
-import java.util.Collections;
+import java.util.Arrays;
 import java.util.List;
 
 public class RepositoryManager {
@@ -45,7 +46,8 @@ public class RepositoryManager {
     public RepositoryManager(SecretClient secretClient,
                              GitConfiguration gitCfg,
                              RepositoryCacheConfiguration cacheCfg,
-                             ObjectMapper objectMapper) throws IOException {
+                             ObjectMapper objectMapper,
+                             DependencyManager dependencyManager) throws IOException {
 
         this.secretClient = secretClient;
         this.gitCfg = gitCfg;
@@ -60,7 +62,7 @@ public RepositoryManager(SecretClient secretClient,
                 .sshTimeoutRetryCount(gitCfg.getSshTimeoutRetryCount())
                 .build();
 
-        List<RepositoryProvider> providers = Collections.singletonList(new GitCliRepositoryProvider(clientCfg));
+        List<RepositoryProvider> providers = Arrays.asList(new MavenRepositoryProvider(dependencyManager), new GitCliRepositoryProvider(clientCfg));
         this.providers = new RepositoryProviders(providers);
 
         this.repositoryCache = new RepositoryCache(cacheCfg.getCacheDir(),
diff --git a/cli/src/main/java/com/walmartlabs/concord/cli/runner/CliRepositoryExporter.java b/cli/src/main/java/com/walmartlabs/concord/cli/runner/CliRepositoryExporter.java
index 7237f1e904..5790a647fa 100644
--- a/cli/src/main/java/com/walmartlabs/concord/cli/runner/CliRepositoryExporter.java
+++ b/cli/src/main/java/com/walmartlabs/concord/cli/runner/CliRepositoryExporter.java
@@ -29,8 +29,9 @@
 import java.net.URLEncoder;
 import java.nio.file.Path;
 import java.time.Duration;
-import java.util.Collections;
+import java.util.Arrays;
 import java.util.Objects;
+import com.walmartlabs.concord.dependencymanager.DependencyManager;
 
 public class CliRepositoryExporter implements RepositoryExporter {
 
@@ -46,7 +47,7 @@ public class CliRepositoryExporter implements RepositoryExporter {
 
     private final RepositoryProviders providers;
 
-    public CliRepositoryExporter(Path repoCacheDir) {
+    public CliRepositoryExporter(Path repoCacheDir, DependencyManager dependencyManager) {
         this.repoCacheDir = repoCacheDir;
 
         GitClientConfiguration clientCfg = GitClientConfiguration.builder()
@@ -59,7 +60,7 @@ public CliRepositoryExporter(Path repoCacheDir) {
                 .sshTimeoutRetryCount(SSH_TIMEOUT_RETRY_COUNT)
                 .build();
 
-        this.providers = new RepositoryProviders(Collections.singletonList(new GitCliRepositoryProvider(clientCfg)));
+        this.providers = new RepositoryProviders(Arrays.asList(new MavenRepositoryProvider(dependencyManager), new GitCliRepositoryProvider(clientCfg)));
     }
 
     @Override
diff --git a/console2/src/components/molecules/RepositoryForm/index.tsx b/console2/src/components/molecules/RepositoryForm/index.tsx
index aa1e7d5550..b23b667b03 100644
--- a/console2/src/components/molecules/RepositoryForm/index.tsx
+++ b/console2/src/components/molecules/RepositoryForm/index.tsx
@@ -71,7 +71,7 @@ interface State {
 
 const sourceOptions = [
     {
-        text: 'Branch/tag',
+        text: 'Branch/tag/version',
         value: RepositorySourceType.BRANCH_OR_TAG
     },
     {
@@ -224,7 +224,7 @@ class RepositoryForm extends React.Component<InjectedFormikProps<Props, FormValu
                         {values.sourceType === RepositorySourceType.BRANCH_OR_TAG && (
                             <FormikInput
                                 name="branch"
-                                label="Branch/Tag"
+                                label="Branch/Tag/Version"
                                 fluid={true}
                                 required={true}
                             />
diff --git a/console2/src/validation.ts b/console2/src/validation.ts
index c9b1b86099..274a92083d 100644
--- a/console2/src/validation.ts
+++ b/console2/src/validation.ts
@@ -30,7 +30,7 @@ export const REPOSITORY_SSH_URL_PATTERN = /^(ssh:\/\/)?([a-zA-Z0-9\-_.]+)@([^:]+
 const requiredError = () => 'Required';
 const tooLongError = (n: number) => `Must be not more than ${n} characters.`;
 const invalidRepositoryUrlError = () =>
-    "Invalid repository URL: must begin with 'https://', 'ssh://' or use 'user@host:path' scheme.";
+    "Invalid repository URL: must begin with 'https://', 'mvn://', 'ssh://' or use 'user@host:path' scheme.";
 const invalidCommitIdError = () => 'Invalid commit ID: must be a valid revision.';
 const concordKeyPatternError = () =>
     "Must start with a digit or a letter, may contain only digits, letters, underscores, hyphens, tildes, '.' or '@' or. Must be between 3 and 128 characters in length.";
@@ -70,7 +70,7 @@ const repositoryUrlValidator = (v?: string) => {
         return requiredError();
     }
 
-    if (!v.startsWith('https://') && !v.match(REPOSITORY_SSH_URL_PATTERN)) {
+    if (!v.startsWith('https://') && !v.match(REPOSITORY_SSH_URL_PATTERN) && !v.startsWith('mvn://')) {
         return invalidRepositoryUrlError();
     }
 
diff --git a/repository/pom.xml b/repository/pom.xml
index 400679aaaa..fb88721251 100644
--- a/repository/pom.xml
+++ b/repository/pom.xml
@@ -26,6 +26,10 @@
             <groupId>com.google.guava</groupId>
             <artifactId>guava</artifactId>
         </dependency>
+        <dependency>
+            <groupId>com.walmartlabs.concord</groupId>
+            <artifactId>concord-dependency-manager</artifactId>
+        </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-api</artifactId>
diff --git a/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java b/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
new file mode 100644
index 0000000000..e77e50fdbd
--- /dev/null
+++ b/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
@@ -0,0 +1,89 @@
+package com.walmartlabs.concord.repository;
+
+import com.walmartlabs.concord.common.IOUtils;
+import com.walmartlabs.concord.dependencymanager.DependencyManager;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import java.io.IOException;
+import java.net.URI;
+import java.net.URISyntaxException;
+import java.nio.file.Path;
+import java.nio.file.StandardCopyOption;
+import java.util.List;
+
+/**
+ * *****
+ * Concord
+ * -----
+ * Copyright (C)  - 2025 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
+ * <p>
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * <p>
+ * 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.
+ * =====
+ */
+public class MavenRepositoryProvider implements RepositoryProvider {
+
+    private static final String URL_PREFIX = "mvn://";
+    private static final Logger log = LoggerFactory.getLogger(MavenRepositoryProvider.class);
+    private final DependencyManager dependencyManager;
+
+    public MavenRepositoryProvider(DependencyManager dependencyManager) {
+        this.dependencyManager = dependencyManager;
+    }
+
+    /**
+     * @param url maven repo url in format mvn://groupId:artifactId
+     * @return boolean can handle or not
+     */
+    @Override
+    public boolean canHandle(String url) {
+        return url.startsWith(URL_PREFIX);
+    }
+
+    /**
+     * @param request fetchRequest
+     * @return fetchResult
+     */
+    @Override
+    public FetchResult fetch(FetchRequest request) {
+        Path dst = request.destination();
+        try {
+            URI uri = new URI(request.url().concat(":").concat(request.version().value()));
+            Path dependencyPath = dependencyManager.resolveSingle(uri).getPath();
+            IOUtils.copy(dependencyPath, dst, StandardCopyOption.REPLACE_EXISTING);
+            return null;
+        } catch (URISyntaxException | IOException e) {
+            try {
+                IOUtils.deleteRecursively(request.destination());
+            } catch (IOException ee) {
+                log.warn("fetch ['{}', '{}', '{}'] -> cleanup error: {}",
+                        request.url(), request.version(), request.destination(), e.getMessage());
+            }
+            throw new RepositoryException("Error while fetching a repository", e);
+        }
+    }
+
+    /**
+     * @param src source of the fetched repo
+     * @param dst destination to be copied to
+     * @param ignorePatterns ignore some files while copying
+     * @return  snapshot of copied files
+     * @throws IOException exception occurred during copy
+     */
+    @Override
+    public Snapshot export(Path src, Path dst, List<String> ignorePatterns) throws IOException {
+        LastModifiedSnapshot snapshot = new LastModifiedSnapshot();
+        IOUtils.unzip(src, dst, false, snapshot, StandardCopyOption.REPLACE_EXISTING);
+        return snapshot;
+    }
+}
diff --git a/server/impl/src/main/java/com/walmartlabs/concord/server/repository/RepositoryManager.java b/server/impl/src/main/java/com/walmartlabs/concord/server/repository/RepositoryManager.java
index cbff486af0..0efc396551 100644
--- a/server/impl/src/main/java/com/walmartlabs/concord/server/repository/RepositoryManager.java
+++ b/server/impl/src/main/java/com/walmartlabs/concord/server/repository/RepositoryManager.java
@@ -22,6 +22,7 @@
 
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.walmartlabs.concord.common.IOUtils;
+import com.walmartlabs.concord.dependencymanager.DependencyManager;
 import com.walmartlabs.concord.process.loader.ProjectLoader;
 import com.walmartlabs.concord.repository.*;
 import com.walmartlabs.concord.sdk.Secret;
@@ -60,7 +61,8 @@ public RepositoryManager(ObjectMapper objectMapper,
                              GitConfiguration gitCfg,
                              RepositoryConfiguration repoCfg,
                              ProjectDao projectDao,
-                             SecretManager secretManager) throws IOException {
+                             SecretManager secretManager,
+                             DependencyManager dependencyManager) throws IOException {
 
         GitClientConfiguration gitCliCfg = GitClientConfiguration.builder()
                 .oauthToken(gitCfg.getOauthToken())
@@ -72,7 +74,7 @@ public RepositoryManager(ObjectMapper objectMapper,
                 .sshTimeoutRetryCount(gitCfg.getSshTimeoutRetryCount())
                 .build();
 
-        List<RepositoryProvider> providers = Arrays.asList(new ClasspathRepositoryProvider(), new GitCliRepositoryProvider(gitCliCfg));
+        List<RepositoryProvider> providers = Arrays.asList(new ClasspathRepositoryProvider(), new MavenRepositoryProvider(dependencyManager), new GitCliRepositoryProvider(gitCliCfg));
 
         this.gitCfg = gitCfg;
         this.providers = new RepositoryProviders(providers);

From e8148c529d8451907ec4d83bd700fb47a0ca52ae Mon Sep 17 00:00:00 2001
From: a0k04u1 <amith.k.b@walmart.com>
Date: Thu, 2 Jan 2025 23:37:20 +0530
Subject: [PATCH 2/8] revert cliRepoProvider

---
 .../concord/cli/runner/CliRepositoryExporter.java         | 7 +++----
 .../concord/repository/MavenRepositoryProvider.java       | 8 ++++----
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/cli/src/main/java/com/walmartlabs/concord/cli/runner/CliRepositoryExporter.java b/cli/src/main/java/com/walmartlabs/concord/cli/runner/CliRepositoryExporter.java
index 5790a647fa..7237f1e904 100644
--- a/cli/src/main/java/com/walmartlabs/concord/cli/runner/CliRepositoryExporter.java
+++ b/cli/src/main/java/com/walmartlabs/concord/cli/runner/CliRepositoryExporter.java
@@ -29,9 +29,8 @@
 import java.net.URLEncoder;
 import java.nio.file.Path;
 import java.time.Duration;
-import java.util.Arrays;
+import java.util.Collections;
 import java.util.Objects;
-import com.walmartlabs.concord.dependencymanager.DependencyManager;
 
 public class CliRepositoryExporter implements RepositoryExporter {
 
@@ -47,7 +46,7 @@ public class CliRepositoryExporter implements RepositoryExporter {
 
     private final RepositoryProviders providers;
 
-    public CliRepositoryExporter(Path repoCacheDir, DependencyManager dependencyManager) {
+    public CliRepositoryExporter(Path repoCacheDir) {
         this.repoCacheDir = repoCacheDir;
 
         GitClientConfiguration clientCfg = GitClientConfiguration.builder()
@@ -60,7 +59,7 @@ public CliRepositoryExporter(Path repoCacheDir, DependencyManager dependencyMana
                 .sshTimeoutRetryCount(SSH_TIMEOUT_RETRY_COUNT)
                 .build();
 
-        this.providers = new RepositoryProviders(Arrays.asList(new MavenRepositoryProvider(dependencyManager), new GitCliRepositoryProvider(clientCfg)));
+        this.providers = new RepositoryProviders(Collections.singletonList(new GitCliRepositoryProvider(clientCfg)));
     }
 
     @Override
diff --git a/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java b/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
index e77e50fdbd..acf34f586b 100644
--- a/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
+++ b/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
@@ -16,14 +16,14 @@
  * *****
  * Concord
  * -----
- * Copyright (C)  - 2025 Walmart Inc.
+ * Copyright (C) 2025 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
- * <p>
- * http://www.apache.org/licenses/LICENSE-2.0
- * <p>
+ * 
+ *      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.

From 975ec6ea6d39ca0dc28adb800df5cb1ad1a1d121 Mon Sep 17 00:00:00 2001
From: a0k04u1 <amith.k.b@walmart.com>
Date: Fri, 3 Jan 2025 03:40:51 +0530
Subject: [PATCH 3/8] support up

---
 .../com/walmartlabs/concord/agent/JobRequest.java     |  1 +
 .../java/com/walmartlabs/concord/agent/Worker.java    |  2 +-
 .../src/components/molecules/RepositoryForm/index.tsx |  4 ++--
 .../concord/repository/MavenRepositoryProvider.java   | 11 +++++++----
 .../processors/RepositoryInfoUpdateProcessor.java     |  4 ++--
 5 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/agent/src/main/java/com/walmartlabs/concord/agent/JobRequest.java b/agent/src/main/java/com/walmartlabs/concord/agent/JobRequest.java
index ef5deebd3e..ebc5d38dd4 100644
--- a/agent/src/main/java/com/walmartlabs/concord/agent/JobRequest.java
+++ b/agent/src/main/java/com/walmartlabs/concord/agent/JobRequest.java
@@ -147,6 +147,7 @@ public String toString() {
                 ", repoUrl='" + repoUrl + '\'' +
                 ", repoPath='" + repoPath + '\'' +
                 ", commitId='" + commitId + '\'' +
+                ", repoBranch='" + repoBranch + '\'' +
                 ", secretName='" + secretName + '\'' +
                 ", imports='" + imports + '\'' +
                 '}';
diff --git a/agent/src/main/java/com/walmartlabs/concord/agent/Worker.java b/agent/src/main/java/com/walmartlabs/concord/agent/Worker.java
index c5218430fc..9f0e375c2f 100644
--- a/agent/src/main/java/com/walmartlabs/concord/agent/Worker.java
+++ b/agent/src/main/java/com/walmartlabs/concord/agent/Worker.java
@@ -142,7 +142,7 @@ private void onStatusChange(UUID instanceId, StatusEnum status) {
     }
 
     private void fetchRepo(JobRequest r) throws Exception {
-        if (r.getRepoUrl() == null || r.getCommitId() == null) {
+        if (r.getRepoUrl() == null || (r.getCommitId() == null && r.getRepoBranch() == null)) {
             return;
         }
 
diff --git a/console2/src/components/molecules/RepositoryForm/index.tsx b/console2/src/components/molecules/RepositoryForm/index.tsx
index b23b667b03..d4b4fe5b19 100644
--- a/console2/src/components/molecules/RepositoryForm/index.tsx
+++ b/console2/src/components/molecules/RepositoryForm/index.tsx
@@ -348,10 +348,10 @@ const validator = async (values: FormValues, props: Props) => {
     }
 
     if (!values.withSecret) {
-        if (!values.url.startsWith('https://')) {
+        if (!values.url.startsWith('https://') && !values.url.startsWith('mvn://')) {
             return Promise.resolve({
                 url:
-                    "Invalid repository URL: must begin with 'https://'. SSH repository URLs require additional credentials to be specified."
+                    "Invalid repository URL: must begin with 'https://' or 'mvn://'. SSH repository URLs require additional credentials to be specified."
             });
         }
     } else {
diff --git a/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java b/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
index acf34f586b..e09205b877 100644
--- a/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
+++ b/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
@@ -10,6 +10,7 @@
 import java.net.URISyntaxException;
 import java.nio.file.Path;
 import java.nio.file.StandardCopyOption;
+import java.util.ArrayList;
 import java.util.List;
 
 /**
@@ -21,9 +22,9 @@
  * 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.
@@ -60,7 +61,7 @@ public FetchResult fetch(FetchRequest request) {
         try {
             URI uri = new URI(request.url().concat(":").concat(request.version().value()));
             Path dependencyPath = dependencyManager.resolveSingle(uri).getPath();
-            IOUtils.copy(dependencyPath, dst, StandardCopyOption.REPLACE_EXISTING);
+            IOUtils.unzip(dependencyPath, dst, false, StandardCopyOption.REPLACE_EXISTING);
             return null;
         } catch (URISyntaxException | IOException e) {
             try {
@@ -83,7 +84,9 @@ public FetchResult fetch(FetchRequest request) {
     @Override
     public Snapshot export(Path src, Path dst, List<String> ignorePatterns) throws IOException {
         LastModifiedSnapshot snapshot = new LastModifiedSnapshot();
-        IOUtils.unzip(src, dst, false, snapshot, StandardCopyOption.REPLACE_EXISTING);
+        List<String> allIgnorePatterns = new ArrayList<>();
+        allIgnorePatterns.addAll(ignorePatterns);
+        IOUtils.copy(src, dst, allIgnorePatterns, snapshot, StandardCopyOption.REPLACE_EXISTING);
         return snapshot;
     }
 }
diff --git a/server/impl/src/main/java/com/walmartlabs/concord/server/process/pipelines/processors/RepositoryInfoUpdateProcessor.java b/server/impl/src/main/java/com/walmartlabs/concord/server/process/pipelines/processors/RepositoryInfoUpdateProcessor.java
index 1e8705bbaf..5c2280e750 100644
--- a/server/impl/src/main/java/com/walmartlabs/concord/server/process/pipelines/processors/RepositoryInfoUpdateProcessor.java
+++ b/server/impl/src/main/java/com/walmartlabs/concord/server/process/pipelines/processors/RepositoryInfoUpdateProcessor.java
@@ -48,8 +48,8 @@ public Payload process(Chain chain, Payload payload) {
             return chain.process(payload);
         }
 
-        String commitId = null;
-        String commitBranch = null;
+        String commitId = i.getCommitId();
+        String commitBranch = i.getBranch();
 
         RepositoryProcessor.CommitInfo ci = i.getCommitInfo();
         if (ci != null) {

From 6c9c2d5c45c186d205e2f9f87eb74b40d4432f82 Mon Sep 17 00:00:00 2001
From: a0k04u1 <amith.k.b@walmart.com>
Date: Fri, 3 Jan 2025 23:49:16 +0530
Subject: [PATCH 4/8] fix UI repo list

---
 .../components/molecules/GitHubLink/index.tsx |  5 +++--
 .../molecules/RepositoryList/index.tsx        | 20 +++++++++++++++----
 2 files changed, 19 insertions(+), 6 deletions(-)

diff --git a/console2/src/components/molecules/GitHubLink/index.tsx b/console2/src/components/molecules/GitHubLink/index.tsx
index 10a8eb530e..1a017ae097 100644
--- a/console2/src/components/molecules/GitHubLink/index.tsx
+++ b/console2/src/components/molecules/GitHubLink/index.tsx
@@ -23,6 +23,7 @@ import { REPOSITORY_SSH_URL_PATTERN } from '../../../validation';
 
 interface Props {
     url: string;
+    link?: string;
     path?: string;
     commitId?: string;
     text?: string;
@@ -59,9 +60,9 @@ const normalizePath = (s: string): string => {
 
 class GitHubLink extends React.PureComponent<Props> {
     render() {
-        const { url, commitId, path, text, branch } = this.props;
+        const { url, link, commitId, path, text, branch } = this.props;
 
-        let s = gitUrlParse(url);
+        let s = !link ? gitUrlParse(url) : link;
         if (!s) {
             return url;
         }
diff --git a/console2/src/components/molecules/RepositoryList/index.tsx b/console2/src/components/molecules/RepositoryList/index.tsx
index c550a0637a..34bbfe166d 100644
--- a/console2/src/components/molecules/RepositoryList/index.tsx
+++ b/console2/src/components/molecules/RepositoryList/index.tsx
@@ -26,6 +26,7 @@ import { ConcordKey } from '../../../api/common';
 import { RepositoryEntry } from '../../../api/org/project/repository';
 import { GitHubLink } from '../../molecules';
 import { RepositoryActionDropdown } from '../../organisms';
+import { gitUrlParse } from "../GitHubLink";
 
 interface ExternalProps {
     orgName: ConcordKey;
@@ -44,7 +45,7 @@ const RepositoryList = ({ orgName, projectName, data, loading, refresh }: Extern
                         <Table.HeaderCell collapsing={true} />
                         <Table.HeaderCell collapsing={true}>Name</Table.HeaderCell>
                         <Table.HeaderCell>Repository URL</Table.HeaderCell>
-                        <Table.HeaderCell collapsing={true}>Branch/Commit ID</Table.HeaderCell>
+                        <Table.HeaderCell collapsing={true}>Branch/Commit ID/Version</Table.HeaderCell>
                         <Table.HeaderCell singleLine={true}>Path</Table.HeaderCell>
                         <Table.HeaderCell collapsing={true} style={{ width: '8%' }}>
                             Secret
@@ -72,9 +73,14 @@ const RepositoryList = ({ orgName, projectName, data, loading, refresh }: Extern
 };
 
 const renderRepoPath = (r: RepositoryEntry) => {
+    const urlLink = gitUrlParse(r.url);
+    if (!urlLink) {
+        return r.path;
+    }
     if (r.commitId) {
         return (
             <GitHubLink
+                link={urlLink}
                 url={r.url!}
                 commitId={r.commitId}
                 path={r.path || '/'}
@@ -82,14 +88,20 @@ const renderRepoPath = (r: RepositoryEntry) => {
             />
         );
     }
-    return <GitHubLink url={r.url!} branch={r.branch} path={r.path || '/'} text={r.path || '/'} />;
+    return <GitHubLink url={r.url!} link={urlLink} branch={r.branch} path={r.path || '/'} text={r.path || '/'} />;
 };
 
 const renderRepoCommitIdOrBranch = (r: RepositoryEntry) => {
+
+    const urlLink = gitUrlParse(r.url);
+    if (!urlLink) {
+        return r.branch;
+    }
+
     if (r.commitId) {
-        return <GitHubLink url={r.url!} commitId={r.commitId} text={r.commitId} />;
+        return <GitHubLink url={r.url!} link={urlLink} commitId={r.commitId} text={r.commitId}/>;
     }
-    return <GitHubLink url={r.url!} branch={r.branch} text={r.branch} />;
+    return <GitHubLink url={r.url!} link={urlLink} branch={r.branch} text={r.branch}/>;
 };
 
 const renderTableRow = (

From 11b7ade59ec7521a897d3f09ced8bddf3ebb7b02 Mon Sep 17 00:00:00 2001
From: a0k04u1 <amith.k.b@walmart.com>
Date: Sat, 4 Jan 2025 21:01:30 +0530
Subject: [PATCH 5/8] fix classpath repo export

---
 agent/src/main/java/com/walmartlabs/concord/agent/Worker.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/agent/src/main/java/com/walmartlabs/concord/agent/Worker.java b/agent/src/main/java/com/walmartlabs/concord/agent/Worker.java
index 9f0e375c2f..be3f6f5ac2 100644
--- a/agent/src/main/java/com/walmartlabs/concord/agent/Worker.java
+++ b/agent/src/main/java/com/walmartlabs/concord/agent/Worker.java
@@ -142,7 +142,9 @@ private void onStatusChange(UUID instanceId, StatusEnum status) {
     }
 
     private void fetchRepo(JobRequest r) throws Exception {
-        if (r.getRepoUrl() == null || (r.getCommitId() == null && r.getRepoBranch() == null)) {
+        if (r.getRepoUrl() == null
+                || (r.getCommitId() == null && r.getRepoBranch() == null)
+                || r.getRepoUrl().startsWith("classpath://")) {
             return;
         }
 

From 11bef188866fb47ee604500cf111c168c81468ec Mon Sep 17 00:00:00 2001
From: a0k04u1 <amith.k.b@walmart.com>
Date: Sat, 4 Jan 2025 23:05:33 +0530
Subject: [PATCH 6/8] up tests

---
 .../concord/it/server/MavenRepoIT.java        |  76 ++++++++++++++++++
 .../server/mvnRepoFiles/mvn-concord-0.0.1.zip | Bin 0 -> 204 bytes
 2 files changed, 76 insertions(+)
 create mode 100644 it/server/src/test/java/com/walmartlabs/concord/it/server/MavenRepoIT.java
 create mode 100644 it/server/src/test/resources/com/walmartlabs/concord/it/server/mvnRepoFiles/mvn-concord-0.0.1.zip

diff --git a/it/server/src/test/java/com/walmartlabs/concord/it/server/MavenRepoIT.java b/it/server/src/test/java/com/walmartlabs/concord/it/server/MavenRepoIT.java
new file mode 100644
index 0000000000..1664f75472
--- /dev/null
+++ b/it/server/src/test/java/com/walmartlabs/concord/it/server/MavenRepoIT.java
@@ -0,0 +1,76 @@
+package com.walmartlabs.concord.it.server;
+
+/*-
+ * *****
+ * Concord
+ * -----
+ * Copyright (C) 2017 - 2018 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 com.walmartlabs.concord.client2.*;
+import com.walmartlabs.concord.common.IOUtils;
+import org.junit.jupiter.api.Test;
+
+import java.nio.file.Path;
+import java.nio.file.Paths;
+import java.nio.file.StandardCopyOption;
+import java.util.Collections;
+
+import static com.walmartlabs.concord.it.common.ServerClient.assertLog;
+import static com.walmartlabs.concord.it.common.ServerClient.waitForCompletion;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+public class MavenRepoIT extends AbstractServerIT {
+
+    @Test
+    public void test() throws Exception {
+        // prepare local mvn repo
+        Path mvnDirectory = Path.of(System.getProperty("user.home"))
+                .resolve(".m2/repository/com/walmartlabs/concord/mvn-concord/0.0.1");
+
+        Path src = Paths.get(MavenRepoIT.class.getResource("mvnRepoFiles").toURI());
+        IOUtils.copy(src, mvnDirectory, StandardCopyOption.REPLACE_EXISTING);
+
+        String url = "mvn://com.walmartlabs.concord:mvn-concord:zip";
+        String projectName = "project_" + randomString();
+        String repoName = "repo_" + randomString();
+
+        ProjectsApi projectsApi = new ProjectsApi(getApiClient());
+        projectsApi.createOrUpdateProject("Default", new ProjectEntry()
+                .name(projectName)
+                .acceptsRawPayload(false)
+                .repositories(Collections.singletonMap(repoName, new RepositoryEntry()
+                        .name(repoName)
+                        .url(url)
+                        .branch("0.0.1"))));
+
+        // ---
+
+        StartProcessResponse spr = start("Default", projectName, repoName, null, null);
+        assertTrue(spr.getOk());
+
+        // ---
+
+        ProcessEntry pe = waitForCompletion(getApiClient(), spr.getInstanceId());
+        assertEquals(ProcessEntry.StatusEnum.FINISHED, pe.getStatus());
+
+        // ---
+
+        byte[] ab = getLog(pe.getInstanceId());
+        assertLog(".*OK.*", ab);
+    }
+}
diff --git a/it/server/src/test/resources/com/walmartlabs/concord/it/server/mvnRepoFiles/mvn-concord-0.0.1.zip b/it/server/src/test/resources/com/walmartlabs/concord/it/server/mvnRepoFiles/mvn-concord-0.0.1.zip
new file mode 100644
index 0000000000000000000000000000000000000000..c5a74a77ac041e9e481985bf8d9fb7c067ff45c9
GIT binary patch
literal 204
zcmWIWW@Zs#U|`^2aM_>|Wq&QyNCC)G1Y&Lm8HVKiyyX0%6urvaoX`+X2Ih{W%Jfbk
zF0J5ZU}X8u$iM*B?0x#AzW%mHK>;3~zTP@#H5m;>MM^}AK3!fbGUd<SsSE+$j7)OO
pxU7)?TE@TtG>c(LBZ!4!DJ#TMGz$Z~S=m5J8G$emNT-503;>NdEHwZC

literal 0
HcmV?d00001


From ab33af3e80f1ce0d2af578e4287dfddcad9ff12b Mon Sep 17 00:00:00 2001
From: a0k04u1 <amith.k.b@walmart.com>
Date: Sat, 4 Jan 2025 23:18:50 +0530
Subject: [PATCH 7/8] up

---
 .../concord/repository/MavenRepositoryProvider.java           | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java b/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
index e09205b877..7e0ae5fa44 100644
--- a/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
+++ b/repository/src/main/java/com/walmartlabs/concord/repository/MavenRepositoryProvider.java
@@ -43,7 +43,7 @@ public MavenRepositoryProvider(DependencyManager dependencyManager) {
     }
 
     /**
-     * @param url maven repo url in format mvn://groupId:artifactId
+     * @param url maven repo url in format mvn://groupId:artifactId:extension
      * @return boolean can handle or not
      */
     @Override
@@ -79,7 +79,7 @@ public FetchResult fetch(FetchRequest request) {
      * @param dst destination to be copied to
      * @param ignorePatterns ignore some files while copying
      * @return  snapshot of copied files
-     * @throws IOException exception occurred during copy
+     * @throws IOException exception during IO operation
      */
     @Override
     public Snapshot export(Path src, Path dst, List<String> ignorePatterns) throws IOException {

From 92ed82936951b60016d964debf509e8dcae232d8 Mon Sep 17 00:00:00 2001
From: a0k04u1 <amith.k.b@walmart.com>
Date: Sun, 5 Jan 2025 11:52:18 +0530
Subject: [PATCH 8/8] fix ITs

---
 it/server/pom.xml                                        | 7 ++++++-
 .../com/walmartlabs/concord/it/server/MavenRepoIT.java   | 9 +++++++--
 2 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/it/server/pom.xml b/it/server/pom.xml
index a45cefdf03..40cff907d7 100644
--- a/it/server/pom.xml
+++ b/it/server/pom.xml
@@ -34,7 +34,7 @@
         <oldap.image>osixia/openldap</oldap.image>
         <server.image>walmartlabs/concord-server</server.image>
         <skip.socat>true</skip.socat>
-        <socat.image>bobrik/socat</socat.image>
+        <socat.image>alpine/socat</socat.image>
         <tmp.dir.src.mount>${java.io.tmpdir}</tmp.dir.src.mount>
         <tmp.dir>${java.io.tmpdir}</tmp.dir>
     </properties>
@@ -250,6 +250,7 @@
                     <systemProperties>
                         <java.io.tmpdir>${tmp.dir}</java.io.tmpdir>
                         <isDocker>${is.docker.profile}</isDocker>
+                        <local.mvn.repo>${local.repository.src.mount}</local.mvn.repo>
                     </systemProperties>
                 </configuration>
             </plugin>
@@ -529,6 +530,9 @@
                                                 <volume>${base.dir}/src/test/resources/server.conf:/opt/concord/conf/server.conf:ro</volume>
                                                 <!-- provide the default process variables cfg -->
                                                 <volume>${base.dir}/src/test/resources/default_vars.yml:/opt/concord/conf/default_vars.yml:ro</volume>
+                                                <!-- share host artifacts -->
+                                                <volume>${local.repository.src.mount}:/host/.m2/repository:ro</volume>
+                                                <volume>${base.dir}/src/test/resources/mvn.json:/opt/concord/conf/mvn.json:ro</volume>
                                             </bind>
                                         </volumes>
                                         <env>
@@ -543,6 +547,7 @@
                                             <NODEROSTER_DB_PASSWORD>it</NODEROSTER_DB_PASSWORD>
                                             <NODEROSTER_DB_URL>jdbc:postgresql://${it.db.addr}/postgres</NODEROSTER_DB_URL>
                                             <NODEROSTER_DB_USERNAME>postgres</NODEROSTER_DB_USERNAME>
+                                            <CONCORD_MAVEN_CFG>/opt/concord/conf/mvn.json</CONCORD_MAVEN_CFG>
                                         </env>
                                         <wait>
                                             <!--suppress MavenModelInspection -->
diff --git a/it/server/src/test/java/com/walmartlabs/concord/it/server/MavenRepoIT.java b/it/server/src/test/java/com/walmartlabs/concord/it/server/MavenRepoIT.java
index 1664f75472..229261fad6 100644
--- a/it/server/src/test/java/com/walmartlabs/concord/it/server/MavenRepoIT.java
+++ b/it/server/src/test/java/com/walmartlabs/concord/it/server/MavenRepoIT.java
@@ -39,8 +39,13 @@ public class MavenRepoIT extends AbstractServerIT {
     @Test
     public void test() throws Exception {
         // prepare local mvn repo
-        Path mvnDirectory = Path.of(System.getProperty("user.home"))
-                .resolve(".m2/repository/com/walmartlabs/concord/mvn-concord/0.0.1");
+        Path localMavenRepo;
+        try {
+            localMavenRepo = Path.of(System.getProperty("local.mvn.repo"));
+        } catch (NullPointerException e) {
+            localMavenRepo = Path.of(System.getProperty("user.home")).resolve(".m2/repository");
+        }
+        Path mvnDirectory = localMavenRepo.resolve("com/walmartlabs/concord/mvn-concord/0.0.1");
 
         Path src = Paths.get(MavenRepoIT.class.getResource("mvnRepoFiles").toURI());
         IOUtils.copy(src, mvnDirectory, StandardCopyOption.REPLACE_EXISTING);