factor out repo cleanup

Author: btraven00

source/_templates/plugin_base.rst.j2

@@ -12,13 +12,11 @@ Snakemake {% block type %}{% endblock %} plugin: {{ plugin_name }}
    :target: #
 {% endif -%}
 {% if repository_type == "github" and repository is not none -%}
-{% set repo = repository.replace("https://github.com/", "").replace(".git", "").rstrip("/") -%}
-.. image:: https://img.shields.io/github/last-commit/{{ repo }}
+.. image:: https://img.shields.io/github/last-commit/{{ repo_shortname }}
    :alt: GitHub - Last commit
    :target: {{ repository }}
 {% elif repository_type == "gitlab" and repository is not none -%}
-{% set repo = repository.replace("https://gitlab.com/", "").replace(".git", "").rstrip("/") -%}
-.. image:: https://img.shields.io/gitlab/last-commit/{{ repo }}
+.. image:: https://img.shields.io/gitlab/last-commit/{{ repo_shortname }}
    :alt: GitLab - Last commit
    :target: {{ repository }}
 {% elif commit_info is not none -%}

source/collect_plugins.py

@@ -237,6 +237,10 @@ def get_setting_meta(setting, key, default="", verb=False):
             repository = project_urls.get("Repository") or project_urls.get(
                 "repository"
             )
+            # Clean up repository URL early - remove .git suffix and trailing slashes
+            if repository:
+                repository = repository.replace(".git", "").rstrip("/")
+
             repository_type = None
             if repository is None:
                 docs_warning = (
@@ -310,11 +314,15 @@ def get_setting_meta(setting, key, default="", verb=False):
                 else:
                     error += "\n\nPlease contact the plugin authors."
 
+            # Get repository shortname for shields.io badges
+            repo_shortname = get_repo_shortname(repository) if repository else None
+
             rendered = templates.get_template(f"{plugin_type}_plugin.rst.j2").render(
                 plugin_name=plugin_name,
                 package_name=package,
                 authors=authors,
                 repository=repository,
+                repo_shortname=repo_shortname,
                 repository_type=repository_type,
                 commit_info=commit_info,
                 commit_url=commit_url,
@@ -400,6 +408,32 @@ def _commit_url(
         return repository
 
 
+def get_repo_shortname(repository: Optional[str]) -> str:
+    """Extract the shortname from repository URL for shields.io badges.
+
+    Removes protocol and known forge prefix (https://github.com/ or https://gitlab.com/),
+    returning just the user/repo path.
+
+    Args:
+        repository: Full repository URL (e.g., https://github.com/user/repo)
+
+    Returns:
+        Repository shortname (e.g., user/repo)
+    """
+    if not repository:
+        return ""
+
+    # Remove protocol and domain prefixes
+    cleaned = (
+        repository.replace("https://github.com/", "")
+        .replace("https://gitlab.com/", "")
+        .replace("http://github.com/", "")
+        .replace("http://gitlab.com/", "")
+    )
+
+    return cleaned
+
+
 def _commit_age_color(date_str: str) -> str:
     """Return a shields.io color hex based on how old the commit date is."""
     dt = datetime.fromisoformat(date_str.replace("Z", "+00:00"))

source/test_collect_plugins.py

@@ -6,6 +6,7 @@
     _convert_markdown_to_rst,
     _plugin_min_snakemake,
     _commit_url,
+    get_repo_shortname,
 )
 
 
@@ -253,3 +254,42 @@ def test_commit_url_unknown_type_non_none():
     """Test unknown repository type string returns base URL."""
     url = _commit_url("https://bitbucket.org/user/repo", "bitbucket", "abc1234")
     assert url == "https://bitbucket.org/user/repo"
+
+
+# Repository shortname extraction tests
+
+
+def test_get_repo_shortname_github_https():
+    """Test extracting shortname from GitHub HTTPS URL."""
+    shortname = get_repo_shortname("https://github.com/user/repo")
+    assert shortname == "user/repo"
+
+
+def test_get_repo_shortname_gitlab_https():
+    """Test extracting shortname from GitLab HTTPS URL."""
+    shortname = get_repo_shortname("https://gitlab.com/user/repo")
+    assert shortname == "user/repo"
+
+
+def test_get_repo_shortname_http_protocol():
+    """Test extracting shortname from HTTP protocol URL."""
+    shortname = get_repo_shortname("http://github.com/user/repo")
+    assert shortname == "user/repo"
+
+
+def test_get_repo_shortname_empty_string():
+    """Test extracting shortname from empty string."""
+    shortname = get_repo_shortname("")
+    assert shortname == ""
+
+
+def test_get_repo_shortname_none():
+    """Test extracting shortname from None value."""
+    shortname = get_repo_shortname(None)
+    assert shortname == ""
+
+
+def test_get_repo_shortname_other_domain():
+    """Test URL from other domain remains unchanged."""
+    shortname = get_repo_shortname("https://bitbucket.org/user/repo")
+    assert shortname == "https://bitbucket.org/user/repo"