gitacross 1.1.0__tar.gz → 2.0.0__tar.gz

{gitacross-1.1.0/src/gitacross.egg-info → gitacross-2.0.0}/PKG-INFO

@@ -1,8 +1,8 @@
 Metadata-Version: 2.4
 Name: gitacross
-Version: 1.1.0
+Version: 2.0.0
 Summary: Mirror releases and git commits across platforms (Gitea, GitHub, local) with transform pipelines.
-Author: GitAcross Authors
+Author: Matthew Deik
 License-Expression: MIT
 Keywords: git,gitea,github,mirror,sync,release
 Classifier: Development Status :: 4 - Beta
@@ -162,7 +162,7 @@ Set to `false` to pause a project without removing it from the config. Default `
 
 #### `preserve_description`
 
-Copy the source release notes/body to the target release. Default `true` (alias: `preserve_release_description`). Set at the project or endpoint level; `false` leaves the target release description empty.
+Copy the source release notes/body to the target release. Default `true`. Set at the project or endpoint level; `false` leaves the target release description empty.
 
 #### `release_description`
 
@@ -174,7 +174,7 @@ Custom commit message for the target commits (alias: `commit_template`). Default
 
 #### `sync_assets`
 
-Mirror prebuilt release packages from the source to the target release — so you only need CI on the source platform (aliases: `preserve_assets`, `include_assets`). Project-level field:
+Mirror prebuilt release packages from the source to the target release — so you only need CI on the source platform. Project-level field:
 
 | Value | Behaviour |
 |---|---|
@@ -489,7 +489,7 @@ results = gitacross.run("config.yml", reset=True)
 <summary>Full control — loop over projects yourself</summary>
 
 ```python
-config = gitacross.Config.from_path("config.yml")
+config = gitacross.Config("config.yml")
 
 for project in config.projects:
     if project.enabled:
@@ -535,12 +535,44 @@ if not report.is_valid:
 
 </details>
 
+<details>
+<summary>Sync from YAML held in a variable — no file needed</summary>
+
+```python
+# ${VAR} tokens still resolve from the environment
+config = gitacross.Config.from_yaml_string("""
+projects:
+  - name: my-mirror
+    source:
+      type: gitea
+      repo: owner/repo
+      api: https://gitea.example.com/api/v1
+      token: ${GITEA_TOKEN}
+    target:
+      type: github
+      repo: owner/repo
+      api: https://api.github.com
+      token: ${GITHUB_TOKEN}
+""")
+gitacross.run(config, dry_run=True)
+```
+
+</details>
+
+All public symbols are importable directly from `gitacross`:
+
 | Symbol | What it does |
 |---|---|
-| `run(config_path, project=None, dry_run=False, reset=False, work_dir=".gitsync")` | Sync from a config file — the primary entry point. Returns one dict per project: `project`, `synced`, `releases_synced`, `releases`, `error` |
+| `run(config, project=None, dry_run=False, reset=False, work_dir=".gitsync")` | **Primary entry point.** Sync from a config — a `Config` instance, a path, or an open file object
 | `sync_project(project, work_dir=".gitsync", dry_run=False)` | Sync one project's new releases (respects `project.enabled`); state and cache live in `work_dir`. Returns dicts with `tag`, `source_commit`, `target_commit`, `source_date` |
