xync-db 0.5.5.dev9__tar.gz → 0.5.5.dev11__tar.gz

{xync_db-0.5.5.dev9 → xync_db-0.5.5.dev11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xync-db
-Version: 0.5.5.dev9
+Version: 0.5.5.dev11
 Summary: Xync DB schema: Tortoise ORM models, enums, pydantic DTOs.
 Author-email: Mike Artemiev <mixartemev@gmail.com>
 License: LicenseRef-EULA

xync_db-0.5.5.dev11/src/xync_db/migrations/models/0012_country_ccn3.py

@@ -0,0 +1,42 @@
+"""`country.code` → `country.ccn3`: колонка была телефонным кодом (нигде в логике не читалась),
+становится числовым ISO 3166-1. Значения переустанавливает канонический сидер стран
+(`a_country_seed.country_seed`), который заодно доводит уже развёрнутую БД до полного набора
+mledoze (250 стран + 2 meta) с реальными валютами вместо USD-заглушек.
+
+Fresh-миграции получают колонку `ccn3` сразу из `0001_initial`, поэтому здесь — транзитное
+переименование ТОЛЬКО для БД, созданных до этого изменения (guarded: срабатывает лишь пока в
+таблице ещё есть колонка `code`; на свежей схеме — no-op, без повторного ресида).
+"""
+
+from typing import ClassVar
+
+from tortoise.migrations.migration import Migration as BaseMigration
+from tortoise.migrations.operations import RunPython
+
+from xync_db.migrations.models.seed.a_country_seed import country_seed
+
+
+async def country_ccn3(_apps, schema_editor) -> None:
+    db = schema_editor.client
+    has_code = await db.execute_query_dict(
+        "SELECT 1 FROM information_schema.columns WHERE table_name = 'country' AND column_name = 'code'"
+    )
+    if not has_code:  # свежая схема — колонка уже `ccn3` (0001), переносить нечего
+        return
+    await db.execute_script('ALTER TABLE "country" RENAME COLUMN "code" TO "ccn3";')
+    await country_seed(db)
+
+
+async def country_ccn3_down(_apps, schema_editor) -> None:
+    db = schema_editor.client
+    has_ccn3 = await db.execute_query_dict(
+        "SELECT 1 FROM information_schema.columns WHERE table_name = 'country' AND column_name = 'ccn3'"
+    )
+    if has_ccn3:
+        await db.execute_script('ALTER TABLE "country" RENAME COLUMN "ccn3" TO "code";')
+
+
+class Migration(BaseMigration):
+    atomic = False
+    dependencies: ClassVar = [("models", "0011_pair_scale")]
+    operations: ClassVar = [RunPython(country_ccn3, country_ccn3_down)]

{xync_db-0.5.5.dev9 → xync_db-0.5.5.dev11}/src/xync_db/models/CLAUDE.md

@@ -110,6 +110,13 @@ actor.set_user(username_id):
 ### `exclude(**kwargs)` negates each kwarg separately (NOT Django's `NOT(AND)`)
 Tortoise `QuerySet.exclude(a=1, b=2)` → `NOT(a=1) AND NOT(b=2)` = `a!=1 AND b!=2` (drops a row if EITHER holds), NOT Django's `NOT(a=1 AND b=2)`. Source: `tortoise.queryset.QuerySet._filter_or_exclude` appends `~Q(**{k:v})` per kwarg, AND-joined. To exclude the **intersection**, wrap in ONE `Q`: `exclude(Q(a=1, b=2))` → `NOT(a=1 AND b=2)`. Real bug: `User.get_hot_ads` `exclude(hex__isnull=True, ad__maker__ex_id__in=SHARE_HEX_EX_IDS)` hid all hex-less ads AND all share-hex ads separately instead of only the unshared-share-hex ones; fixed with `exclude(Q(...))` + `from tortoise.expressions import Q`.
 
+### Filtering by an m2m field multiplies rows — add `.distinct()`
+`filter(credexs__cred__ovr_pm_id=0)` is a JOIN through `myad_cred`, so a `MyAd` with TWO matching
+credentials comes back **twice**. Real bug: `/hot/ads?mine=true` showed every 2-payment ad doubled
+(the taker path hid it — `_dedup_hot_by_slot` collapses by slot) and `BaseAgentClient.ads_share`
+shared it twice. Fixed with `.distinct()` in `MyAd.hot_mads_query` (test `test_hot_mads_distinct.py`);
+`unshared_ad_ids` already had it. Rule: any queryset filtered through an m2m relation needs `.distinct()`.
+
 ### id=0 rows are unhashable — never `set[Model]`
 The XyncPay `Pm` row has **pk id=0** (norm='xyncpay'). Tortoise's `Model.__hash__` does `if not self.pk: raise TypeError(... unhashable)` — and `not 0` is `True`, so a fully-saved id=0 row is unhashable; putting it in a `set[Model]` (literal, `set(...)`, comprehension, or a pydantic `set[Model]` field) raises at construction. Rule: never build a `set` of model instances if any row can be id=0 (Pm especially, ovr_pm_id=0). Diff/dedup m2m by `pm.id` instead (`{p.id: p for p in pms}`, `{p.id for p in ...}`) — int sets are always hashable. Bit us in `Ad.upsert`/`_sync_pms` (the pms m2m path: `AdXIn.pms` is `list[Pm]` not `set`, diffs by id). Test `test_ad_upsert_pms_sync`.
 

{xync_db-0.5.5.dev9 → xync_db-0.5.5.dev11}/src/xync_db/models/__init__.py

@@ -1421,6 +1421,27 @@ class Ad(Model, TsTrait):
         "price",
     }
 
