xync-db 0.2.2.dev17__tar.gz → 0.3.1__tar.gz

{xync_db-0.2.2.dev17 → xync_db-0.3.1}/PKG-INFO

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

xync_db-0.3.1/src/xync_db/migrations/models/0006_credex_orphan.py

@@ -0,0 +1,47 @@
+"""Per-CredEx exid-overwrite trigger → `credexs_orphan` channel.
+
+`CredEx.upsert` keys on (cred, ex) and overwrites `exid`. When it overwrites a REAL (non-zero) exid
+with a different value, the previously-registered credential keeps living on the exchange as an
+orphaned duplicate (the DB no longer tracks it). We don't silently leak it: a BEFORE-UPDATE trigger
+catches the change and emits `[person_id, ex_id, OLD.exid, OLD.cred_id]` so service-api can ask the
+owner whether to delete the stale one (handler `api.handlers.creds.credex_orphan`, mirror of the
+`is_you` consent flow). On "yes" the answer routes back through `CredEx.orphan_del` →
+`credexs_orphan_del` → `credex_orphan_del` handler → `acl.cred_del(OLD.exid)` + a success/error report.
+
+`OLD.cred_id` is in the payload because the orphan-consent question shows the credential's `detail`,
+which the handler can't recover from `[person_id, ex_id, OLD.exid]` alone: post-overwrite the (cred,
+ex) row already holds the NEW exid, and a person may own several creds on one exchange.
+
+Fires ONLY on a genuine real→different overwrite:
+  - `OLD.exid <> 0` skips the normal publish (stub 0 → real exid in `credex_new`);
+  - `NEW.exid <> OLD.exid` skips no-op re-writes (parse re-saving the same exid).
+The user re-registers immediately (the overwrite proceeds — `RETURN NEW`); orphan cleanup is async
+and consent-gated. person via the parent cred; NULL ⇒ no owner to ask ⇒ skip.
+"""
+
+from typing import ClassVar
+
+from tortoise.migrations.migration import Migration as BaseMigration
+from tortoise.migrations.operations import RunSQL
+
+CREDEX_ORPHAN_SQL = """
+CREATE OR REPLACE FUNCTION credex_orphan() returns trigger as $credex_orphan_trg$
+DECLARE
+    pid INT;
+BEGIN
+    IF OLD.exid <> 0 AND NEW.exid <> OLD.exid THEN
+        SELECT person_id INTO pid FROM cred WHERE id = OLD.cred_id;
+        IF pid IS NOT NULL THEN
+            PERFORM pg_notify('credexs_orphan', jsonb_build_array(pid, OLD.ex_id, OLD.exid, OLD.cred_id)::varchar);
+        END IF;
+    END IF;
+    RETURN NEW;
+END
+$credex_orphan_trg$ LANGUAGE plpgsql;
+CREATE OR REPLACE TRIGGER credex_orphan BEFORE UPDATE ON "credex" FOR EACH ROW EXECUTE FUNCTION credex_orphan();
+"""
+
+
+class Migration(BaseMigration):
+    dependencies: ClassVar = [("models", "0005_credex_del")]
+    operations: ClassVar = [RunSQL(CREDEX_ORPHAN_SQL)]

{xync_db-0.2.2.dev17 → xync_db-0.3.1}/src/xync_db/models/__init__.py

@@ -97,6 +97,11 @@ from xync_db.typs import HotAd, HotAds, OrderResult
 # both are stubbed out until the porting cleanup decides their final shape.
 # from xync_db.typs.db import AdNewFlatRaw
 
+# ex_id бирж с share-hex: у их объяв обязателен MyAd.hex, иначе url-кнопку на объяву не построить, поэтому
+# незашаренные объявы этих бирж прячем из выдачи тейкеру / алертим админу. Пока только Bybit (4); единственное
+# место смены — на случай добавления других share-бирж.
+SHARE_HEX_EX_IDS = [4]
+
 
 class Country(BaseCountry):
     cur: ForeignKeyRelation[Cur] = ForeignKeyField("models.Cur", "countries", on_update=CASCADE)
@@ -676,6 +681,11 @@ class User(TgUser, TsTrait):
     ) -> tuple[list[MyAd], list[MyAd], dict[int, float], dict[int, dict[int, int]]]:
         owner = {"ad__maker__person__user_id": self.id} if mine else {"ad__maker__person__user_id__not": self.id}
         adq = MyAd.hot_mads_query(ex_ids).filter(**owner)
+        if not mine:
+            # тейкеру прячем незашаренные объявы share-hex-бирж (hex IS NULL AND ex∈SHARE_HEX_EX_IDS): без
+            # share-hex url-кнопку не построить. Сужено до этих бирж, чтобы не зацепить hot-биржи без share-hex.
+            # Свои (mine, страница settings) оставляем — их видно для шеринга.
+            adq = adq.exclude(hex__isnull=True, ad__maker__ex_id__in=SHARE_HEX_EX_IDS)
         bads = await adq.filter(ad__pair_side__is_sell=False)
         # остаток каждой монеты (asset.free) у акторов персон этого юзера, по бирже и монете  # noqa: RUF003
         coin_balances: dict[int, dict[int, int]] = {}
@@ -1596,6 +1606,21 @@ class CredEx(Model):
         `acl.cred_del(exid)`. Каскадное удаление при сносе самого Cred идёт другим путём (`creds_del`)."""
         return bool(await cls.filter(cred_id=cred_id, ex_id=ex_id).delete())
 
+    @classmethod
+    async def orphan_del(cls, *, person_id: int, ex_id: int, exid: int) -> None:
+        """Удалить ОСИРОТЕВШИЙ exid на бирже — после согласия владельца (web-контроллер / бот-колбэк).
+        Перезапись exid у строки (cred, ex) на другой оставляет старый реквизит висеть на бирже дубликатом;
+        BEFORE-UPDATE триггер `credex_orphan` (миграция 0006) ловит это и шлёт `credexs_orphan` → handler
+        спрашивает владельца. На «да» зовут СЮДА: свой канал `credexs_orphan_del` (НЕ переиспользуем
+        `credexs_del`) — его handler `credex_orphan_del` делает `acl.cred_del(exid)` по (person, ex) И
+        возвращает владельцу результат (успех/ошибка) тем же Notify; у обычного `credex_del` отчёта нет,
+        поэтому каналы разделены. Строки CredEx для старого exid уже нет, и она не нужна. Единый эмиттер,
+        чтобы канал/полезную нагрузку не дублировать в web- и bot-зонах."""
+        await connections.get("default").execute_query(
+            "SELECT pg_notify('credexs_orphan_del', jsonb_build_array($1::int, $2::int, $3::bigint)::varchar)",
+            [person_id, ex_id, exid],
+        )
+
     class Meta:
         table_description = "Credential on Exchange"
         unique_together = (("ex", "exid"),)