-| `lint_config(config_path, print_output=True)` | Lint a config file → `LintReport` (`.is_valid`, `.errors`) |
-| `fix_config(config_path, write_back=True, print_output=True)` | Fix misplaced/redundant options → `FixReport` |
-| `Config(config_path)` / `Config.from_path(config_path)` | Load a config file; exposes `.projects` |
-| `ProjectConfig` | One mirror: `name`, `enabled`, `source`, `target`, `renderer`, `retry`, `sync_assets`, `stream_assets`, `preserve_description`, `commit_message`, `release_description` |
-| `ConfigLinter`, `ConfigFixer`, `LintIssue`, `FixIssue`, `LintReport`, `FixReport`, `LintSeverity` | Building blocks for programmatic linting and fixing |
+| `lint_config(config, print_output=True)` | Lint a config (path or open file object) → `LintReport` |
+| `fix_config(config, write_back=True, print_output=True)` | Fix misplaced/redundant options (path only — writes back to the file) → `FixReport` |
+| `Config(config_source)` | Load a config from a path or open file object; exposes `.projects`. `Config.from_yaml_string(content)` loads a config from raw YAML text (`str` or `bytes`) — no file or stream needed |
+| `ProjectConfig(raw)` | Build one mirror project from a raw config dict (see the “No config file” example). Fields: `name`, `enabled`, `source`, `target`, `renderer`, `retry`, `preserve_description`, `sync_assets`, `stream_assets`, `commit_message`, `release_description` |
+| `ConfigLinter()` | Collect lint issues programmatically: `lint_file(config)`, `lint_yaml_string(content)`; results accumulate in `.issues` |
+| `ConfigFixer()` | Fix a config programmatically: `fix_yaml_string(content)` → `FixReport`; actions recorded in `.fixes` |
+| `LintIssue(severity, message, project=None, key=None)` | One lint finding |
+| `FixIssue(message, project=None)` | One applied fix |
+| `LintReport(issues)` | Lint results: `.issues`, `.errors`, `.warnings`, `.redundant`, `.is_valid`, `.format_text()` |
+| `FixReport(fixes, content, is_valid, error=None)` | Fix results: `.fixes`, `.content`, `.is_valid`, `.error`, `.format_text()` |
+| `LintSeverity` | Severity levels used by `LintIssue`: `LintSeverity.ERROR`, `LintSeverity.WARNING`, `LintSeverity.REDUNDANT` |

{gitacross-1.1.0 → gitacross-2.0.0}/README.md

@@ -134,7 +134,7 @@ Set to `false` to pause a project without removing it from the config. Default `
 
 #### `preserve_description`
 
-Copy the source release notes/body to the target release. Default `true` (alias: `preserve_release_description`). Set at the project or endpoint level; `false` leaves the target release description empty.
+Copy the source release notes/body to the target release. Default `true`. Set at the project or endpoint level; `false` leaves the target release description empty.
 
 #### `release_description`
 
@@ -146,7 +146,7 @@ Custom commit message for the target commits (alias: `commit_template`). Default
 
 #### `sync_assets`
 
-Mirror prebuilt release packages from the source to the target release — so you only need CI on the source platform (aliases: `preserve_assets`, `include_assets`). Project-level field:
+Mirror prebuilt release packages from the source to the target release — so you only need CI on the source platform. Project-level field:
 
 | Value | Behaviour |
 |---|---|
@@ -461,7 +461,7 @@ results = gitacross.run("config.yml", reset=True)
 <summary>Full control — loop over projects yourself</summary>
 
 ```python
-config = gitacross.Config.from_path("config.yml")
+config = gitacross.Config("config.yml")
 
 for project in config.projects:
     if project.enabled:
@@ -507,12 +507,44 @@ if not report.is_valid:
 
 </details>
 
+<details>
+<summary>Sync from YAML held in a variable — no file needed</summary>
+
+```python
+# ${VAR} tokens still resolve from the environment
+config = gitacross.Config.from_yaml_string("""
+projects:
+  - name: my-mirror
+    source:
+      type: gitea
+      repo: owner/repo
+      api: https://gitea.example.com/api/v1
+      token: ${GITEA_TOKEN}
+    target:
+      type: github
+      repo: owner/repo
+      api: https://api.github.com
+      token: ${GITHUB_TOKEN}
+""")
+gitacross.run(config, dry_run=True)
+```
+
+</details>
+
+All public symbols are importable directly from `gitacross`:
+
 | Symbol | What it does |
 |---|---|
