scmrepo 3.3.11__tar.gz → 3.4.0__tar.gz

{scmrepo-3.3.11 → scmrepo-3.4.0}/.pre-commit-config.yaml

@@ -20,7 +20,7 @@ repos:
       - id: sort-simple-yaml
       - id: trailing-whitespace
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: 'v0.11.7'
+    rev: 'v0.12.3'
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]

{scmrepo-3.3.11/src/scmrepo.egg-info → scmrepo-3.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: scmrepo
-Version: 3.3.11
+Version: 3.4.0
 Summary: scmrepo
 Author-email: Iterative <support@dvc.org>
 License: Apache-2.0
@@ -17,7 +17,7 @@ Requires-Python: >=3.9
 Description-Content-Type: text/x-rst
 License-File: LICENSE
 Requires-Dist: gitpython>3
-Requires-Dist: dulwich>=0.22.1
+Requires-Dist: dulwich>=0.23.1
 Requires-Dist: pygit2>=1.14.0
 Requires-Dist: pygtrie>=2.3.2
 Requires-Dist: fsspec[tqdm]>=2024.2.0
@@ -30,7 +30,7 @@ Provides-Extra: tests
 Requires-Dist: aioresponses<0.8,>=0.7; extra == "tests"
 Requires-Dist: paramiko<4,>=3.4.0; extra == "tests"
 Requires-Dist: pytest<9,>=7; extra == "tests"
-Requires-Dist: pytest-asyncio<1,>=0.23.2; extra == "tests"
+Requires-Dist: pytest-asyncio<2,>=0.23.2; extra == "tests"
 Requires-Dist: pytest-cov>=4.1.0; extra == "tests"
 Requires-Dist: pytest-docker<4,>=1; extra == "tests"
 Requires-Dist: pytest-mock; extra == "tests"
@@ -38,7 +38,7 @@ Requires-Dist: pytest-sugar; extra == "tests"
 Requires-Dist: pytest-test-utils<0.2,>=0.1.0; extra == "tests"
 Requires-Dist: proxy.py; extra == "tests"
 Provides-Extra: dev
-Requires-Dist: mypy==1.15.0; extra == "dev"
+Requires-Dist: mypy==1.17.0; extra == "dev"
 Requires-Dist: scmrepo[tests]; extra == "dev"
 Requires-Dist: types-certifi; extra == "dev"
 Requires-Dist: types-mock; extra == "dev"

{scmrepo-3.3.11 → scmrepo-3.4.0}/pyproject.toml

@@ -23,7 +23,7 @@ requires-python = ">=3.9"
 dynamic = ["version"]
 dependencies = [
     "gitpython>3",
-    "dulwich>=0.22.1",
+    "dulwich>=0.23.1",
     "pygit2>=1.14.0",
     "pygtrie>=2.3.2",
     "fsspec[tqdm]>=2024.2.0",
@@ -43,7 +43,7 @@ tests = [
     "aioresponses>=0.7,<0.8",
     "paramiko>=3.4.0,<4",
     "pytest>=7,<9",
-    "pytest-asyncio>=0.23.2,<1",
+    "pytest-asyncio>=0.23.2,<2",
     "pytest-cov>=4.1.0",
     "pytest-docker>=1,<4",
     "pytest-mock",
@@ -52,7 +52,7 @@ tests = [
     "proxy.py",
 ]
 dev = [
-    "mypy==1.15.0",
+    "mypy==1.17.0",
     "scmrepo[tests]",
     "types-certifi",
     "types-mock",
@@ -147,6 +147,7 @@ ignore = [
     "RET503", # implicit-return
     "SIM117", # multiple-with-statements
     "N818", # error-suffix-on-exception-name
+    "PLC0415", # import-outside-top-level
 ]
 select = [
     "A", # flake8-buitlins
@@ -191,8 +192,9 @@ select = [
 
 [tool.ruff.lint.per-file-ignores]
 "noxfile.py" = ["D", "PTH"]
-"tests/**" = ["S", "ARG001", "ARG002", "ANN"]
+"tests/**" = ["S", "ARG001", "ARG002", "ANN", "TID251", "TID253"]
 "docs/**" = ["INP"]
+"src/scmrepo/git/backend/gitpython.py" = ["TID251"]
 
 [tool.ruff.lint.flake8-pytest-style]
 fixture-parentheses = false
@@ -202,6 +204,14 @@ parametrize-names-type = "csv"
 [tool.ruff.lint.flake8-type-checking]
 strict = true
 
+[tool.ruff.lint.flake8-tidy-imports.banned-api]
+"git".msg = "importing from 'git' is not allowed except inside `gitpython` backend"
+
+[tool.ruff.lint.flake8-tidy-imports]
+# Ban certain modules from being imported at module level, instead requiring
+# that they're imported lazily (e.g., within a function definition).
+banned-module-level-imports = ["git"]
+
 [tool.ruff.lint.isort]
 known-first-party = ["scmrepo"]
 

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo/asyn.py

@@ -6,7 +6,6 @@ import threading
 from typing import Any, Optional
 
 from fsspec.asyn import (  # noqa: F401, pylint:disable=unused-import
-    _selector_policy,
     sync,
     sync_wrapper,
 )
@@ -23,8 +22,7 @@ def get_loop() -> asyncio.AbstractEventLoop:
     if default_loop[0] is None:
         with lock:
             if default_loop[0] is None:
-                with _selector_policy():
-                    default_loop[0] = asyncio.new_event_loop()
+                default_loop[0] = asyncio.new_event_loop()
                 loop = default_loop[0]
                 th = threading.Thread(
                     target=loop.run_forever,  # type: ignore[attr-defined]

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo/git/__init__.py

@@ -10,6 +10,7 @@ from contextlib import contextmanager
 from functools import partialmethod
 from typing import (
     TYPE_CHECKING,
+    Any,
     Callable,
     ClassVar,
     Optional,
@@ -295,7 +296,7 @@ class Git(Base):
     # See:
     # https://github.com/iterative/dvc/issues/5641
     # https://github.com/iterative/dvc/issues/7458
-    def _backend_func(self, name, *args, **kwargs):
+    def _backend_func(self, name, *args, **kwargs) -> Any:
         backends: Iterable[str] = kwargs.pop("backends", self.backends)
         for key in backends:
             if self._last_backend is not None and key != self._last_backend:

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo/git/backend/dulwich/__init__.py

@@ -17,6 +17,7 @@ from typing import (
 )
 
 from dulwich.config import ConfigFile, StackedConfig
+from dulwich.walk import ORDER_DATE
 from funcy import cached_property, reraise
 
 from scmrepo.exceptions import AuthError, CloneError, InvalidRemote, RevError, SCMError
@@ -39,7 +40,7 @@ logger = logging.getLogger(__name__)
 
 
 class DulwichObject(GitObject):
-    def __init__(self, repo, name, mode, sha):
+    def __init__(self, repo, name, mode, sha) -> None:
         self.repo = repo
         self._name = name
         self._mode = mode
@@ -136,7 +137,7 @@ def _get_ssh_vendor() -> "SSHVendor":
 
 
 class DulwichConfig(Config):
-    def __init__(self, config: Union["ConfigFile", "StackedConfig"]):
+    def __init__(self, config: Union["ConfigFile", "StackedConfig"]) -> None:
         self._config = config
 
     @property
@@ -183,7 +184,7 @@ class DulwichBackend(BaseGitBackend):  # pylint:disable=abstract-method
 
     def __init__(  # pylint:disable=W0231
         self, root_dir=os.curdir, search_parent_directories=True
-    ):
+    ) -> None:
         from dulwich.errors import NotGitRepository
         from dulwich.repo import Repo
 
@@ -473,7 +474,25 @@ class DulwichBackend(BaseGitBackend):  # pylint:disable=abstract-method
         return sorted(ref[len(base) :] for ref in self.iter_refs(base))
 
     def list_all_commits(self) -> Iterable[str]:
-        raise NotImplementedError
+        from dulwich.objects import Tag
+
+        repo = self.repo
+        starting_points: list[bytes] = []
+
+        # HEAD
+        head_rev = self.get_ref("HEAD")
+        if head_rev:
+            starting_points.append(head_rev.encode("utf-8"))
+
+        # Branches and remotes
+        for ref in repo.refs:
+            if ref.startswith((b"refs/heads/", b"refs/remotes/", b"refs/tags/")):
+                if isinstance(repo.refs[ref], Tag):
+                    ref = self.repo.get_peeled(repo.refs[ref])
+                starting_points.append(repo.refs[ref])
+
+        walker = self.repo.get_walker(include=starting_points, order=ORDER_DATE)
+        return [e.commit.id.decode() for e in walker]
 
     def get_tree_obj(self, rev: str, **kwargs) -> DulwichObject:
         from dulwich.objectspec import parse_tree
@@ -990,7 +1009,7 @@ def ls_remote(url: str) -> dict[str, str]:
     from dulwich.client import HTTPUnauthorized
 
     try:
-        refs = porcelain.ls_remote(url)
+        refs = porcelain.ls_remote(url).refs
         return {os.fsdecode(ref): sha.decode("ascii") for ref, sha in refs.items()}
     except HTTPUnauthorized as exc:
         raise AuthError(url) from exc

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo/git/backend/dulwich/asyncssh_vendor.py

@@ -71,7 +71,9 @@ class _StderrWrapper:
 
 
 class AsyncSSHWrapper(BaseAsyncObject):
-    def __init__(self, conn: "SSHClientConnection", proc: "SSHClientProcess", **kwargs):
+    def __init__(
+        self, conn: "SSHClientConnection", proc: "SSHClientProcess", **kwargs
+    ) -> None:
         super().__init__(**kwargs)
         self.conn: SSHClientConnection = conn
         self.proc: SSHClientProcess = proc
@@ -109,7 +111,7 @@ class InteractiveSSHClient(SSHClient):
     _keys_to_try: Optional[list["FilePath"]] = None
     _passphrases: dict[str, str]
 
-    def __init__(self, *args, **kwargs):
+    def __init__(self, *args, **kwargs) -> None:
         super(*args, **kwargs)
         self._passphrases: dict[str, str] = {}
 

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo/git/backend/dulwich/client.py

@@ -14,7 +14,7 @@ class GitCredentialsHTTPClient(Urllib3HttpGitClient):  # pylint: disable=abstrac
         password=None,
         config=None,
         **kwargs,
-    ):
+    ) -> None:
         super().__init__(
             base_url=base_url,
             username=username,
@@ -29,6 +29,7 @@ class GitCredentialsHTTPClient(Urllib3HttpGitClient):  # pylint: disable=abstrac
         url: str,
         headers: Optional[dict[str, str]] = None,
         data: Optional[Union[bytes, Iterator[bytes]]] = None,
+        raise_for_status: bool = True,
     ):
         cached_chunks: list[bytes] = []
 
@@ -48,7 +49,10 @@ class GitCredentialsHTTPClient(Urllib3HttpGitClient):  # pylint: disable=abstrac
 
         try:
             result = super()._http_request(
-                url, headers=headers, data=None if data is None else _cached_data()
+                url,
+                headers=headers,
+                data=None if data is None else _cached_data(),
+                raise_for_status=raise_for_status,
             )
         except HTTPUnauthorized:
             auth_header = self._get_auth()
@@ -59,7 +63,10 @@ class GitCredentialsHTTPClient(Urllib3HttpGitClient):  # pylint: disable=abstrac
             else:
                 headers = auth_header
             result = super()._http_request(
-                url, headers=headers, data=None if data is None else _cached_data()
+                url,
+                headers=headers,
+                data=None if data is None else _cached_data(),
+                raise_for_status=raise_for_status,
             )
         if self._store_credentials is not None:
             self._store_credentials.approve()

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo/git/backend/pygit2/__init__.py

@@ -452,7 +452,32 @@ class Pygit2Backend(BaseGitBackend):  # pylint:disable=abstract-method
         return sorted(ref[len(base) :] for ref in self.iter_refs(base))
 
     def list_all_commits(self) -> Iterable[str]:
-        raise NotImplementedError
+        import pygit2
+        from pygit2.enums import SortMode
+
+        # Add HEAD
+        starting_points: list[Union[Oid, str]] = []
+        if not self.repo.head_is_unborn:
+            starting_points.append(self.repo.head.target)
+
+        # Add all branches, remotes, and tags
+        for ref in self.repo.references:
+            if ref.startswith(("refs/heads/", "refs/remotes/")):
+                oid = self.repo.revparse_single(ref).id
+                starting_points.append(oid)
+            elif ref.startswith("refs/tags/"):
+                tag_obj = self.repo.revparse_single(ref)
+                if isinstance(tag_obj, pygit2.Tag):
+                    starting_points.append(tag_obj.target)
+                else:
+                    starting_points.append(tag_obj.id)
+
+        # Walk all commits
+        walker = self.repo.walk(None)
+        for oid in starting_points:
+            walker.push(oid)
+        walker.sort(SortMode.TIME)
+        return [str(commit.id) for commit in walker]
 
     def get_tree_obj(self, rev: str, **kwargs) -> Pygit2Object:
         tree = self.repo[rev].tree

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo/git/backend/pygit2/filter.py

@@ -13,7 +13,7 @@ logger = logging.getLogger(__name__)
 class LFSFilter(Filter):
     attributes = "filter=*"
 
-    def __init__(self, *args, **kwargs):
+    def __init__(self, *args, **kwargs) -> None:
         self._smudge_buf: Optional[io.BytesIO] = None
         self._smudge_root: Optional[str] = None
 

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo/git/credentials.py

@@ -105,7 +105,7 @@ class GitCredentialHelper(CredentialHelper):
     >>> password = credentials.password
     """
 
-    def __init__(self, command: str, use_http_path: bool = False):
+    def __init__(self, command: str, use_http_path: bool = False) -> None:
         super().__init__()
         self._command = command
         self._run_kwargs: dict[str, Any] = {}
@@ -136,7 +136,7 @@ class GitCredentialHelper(CredentialHelper):
         if not shutil.which(executable) and shutil.which("git"):
             # If the helper cannot be found in PATH, it might be
             # a C git helper in GIT_EXEC_PATH
-            git_exec_path = subprocess.check_output(  # noqa: S603
+            git_exec_path = subprocess.check_output(
                 ("git", "--exec-path"),
                 text=True,
             ).strip()
@@ -353,7 +353,7 @@ def _input_tty(prompt: str = "Username: ") -> str:
 class MemoryCredentialHelper(CredentialHelper):
     """Memory credential helper that supports optional interactive input."""
 
-    def __init__(self):
+    def __init__(self) -> None:
         super().__init__()
         self._credentials: dict[_CredentialKey, Credential] = {}
 
@@ -535,7 +535,7 @@ class Credential(Mapping[str, str]):
         path: Optional[str] = None,
         username: Optional[str] = None,
         password: Optional[str] = None,
-        password_expiry_utc: Optional[int] = None,
+        password_expiry_utc: Optional[str] = None,
         url: Optional[str] = None,
     ):
         self.protocol = protocol

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo/git/lfs/smudge.py

@@ -17,7 +17,7 @@ def smudge(
     batch_size: Optional[int] = None,
 ) -> BinaryIO:
     """Wrap the specified binary IO stream and run LFS smudge if necessary."""
-    reader = io.BufferedReader(fobj)  # type: ignore[arg-type]
+    reader = io.BufferedReader(fobj)  # type: ignore[type-var]
     data = reader.peek(100)
     if any(data.startswith(header) for header in HEADERS):
         # read the pointer data into memory since the raw stream is unseekable

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo/progress.py

@@ -4,7 +4,7 @@ from funcy import compose
 
 
 def code2desc(op_code):
-    from git import RootUpdateProgress as OP  # noqa: N814
+    from git import RootUpdateProgress as OP  # noqa: N814, TID251
 
     ops = {
         OP.COUNTING: "Counting",
@@ -44,16 +44,20 @@ class GitProgressEvent(NamedTuple):
 
 class GitProgressReporter:
     def __init__(self, fn) -> None:
-        from git.util import CallableRemoteProgress
-
-        self._reporter = CallableRemoteProgress(self.wrap_fn(fn))
+        try:
+            from git.util import CallableRemoteProgress  # noqa: TID251
+        except ImportError:
+            self._reporter = None
+        else:
+            self._reporter = CallableRemoteProgress(self.wrap_fn(fn))
 
     def __call__(self, msg: Union[str, bytes]) -> None:
-        self._reporter._parse_progress_line(
-            msg.decode("utf-8", errors="replace").strip()
-            if isinstance(msg, bytes)
-            else msg
-        )
+        if self._reporter is not None:
+            self._reporter._parse_progress_line(
+                msg.decode("utf-8", errors="replace").strip()
+                if isinstance(msg, bytes)
+                else msg
+            )
 
     @staticmethod
     def wrap_fn(fn):

{scmrepo-3.3.11 → scmrepo-3.4.0/src/scmrepo.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: scmrepo
-Version: 3.3.11
+Version: 3.4.0
 Summary: scmrepo
 Author-email: Iterative <support@dvc.org>
 License: Apache-2.0
@@ -17,7 +17,7 @@ Requires-Python: >=3.9
 Description-Content-Type: text/x-rst
 License-File: LICENSE
 Requires-Dist: gitpython>3
-Requires-Dist: dulwich>=0.22.1
+Requires-Dist: dulwich>=0.23.1
 Requires-Dist: pygit2>=1.14.0
 Requires-Dist: pygtrie>=2.3.2
 Requires-Dist: fsspec[tqdm]>=2024.2.0
@@ -30,7 +30,7 @@ Provides-Extra: tests
 Requires-Dist: aioresponses<0.8,>=0.7; extra == "tests"
 Requires-Dist: paramiko<4,>=3.4.0; extra == "tests"
 Requires-Dist: pytest<9,>=7; extra == "tests"
-Requires-Dist: pytest-asyncio<1,>=0.23.2; extra == "tests"
+Requires-Dist: pytest-asyncio<2,>=0.23.2; extra == "tests"
 Requires-Dist: pytest-cov>=4.1.0; extra == "tests"
 Requires-Dist: pytest-docker<4,>=1; extra == "tests"
 Requires-Dist: pytest-mock; extra == "tests"
@@ -38,7 +38,7 @@ Requires-Dist: pytest-sugar; extra == "tests"
 Requires-Dist: pytest-test-utils<0.2,>=0.1.0; extra == "tests"
 Requires-Dist: proxy.py; extra == "tests"
 Provides-Extra: dev
-Requires-Dist: mypy==1.15.0; extra == "dev"
+Requires-Dist: mypy==1.17.0; extra == "dev"
 Requires-Dist: scmrepo[tests]; extra == "dev"
 Requires-Dist: types-certifi; extra == "dev"
 Requires-Dist: types-mock; extra == "dev"

{scmrepo-3.3.11 → scmrepo-3.4.0}/src/scmrepo.egg-info/requires.txt

@@ -1,5 +1,5 @@
 gitpython>3
-dulwich>=0.22.1
+dulwich>=0.23.1
 pygit2>=1.14.0
 pygtrie>=2.3.2
 fsspec[tqdm]>=2024.2.0
@@ -10,7 +10,7 @@ aiohttp-retry>=2.5.0
 tqdm
 
 [dev]
-mypy==1.15.0
+mypy==1.17.0
 scmrepo[tests]
 types-certifi
 types-mock
@@ -21,7 +21,7 @@ types-tqdm
 aioresponses<0.8,>=0.7
 paramiko<4,>=3.4.0
 pytest<9,>=7
-pytest-asyncio<1,>=0.23.2
+pytest-asyncio<2,>=0.23.2
 pytest-cov>=4.1.0
 pytest-docker<4,>=1
 pytest-mock

{scmrepo-3.3.11 → scmrepo-3.4.0}/tests/test_git.py

@@ -1,5 +1,6 @@
 import os
 import shutil
+import time
 from pathlib import Path
 from typing import Any, Optional
 
@@ -422,21 +423,170 @@ def test_iter_remote_refs(
     } == set(git.iter_remote_refs(remote))
 
 
-@pytest.mark.skip_git_backend("dulwich", "pygit2")
+def _gen(scm: Git, s: str, commit_timestamp: Optional[float] = None) -> str:
+    with open(s, mode="w") as f:
+        f.write(s)
+    scm.dulwich.add([s])
+    scm.dulwich.repo.do_commit(
+        message=s.encode("utf-8"), commit_timestamp=commit_timestamp
+    )
+    return scm.get_rev()
+
+
 def test_list_all_commits(tmp_dir: TmpDir, scm: Git, git: Git, matcher: type[Matcher]):
-    def _gen(s):
-        tmp_dir.gen(s, s)
-        scm.add_commit(s, message=s)
-        return scm.get_rev()
+    assert git.list_all_commits() == []
+    # https://github.com/libgit2/libgit2/issues/6336
+    now = time.time()
+
+    rev_a = _gen(scm, "a", commit_timestamp=now - 10)
+    rev_b = _gen(scm, "b", commit_timestamp=now - 8)
+    rev_c = _gen(scm, "c", commit_timestamp=now - 5)
+    rev_d = _gen(scm, "d", commit_timestamp=now - 2)
+
+    assert git.list_all_commits() == [rev_d, rev_c, rev_b, rev_a]
+
+    scm.gitpython.git.reset(rev_b, hard=True)
+    assert git.list_all_commits() == [rev_b, rev_a]
 
-    rev_a = _gen("a")
-    rev_b = _gen("b")
+
+def test_list_all_commits_branch(
+    tmp_dir: TmpDir, scm: Git, git: Git, matcher: type[Matcher]
+):
+    revs = {}
+    now = time.time()
+
+    revs["1"] = _gen(scm, "a", commit_timestamp=now - 10)
+
+    scm.checkout("branch", create_new=True)
+    revs["3"] = _gen(scm, "c", commit_timestamp=now - 9)
+
+    scm.checkout("master")
+    revs["2"] = _gen(scm, "b", commit_timestamp=now - 7)
+
+    scm.checkout("branch")
+    revs["5"] = _gen(scm, "e", commit_timestamp=now - 6)
+
+    scm.checkout("master")
+    revs["4"] = _gen(scm, "d", commit_timestamp=now - 5)
+
+    scm.checkout("branch")
+    revs["6"] = _gen(scm, "f", commit_timestamp=now - 4)
+
+    scm.checkout("master")
+    revs["7"] = _gen(scm, "g", commit_timestamp=now - 3)
+    revs["8"] = scm.merge("branch", msg="merge branch")
+
+    inv_map = {v: k for k, v in revs.items()}
+    assert [inv_map[k] for k in git.list_all_commits()] == [
+        "8",
+        "7",
+        "6",
+        "4",
+        "5",
+        "2",
+        "3",
+        "1",
+    ]
+
+
+def test_list_all_tags(tmp_dir: TmpDir, scm: Git, git: Git, matcher: type[Matcher]):
+    rev_a = _gen(scm, "a")
     scm.tag("tag")
-    rev_c = _gen("c")
+    rev_b = _gen(scm, "b")
+    scm.tag("annotated", annotated=True, message="Annotated Tag")
+    rev_c = _gen(scm, "c")
+    rev_d = _gen(scm, "d")
+    assert git.list_all_commits() == matcher.unordered(rev_d, rev_c, rev_b, rev_a)
+
+    rev_e = _gen(scm, "e")
+    scm.tag(
+        "annotated2",
+        target="refs/tags/annotated",
+        annotated=True,
+        message="Annotated Tag",
+    )
+    assert git.list_all_commits() == matcher.unordered(
+        rev_e, rev_d, rev_c, rev_b, rev_a
+    )
+
+    rev_f = _gen(scm, "f")
+    scm.tag(
+        "annotated3",
+        target="refs/tags/annotated2",
+        annotated=True,
+        message="Annotated Tag 3",
+    )
+    assert git.list_all_commits() == matcher.unordered(
+        rev_f, rev_e, rev_d, rev_c, rev_b, rev_a
+    )
+
     scm.gitpython.git.reset(rev_a, hard=True)
-    scm.set_ref("refs/foo/bar", rev_c)
+    assert git.list_all_commits() == matcher.unordered(rev_b, rev_a)
+
+
+def test_list_all_commits_dangling_annotated_tag(tmp_dir: TmpDir, scm: Git, git: Git):
+    rev_a = _gen(scm, "a")
+    scm.tag("annotated", annotated=True, message="Annotated Tag")
+
+    _gen(scm, "b")
 
-    assert git.list_all_commits() == matcher.unordered(rev_a, rev_b)
+    # Delete branch pointing to rev_a
+    scm.checkout(rev_a)
+    scm.gitpython.repo.delete_head("master", force=True)
+
+    assert git.list_all_commits() == [rev_a]  # Only reachable via the tag
+
+
+def test_list_all_commits_orphan(
+    tmp_dir: TmpDir, scm: Git, git: Git, matcher: type[Matcher]
+):
+    rev_a = _gen(scm, "a")
+
+    # Make an orphan branch
+    scm.gitpython.git.checkout("--orphan", "orphan-branch")
+    rev_orphan = _gen(scm, "orphanfile")
+
+    assert rev_orphan != rev_a
+    assert git.list_all_commits() == matcher.unordered(rev_orphan, rev_a)
+
+
+def test_list_all_commits_refs(
+    tmp_dir: TmpDir, scm: Git, git: Git, matcher: type[Matcher]
+):
+    assert git.list_all_commits() == []
+
+    rev_a = _gen(scm, "a")
+
+    assert git.list_all_commits() == [rev_a]
+    rev_b = _gen(scm, "b")
+    scm.set_ref("refs/remotes/origin/feature", rev_b)
+    assert git.list_all_commits() == matcher.unordered(rev_b, rev_a)
+
+    # also add refs/exps/foo/bar
+    rev_c = _gen(scm, "c")
+    scm.set_ref("refs/exps/foo/bar", rev_c)
+    assert git.list_all_commits() == matcher.unordered(rev_c, rev_b, rev_a)
+
+    # Dangling/broken ref ---
+    scm.set_ref("refs/heads/bad-ref", "deadbeefdeadbeefdeadbeefdeadbeefdeadbeef")
+    with pytest.raises(Exception):  # noqa: B017, PT011
+        git.list_all_commits()
+    scm.remove_ref("refs/heads/bad-ref")
+
+    scm.gitpython.git.reset(rev_a, hard=True)
+    assert git.list_all_commits() == matcher.unordered(rev_b, rev_a)
+
+
+def test_list_all_commits_detached_head(
+    tmp_dir: TmpDir, scm: Git, git: Git, matcher: type[Matcher]
+):
+    rev_a = _gen(scm, "a")
+    rev_b = _gen(scm, "b")
+    rev_c = _gen(scm, "c")
+    scm.checkout(rev_b)
+
+    assert scm.pygit2.repo.head_is_detached
+    assert git.list_all_commits() == matcher.unordered(rev_c, rev_b, rev_a)
 
 
 @pytest.mark.skip_git_backend("pygit2")
@@ -771,7 +921,7 @@ def test_ignored(tmp_dir: TmpDir, scm: Git, git: Git, git_backend: str):
     assert not git.is_ignored(tmp_dir / "dir1" / "file2.txt")
 
 
-@pytest.mark.skip_git_backend("pygit2", "gitpython")
+@pytest.mark.skip_git_backend("pygit2", "gitpython", "dulwich")
 def test_ignored_dir_unignored_subdirs(tmp_dir: TmpDir, scm: Git, git: Git):
     tmp_dir.gen({".gitignore": "data/**\n!data/**/\n!data/**/*.csv"})
     scm.add([".gitignore"])