mews.packages: improve text formatting

src/altrepo/parser/methods.py

@@ -34,14 +34,14 @@ class NewsInfo:
         if not url:
             return None
         html = await self.client.get(url, "koi8-r")
-        return await packages_parser(html)
+        return await packages_parser(html, url)
         
     async def p11(self) -> models.PackagesModel | None:
         url = (await self.news_urls()).p11
         if not url:
             return None
         html = await self.client.get(url, "koi8-r")
-        return await packages_parser(html)
+        return await packages_parser(html, url)
 
 
 class ALTRepoParser:

src/altrepo/parser/models.py

@@ -39,6 +39,7 @@ class PackageElementModel(BaseModel):
 
 
 class PackagesModel(BaseModel):
+    url: str
     added: List[PackageElementModel] | None = None
     removed: List[RemovedPackageElementModel] | None = None
     updated: List[PackageElementModel] | None = None

src/altrepo/parser/news/packages.py

@@ -4,7 +4,7 @@ import re
 
 from .. import models
 
-async def packages_parser(html: str):
+async def packages_parser(html: str, url:  str):
     soup = BeautifulSoup(html, "html.parser")
     pre_tag = soup.find("pre")
     if not pre_tag:
@@ -69,6 +69,8 @@ async def packages_parser(html: str):
     if current_package and current_section:
         sections[current_section].append(current_package)
     
+    sections["url"] = url
+
     return models.PackagesModel(**sections)
 
 

src/handlers/news.py

@@ -23,13 +23,15 @@ async def news_handler(m: Message, news_type: str | None, translate: str | None)
             packages_data = await altrepo.parser.news.sisyphus()
         elif news_type == "p11":
             packages_data = await altrepo.parser.news.p11()
+        else:
+            return
         
         if not packages_data:
             await m.answer("Новостей пока нет.")
             return
         
-        added, removed, updated = await format_packages(packages_data, bool(translate))
+        added, removed, updated, info_message = await format_packages(packages_data, bool(translate))
         
-        await m.answer(f"{added}{removed}")
+        await m.answer(f"{added}{removed}{info_message}")
         for msg in updated:
             res = await m.answer(msg)

src/services/news/packages.py

 from telegrinder.tools.formatting import HTMLFormatter, link, bold
 
+import random
+from collections import defaultdict
+
 from altrepo.parser.models import PackagesModel, PackageElementModel, RemovedPackageElementModel
 
 from config import PACKAGES_URL
@@ -8,6 +11,7 @@ from services.utils import translate_async, chunk_list
 
 repo = PACKAGES_URL.format(repo="sisyphus")
 
+
 async def format_packages(packages: PackagesModel, translate: bool | None = None):
 
     added_message = ""
@@ -25,13 +29,18 @@ async def format_packages(packages: PackagesModel, translate: bool | None = None
             removed_message += await _format_removed_package(package)
 
     if packages.updated:
-        for packages in chunk_list(packages.updated, 20):
-            updated_message = "Обновлены:\n"
-            for package in packages:
-                updated_message += await _format_package(package, translate)
-            updated_messages.append(updated_message)
+        updated_messages = await _format_updated_packages(packages, translate)
+
+    info_message = (
+        "Источник: " +
+        HTMLFormatter(link(packages.url, text="sisyphus-cybertalk")) +
+        "\n\n"
+        "Сформировано при помощи " +
+        HTMLFormatter(bold(link("https://t.me/altlinux_packages_bot", text="ALT Packages Bot")))
+    )
+
+    return added_message, removed_message, updated_messages, info_message
 
-    return added_message, removed_message, updated_messages
 
 async def _format_package(package: PackageElementModel, translate: bool):
     message = HTMLFormatter(
@@ -47,3 +56,39 @@ async def _format_removed_package(package: RemovedPackageElementModel):
     message = HTMLFormatter(
         bold(link(f"{repo}srpms/{package.name}", text=package.name))) + "\n\n"
     return message
+
+
+async def _format_updated_packages(packages_model: PackagesModel, translate: bool):
+    updated = packages_model.updated or []
+    total_updated = len(updated)
+
+    grouped = defaultdict(list)
+    for pkg in updated:
+        grouped[(pkg.maintainer_nick, pkg.maintainer_name)].append(pkg)
+    sorted_groups = sorted(grouped.items(), key=lambda x: x[0][0])
+
+    result_pages = []
+
+    for chunk in chunk_list(sorted_groups, 8):
+        lines = [f"Обновлено {total_updated} пакетов, в том числе:"]
+        for (nick, name), pkgs in chunk:
+            selected_pkgs = random.sample(pkgs, min(8, len(pkgs)))
+
+            maintainer_display = (
+                await translate_async(name) if translate else name
+            ) + f" ({nick})"
+
+            for pkg in selected_pkgs:
+                description = (
+                    await translate_async(pkg.description)
+                    if translate else pkg.description
+                )
+                pkg_link = link(f"{repo}srpms/{pkg.name}", text=pkg.name)
+                pkg_name = HTMLFormatter(bold(pkg_link))
+                lines.append(f"{pkg_name} — {description}")
+
+            lines.append(f"* {maintainer_display}\n")
+
+        result_pages.append("\n".join(lines).strip())
+
+    return result_pages