-| `run(config_path, project=None, dry_run=False, reset=False, work_dir=".gitsync")` | Sync from a config file — the primary entry point. Returns one dict per project: `project`, `synced`, `releases_synced`, `releases`, `error` |
+| `run(config, project=None, dry_run=False, reset=False, work_dir=".gitsync")` | **Primary entry point.** Sync from a config — a `Config` instance, a path, or an open file object
 | `sync_project(project, work_dir=".gitsync", dry_run=False)` | Sync one project's new releases (respects `project.enabled`); state and cache live in `work_dir`. Returns dicts with `tag`, `source_commit`, `target_commit`, `source_date` |
-| `lint_config(config_path, print_output=True)` | Lint a config file → `LintReport` (`.is_valid`, `.errors`) |
-| `fix_config(config_path, write_back=True, print_output=True)` | Fix misplaced/redundant options → `FixReport` |
-| `Config(config_path)` / `Config.from_path(config_path)` | Load a config file; exposes `.projects` |
-| `ProjectConfig` | One mirror: `name`, `enabled`, `source`, `target`, `renderer`, `retry`, `sync_assets`, `stream_assets`, `preserve_description`, `commit_message`, `release_description` |
-| `ConfigLinter`, `ConfigFixer`, `LintIssue`, `FixIssue`, `LintReport`, `FixReport`, `LintSeverity` | Building blocks for programmatic linting and fixing |
+| `lint_config(config, print_output=True)` | Lint a config (path or open file object) → `LintReport` |
+| `fix_config(config, write_back=True, print_output=True)` | Fix misplaced/redundant options (path only — writes back to the file) → `FixReport` |
+| `Config(config_source)` | Load a config from a path or open file object; exposes `.projects`. `Config.from_yaml_string(content)` loads a config from raw YAML text (`str` or `bytes`) — no file or stream needed |
+| `ProjectConfig(raw)` | Build one mirror project from a raw config dict (see the “No config file” example). Fields: `name`, `enabled`, `source`, `target`, `renderer`, `retry`, `preserve_description`, `sync_assets`, `stream_assets`, `commit_message`, `release_description` |
+| `ConfigLinter()` | Collect lint issues programmatically: `lint_file(config)`, `lint_yaml_string(content)`; results accumulate in `.issues` |
+| `ConfigFixer()` | Fix a config programmatically: `fix_yaml_string(content)` → `FixReport`; actions recorded in `.fixes` |
+| `LintIssue(severity, message, project=None, key=None)` | One lint finding |
+| `FixIssue(message, project=None)` | One applied fix |
+| `LintReport(issues)` | Lint results: `.issues`, `.errors`, `.warnings`, `.redundant`, `.is_valid`, `.format_text()` |
+| `FixReport(fixes, content, is_valid, error=None)` | Fix results: `.fixes`, `.content`, `.is_valid`, `.error`, `.format_text()` |
+| `LintSeverity` | Severity levels used by `LintIssue`: `LintSeverity.ERROR`, `LintSeverity.WARNING`, `LintSeverity.REDUNDANT` |

{gitacross-1.1.0 → gitacross-2.0.0}/pyproject.toml

