fix: Use more fallback env variables for slug

Author: calvin-codecov

codecov-cli/codecov_cli/helpers/ci_adapters/harness.py

@@ -1,6 +1,7 @@
 import os
 
 from codecov_cli.helpers.ci_adapters.base import CIAdapterBase
+from codecov_cli.helpers.git import parse_slug
 
 # https://developer.harness.io/docs/continuous-integration/troubleshoot-ci/ci-env-var/
 
@@ -29,7 +30,31 @@ def _get_build_url(self):
         return os.getenv("CI_BUILD_LINK")
 
     def _get_slug(self):
-        return os.getenv("CI_REPO")
+        # DRONE_REPO is the canonical org/repo slug on Harness CI (Drone-compatible).
+        # CI_REPO is often equivalent but some setups only set the repo name; then use
+        # DRONE_REPO_NAMESPACE + DRONE_REPO_NAME (or short CI_REPO as the name part).
+        if drone_repo := os.getenv("DRONE_REPO"):
+            return drone_repo
+        ci_repo = os.getenv("CI_REPO")
+        if ci_repo and "/" in ci_repo:
+            return ci_repo
+        namespace = os.getenv("DRONE_REPO_NAMESPACE") or os.getenv("CI_REPO_NAMESPACE")
+        name = os.getenv("DRONE_REPO_NAME") or (
+            ci_repo if ci_repo and "/" not in ci_repo else None
+        )
+        if namespace and name:
+            return f"{namespace}/{name}"
+        # Many Harness builds omit CI_REPO / *_NAMESPACE but still set clone/remotes:
+        # https://developer.harness.io/docs/continuous-integration/troubleshoot-ci/ci-env-var/
+        for env_var in (
+            "DRONE_GIT_HTTP_URL",
+            "CI_REPO_REMOTE",
+            "DRONE_REMOTE_URL",
+        ):
+            if url := os.getenv(env_var):
+                if slug := parse_slug(url):
+                    return slug
+        return None
 
     def _get_service(self):
         return "harness"

codecov-cli/tests/ci_adapters/test_harnessci.py

@@ -12,10 +12,17 @@ class HarnessEnvEnum(str, Enum):
     CI_BUILD_LINK = "CI_BUILD_LINK"
     CI_BUILD_NUMBER = "CI_BUILD_NUMBER"
     CI_REPO = "CI_REPO"
+    CI_REPO_NAMESPACE = "CI_REPO_NAMESPACE"
     DRONE = "DRONE"
     DRONE_COMMIT_BRANCH = "DRONE_COMMIT_BRANCH"
     DRONE_COMMIT_SHA = "DRONE_COMMIT_SHA"
     DRONE_PULL_REQUEST = "DRONE_PULL_REQUEST"
+    DRONE_GIT_HTTP_URL = "DRONE_GIT_HTTP_URL"
+    DRONE_REMOTE_URL = "DRONE_REMOTE_URL"
+    DRONE_REPO = "DRONE_REPO"
+    DRONE_REPO_NAME = "DRONE_REPO_NAME"
+    DRONE_REPO_NAMESPACE = "DRONE_REPO_NAMESPACE"
+    CI_REPO_REMOTE = "CI_REPO_REMOTE"
 
 class TestHarnessCI(object):
     @pytest.mark.parametrize(
@@ -93,8 +100,49 @@ def test_build_url(self, env_dict, expected, mocker):
     @pytest.mark.parametrize(
         "env_dict,expected",
         [
-        ({}, None),
-        ({HarnessEnvEnum.CI_REPO: "repo"}, "repo"),
+            ({}, None),
+            ({HarnessEnvEnum.DRONE_REPO: "owner/repo"}, "owner/repo"),
+            ({HarnessEnvEnum.CI_REPO: "owner/repo"}, "owner/repo"),
+            (
+                {
+                    HarnessEnvEnum.DRONE_REPO_NAMESPACE: "owner",
+                    HarnessEnvEnum.DRONE_REPO_NAME: "repo",
+                },
+                "owner/repo",
+            ),
+            (
+                {
+                    HarnessEnvEnum.DRONE_REPO_NAMESPACE: "owner",
+                    HarnessEnvEnum.CI_REPO: "repo",
+                },
+                "owner/repo",
+            ),
+            (
+                {
+                    HarnessEnvEnum.CI_REPO_NAMESPACE: "owner",
+                    HarnessEnvEnum.CI_REPO: "repo",
+                },
+                "owner/repo",
+            ),
+            ({HarnessEnvEnum.CI_REPO: "repo"}, None),
+            (
+                {
+                    HarnessEnvEnum.DRONE_GIT_HTTP_URL: "https://github.com/myorg/myrepo.git",
+                },
+                "myorg/myrepo",
+            ),
+            (
+                {
+                    HarnessEnvEnum.CI_REPO_REMOTE: "https://gitlab.com/mygroup/myrepo.git",
+                },
+                "mygroup/myrepo",
+            ),
+            (
+                {
+                    HarnessEnvEnum.DRONE_REMOTE_URL: "git@github.com:acme/coverage.git",
+                },
+                "acme/coverage",
+            ),
         ],
     )
     def test_slug(self, env_dict, expected, mocker):