+    @classmethod
+    def maker_hot_ads(cls, maker_id: int, cur_id: int, coin_id: int, is_sell: bool):
+        """ВСЕ hot-объявы (с XyncPay-credex) мейкера в ОДНОМ направлении (пара+сторона) — ими ads_fresh
+        и обновляет цены, и считает занятость слотов. Два обязательных свойства выборки:
+          - `.distinct()`: credexs — m2m, объява с ДВУМЯ XyncPay-платёжками приезжает JOIN'ом дважды
+            (тот же класс бага, что чинил distinct в `hot_mads_query`);
+          - БЕЗ `.limit()`: срезать выборку бюджетом `same_dir_ad` нельзя — хвост направления тогда не
+            обновляется (висит на бирже со старым курсом) И не попадает в занятые слоты, а ads_fresh
+            досоздаёт в «свободный» слот дубль уже стоящей объявы (Bybit отбивает его 90043).
+        `exid=0` (заготовка, на бирже ещё нет) и `my_ad.blocked` — вне hot-оборота."""
+        return cls.filter(
+            exid__not=0,
+            my_ad__blocked=False,
+            my_ad__credexs__cred__ovr_pm_id=0,
+            my_ad__credexs__cred__pmcur__cur_id=cur_id,
+            pair_side__pair__coin_id=coin_id,
+            pair_side__pair__cur_id=cur_id,
+            pair_side__is_sell=is_sell,
+            maker_id=maker_id,
+        ).distinct()
+
     @staticmethod
     def slot_of(min_fiat: int, filtered: bool, minimum: int) -> tuple[int, int] | None:
         """Слот hot-объявы — пара `(множитель_мин_лимита, фильтрованность)` по её нижнему лимиту и
@@ -1567,8 +1588,12 @@ class MyAd(Model):  # Road
             fltr["ad__status"] = AdStatus.active
         if ex_ids:
             fltr["ad__maker__ex_id__in"] = ex_ids
-        return cls.filter(**fltr).prefetch_related(
-            "ad__pair_side__pair__coin", "ad__pair_side__pair__cur", "ad__maker__ex"
+        # distinct: credexs — m2m, и объява с двумя платёжками (2 credex с ovr_pm_id=0) иначе  # noqa: RUF003
+        # приезжает JOIN'ом дважды (дубли строк в /hot/ads?mine=true и двойной ads_share)  # noqa: RUF003
+        return (
+            cls.filter(**fltr)
+            .distinct()
+            .prefetch_related("ad__pair_side__pair__coin", "ad__pair_side__pair__cur", "ad__maker__ex")
         )
 
     @classmethod