@@ -4,13 +4,13 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "gitacross"
-version = "1.1.0"
+version = "2.0.0"
 description = "Mirror releases and git commits across platforms (Gitea, GitHub, local) with transform pipelines."
 readme = "README.md"
 requires-python = ">=3.8"
 license = "MIT"
 authors = [
-    { name = "GitAcross Authors" }
+    { name = "Matthew Deik" }
 ]
 keywords = ["git", "gitea", "github", "mirror", "sync", "release"]
 classifiers = [

{gitacross-1.1.0 → gitacross-2.0.0}/src/gitacross/__init__.py

@@ -11,7 +11,7 @@ Quickstart (Python API)::
     results = gitacross.run("config.yml", project="my-mirror", dry_run=True)
 
     # Lower-level: build objects yourself for full control:
-    config = gitacross.Config.from_path("config.yml")
+    config = gitacross.Config("config.yml")
     for project in config.projects:
         if project.enabled:
             gitacross.sync_project(project, ".gitsync")
@@ -21,8 +21,12 @@ Quickstart (Python API)::
     if not report.is_valid:
         gitacross.fix_config("config.yml")
 """
+from __future__ import annotations
 
-__version__ = "0.1.0"
+import re
+from importlib.metadata import PackageNotFoundError
+from importlib.metadata import version as _installed_version
+from pathlib import Path
 
 from .config import Config, ProjectConfig
 from .linter import (
@@ -36,20 +40,35 @@ from .linter import (
     fix_config,
     lint_config,
 )
-from .main import run, sync_project, main
+from .main import main, run, sync_project
+
+
+def _package_version() -> str:
+    """Return the package version, stored only in pyproject.toml.
+
+    Installed: read from the distribution metadata, which setuptools populates
+    from ``[project].version``. Source checkout (package not installed): parse
+    the literal out of ``pyproject.toml``. Falls back to ``"0.0.0"`` if neither
+    is available.
+    """
+    try:
+        return _installed_version("gitacross")
+    except PackageNotFoundError:
+        pass
+    pyproject = Path(__file__).resolve().parent.parent.parent / "pyproject.toml"
+    if pyproject.is_file():
+        match = re.search(
+            r'^version\s*=\s*"([^"]+)"', pyproject.read_text(encoding="utf-8"), re.MULTILINE
+        )
+        if match:
+            return match.group(1)
+    return "0.0.0"
+
+
+__version__ = _package_version()
 
 __all__ = [
-    "__version__",
-    # Config
     "Config",
-    "ProjectConfig",
-    # High-level API
-    "run",
-    # Lower-level API
-    "sync_project",
-    # CLI entry point (exposed for completeness)
-    "main",
-    # Linter
     "ConfigFixer",
     "ConfigLinter",
     "FixIssue",
@@ -57,6 +76,11 @@ __all__ = [
     "LintIssue",
     "LintReport",
     "LintSeverity",
+    "ProjectConfig",
+    "__version__",
     "fix_config",
     "lint_config",
+    "main",
+    "run",
+    "sync_project",
 ]

{gitacross-1.1.0 → gitacross-2.0.0}/src/gitacross/__main__.py

@@ -1,4 +1,6 @@
 """Entry point when invoked with `python -m gitacross`."""
+from __future__ import annotations
+
 
 from .main import main
 

{gitacross-1.1.0 → gitacross-2.0.0}/src/gitacross/config.py

@@ -1,11 +1,15 @@
+from __future__ import annotations
+
 import logging
 import os
+from pathlib import Path
+from typing import final
 
 import yaml
 
 logger = logging.getLogger(__name__)
 
-_VALID_PROJECT_KEYS = {
+VALID_PROJECT_KEYS = {
     "name",
     "enabled",
     "source",
@@ -13,10 +17,7 @@ _VALID_PROJECT_KEYS = {
     "renderer",
     "retry",
     "preserve_description",
-    "preserve_release_description",
     "sync_assets",
-    "preserve_assets",
-    "include_assets",
     "stream_assets",
     "commit_message",
     "commit_template",
@@ -25,7 +26,7 @@ _VALID_PROJECT_KEYS = {
     "description_template",
 }
 
-_KNOWN_SOURCE_KEYS = {
+KNOWN_SOURCE_KEYS = {
     "mode": "source",
     "sync_from": "source",
     "include_prereleases": "source",
@@ -33,7 +34,7 @@ _KNOWN_SOURCE_KEYS = {
     "tag_pattern": "source",
 }
 
-_KNOWN_ENDPOINT_KEYS = {
+KNOWN_ENDPOINT_KEYS = {
     "repo": "source or target",
     "api": "source or target",
     "token": "source or target",
@@ -41,25 +42,90 @@ _KNOWN_ENDPOINT_KEYS = {
 }
 
 
+@final
 class Config:
-    def __init__(self, config_path):
-        with open(config_path) as f:
-            raw = yaml.safe_load(f)
+    """Load and parse a config from a file path or an already-open file object.
+
+    ``config_source`` may be a ``str``/:class:`~pathlib.Path` (opened and read
+    here) or any file-like object with a ``read()`` method (e.g. the result of
+    ``open()`` or an ``io.StringIO``), matching what ``yaml.safe_load`` accepts.
+    To parse YAML held directly in a variable, use :meth:`from_yaml_string`.
+    """
+
+    def __init__(self, config_source):
+        if isinstance(config_source, (str, Path)):
+            try:
+                with open(config_source) as f:
+                    raw = yaml.safe_load(f)
+            except FileNotFoundError as exc:
+                raise Config._missing_source_error(exc, config_source) from exc
+        elif hasattr(config_source, "read"):
+            # Open file-like object (StringIO, an open() handle, BytesIO, ...)
+            raw = yaml.safe_load(config_source)
+        else:
+            raise TypeError(
+                "config_source must be a path (str/Path) or an open file-like "
+                + f"object, got {type(config_source).__name__}"
+            )
+        self.projects = self._projects_from_parsed(raw)
+
+    @classmethod
+    def from_yaml_string(cls, content):
+        """Build a :class:`Config` from raw YAML text (``str`` or ``bytes``).
+
+        Equivalent to wrapping *content* in ``io.StringIO``/``io.BytesIO`` and
+        passing that to :class:`Config`, but hands the string straight to the
+        YAML parser — no intermediate file or stream object:
+
+        >>> config = Config.from_yaml_string("projects: []")
+
+        Raises:
+            yaml.YAMLError: If *content* is not valid YAML.
+            TypeError: If *content* is not a ``str`` or ``bytes``.
+        """
+        if not isinstance(content, (str, bytes)):
+            raise TypeError(
+                "content must be a str or bytes holding YAML, got "
+                + type(content).__name__
+            )
+        config = cls.__new__(cls)
+        config.projects = cls._projects_from_parsed(yaml.safe_load(content))
+        return config
+
+    @staticmethod
+    def _projects_from_parsed(raw) -> list[ProjectConfig]:
+        """Normalize one parsed YAML document into project configs.
+
+        Single source of truth for the accepted top-level shapes: a ``projects``
+        mapping, a flat list of project mappings, or nothing (empty config).
+        Shared by :meth:`__init__` and :meth:`from_yaml_string` so both accept
+        exactly the same documents.
+        """
         if isinstance(raw, list):
             # Flat list at top level: [ {name:..., source:..., ...} ]
             raw = {"projects": raw}
         elif raw is None:
             raw = {"projects": []}
-        self.projects = [ProjectConfig(p) for p in (raw.get("projects") or [])]
-
-    @classmethod
-    def from_path(cls, config_path):
-        return cls(config_path)
+        return [ProjectConfig(p) for p in (raw.get("projects") or [])]
+
+    @staticmethod
+    def _missing_source_error(exc: FileNotFoundError, config_source) -> FileNotFoundError:
+        """Re-raise a failed path open, hinting when the string looks like YAML
+        content rather than a path. Cosmetic only — never affects dispatch."""
+        text = os.fspath(config_source)
+        hint = (
+            " — did you mean to pass YAML content? Use "
+            + "Config.from_yaml_string(...) or wrap it in io.StringIO(...)."
+            if "\n" in text or ": " in text
+            else ""
+        )
+        return FileNotFoundError(f"{exc}{hint}")
 
     def __repr__(self):
         return f"Config(projects={len(self.projects)})"
 
 
+@final
 class _EndpointConfig:
     """Config for a source or target endpoint.
 
@@ -116,10 +182,11 @@ class _EndpointConfig:
         return f"https://{self.owner}:{self.token}@{host}/{self.repo}.git"
 
 
+@final
 class ProjectConfig:
     def __init__(self, raw):
         if not isinstance(raw, dict):
-            raise ValueError(
+            raise TypeError(
                 f"Project entry must be a map/dict (got {type(raw).__name__})"
             )
         name = raw.get("name")
@@ -136,24 +203,24 @@ class ProjectConfig:
 
         # Check for misplaced or unknown keys at the project level
         for k in raw:
-            if k in _VALID_PROJECT_KEYS:
+            if k in VALID_PROJECT_KEYS:
                 continue
-            if k in _KNOWN_SOURCE_KEYS:
+            if k in KNOWN_SOURCE_KEYS:
                 logger.warning(
                     "Project '%s': '%s' was specified at the project level, but must be configured under '%s:' (e.g. %s.%s: %s).",
                     self.name,
                     k,
-                    _KNOWN_SOURCE_KEYS[k],
-                    _KNOWN_SOURCE_KEYS[k],
+                    KNOWN_SOURCE_KEYS[k],
+                    KNOWN_SOURCE_KEYS[k],
                     k,
                     raw[k],
                 )
-            elif k in _KNOWN_ENDPOINT_KEYS:
+            elif k in KNOWN_ENDPOINT_KEYS:
                 logger.warning(
                     "Project '%s': '%s' was specified at the project level, but belongs under '%s:'.",
                     self.name,
                     k,
-                    _KNOWN_ENDPOINT_KEYS[k],
+                    KNOWN_ENDPOINT_KEYS[k],
                 )
             else:
                 logger.warning(
@@ -163,18 +230,16 @@ class ProjectConfig:
                 )
 
         # preserve_description — single cascade: project → source → target → True
-        # Supports aliases: preserve_release_description (legacy)
         self.preserve_description = _cascade(
             raw, raw_source, raw_target,
-            keys=["preserve_description", "preserve_release_description"],
+            keys=["preserve_description"],
             default=True,
         )
 
         # sync_assets — single cascade: project → source → target → False
-        # Supports aliases: preserve_assets, include_assets (legacy)
         self.sync_assets = _cascade(
             raw, raw_source, raw_target,
-            keys=["sync_assets", "preserve_assets", "include_assets"],
+            keys=["sync_assets"],
             default=False,
         )
 
@@ -202,6 +267,7 @@ class ProjectConfig:
         )
 
 
+@final
 class _AuthorConfig:
     def __init__(self, raw):
         self.name = raw.get("name", "")
@@ -212,6 +278,7 @@ class _AuthorConfig:
         return bool(self.name) or bool(self.email)
 
 
+@final
 class _RendererConfig:
     def __init__(self, raw):
         self.ignore = raw.get("ignore", [])
@@ -220,6 +287,7 @@ class _RendererConfig:
         self.author = _AuthorConfig(raw_author) if raw_author else _AuthorConfig({})
 
 
+@final
 class _RetryConfig:
     def __init__(self, raw):
         self.max_attempts = raw.get("max_attempts", 3)

{gitacross-1.1.0 → gitacross-2.0.0}/src/gitacross/git.py

@@ -1,3 +1,5 @@
+from __future__ import annotations
+
 import io
 import logging
 import os
@@ -5,6 +7,7 @@ import re
 import subprocess
 import tarfile
 from pathlib import Path
+from typing import final
 
 logger = logging.getLogger(__name__)
 
@@ -59,6 +62,7 @@ def _git(*args, check=True, input_data=None, text=True, env=None):
     return result
 
 
+@final
 class GitRepo:
     """A git repository for release operations.
 
@@ -85,17 +89,17 @@ class GitRepo:
             # Update remote URL *before* fetching so token rotation takes effect
             current = _git("-C", str(path), "remote", "get-url", "origin", check=False)
             if current.returncode == 0 and current.stdout.strip() != url:
-                _git("-C", str(path), "remote", "set-url", "origin", url)
+                _ = _git("-C", str(path), "remote", "set-url", "origin", url)
                 logger.info("Updated remote URL for mirror at %s", dest)
-            _git("-C", str(path), "fetch", "--tags", "--prune", "origin")
+            _ = _git("-C", str(path), "fetch", "--tags", "--prune", "origin")
             logger.info("Updated mirror at %s", dest)
         else:
             path.parent.mkdir(parents=True, exist_ok=True)
-            _git("clone", "--mirror", url, str(path))
+            _ = _git("clone", "--mirror", url, str(path))
             logger.info("Cloned mirror from %s", _redact(url))
         # Ensure author identity for automated commits
-        _git("-C", str(path), "config", "user.name", "GitAcross")
-        _git("-C", str(path), "config", "user.email", "sync@gitacross")
+        _ = _git("-C", str(path), "config", "user.name", "GitAcross")
+        _ = _git("-C", str(path), "config", "user.email", "sync@gitacross")
         return cls(path, is_bare=True)
 
     @classmethod
@@ -163,6 +167,11 @@ class GitRepo:
 
     def export_commit(self, commit_sha, dest):
         """Export a commit's file tree into *dest* (empty dir recommended)."""
+        # git archive of an empty tree emits a dangling pax_global_header that
+        # tarfile can't read, so skip archiving when the tree has no files.
+        tree = self._g("ls-tree", "-r", "--name-only", commit_sha, check=False)
+        if tree.returncode != 0 or not tree.stdout.strip():
+            return
         result = self._g("archive", commit_sha, "--format=tar", text=False)
         with tarfile.open(fileobj=io.BytesIO(result.stdout), mode="r|") as tar:
             tar.extractall(path=str(dest))
@@ -180,7 +189,7 @@ class GitRepo:
         # Already exists locally
         local = self._g("show-ref", "--verify", f"refs/heads/{branch}", check=False)
         if local.returncode == 0:
-            self._g("symbolic-ref", "HEAD", f"refs/heads/{branch}")
+            _ = self._g("symbolic-ref", "HEAD", f"refs/heads/{branch}")
             return
 
         # Exists on remote
@@ -188,12 +197,12 @@ class GitRepo:
             "show-ref", "--verify", f"refs/remotes/origin/{branch}", check=False
         )
         if remote.returncode == 0:
