feat: store and show pkgbuild (implements #157)

docs/ahriman.application.handlers.rst

@@ -92,14 +92,6 @@ ahriman.application.handlers.patch module
    :no-undoc-members:
    :show-inheritance:
 
-ahriman.application.handlers.pkgbuild module
---------------------------------------------
-
-.. automodule:: ahriman.application.handlers.pkgbuild
-   :members:
-   :no-undoc-members:
-   :show-inheritance:
-
 ahriman.application.handlers.rebuild module
 -------------------------------------------
 

docs/ahriman.core.alpm.rst

@@ -28,14 +28,6 @@ ahriman.core.alpm.pacman\_database module
    :no-undoc-members:
    :show-inheritance:
 
-ahriman.core.alpm.pacman\_handle module
----------------------------------------
-
-.. automodule:: ahriman.core.alpm.pacman_handle
-   :members:
-   :no-undoc-members:
-   :show-inheritance:
-
 ahriman.core.alpm.pkgbuild\_parser module
 -----------------------------------------
 

docs/ahriman.core.database.migrations.rst

@@ -140,14 +140,6 @@ ahriman.core.database.migrations.m016\_archive module
    :no-undoc-members:
    :show-inheritance:
 
-ahriman.core.database.migrations.m017\_pkgbuild module
-------------------------------------------------------
-
-.. automodule:: ahriman.core.database.migrations.m017_pkgbuild
-   :members:
-   :no-undoc-members:
-   :show-inheritance:
-
 Module contents
 ---------------
 

docs/ahriman.core.formatters.rst

@@ -76,14 +76,6 @@ ahriman.core.formatters.patch\_printer module
    :no-undoc-members:
    :show-inheritance:
 
-ahriman.core.formatters.pkgbuild\_printer module
-------------------------------------------------
-
-.. automodule:: ahriman.core.formatters.pkgbuild_printer
-   :members:
-   :no-undoc-members:
-   :show-inheritance:
-
 ahriman.core.formatters.printer module
 --------------------------------------
 

src/ahriman/application/handlers/change.py

@@ -54,7 +54,7 @@ class Change(Handler):
             case Action.List:
                 changes = client.package_changes_get(args.package)
                 ChangesPrinter(changes)(verbose=True, separator="")
-                Change.check_status(args.exit_code, changes.changes is not None)
+                Change.check_status(args.exit_code, not changes.is_empty)
             case Action.Remove:
                 client.package_changes_update(args.package, Changes())
 

src/ahriman/application/handlers/pkgbuild.py

@@ -1,101 +0,0 @@
-#
-# Copyright (c) 2021-2026 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-import argparse
-
-from dataclasses import replace
-
-from ahriman.application.application import Application
-from ahriman.application.handlers.handler import Handler, SubParserAction
-from ahriman.core.configuration import Configuration
-from ahriman.core.formatters import PkgbuildPrinter
-from ahriman.models.action import Action
-from ahriman.models.repository_id import RepositoryId
-
-
-class Pkgbuild(Handler):
-    """
-    package pkgbuild handler
-    """
-
-    ALLOW_MULTI_ARCHITECTURE_RUN = False  # conflicting io
-
-    @classmethod
-    def run(cls, args: argparse.Namespace, repository_id: RepositoryId, configuration: Configuration, *,
-            report: bool) -> None:
-        """
-        callback for command line
-
-        Args:
-            args(argparse.Namespace): command line args
-            repository_id(RepositoryId): repository unique identifier
-            configuration(Configuration): configuration instance
-            report(bool): force enable or disable reporting
-        """
-        application = Application(repository_id, configuration, report=True)
-        client = application.repository.reporter
-
-        match args.action:
-            case Action.List:
-                changes = client.package_changes_get(args.package)
-                PkgbuildPrinter(changes)(verbose=True, separator="")
-                Pkgbuild.check_status(args.exit_code, changes.pkgbuild is not None)
-            case Action.Remove:
-                changes = client.package_changes_get(args.package)
-                client.package_changes_update(args.package, replace(changes, pkgbuild=None))
-
-    @staticmethod
-    def _set_package_pkgbuild_parser(root: SubParserAction) -> argparse.ArgumentParser:
-        """
-        add parser for package pkgbuild subcommand
-
-        Args:
-            root(SubParserAction): subparsers for the commands
-
-        Returns:
-            argparse.ArgumentParser: created argument parser
-        """
-        parser = root.add_parser("package-pkgbuild", help="get package pkgbuild",
-                                 description="retrieve package PKGBUILD stored in database",
-                                 epilog="This command requests package status from the web interface "
-                                        "if it is available.")
-        parser.add_argument("package", help="package base")
-        parser.add_argument("-e", "--exit-code", help="return non-zero exit status if result is empty",
-                            action="store_true")
-        parser.set_defaults(action=Action.List, lock=None, quiet=True, report=False, unsafe=True)
-        return parser
-
-    @staticmethod
-    def _set_package_pkgbuild_remove_parser(root: SubParserAction) -> argparse.ArgumentParser:
-        """
-        add parser for package pkgbuild remove subcommand
-
-        Args:
-            root(SubParserAction): subparsers for the commands
-
-        Returns:
-            argparse.ArgumentParser: created argument parser
-        """
-        parser = root.add_parser("package-pkgbuild-remove", help="remove package pkgbuild",
-                                 description="remove the package PKGBUILD stored remotely")
-        parser.add_argument("package", help="package base")
-        parser.set_defaults(action=Action.Remove, exit_code=False, lock=None, quiet=True, report=False, unsafe=True)
-        return parser
-
-    arguments = [_set_package_pkgbuild_parser, _set_package_pkgbuild_remove_parser]

src/ahriman/core/alpm/pacman.py

@@ -24,11 +24,10 @@ import tarfile
 from collections.abc import Iterable, Iterator
 from functools import cached_property
 from pathlib import Path
-from pyalpm import DB, Package, SIG_DATABASE_OPTIONAL, SIG_PACKAGE_OPTIONAL  # type: ignore[import-not-found]
+from pyalpm import DB, Handle, Package, SIG_DATABASE_OPTIONAL, SIG_PACKAGE_OPTIONAL  # type: ignore[import-not-found]
 from string import Template
 
 from ahriman.core.alpm.pacman_database import PacmanDatabase
-from ahriman.core.alpm.pacman_handle import PacmanHandle
 from ahriman.core.configuration import Configuration
 from ahriman.core.log import LazyLogging
 from ahriman.core.utils import trim_package
@@ -62,16 +61,16 @@ class Pacman(LazyLogging):
         self.refresh_database = refresh_database
 
     @cached_property
-    def handle(self) -> PacmanHandle:
+    def handle(self) -> Handle:
         """
         pyalpm handle
 
         Returns:
-            PacmanHandle: generated pyalpm handle instance
+            Handle: generated pyalpm handle instance
         """
         return self.__create_handle(refresh_database=self.refresh_database)
 
-    def __create_handle(self, *, refresh_database: PacmanSynchronization) -> PacmanHandle:
+    def __create_handle(self, *, refresh_database: PacmanSynchronization) -> Handle:
         """
         create lazy handle function
 
@@ -79,14 +78,14 @@ class Pacman(LazyLogging):
             refresh_database(PacmanSynchronization): synchronize local cache to remote
 
         Returns:
-            PacmanHandle: fully initialized pacman handle
+            Handle: fully initialized pacman handle
         """
         pacman_root = self.configuration.getpath("alpm", "database")
         use_ahriman_cache = self.configuration.getboolean("alpm", "use_ahriman_cache")
 
         database_path = self.repository_paths.pacman if use_ahriman_cache else pacman_root
         root = self.configuration.getpath("alpm", "root")
-        handle = PacmanHandle(str(root), str(database_path))
+        handle = Handle(str(root), str(database_path))
 
         for repository in self.configuration.getlist("alpm", "repositories"):
             database = self.database_init(handle, repository, self.repository_id.architecture)
@@ -100,12 +99,12 @@ class Pacman(LazyLogging):
 
         return handle
 
-    def database_copy(self, handle: PacmanHandle, database: DB, pacman_root: Path, *, use_ahriman_cache: bool) -> None:
+    def database_copy(self, handle: Handle, database: DB, pacman_root: Path, *, use_ahriman_cache: bool) -> None:
         """
         copy database from the operating system root to the ahriman local home
 
         Args:
-            handle(PacmanHandle): pacman handle which will be used for database copying
+            handle(Handle): pacman handle which will be used for database copying
             database(DB): pacman database instance to be copied
             pacman_root(Path): operating system pacman root
             use_ahriman_cache(bool): use local ahriman cache instead of system one
@@ -134,12 +133,12 @@ class Pacman(LazyLogging):
         with self.repository_paths.preserve_owner():
             shutil.copy(src, dst)
 
-    def database_init(self, handle: PacmanHandle, repository: str, architecture: str) -> DB:
+    def database_init(self, handle: Handle, repository: str, architecture: str) -> DB:
         """
         create database instance from pacman handler and set its properties
 
         Args:
-            handle(PacmanHandle): pacman handle which will be used for database initializing
+            handle(Handle): pacman handle which will be used for database initializing
             repository(str): pacman repository name (e.g. core)
             architecture(str): repository architecture
 
@@ -165,12 +164,12 @@ class Pacman(LazyLogging):
 
         return database
 
-    def database_sync(self, handle: PacmanHandle, *, force: bool) -> None:
+    def database_sync(self, handle: Handle, *, force: bool) -> None:
         """
         sync local database
 
         Args:
-            handle(PacmanHandle): pacman handle which will be used for database sync
+            handle(Handle): pacman handle which will be used for database sync
             force(bool): force database synchronization (same as ``pacman -Syy``)
         """
         self.logger.info("refresh ahriman's home pacman database (force refresh %s)", force)

src/ahriman/core/alpm/pacman_handle.py

@@ -1,81 +0,0 @@
-#
-# Copyright (c) 2021-2026 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-from pathlib import Path
-from pyalpm import Handle, Package  # type: ignore[import-not-found]
-from tempfile import TemporaryDirectory
-from typing import Any, ClassVar, Self
-
-
-class PacmanHandle:
-    """
-    lightweight wrapper for pacman handle to be used for direct alpm operations (e.g. package load)
-
-    Attributes:
-        handle(Handle): pyalpm handle instance
-    """
-
-    _ephemeral: ClassVar[Self | None] = None
-
-    def __init__(self, *args: Any, **kwargs: Any) -> None:
-        """
-        Args:
-            *args(Any): positional arguments for :class:`pyalpm.Handle`
-            **kwargs(Any): keyword arguments for :class:`pyalpm.Handle`
-        """
-        self.handle = Handle(*args, **kwargs)
-
-    @classmethod
-    def ephemeral(cls) -> Self:
-        """
-        create temporary instance with no access to real databases
-
-        Returns:
-            Self: loaded class
-        """
-        if cls._ephemeral is None:
-            # handle creates alpm version file, but we don't use it
-            # so it is ok to just remove it
-            with TemporaryDirectory(ignore_cleanup_errors=True) as dir_name:
-                cls._ephemeral = cls("/", dir_name)
-        return cls._ephemeral
-
-    def package_load(self, path: Path) -> Package:
-        """
-        load package from path to the archive
-
-        Args:
-            path(Path): path to package archive
-
-        Returns:
-            Package: package instance
-        """
-        return self.handle.load_pkg(str(path))
-
-    def __getattr__(self, item: str) -> Any:
-        """
-        proxy methods for :class:`pyalpm.Handle`, because it doesn't allow subclassing
-
-        Args:
-            item(str): property name
-
-        Returns:
-            Any: attribute by its name
-        """
-        return self.handle.__getattribute__(item)

src/ahriman/core/database/migrations/m005_make_opt_depends.py

@@ -19,9 +19,11 @@
 #
 from sqlite3 import Connection
 
+from ahriman.core.alpm.pacman import Pacman
 from ahriman.core.configuration import Configuration
 from ahriman.core.utils import package_like
 from ahriman.models.package import Package
+from ahriman.models.pacman_synchronization import PacmanSynchronization
 
 
 __all__ = ["migrate_data", "steps"]
@@ -59,9 +61,12 @@ def migrate_package_depends(connection: Connection, configuration: Configuration
     if not configuration.repository_paths.repository.is_dir():
         return
 
+    _, repository_id = configuration.check_loaded()
+    pacman = Pacman(repository_id, configuration, refresh_database=PacmanSynchronization.Disabled)
+
     package_list = []
     for full_path in filter(package_like, configuration.repository_paths.repository.iterdir()):
-        base = Package.from_archive(full_path)
+        base = Package.from_archive(full_path, pacman)
         for package, description in base.packages.items():
             package_list.append({
                 "make_depends": description.make_depends,

src/ahriman/core/database/migrations/m007_check_depends.py

@@ -19,9 +19,11 @@
 #
 from sqlite3 import Connection
 
+from ahriman.core.alpm.pacman import Pacman
 from ahriman.core.configuration import Configuration
 from ahriman.core.utils import package_like
 from ahriman.models.package import Package
+from ahriman.models.pacman_synchronization import PacmanSynchronization
 
 
 __all__ = ["migrate_data", "steps"]
@@ -56,9 +58,12 @@ def migrate_package_check_depends(connection: Connection, configuration: Configu
     if not configuration.repository_paths.repository.is_dir():
         return
 
+    _, repository_id = configuration.check_loaded()
+    pacman = Pacman(repository_id, configuration, refresh_database=PacmanSynchronization.Disabled)
+
     package_list = []
     for full_path in filter(package_like, configuration.repository_paths.repository.iterdir()):
-        base = Package.from_archive(full_path)
+        base = Package.from_archive(full_path, pacman)
         for package, description in base.packages.items():
             package_list.append({
                 "check_depends": description.check_depends,

src/ahriman/core/database/migrations/m008_packagers.py

@@ -19,9 +19,11 @@
 #
 from sqlite3 import Connection
 
+from ahriman.core.alpm.pacman import Pacman
 from ahriman.core.configuration import Configuration
 from ahriman.core.utils import package_like
 from ahriman.models.package import Package
+from ahriman.models.pacman_synchronization import PacmanSynchronization
 
 
 __all__ = ["migrate_data", "steps"]
@@ -62,9 +64,12 @@ def migrate_package_base_packager(connection: Connection, configuration: Configu
     if not configuration.repository_paths.repository.is_dir():
         return
 
+    _, repository_id = configuration.check_loaded()
+    pacman = Pacman(repository_id, configuration, refresh_database=PacmanSynchronization.Disabled)
+
     package_list = []
     for full_path in filter(package_like, configuration.repository_paths.repository.iterdir()):
-        package = Package.from_archive(full_path)
+        package = Package.from_archive(full_path, pacman)
         package_list.append({
             "package_base": package.base,
             "packager": package.packager,

src/ahriman/core/database/migrations/m016_archive.py

@@ -20,11 +20,13 @@
 from dataclasses import replace
 from sqlite3 import Connection
 
+from ahriman.core.alpm.pacman import Pacman
 from ahriman.core.configuration import Configuration
 from ahriman.core.repository import Explorer
 from ahriman.core.sign.gpg import GPG
 from ahriman.core.utils import atomic_move, package_like, symlink_relative
 from ahriman.models.package import Package
+from ahriman.models.pacman_synchronization import PacmanSynchronization
 from ahriman.models.repository_paths import RepositoryPaths
 
 
@@ -43,27 +45,29 @@ def migrate_data(connection: Connection, configuration: Configuration) -> None:
 
     for repository_id in Explorer.repositories_extract(configuration):
         paths = replace(configuration.repository_paths, repository_id=repository_id)
+        pacman = Pacman(repository_id, configuration, refresh_database=PacmanSynchronization.Disabled)
 
         # create archive directory if required
         if not paths.archive.is_dir():
             with paths.preserve_owner():
                 paths.archive.mkdir(mode=0o755, parents=True)
 
-        move_packages(paths)
+        move_packages(paths, pacman)
 
 
-def move_packages(repository_paths: RepositoryPaths) -> None:
+def move_packages(repository_paths: RepositoryPaths, pacman: Pacman) -> None:
     """
     move packages from repository to archive and create symbolic links
 
     Args:
         repository_paths(RepositoryPaths): repository paths instance
+        pacman(Pacman): alpm wrapper instance
     """
     for archive in filter(package_like, repository_paths.repository.iterdir()):
         if not archive.is_file(follow_symlinks=False):
             continue  # skip symbolic links if any
 
-        package = Package.from_archive(archive)
+        package = Package.from_archive(archive, pacman)
         artifacts = [archive]
         # check if there are signatures for this package and append it here too
         if (signature := GPG.signature(archive)).exists():

src/ahriman/core/formatters/__init__.py

@@ -26,7 +26,6 @@ from ahriman.core.formatters.event_stats_printer import EventStatsPrinter
 from ahriman.core.formatters.package_printer import PackagePrinter
 from ahriman.core.formatters.package_stats_printer import PackageStatsPrinter
 from ahriman.core.formatters.patch_printer import PatchPrinter
-from ahriman.core.formatters.pkgbuild_printer import PkgbuildPrinter
 from ahriman.core.formatters.printer import Printer
 from ahriman.core.formatters.repository_printer import RepositoryPrinter
 from ahriman.core.formatters.repository_stats_printer import RepositoryStatsPrinter

src/ahriman/core/formatters/changes_printer.py

@@ -45,7 +45,7 @@ class ChangesPrinter(Printer):
         Returns:
             list[Property]: list of content properties
         """
-        if self.changes.changes is None:
+        if self.changes.is_empty:
             return []
         return [Property("", self.changes.changes, is_required=True, indent=0)]
 
@@ -57,6 +57,6 @@ class ChangesPrinter(Printer):
         Returns:
             str | None: content title if it can be generated and ``None`` otherwise
         """
-        if self.changes.changes is None:
+        if self.changes.is_empty:
             return None
         return self.changes.last_commit_sha

src/ahriman/core/formatters/pkgbuild_printer.py

@@ -1,62 +0,0 @@
-#
-# Copyright (c) 2021-2026 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-from ahriman.core.formatters.printer import Printer
-from ahriman.models.changes import Changes
-from ahriman.models.property import Property
-
-
-class PkgbuildPrinter(Printer):
-    """
-    print content of the pkgbuild stored in changes
-
-    Attributes:
-        changes(Changes): package changes
-    """
-
-    def __init__(self, changes: Changes) -> None:
-        """
-        Args:
-            changes(Changes): package changes
-        """
-        Printer.__init__(self)
-        self.changes = changes
-
-    def properties(self) -> list[Property]:
-        """
-        convert content into printable data
-
-        Returns:
-            list[Property]: list of content properties
-        """
-        if self.changes.pkgbuild is None:
-            return []
-        return [Property("", self.changes.pkgbuild, is_required=True, indent=0)]
-
-    # pylint: disable=redundant-returns-doc
-    def title(self) -> str | None:
-        """
-        generate entry title from content
-
-        Returns:
-            str | None: content title if it can be generated and ``None`` otherwise
-        """
-        if self.changes.pkgbuild is None:
-            return None
-        return self.changes.last_commit_sha

src/ahriman/core/housekeeping/archive_rotation_trigger.py

@@ -17,10 +17,14 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
+from collections.abc import Callable
+from functools import cmp_to_key
+
 from ahriman.core import context
+from ahriman.core.alpm.pacman import Pacman
 from ahriman.core.configuration import Configuration
-from ahriman.core.repository import Repository
 from ahriman.core.triggers import Trigger
+from ahriman.core.utils import package_like
 from ahriman.models.package import Package
 from ahriman.models.repository_id import RepositoryId
 from ahriman.models.result import Result
@@ -74,20 +78,27 @@ class ArchiveRotationTrigger(Trigger):
         """
         return list(cls.CONFIGURATION_SCHEMA.keys())
 
-    def archives_remove(self, package: Package, repository: Repository) -> None:
+    def archives_remove(self, package: Package, pacman: Pacman) -> None:
         """
         remove older versions of the specified package
 
         Args:
             package(Package): package which has been updated to check for older versions
-            repository(Repository): repository instance
+            pacman(Pacman): alpm wrapper instance
         """
         # explicit guard to skip process in case if rotation is disabled
         # this guard is supposed to speedup process
         if self.keep_built_packages == 0:
             return
 
-        to_remove = repository.package_archives(package.base)
+        packages: dict[tuple[str, str], Package] = {}
+        # we can't use here load_archives, because it ignores versions
+        for full_path in filter(package_like, self.paths.archive_for(package.base).iterdir()):
+            local = Package.from_archive(full_path, pacman)
+            packages.setdefault((local.base, local.version), local).packages.update(local.packages)
+
+        comparator: Callable[[Package, Package], int] = lambda left, right: left.vercmp(right.version)
+        to_remove = sorted(packages.values(), key=cmp_to_key(comparator))
 
         for single in to_remove[:-self.keep_built_packages]:
             self.logger.info("removing version %s of package %s", single.version, single.base)
@@ -104,7 +115,7 @@ class ArchiveRotationTrigger(Trigger):
             packages(list[Package]): list of all available packages
         """
         ctx = context.get()
-        repository = ctx.get(Repository)
+        pacman = ctx.get(Pacman)
 
         for package in result.success:
-            self.archives_remove(package, repository)
+            self.archives_remove(package, pacman)

src/ahriman/core/repository/executor.py

@@ -57,7 +57,7 @@ class Executor(PackageInfo, Cleaner):
 
         for path in filter(package_like, archive.iterdir()):
             # check if package version is the same
-            built = Package.from_archive(path)
+            built = Package.from_archive(path, self.pacman)
             if built.version != package.version:
                 continue
 
@@ -117,7 +117,7 @@ class Executor(PackageInfo, Cleaner):
         else:
             built = task.build(path, PACKAGER=packager)
 
-        package.with_packages(built)
+        package.with_packages(built, self.pacman)
         for src in built:
             dst = self.paths.packages / src.name
             atomic_move(src, dst)

src/ahriman/core/repository/package_info.py

@@ -19,36 +19,23 @@
 #
 import copy
 
-from collections.abc import Callable, Iterable
-from functools import cmp_to_key
+from collections.abc import Iterable
 from pathlib import Path
 from tempfile import TemporaryDirectory
 
-from ahriman.core.alpm.pacman import Pacman
 from ahriman.core.build_tools.package_version import PackageVersion
 from ahriman.core.build_tools.sources import Sources
-from ahriman.core.configuration import Configuration
-from ahriman.core.log import LazyLogging
-from ahriman.core.status import Client
+from ahriman.core.repository.repository_properties import RepositoryProperties
 from ahriman.core.utils import package_like
 from ahriman.models.changes import Changes
 from ahriman.models.package import Package
 
 
-class PackageInfo(LazyLogging):
+class PackageInfo(RepositoryProperties):
     """
     handler for the package information
-
-    Attributes:
-        configuration(Configuration): configuration instance
-        pacman(Pacman): alpm wrapper instance
-        reporter(Client): build status reporter instance
     """
 
-    configuration: Configuration
-    pacman: Pacman
-    reporter: Client
-
     def full_depends(self, package: Package, packages: Iterable[Package]) -> list[str]:
         """
         generate full dependencies list including transitive dependencies
@@ -99,7 +86,7 @@ class PackageInfo(LazyLogging):
         # we are iterating over bases, not single packages
         for full_path in packages:
             try:
-                local = Package.from_archive(full_path)
+                local = Package.from_archive(full_path, self.pacman)
                 if (source := sources.get(local.base)) is not None:  # update source with remote
                     local.remote = source
 
@@ -115,29 +102,6 @@ class PackageInfo(LazyLogging):
                 self.logger.exception("could not load package from %s", full_path)
         return list(result.values())
 
-    def package_archives(self, package_base: str) -> list[Package]:
-        """
-        load list of packages known for this package base. This method unlike
-        :func:`ahriman.core.repository.package_info.PackageInfo.load_archives` scans archive directory and loads all
-        versions available for the ``package_base``
-
-        Args:
-            package_base(str): package base
-
-        Returns:
-            list[Package]: list of packages belonging to this base, sorted by version by ascension
-        """
-        paths = self.configuration.repository_paths
-
-        packages: dict[tuple[str, str], Package] = {}
-        # we can't use here load_archives, because it ignores versions
-        for full_path in filter(package_like, paths.archive_for(package_base).iterdir()):
-            local = Package.from_archive(full_path)
-            packages.setdefault((local.base, local.version), local).packages.update(local.packages)
-
-        comparator: Callable[[Package, Package], int] = lambda left, right: left.vercmp(right.version)
-        return sorted(packages.values(), key=cmp_to_key(comparator))
-
     def package_changes(self, package: Package, last_commit_sha: str) -> Changes | None:
         """
         extract package change for the package since last commit if available
@@ -152,7 +116,7 @@ class PackageInfo(LazyLogging):
         with TemporaryDirectory(ignore_cleanup_errors=True) as dir_name:
             dir_path = Path(dir_name)
             patches = self.reporter.package_patches_get(package.base, None)
-            current_commit_sha = Sources.load(dir_path, package, patches, self.configuration.repository_paths)
+            current_commit_sha = Sources.load(dir_path, package, patches, self.paths)
 
             if current_commit_sha != last_commit_sha:
                 return Sources.changes(dir_path, last_commit_sha)
@@ -168,7 +132,7 @@ class PackageInfo(LazyLogging):
         Returns:
             list[Package]: list of packages properties
         """
-        packages = self.load_archives(filter(package_like, self.configuration.repository_paths.repository.iterdir()))
+        packages = self.load_archives(filter(package_like, self.paths.repository.iterdir()))
         if filter_packages:
             packages = [package for package in packages if package.base in filter_packages]
 
@@ -181,7 +145,7 @@ class PackageInfo(LazyLogging):
         Returns:
             list[Path]: list of filenames from the directory
         """
-        return list(filter(package_like, self.configuration.repository_paths.packages.iterdir()))
+        return list(filter(package_like, self.paths.packages.iterdir()))
 
     def packages_depend_on(self, packages: list[Package], depends_on: Iterable[str] | None) -> list[Package]:
         """

src/ahriman/core/status/watcher.py

@@ -23,7 +23,6 @@ from typing import Any, Self
 
 from ahriman.core.exceptions import UnknownPackageError
 from ahriman.core.log import LazyLogging
-from ahriman.core.repository.package_info import PackageInfo
 from ahriman.core.status import Client
 from ahriman.models.build_status import BuildStatus, BuildStatusEnum
 from ahriman.models.changes import Changes
@@ -40,18 +39,15 @@ class Watcher(LazyLogging):
 
     Attributes:
         client(Client): reporter instance
-        package_info(PackageInfo): package info instance
         status(BuildStatus): daemon status
     """
 
-    def __init__(self, client: Client, package_info: PackageInfo) -> None:
+    def __init__(self, client: Client) -> None:
         """
         Args:
             client(Client): reporter instance
-            package_info(PackageInfo): package info instance
         """
         self.client = client
-        self.package_info = package_info
 
         self._lock = Lock()
         self._known: dict[str, tuple[Package, BuildStatus]] = {}
@@ -84,15 +80,6 @@ class Watcher(LazyLogging):
 
     logs_rotate: Callable[[int], None]
 
-    def package_archives(self, package_base: str) -> list[Package]:
-        """
-        get known package archives
-
-        Returns:
-            list[Package]: list of built package for this package base
-        """
-        return self.package_info.package_archives(package_base)
-
     package_changes_get: Callable[[str], Changes]
 
     package_changes_update: Callable[[str, Changes], None]

src/ahriman/models/changes.py

@@ -38,6 +38,16 @@ class Changes:
     changes: str | None = None
     pkgbuild: str | None = None
 
+    @property
+    def is_empty(self) -> bool:
+        """
+        validate that changes are not empty
+
+        Returns:
+            bool: ``True`` in case if changes are not set and ``False`` otherwise
+        """
+        return self.changes is None
+
     @classmethod
     def from_json(cls, dump: dict[str, Any]) -> Self:
         """

src/ahriman/models/package.py

@@ -26,7 +26,6 @@ from pyalpm import vercmp  # type: ignore[import-not-found]
 from typing import Any, Self
 
 from ahriman.core.alpm.pacman import Pacman
-from ahriman.core.alpm.pacman_handle import PacmanHandle
 from ahriman.core.alpm.remote import AUR, Official, OfficialSyncdb
 from ahriman.core.log import LazyLogging
 from ahriman.core.utils import dataclass_view, full_version, list_flatmap, parse_version, srcinfo_property_list
@@ -187,17 +186,18 @@ class Package(LazyLogging):
         return sorted(packages)
 
     @classmethod
-    def from_archive(cls, path: Path) -> Self:
+    def from_archive(cls, path: Path, pacman: Pacman) -> Self:
         """
         construct package properties from package archive
 
         Args:
             path(Path): path to package archive
+            pacman(Pacman): alpm wrapper instance
 
         Returns:
             Self: package properties
         """
-        package = PacmanHandle.ephemeral().package_load(path)
+        package = pacman.handle.load_pkg(str(path))
         description = PackageDescription.from_package(package, path)
         return cls(
             base=package.base or package.name,
@@ -400,16 +400,17 @@ class Package(LazyLogging):
         """
         return dataclass_view(self)
 
-    def with_packages(self, packages: Iterable[Path]) -> None:
+    def with_packages(self, packages: Iterable[Path], pacman: Pacman) -> None:
         """
         replace packages descriptions with ones from archives
 
         Args:
             packages(Iterable[Path]): paths to package archives
+            pacman(Pacman): alpm wrapper instance
         """
         self.packages = {}  # reset state
         for package in packages:
-            archive = self.from_archive(package)
+            archive = self.from_archive(package, pacman)
             if archive.base != self.base:
                 continue
 

src/ahriman/web/views/v1/packages/archives.py

@@ -1,65 +0,0 @@
-#
-# Copyright (c) 2021-2026 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-from aiohttp.web import Response
-from typing import ClassVar
-
-from ahriman.models.user_access import UserAccess
-from ahriman.web.apispec.decorators import apidocs
-from ahriman.web.schemas import PackageNameSchema, PackageSchema, RepositoryIdSchema
-from ahriman.web.views.base import BaseView
-from ahriman.web.views.status_view_guard import StatusViewGuard
-
-
-class Archives(StatusViewGuard, BaseView):
-    """
-       package archives web view
-
-       Attributes:
-           GET_PERMISSION(UserAccess): (class attribute) get permissions of self
-       """
-
-    GET_PERMISSION: ClassVar[UserAccess] = UserAccess.Reporter
-    ROUTES = ["/api/v1/packages/{package}/archives"]
-
-    @apidocs(
-        tags=["Packages"],
-        summary="Get package archives",
-        description="Retrieve built package archives for the base",
-        permission=GET_PERMISSION,
-        error_404_description="Package base and/or repository are unknown",
-        schema=PackageSchema(many=True),
-        match_schema=PackageNameSchema,
-        query_schema=RepositoryIdSchema,
-    )
-    async def get(self) -> Response:
-        """
-        get package changes
-
-        Returns:
-            Response: 200 with package change on success
-
-        Raises:
-            HTTPNotFound: if package base is unknown
-        """
-        package_base = self.request.match_info["package"]
-
-        archives = self.service(package_base=package_base).package_archives(package_base)
-
-        return self.json_response([archive.view() for archive in archives])

src/ahriman/web/web.py

@@ -23,14 +23,12 @@ import logging
 import socket
 
 from aiohttp.web import Application, normalize_path_middleware, run_app
-from pathlib import Path
 
 from ahriman.core.auth import Auth
 from ahriman.core.configuration import Configuration
 from ahriman.core.database import SQLite
 from ahriman.core.distributed import WorkersCache
 from ahriman.core.exceptions import InitializeError
-from ahriman.core.repository.package_info import PackageInfo
 from ahriman.core.spawn import Spawn
 from ahriman.core.status import Client
 from ahriman.core.status.watcher import Watcher
@@ -80,33 +78,6 @@ def _create_socket(configuration: Configuration, application: Application) -> so
     return sock
 
 
-def _create_watcher(path: Path, repository_id: RepositoryId) -> Watcher:
-    """
-    build watcher for selected repository
-
-    Args:
-        path(Path): path to configuration file
-        repository_id(RepositoryId): repository unique identifier
-
-    Returns:
-        Watcher: watcher instance
-    """
-    logging.getLogger(__name__).info("load repository %s", repository_id)
-    # load settings explicitly for architecture if any
-    configuration = Configuration.from_path(path, repository_id)
-
-    # load database instance, because it holds identifier
-    database = SQLite.load(configuration)
-    # explicitly load local client
-    client = Client.load(repository_id, configuration, database, report=False)
-
-    # load package info wrapper
-    package_info = PackageInfo()
-    package_info.configuration = configuration
-
-    return Watcher(client, package_info)
-
-
 async def _on_shutdown(application: Application) -> None:
     """
     web application shutdown handler
@@ -197,11 +168,18 @@ def setup_server(configuration: Configuration, spawner: Spawn, repositories: lis
     # package cache
     if not repositories:
         raise InitializeError("No repositories configured, exiting")
+    watchers: dict[RepositoryId, Watcher] = {}
     configuration_path, _ = configuration.check_loaded()
-    application[WatcherKey] = {
-        repository_id: _create_watcher(configuration_path, repository_id)
-        for repository_id in repositories
-    }
+    for repository_id in repositories:
+        application.logger.info("load repository %s", repository_id)
+        # load settings explicitly for architecture if any
+        repository_configuration = Configuration.from_path(configuration_path, repository_id)
+        # load database instance, because it holds identifier
+        database = SQLite.load(repository_configuration)
+        # explicitly load local client
+        client = Client.load(repository_id, repository_configuration, database, report=False)
+        watchers[repository_id] = Watcher(client)
+    application[WatcherKey] = watchers
     # workers cache
     application[WorkersKey] = WorkersCache(configuration)
     # process spawner

tests/ahriman/application/handlers/test_handler_pkgbuild.py

@@ -1,100 +0,0 @@
-import argparse
-import pytest
-
-from pytest_mock import MockerFixture
-
-from ahriman.application.handlers.pkgbuild import Pkgbuild
-from ahriman.core.configuration import Configuration
-from ahriman.core.database import SQLite
-from ahriman.core.repository import Repository
-from ahriman.models.action import Action
-from ahriman.models.changes import Changes
-
-
-def _default_args(args: argparse.Namespace) -> argparse.Namespace:
-    """
-    default arguments for these test cases
-
-    Args:
-        args(argparse.Namespace): command line arguments fixture
-
-    Returns:
-        argparse.Namespace: generated arguments for these test cases
-    """
-    args.action = Action.List
-    args.exit_code = False
-    args.package = "package"
-    return args
-
-
-def test_run(args: argparse.Namespace, configuration: Configuration, repository: Repository,
-             mocker: MockerFixture) -> None:
-    """
-    must run command
-    """
-    args = _default_args(args)
-    mocker.patch("ahriman.core.repository.Repository.load", return_value=repository)
-    application_mock = mocker.patch("ahriman.core.status.local_client.LocalClient.package_changes_get",
-                                    return_value=Changes("sha", "change", "pkgbuild content"))
-    check_mock = mocker.patch("ahriman.application.handlers.handler.Handler.check_status")
-    print_mock = mocker.patch("ahriman.core.formatters.Printer.print")
-
-    _, repository_id = configuration.check_loaded()
-    Pkgbuild.run(args, repository_id, configuration, report=False)
-    application_mock.assert_called_once_with(args.package)
-    check_mock.assert_called_once_with(False, True)
-    print_mock.assert_called_once_with(verbose=True, log_fn=pytest.helpers.anyvar(int), separator="")
-
-
-def test_run_empty_exception(args: argparse.Namespace, configuration: Configuration, repository: Repository,
-                             mocker: MockerFixture) -> None:
-    """
-    must raise ExitCode exception on empty pkgbuild result
-    """
-    args = _default_args(args)
-    args.exit_code = True
-    mocker.patch("ahriman.core.repository.Repository.load", return_value=repository)
-    mocker.patch("ahriman.core.status.local_client.LocalClient.package_changes_get", return_value=Changes())
-    check_mock = mocker.patch("ahriman.application.handlers.handler.Handler.check_status")
-
-    _, repository_id = configuration.check_loaded()
-    Pkgbuild.run(args, repository_id, configuration, report=False)
-    check_mock.assert_called_once_with(True, False)
-
-
-def test_run_remove(args: argparse.Namespace, configuration: Configuration, repository: Repository,
-                    mocker: MockerFixture) -> None:
-    """
-    must remove package pkgbuild
-    """
-    args = _default_args(args)
-    args.action = Action.Remove
-    mocker.patch("ahriman.core.repository.Repository.load", return_value=repository)
-    changes = Changes("sha", "change", "pkgbuild content")
-    mocker.patch("ahriman.core.status.local_client.LocalClient.package_changes_get", return_value=changes)
-    update_mock = mocker.patch("ahriman.core.status.local_client.LocalClient.package_changes_update")
-
-    _, repository_id = configuration.check_loaded()
-    Pkgbuild.run(args, repository_id, configuration, report=False)
-    update_mock.assert_called_once_with(args.package, Changes("sha", "change", None))
-
-
-def test_imply_with_report(args: argparse.Namespace, configuration: Configuration, database: SQLite,
-                           mocker: MockerFixture) -> None:
-    """
-    must create application object with native reporting
-    """
-    args = _default_args(args)
-    mocker.patch("ahriman.core.database.SQLite.load", return_value=database)
-    load_mock = mocker.patch("ahriman.core.repository.Repository.load")
-
-    _, repository_id = configuration.check_loaded()
-    Pkgbuild.run(args, repository_id, configuration, report=False)
-    load_mock.assert_called_once_with(repository_id, configuration, database, report=True, refresh_pacman_database=0)
-
-
-def test_disallow_multi_architecture_run() -> None:
-    """
-    must not allow multi architecture run
-    """
-    assert not Pkgbuild.ALLOW_MULTI_ARCHITECTURE_RUN

tests/ahriman/application/test_ahriman.py

@@ -309,44 +309,6 @@ def test_subparsers_package_changes_remove_package_changes(parser: argparse.Argu
     assert dir(args) == dir(reference_args)
 
 
-def test_subparsers_package_pkgbuild(parser: argparse.ArgumentParser) -> None:
-    """
-    package-pkgbuild command must imply action, exit code, lock, quiet, report and unsafe
-    """
-    args = parser.parse_args(["-a", "x86_64", "-r", "repo", "package-pkgbuild", "ahriman"])
-    assert args.action == Action.List
-    assert args.architecture == "x86_64"
-    assert not args.exit_code
-    assert args.lock is None
-    assert args.quiet
-    assert not args.report
-    assert args.repository == "repo"
-    assert args.unsafe
-
-
-def test_subparsers_package_pkgbuild_remove(parser: argparse.ArgumentParser) -> None:
-    """
-    package-pkgbuild-remove command must imply action, lock, quiet, report and unsafe
-    """
-    args = parser.parse_args(["-a", "x86_64", "-r", "repo", "package-pkgbuild-remove", "ahriman"])
-    assert args.action == Action.Remove
-    assert args.architecture == "x86_64"
-    assert args.lock is None
-    assert args.quiet
-    assert not args.report
-    assert args.repository == "repo"
-    assert args.unsafe
-
-
-def test_subparsers_package_pkgbuild_remove_package_pkgbuild(parser: argparse.ArgumentParser) -> None:
-    """
-    package-pkgbuild-remove must have same keys as package-pkgbuild
-    """
-    args = parser.parse_args(["-a", "x86_64", "-r", "repo", "package-pkgbuild-remove", "ahriman"])
-    reference_args = parser.parse_args(["-a", "x86_64", "-r", "repo", "package-pkgbuild", "ahriman"])
-    assert dir(args) == dir(reference_args)
-
-
 def test_subparsers_package_copy_option_architecture(parser: argparse.ArgumentParser) -> None:
     """
     package-copy command must correctly parse architecture list

tests/ahriman/core/alpm/test_pacman_handle.py

@@ -1,37 +0,0 @@
-import pytest
-
-from pathlib import Path
-from unittest.mock import MagicMock
-
-from ahriman.core.alpm.pacman_handle import PacmanHandle
-
-
-def test_package_load() -> None:
-    """
-    must load package from archive path
-    """
-    local = Path("local")
-    instance = PacmanHandle.ephemeral()
-    handle_mock = instance.handle = MagicMock()
-
-    instance.package_load(local)
-    handle_mock.load_pkg.assert_called_once_with(str(local))
-
-    PacmanHandle._ephemeral = None
-
-
-def test_getattr() -> None:
-    """
-    must proxy attribute access to underlying handle
-    """
-    instance = PacmanHandle.ephemeral()
-    assert instance.dbpath
-
-
-def test_getattr_not_found() -> None:
-    """
-    must raise AttributeError for missing handle attributes
-    """
-    instance = PacmanHandle.ephemeral()
-    with pytest.raises(AttributeError):
-        assert instance.random_attribute

tests/ahriman/core/database/migrations/test_m005_make_opt_depends.py

@@ -34,7 +34,8 @@ def test_migrate_package_depends(connection: Connection, configuration: Configur
     package_mock = mocker.patch("ahriman.models.package.Package.from_archive", return_value=package_ahriman)
 
     migrate_package_depends(connection, configuration)
-    package_mock.assert_called_once_with(package_ahriman.packages[package_ahriman.base].filepath)
+    package_mock.assert_called_once_with(
+        package_ahriman.packages[package_ahriman.base].filepath, pytest.helpers.anyvar(int))
     connection.executemany.assert_called_once_with(pytest.helpers.anyvar(str, strict=True), [{
         "make_depends": package_ahriman.packages[package_ahriman.base].make_depends,
         "opt_depends": package_ahriman.packages[package_ahriman.base].opt_depends,

tests/ahriman/core/database/migrations/test_m007_check_depends.py

@@ -34,7 +34,8 @@ def test_migrate_package_depends(connection: Connection, configuration: Configur
     package_mock = mocker.patch("ahriman.models.package.Package.from_archive", return_value=package_ahriman)
 
     migrate_package_check_depends(connection, configuration)
-    package_mock.assert_called_once_with(package_ahriman.packages[package_ahriman.base].filepath)
+    package_mock.assert_called_once_with(
+        package_ahriman.packages[package_ahriman.base].filepath, pytest.helpers.anyvar(int))
     connection.executemany.assert_called_once_with(pytest.helpers.anyvar(str, strict=True), [{
         "check_depends": package_ahriman.packages[package_ahriman.base].check_depends,
         "package": package_ahriman.base,

tests/ahriman/core/database/migrations/test_m008_packagers.py

@@ -34,7 +34,8 @@ def test_migrate_package_base_packager(connection: Connection, configuration: Co
     package_mock = mocker.patch("ahriman.models.package.Package.from_archive", return_value=package_ahriman)
 
     migrate_package_base_packager(connection, configuration)
-    package_mock.assert_called_once_with(package_ahriman.packages[package_ahriman.base].filepath)
+    package_mock.assert_called_once_with(
+        package_ahriman.packages[package_ahriman.base].filepath, pytest.helpers.anyvar(int))
     connection.executemany.assert_called_once_with(pytest.helpers.anyvar(str, strict=True), [{
         "package_base": package_ahriman.base,
         "packager": package_ahriman.packager,

tests/ahriman/core/database/migrations/test_m016_archive.py

@@ -7,6 +7,7 @@ from sqlite3 import Connection
 from typing import Any
 from unittest.mock import call as MockCall
 
+from ahriman.core.alpm.pacman import Pacman
 from ahriman.core.configuration import Configuration
 from ahriman.core.database.migrations.m016_archive import migrate_data, move_packages
 from ahriman.models.package import Package
@@ -27,12 +28,12 @@ def test_migrate_data(connection: Connection, configuration: Configuration, mock
 
     migrate_data(connection, configuration)
     migration_mock.assert_has_calls([
-        MockCall(replace(configuration.repository_paths, repository_id=repository))
+        MockCall(replace(configuration.repository_paths, repository_id=repository), pytest.helpers.anyvar(int))
         for repository in repositories
     ])
 
 
-def test_move_packages(repository_paths: RepositoryPaths, package_ahriman: Package,
+def test_move_packages(repository_paths: RepositoryPaths, pacman: Pacman, package_ahriman: Package,
                        mocker: MockerFixture) -> None:
     """
     must move packages to the archive directory
@@ -56,9 +57,9 @@ def test_move_packages(repository_paths: RepositoryPaths, package_ahriman: Packa
     move_mock = mocker.patch("ahriman.core.database.migrations.m016_archive.atomic_move")
     symlink_mock = mocker.patch("pathlib.Path.symlink_to")
 
-    move_packages(repository_paths)
+    move_packages(repository_paths, pacman)
     archive_mock.assert_has_calls([
-        MockCall(repository_paths.repository / filename)
+        MockCall(repository_paths.repository / filename, pacman)
         for filename in ("file.pkg.tar.xz", "file2.pkg.tar.xz")
     ])
     move_mock.assert_has_calls([

tests/ahriman/core/formatters/conftest.py

@@ -2,26 +2,24 @@ import pytest
 
 from pathlib import Path
 
-from ahriman.core.formatters import (
-    AurPrinter,
-    ChangesPrinter,
-    ConfigurationPathsPrinter,
-    ConfigurationPrinter,
-    EventStatsPrinter,
-    PackagePrinter,
-    PackageStatsPrinter,
-    PatchPrinter,
-    PkgbuildPrinter,
-    RepositoryPrinter,
-    RepositoryStatsPrinter,
-    StatusPrinter,
-    StringPrinter,
-    TreePrinter,
-    UpdatePrinter,
-    UserPrinter,
-    ValidationPrinter,
+from ahriman.core.formatters import \
+    AurPrinter, \
+    ChangesPrinter, \
+    ConfigurationPathsPrinter, \
+    ConfigurationPrinter, \
+    EventStatsPrinter, \
+    PackagePrinter, \
+    PackageStatsPrinter, \
+    PatchPrinter, \
+    RepositoryPrinter, \
+    RepositoryStatsPrinter, \
+    StatusPrinter, \
+    StringPrinter, \
+    TreePrinter, \
+    UpdatePrinter, \
+    UserPrinter, \
+    ValidationPrinter, \
     VersionPrinter
-)
 from ahriman.models.aur_package import AURPackage
 from ahriman.models.build_status import BuildStatus
 from ahriman.models.changes import Changes
@@ -57,17 +55,6 @@ def changes_printer() -> ChangesPrinter:
     return ChangesPrinter(Changes("sha", "changes"))
 
 
-@pytest.fixture
-def pkgbuild_printer() -> PkgbuildPrinter:
-    """
-    fixture for pkgbuild printer
-
-    Returns:
-        PkgbuildPrinter: pkgbuild printer test instance
-    """
-    return PkgbuildPrinter(Changes("sha", "changes", "pkgbuild content"))
-
-
 @pytest.fixture
 def configuration_paths_printer() -> ConfigurationPathsPrinter:
     """

tests/ahriman/core/formatters/test_pkgbuild_printer.py

@@ -1,32 +0,0 @@
-from ahriman.core.formatters import PkgbuildPrinter
-from ahriman.models.changes import Changes
-
-
-def test_properties(pkgbuild_printer: PkgbuildPrinter) -> None:
-    """
-    must return non-empty properties list
-    """
-    assert pkgbuild_printer.properties()
-
-
-def test_properties_empty() -> None:
-    """
-    must return empty properties list if pkgbuild is empty
-    """
-    assert not PkgbuildPrinter(Changes()).properties()
-    assert not PkgbuildPrinter(Changes("sha", "changes")).properties()
-
-
-def test_title(pkgbuild_printer: PkgbuildPrinter) -> None:
-    """
-    must return non-empty title
-    """
-    assert pkgbuild_printer.title()
-
-
-def test_title_empty() -> None:
-    """
-    must return empty title if change is empty
-    """
-    assert not PkgbuildPrinter(Changes()).title()
-    assert not PkgbuildPrinter(Changes("sha")).title()

tests/ahriman/core/housekeeping/test_archive_rotation_trigger.py

@@ -3,11 +3,12 @@ import pytest
 from dataclasses import replace
 from pathlib import Path
 from pytest_mock import MockerFixture
+from typing import Any
 from unittest.mock import call as MockCall
 
+from ahriman.core.alpm.pacman import Pacman
 from ahriman.core.configuration import Configuration
 from ahriman.core.housekeeping import ArchiveRotationTrigger
-from ahriman.core.repository import Repository
 from ahriman.models.package import Package
 from ahriman.models.result import Result
 
@@ -20,24 +21,26 @@ def test_configuration_sections(configuration: Configuration) -> None:
 
 
 def test_archives_remove(archive_rotation_trigger: ArchiveRotationTrigger, package_ahriman: Package,
-                         repository: Repository, mocker: MockerFixture) -> None:
+                         pacman: Pacman, mocker: MockerFixture) -> None:
     """
     must remove older packages
     """
-    packages = []
-    for i in range(5):
-        generated = replace(package_ahriman, version=str(i))
+    def package(version: Any, *args: Any, **kwargs: Any) -> Package:
+        generated = replace(package_ahriman, version=str(version))
         generated.packages = {
-            key: replace(value, filename=str(i))
+            key: replace(value, filename=str(version))
             for key, value in generated.packages.items()
         }
-        packages.append(generated)
+        return generated
 
-    mocker.patch("ahriman.core.repository.package_info.PackageInfo.package_archives", return_value=packages)
+    mocker.patch("pathlib.Path.is_dir", return_value=True)
+    mocker.patch("ahriman.core.housekeeping.archive_rotation_trigger.package_like", return_value=True)
     mocker.patch("pathlib.Path.glob", return_value=[Path(str(i)) for i in range(5)])
+    mocker.patch("pathlib.Path.iterdir", return_value=[Path(str(i)) for i in range(5)])
+    mocker.patch("ahriman.models.package.Package.from_archive", side_effect=package)
     unlink_mock = mocker.patch("pathlib.Path.unlink", autospec=True)
 
-    archive_rotation_trigger.archives_remove(package_ahriman, repository)
+    archive_rotation_trigger.archives_remove(package_ahriman, pacman)
     unlink_mock.assert_has_calls([
         MockCall(Path("0")),
         MockCall(Path("1")),
@@ -45,15 +48,28 @@ def test_archives_remove(archive_rotation_trigger: ArchiveRotationTrigger, packa
 
 
 def test_archives_remove_keep(archive_rotation_trigger: ArchiveRotationTrigger, package_ahriman: Package,
-                              repository: Repository, mocker: MockerFixture) -> None:
+                              pacman: Pacman, mocker: MockerFixture) -> None:
     """
     must keep all packages if set to
     """
-    archives_mock = mocker.patch("ahriman.core.repository.package_info.PackageInfo.package_archives")
+    def package(version: Any, *args: Any, **kwargs: Any) -> Package:
+        generated = replace(package_ahriman, version=str(version))
+        generated.packages = {
+            key: replace(value, filename=str(version))
+            for key, value in generated.packages.items()
+        }
+        return generated
+
+    mocker.patch("pathlib.Path.is_dir", return_value=True)
+    mocker.patch("ahriman.core.housekeeping.archive_rotation_trigger.package_like", return_value=True)
+    mocker.patch("pathlib.Path.glob", return_value=[Path(str(i)) for i in range(5)])
+    mocker.patch("pathlib.Path.iterdir", return_value=[Path(str(i)) for i in range(5)])
+    mocker.patch("ahriman.models.package.Package.from_archive", side_effect=package)
+    unlink_mock = mocker.patch("pathlib.Path.unlink", autospec=True)
 
     archive_rotation_trigger.keep_built_packages = 0
-    archive_rotation_trigger.archives_remove(package_ahriman, repository)
-    archives_mock.assert_not_called()
+    archive_rotation_trigger.archives_remove(package_ahriman, pacman)
+    unlink_mock.assert_not_called()
 
 
 def test_on_result(archive_rotation_trigger: ArchiveRotationTrigger, package_ahriman: Package,

tests/ahriman/core/repository/conftest.py

@@ -6,6 +6,7 @@ from ahriman.core.configuration import Configuration
 from ahriman.core.database import SQLite
 from ahriman.core.repository.cleaner import Cleaner
 from ahriman.core.repository.executor import Executor
+from ahriman.core.repository.package_info import PackageInfo
 from ahriman.core.repository.update_handler import UpdateHandler
 from ahriman.models.pacman_synchronization import PacmanSynchronization
 
@@ -49,6 +50,23 @@ def executor(configuration: Configuration, database: SQLite, mocker: MockerFixtu
                     refresh_pacman_database=PacmanSynchronization.Disabled)
 
 
+@pytest.fixture
+def package_info(configuration: Configuration, database: SQLite) -> PackageInfo:
+    """
+    fixture for package info
+
+    Args:
+        configuration(Configuration): configuration fixture
+        database(SQLite): database fixture
+
+    Returns:
+        PackageInfo: package info test instance
+    """
+    _, repository_id = configuration.check_loaded()
+    return PackageInfo(repository_id, configuration, database, report=False,
+                       refresh_pacman_database=PacmanSynchronization.Disabled)
+
+
 @pytest.fixture
 def update_handler(configuration: Configuration, database: SQLite, mocker: MockerFixture) -> UpdateHandler:
     """

tests/ahriman/core/repository/test_executor.py

@@ -118,7 +118,7 @@ def test_package_build(executor: Executor, package_ahriman: Package, mocker: Moc
     init_mock.assert_called_once_with(pytest.helpers.anyvar(int), pytest.helpers.anyvar(int), None)
     package_mock.assert_called_once_with(Path("local"), executor.architecture, None)
     lookup_mock.assert_called_once_with(package_ahriman)
-    with_packages_mock.assert_called_once_with([Path(package_ahriman.base)])
+    with_packages_mock.assert_called_once_with([Path(package_ahriman.base)], executor.pacman)
     rename_mock.assert_called_once_with(Path(package_ahriman.base), executor.paths.packages / package_ahriman.base)
 
 

tests/ahriman/core/repository/test_package_info.py

@@ -4,12 +4,12 @@ from pathlib import Path
 from pytest_mock import MockerFixture
 from unittest.mock import MagicMock
 
-from ahriman.core.repository import Repository
+from ahriman.core.repository.package_info import PackageInfo
 from ahriman.models.changes import Changes
 from ahriman.models.package import Package
 
 
-def test_full_depends(repository: Repository, package_ahriman: Package, package_python_schedule: Package,
+def test_full_depends(package_info: PackageInfo, package_ahriman: Package, package_python_schedule: Package,
                       pyalpm_package_ahriman: MagicMock) -> None:
     """
     must extract all dependencies from the package
@@ -18,18 +18,18 @@ def test_full_depends(repository: Repository, package_ahriman: Package, package_
 
     database_mock = MagicMock()
     database_mock.pkgcache = [pyalpm_package_ahriman]
-    repository.pacman = MagicMock()
-    repository.pacman.handle.get_syncdbs.return_value = [database_mock]
+    package_info.pacman = MagicMock()
+    package_info.pacman.handle.get_syncdbs.return_value = [database_mock]
 
-    assert repository.full_depends(package_ahriman, [package_python_schedule]) == package_ahriman.depends
+    assert package_info.full_depends(package_ahriman, [package_python_schedule]) == package_ahriman.depends
 
     package_python_schedule.packages[package_python_schedule.base].depends = [package_ahriman.base]
     expected = sorted(set(package_python_schedule.depends + package_ahriman.depends))
-    assert repository.full_depends(package_python_schedule, [package_python_schedule]) == expected
+    assert package_info.full_depends(package_python_schedule, [package_python_schedule]) == expected
 
 
-def test_load_archives(repository: Repository, package_ahriman: Package, package_python_schedule: Package,
-                       mocker: MockerFixture) -> None:
+def test_load_archives(package_ahriman: Package, package_python_schedule: Package,
+                       package_info: PackageInfo, mocker: MockerFixture) -> None:
     """
     must return all packages grouped by package base
     """
@@ -45,7 +45,7 @@ def test_load_archives(repository: Repository, package_ahriman: Package, package
         (package_ahriman, None),
     ])
 
-    packages = repository.load_archives([Path("a.pkg.tar.xz"), Path("b.pkg.tar.xz"), Path("c.pkg.tar.xz")])
+    packages = package_info.load_archives([Path("a.pkg.tar.xz"), Path("b.pkg.tar.xz"), Path("c.pkg.tar.xz")])
     assert len(packages) == 2
     assert {package.base for package in packages} == {package_ahriman.base, package_python_schedule.base}
 
@@ -56,22 +56,22 @@ def test_load_archives(repository: Repository, package_ahriman: Package, package
     assert set(archives) == expected
 
 
-def test_load_archives_failed(repository: Repository, mocker: MockerFixture) -> None:
+def test_load_archives_failed(package_info: PackageInfo, mocker: MockerFixture) -> None:
     """
     must skip packages which cannot be loaded
     """
     mocker.patch("ahriman.models.package.Package.from_archive", side_effect=Exception)
-    assert not repository.load_archives([Path("a.pkg.tar.xz")])
+    assert not package_info.load_archives([Path("a.pkg.tar.xz")])
 
 
-def test_load_archives_not_package(repository: Repository) -> None:
+def test_load_archives_not_package(package_info: PackageInfo) -> None:
     """
     must skip not packages from iteration
     """
-    assert not repository.load_archives([Path("a.tar.xz")])
+    assert not package_info.load_archives([Path("a.tar.xz")])
 
 
-def test_load_archives_different_version(repository: Repository, package_python_schedule: Package,
+def test_load_archives_different_version(package_info: PackageInfo, package_python_schedule: Package,
                                          mocker: MockerFixture) -> None:
     """
     must load packages with different versions choosing maximal
@@ -86,36 +86,12 @@ def test_load_archives_different_version(repository: Repository, package_python_
     single_packages[0].version = "0.0.1-1"
     mocker.patch("ahriman.models.package.Package.from_archive", side_effect=single_packages)
 
-    packages = repository.load_archives([Path("a.pkg.tar.xz"), Path("b.pkg.tar.xz")])
+    packages = package_info.load_archives([Path("a.pkg.tar.xz"), Path("b.pkg.tar.xz")])
     assert len(packages) == 1
     assert packages[0].version == package_python_schedule.version
 
 
-def test_package_archives(repository: Repository, package_ahriman: Package, mocker: MockerFixture) -> None:
-    """
-    must load package archives sorted by version
-    """
-    from dataclasses import replace
-    from typing import Any
-
-    def package(version: Any, *args: Any, **kwargs: Any) -> Package:
-        generated = replace(package_ahriman, version=str(version))
-        generated.packages = {
-            key: replace(value, filename=str(version))
-            for key, value in generated.packages.items()
-        }
-        return generated
-
-    mocker.patch("ahriman.core.repository.package_info.package_like", return_value=True)
-    mocker.patch("pathlib.Path.iterdir", return_value=[Path(str(i)) for i in range(5)])
-    mocker.patch("ahriman.models.package.Package.from_archive", side_effect=package)
-
-    result = repository.package_archives(package_ahriman.base)
-    assert len(result) == 5
-    assert [p.version for p in result] == [str(i) for i in range(5)]
-
-
-def test_package_changes(repository: Repository, package_ahriman: Package, mocker: MockerFixture) -> None:
+def test_package_changes(package_info: PackageInfo, package_ahriman: Package, mocker: MockerFixture) -> None:
     """
     must load package changes
     """
@@ -123,24 +99,23 @@ def test_package_changes(repository: Repository, package_ahriman: Package, mocke
     load_mock = mocker.patch("ahriman.core.build_tools.sources.Sources.load", return_value="sha2")
     changes_mock = mocker.patch("ahriman.core.build_tools.sources.Sources.changes", return_value=changes)
 
-    assert repository.package_changes(package_ahriman, changes.last_commit_sha) == changes
-    load_mock.assert_called_once_with(
-        pytest.helpers.anyvar(int), package_ahriman, [], repository.configuration.repository_paths)
+    assert package_info.package_changes(package_ahriman, changes.last_commit_sha) == changes
+    load_mock.assert_called_once_with(pytest.helpers.anyvar(int), package_ahriman, [], package_info.paths)
     changes_mock.assert_called_once_with(pytest.helpers.anyvar(int), changes.last_commit_sha)
 
 
-def test_package_changes_skip(repository: Repository, package_ahriman: Package, mocker: MockerFixture) -> None:
+def test_package_changes_skip(package_info: PackageInfo, package_ahriman: Package, mocker: MockerFixture) -> None:
     """
     must skip loading package changes if no new commits
     """
     mocker.patch("ahriman.core.build_tools.sources.Sources.load", return_value="sha")
     changes_mock = mocker.patch("ahriman.core.build_tools.sources.Sources.changes")
 
-    assert repository.package_changes(package_ahriman, "sha") is None
+    assert package_info.package_changes(package_ahriman, "sha") is None
     changes_mock.assert_not_called()
 
 
-def test_packages(repository: Repository, package_ahriman: Package, package_python_schedule: Package,
+def test_packages(package_info: PackageInfo, package_ahriman: Package, package_python_schedule: Package,
                   mocker: MockerFixture) -> None:
     """
     must return repository packages
@@ -148,12 +123,12 @@ def test_packages(repository: Repository, package_ahriman: Package, package_pyth
     mocker.patch("pathlib.Path.iterdir")
     load_mock = mocker.patch("ahriman.core.repository.package_info.PackageInfo.load_archives",
                              return_value=[package_ahriman, package_python_schedule])
-    assert repository.packages() == [package_ahriman, package_python_schedule]
+    assert package_info.packages() == [package_ahriman, package_python_schedule]
     # it uses filter object, so we cannot verify argument list =/
     load_mock.assert_called_once_with(pytest.helpers.anyvar(int))
 
 
-def test_packages_filter(repository: Repository, package_ahriman: Package, package_python_schedule: Package,
+def test_packages_filter(package_info: PackageInfo, package_ahriman: Package, package_python_schedule: Package,
                          mocker: MockerFixture) -> None:
     """
     must filter result by bases
@@ -161,33 +136,33 @@ def test_packages_filter(repository: Repository, package_ahriman: Package, packa
     mocker.patch("pathlib.Path.iterdir")
     mocker.patch("ahriman.core.repository.package_info.PackageInfo.load_archives",
                  return_value=[package_ahriman, package_python_schedule])
-    assert repository.packages([package_ahriman.base]) == [package_ahriman]
+    assert package_info.packages([package_ahriman.base]) == [package_ahriman]
 
 
-def test_packages_built(repository: Repository, mocker: MockerFixture) -> None:
+def test_packages_built(package_info: PackageInfo, mocker: MockerFixture) -> None:
     """
     must return build packages
     """
     mocker.patch("pathlib.Path.iterdir", return_value=[Path("a.tar.xz"), Path("b.pkg.tar.xz")])
-    assert repository.packages_built() == [Path("b.pkg.tar.xz")]
+    assert package_info.packages_built() == [Path("b.pkg.tar.xz")]
 
 
-def test_packages_depend_on(repository: Repository, package_ahriman: Package, package_python_schedule: Package,
+def test_packages_depend_on(package_info: PackageInfo, package_ahriman: Package, package_python_schedule: Package,
                             mocker: MockerFixture) -> None:
     """
     must filter packages by depends list
     """
     mocker.patch("ahriman.core.repository.repository.Repository.packages",
                  return_value=[package_ahriman, package_python_schedule])
-    assert repository.packages_depend_on([package_ahriman], {"python-srcinfo"}) == [package_ahriman]
+    assert package_info.packages_depend_on([package_ahriman], {"python-srcinfo"}) == [package_ahriman]
 
 
-def test_packages_depend_on_empty(repository: Repository, package_ahriman: Package, package_python_schedule: Package,
+def test_packages_depend_on_empty(package_info: PackageInfo, package_ahriman: Package, package_python_schedule: Package,
                                   mocker: MockerFixture) -> None:
     """
     must return all packages in case if no filter is provided
     """
     mocker.patch("ahriman.core.repository.repository.Repository.packages",
                  return_value=[package_ahriman, package_python_schedule])
-    assert repository.packages_depend_on([package_ahriman, package_python_schedule], None) == \
+    assert package_info.packages_depend_on([package_ahriman, package_python_schedule], None) == \
         [package_ahriman, package_python_schedule]

tests/ahriman/models/conftest.py

@@ -89,6 +89,22 @@ def pkgbuild_ahriman(resource_path_root: Path) -> Pkgbuild:
     return Pkgbuild.from_file(pkgbuild)
 
 
+@pytest.fixture
+def pyalpm_handle(pyalpm_package_ahriman: MagicMock) -> MagicMock:
+    """
+    mock object for pyalpm
+
+    Args:
+        pyalpm_package_ahriman(MagicMock): mock object for pyalpm package
+
+    Returns:
+        MagicMock: pyalpm mock
+    """
+    mock = MagicMock()
+    mock.handle.load_pkg.return_value = pyalpm_package_ahriman
+    return mock
+
+
 @pytest.fixture
 def pyalpm_package_description_ahriman(package_description_ahriman: PackageDescription) -> MagicMock:
     """

tests/ahriman/models/test_changes.py

@@ -1,6 +1,17 @@
 from ahriman.models.changes import Changes
 
 
+def test_is_empty() -> None:
+    """
+    must check if changes are empty
+    """
+    assert Changes().is_empty
+    assert Changes("sha").is_empty
+
+    assert not Changes("sha", "change").is_empty
+    assert not Changes(None, "change").is_empty  # well, ok
+
+
 def test_changes_from_json_view() -> None:
     """
     must construct same object from json

tests/ahriman/models/test_package.py

@@ -148,14 +148,13 @@ def test_packages_full(package_ahriman: Package) -> None:
     assert package_ahriman.packages_full == [package_ahriman.base, f"{package_ahriman.base}-git"]
 
 
-def test_from_archive(package_ahriman: Package, pyalpm_package_ahriman: MagicMock, mocker: MockerFixture) -> None:
+def test_from_archive(package_ahriman: Package, pyalpm_handle: MagicMock, mocker: MockerFixture) -> None:
     """
     must construct package from alpm library
     """
-    mocker.patch("ahriman.core.alpm.pacman_handle.PacmanHandle.package_load", return_value=pyalpm_package_ahriman)
     mocker.patch("ahriman.models.package_description.PackageDescription.from_package",
                  return_value=package_ahriman.packages[package_ahriman.base])
-    generated = Package.from_archive(Path("path"))
+    generated = Package.from_archive(Path("path"), pyalpm_handle)
     generated.remote = package_ahriman.remote
 
     assert generated == package_ahriman
@@ -166,12 +165,13 @@ def test_from_archive_empty_base(package_ahriman: Package, pyalpm_package_ahrima
     """
     must construct package with empty base from alpm library
     """
+    pyalpm_handle = MagicMock()
     type(pyalpm_package_ahriman).base = PropertyMock(return_value=None)
-    mocker.patch("ahriman.core.alpm.pacman_handle.PacmanHandle.package_load", return_value=pyalpm_package_ahriman)
+    pyalpm_handle.handle.load_pkg.return_value = pyalpm_package_ahriman
 
     mocker.patch("ahriman.models.package_description.PackageDescription.from_package",
                  return_value=package_ahriman.packages[package_ahriman.base])
-    generated = Package.from_archive(Path("path"))
+    generated = Package.from_archive(Path("path"), pyalpm_handle)
     generated.remote = package_ahriman.remote
 
     assert generated == package_ahriman
@@ -362,7 +362,7 @@ def test_vercmp(package_ahriman: Package, mocker: MockerFixture) -> None:
     vercmp_mock.assert_called_once_with(package_ahriman.version, "version")
 
 
-def test_with_packages(package_ahriman: Package, package_python_schedule: Package,
+def test_with_packages(package_ahriman: Package, package_python_schedule: Package, pacman: Pacman,
                        mocker: MockerFixture) -> None:
     """
     must correctly replace packages descriptions
@@ -375,8 +375,8 @@ def test_with_packages(package_ahriman: Package, package_python_schedule: Packag
     result = copy.deepcopy(package_ahriman)
     package_ahriman.packages[package_ahriman.base].architecture = "i686"
 
-    result.with_packages(paths)
+    result.with_packages(paths, pacman)
     from_archive_mock.assert_has_calls([
-        MockCall(path) for path in paths
+        MockCall(path, pacman) for path in paths
     ])
     assert result.packages[result.base] == package_ahriman.packages[package_ahriman.base]