-            self._g("branch", "--force", branch, f"origin/{branch}")
-            self._g("symbolic-ref", "HEAD", f"refs/heads/{branch}")
+            _ = self._g("branch", "--force", branch, f"origin/{branch}")
+            _ = self._g("symbolic-ref", "HEAD", f"refs/heads/{branch}")
             return
 
         # Brand new branch — just set HEAD, the ref is created on first commit
-        self._g("symbolic-ref", "HEAD", f"refs/heads/{branch}")
+        _ = self._g("symbolic-ref", "HEAD", f"refs/heads/{branch}")
 
     def commit(self, work_dir, message, date=None, author_name=None, author_email=None):
         """Stage all files in *work_dir* and commit on current branch.
@@ -204,7 +213,7 @@ class GitRepo:
         If *author_name* and/or *author_email* are provided, they override the
         git identity for this commit (both author and committer).
         """
-        self._gw(work_dir, "add", "-A")
+        _ = self._gw(work_dir, "add", "-A")
         env = dict(os.environ)
         if date:
             env.update({
@@ -239,25 +248,25 @@ class GitRepo:
 
     def tag(self, name, message):
         """Create (or force-update) an annotated tag."""
-        self._g("tag", "-f", name, "-m", message)
+        _ = self._g("tag", "-f", name, "-m", message)
 
     def reset_worktree(self):
         """Populate the working tree to match HEAD. No-op for bare repos."""
         if not self.is_bare:
-            _git("-C", str(self.git_dir.parent), "reset", "--hard", "HEAD")
+            _ = _git("-C", str(self.git_dir.parent), "reset", "--hard", "HEAD")
 
     def push(self, remote, *refs):
         """Push specific refs to remote."""
         # Mirrors (cloned with --mirror) reject explicit refspecs by default.
         # Temporarily disable mirror mode so we can push only the refs we need.
         if self.is_bare:
-            self._g("config", "--local", "remote.origin.mirror", "false")
+            _ = self._g("config", "--local", "remote.origin.mirror", "false")
             try:
-                self._g("push", remote, *refs)
+                _ = self._g("push", remote, *refs)
             finally:
-                self._g("config", "--local", "remote.origin.mirror", "true")
+                _ = self._g("config", "--local", "remote.origin.mirror", "true")
         else:
-            self._g("push", remote, *refs)
+            _ = self._g("push", remote, *refs)
 
     def tag_exists(self, tag):
         """Check if a tag exists locally."""