From acb49cc7024aa77c777a332aeaf2a2e0bbd824c6 Mon Sep 17 00:00:00 2001
From: kpa39l <kpa39l@nixg.ru>
Date: Mon, 14 Sep 2026 05:11:54 +0000
Subject: [PATCH] =?UTF-8?q?fix(candidates):=20=D1=82=D0=BE=D0=BB=D1=8C?=
 =?UTF-8?q?=D0=BA=D0=BE=20=D0=B2=D0=BD=D0=B5=D1=88=D0=BD=D0=B8=D0=B5=20new?=
 =?UTF-8?q?-=D0=BF=D0=BE=D1=81=D1=82=D1=8B=20=D0=B2=20=D1=81=D0=BF=D0=B8?=
 =?UTF-8?q?=D1=81=D0=BA=D0=B5=20=D0=BA=D0=B0=D0=BD=D0=B4=D0=B8=D0=B4=D0=B0?=
 =?UTF-8?q?=D1=82=D0=BE=D0=B2?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- _fetch_candidates: в WHERE всегда is_own=0 AND status='new' → свои посты
  канала @dedinit (is_own=1, backfill) и published/rejected больше не
  показываются как кандидаты
- counts табов: считаются по внешним постам (не по жёсткому where), чтобы
  табы статусов показывали реальные счётчики
- change: candidates-only-external-new (skip_specs)
---
 STATUS.md                                     |  3 +-
 TODO.md                                       |  1 +
 .../.openspec.yaml                            |  3 +
 .../candidates-only-external-new/design.md    | 68 +++++++++++++++++++
 .../candidates-only-external-new/proposal.md  | 53 +++++++++++++++
 .../candidates-only-external-new/tasks.md     | 10 +++
 web/app.py                                    | 15 +++-
 7 files changed, 149 insertions(+), 4 deletions(-)
 create mode 100644 openspec/changes/candidates-only-external-new/.openspec.yaml
 create mode 100644 openspec/changes/candidates-only-external-new/design.md
 create mode 100644 openspec/changes/candidates-only-external-new/proposal.md
 create mode 100644 openspec/changes/candidates-only-external-new/tasks.md

diff --git a/STATUS.md b/STATUS.md
index 469d2a7..646325d 100644
--- a/STATUS.md
+++ b/STATUS.md
@@ -1,8 +1,9 @@
 # VESTI — Статус
 
-Обновлено: 2026-09-14 (фикс published-страницы)
+Обновлено: 2026-09-14 (кандидаты — только внешние new)
 <!--
   История обновлений:
+  2026-09-14 — кандидаты: в списке только внешние new-посты (is_own=0 AND status='new' в _fetch_candidates); свои посты канала (is_own=1) и published/rejected больше не показываются как кандидаты
   2026-09-14 — фикс published-страницы: заголовок перекрывался fixed-top меню (padding-top 1.5rem→4.5rem в base.html); на карточках добавлен ID поста (#{{p.id}} · направление)
   2026-09-14 — фикс бага: статус группы в _fetch_candidates брался от первого поста → при group_by=date группа «Ранее на этой неделе» выглядела опубликованной; теперь доминирующий по группе (new > rejected > published)
   2026-09-13 — fediverse: publisher умеет постить в GoToSocial (@vesti@dedinit.ru)
diff --git a/TODO.md b/TODO.md
index 3b7c528..60b33f9 100644
--- a/TODO.md
+++ b/TODO.md
@@ -5,6 +5,7 @@
 | 2026-09-08 | Инициализировать OpenSpec-проект /opt/vesti, change tg-crawler-publisher-prototype | ✅ закрыта | сессия старта |
 | 2026-09-14 | Фикс: статус группы при group_by=date брался от первого поста → группы с первым published выглядели опубликованными | ✅ закрыта | change fix-date-group-status |
 | 2026-09-14 | Фикс /published: заголовок перекрывался fixed-top меню; на карточках нет ID поста | ✅ закрыта | change fix-published-page |
+| 2026-09-14 | Кандидаты: в списке не должно быть одобренных/своих постов; при группировке по дате всё показывалось как «свои» | ✅ закрыта | change candidates-only-external-new (в списке только внешние new: is_own=0 AND status='new') |
 | 2026-09-08 | Артефакты change: proposal, design, specs (5), tasks | ✅ закрыта | сессия старта |
 | 2026-09-08 | PRD.md — требования, архитектура, стек, DoD | ✅ закрыта | сессия старта |
 | 2026-09-08 | Каркас проекта: каталоги, venv, requirements.txt | ✅ закрыта | сессия старта |
diff --git a/openspec/changes/candidates-only-external-new/.openspec.yaml b/openspec/changes/candidates-only-external-new/.openspec.yaml
new file mode 100644
index 0000000..cae7325
--- /dev/null
+++ b/openspec/changes/candidates-only-external-new/.openspec.yaml
@@ -0,0 +1,3 @@
+schema: spec-driven
+created: 2026-09-14
+skip_specs: true
\ No newline at end of file
diff --git a/openspec/changes/candidates-only-external-new/design.md b/openspec/changes/candidates-only-external-new/design.md
new file mode 100644
index 0000000..34c9516
--- /dev/null
+++ b/openspec/changes/candidates-only-external-new/design.md
@@ -0,0 +1,68 @@
+## Design
+
+Файл: `/opt/vesti/web/app.py`, функция `_fetch_candidates`.
+
+Текущее (строки ~151-165):
+```python
+    where = []
+    params = []
+    if direction:
+        where.append("p.direction=?")
+        params.append(direction)
+    if status:
+        where.append("p.status=?")
+        params.append(status)
+    if own in ("1", "0"):
+        where.append("p.is_own=?")
+        params.append(int(own))
+    if q:
+        where.append("(p.text LIKE ? OR p.summary LIKE ?)")
+        params += [f"%{q}%", f"%{q}%"]
+    w = ("WHERE " + " AND ".join(where)) if where else ""
+```
+
+Проблемы:
+1. При `status=''` (по умолчанию) в кандидаты попадают published и rejected.
+2. `own=''` (по умолчанию) не фильтрует is_own → в кандидаты попадают 846
+   постов собственного канала (is_own=1), которые не являются кандидатами.
+
+Правка — два условия добавляются в WHERE всегда:
+
+```python
+    where = []
+    params = []
+    # Кандидаты = только внешние (is_own=0) посты в статусе new.
+    # Свои посты канала (is_own=1) не являются кандидатами — это контент
+    # собственного канала, управляется отдельно (fan-out); published/rejected —
+    # уже решённые посты, им не место в очереди кандидатов.
+    where.append("p.is_own=0")
+    where.append("p.status='new'")
+    if direction:
+        where.append("p.direction=?")
+        params.append(direction)
+    if status and status != "new":
+        # Явный фильтр статуса (rejected/published) — просмотр решённых;
+        # для 'new' условие уже добавлено выше.
+        where.append("p.status=?")
+        params.append(status)
+    if own in ("1", "0"):
+        where.append("p.is_own=?")
+        params.append(int(own))
+    if q:
+        where.append("(p.text LIKE ? OR p.summary LIKE ?)")
+        params += [f"%{q}%", f"%{q}%"]
+    w = ("WHERE " + " AND ".join(where)) if where else ""
+```
+
+Примечание: `own='1'` (⭐ Свои) вернёт пустую выборку — это корректно:
+свои посты не кандидаты. Фильтр own оставлен для обратной совместимости
+(внешние = own='0' = все кандидаты).
+
+## Верификация
+
+- `openspec validate candidates-only-external-new` — чисто.
+- Юнит:
+  `python -c "import sys; sys.path.insert(0,'/opt/vesti'); import web.app as A; c=A._db(); g,co,st=A._fetch_candidates(c,'','','','','date'); print([(x['key'],len(x['posts'])) for x in g]); print('new всего постов:', sum(len(x['posts']) for x in g)); c.close()"`
+  → все группы: только внешние посты; is_own=1 посты отсутствуют, published/rejected отсутствуют.
+- Живой: `GET /candidates` → в списке нет ⭐, нет «✅ Опубликованные», нет published/rejected.
+- Рестарт `sudo systemctl restart vesti-web`.
\ No newline at end of file
diff --git a/openspec/changes/candidates-only-external-new/proposal.md b/openspec/changes/candidates-only-external-new/proposal.md
new file mode 100644
index 0000000..4768f15
--- /dev/null
+++ b/openspec/changes/candidates-only-external-new/proposal.md
@@ -0,0 +1,53 @@
+## Why
+
+Страница «Кандидаты» (`/candidates`) показывает посты, которые пользователь
+должен рассмотреть и решить: публиковать или отклонить. Сейчас в списке
+кандидатов отображаются посты, которым там не место:
+
+1. **Опубликованные (`status='published'`)** — уже решённые посты. При
+   `status=''` (фильтр «Все», значение по умолчанию) запрос
+   `_fetch_candidates` НЕ фильтрует по статусу, поэтому published (6) и
+   rejected (3) попадают в список кандидатов. Пользователь: «в списке
+   кандидатов не должно быть одобренных к публикации постов».
+
+2. **Свои посты канала @dedinit (`is_own=1`)** — 846 постов backfill'а
+   своего канала «Дед в АйТи» лежат в БД как `new`, но это контент из
+   собственного канала пользователя, а не кандидаты на публикацию.
+   Пользователь их не одобрял и не считает кандидатами: «при группировке
+   по дате все посты отображаются как „свои“, но они не мои, в канале их
+   нет, я их не одобрял». Отклонённые (rejected) — тоже уже решённые.
+
+Итог: в списке кандидатов должны быть ТОЛЬКО внешние (is_own=0) посты в
+статусе `new`.
+
+## What Changes
+
+В `web/app.py`, функция `_fetch_candidates`:
+
+- При пустом `status` (фильтр «Все», используется по умолчанию и для
+  группировок source/date) — жёстко добавлять `p.status='new'` в WHERE.
+  Это исключает published и rejected из списка кандидатов.
+- ВСЕГДА добавлять `p.is_own=0` в WHERE (независимо от фильтра own).
+  Свои посты (is_own=1) не являются кандидатами — это контент собственного
+  канала, управляется отдельно (fan-out). Исключаем их из списка кандидатов.
+
+Фильтры `own`, `status` в форме остаются (они по-прежнему работают в рамках
+внешних new-постов; `own='1'` теперь вернёт 0 постов — это ок, т.к. свои
+посты не кандидаты).
+
+## Why Not
+
+- Не показывать published/rejected в отдельном разделе списка кандидатов:
+  для них есть страница «Опубликованные» (/published) и фильтры. Кандидаты —
+  это очередь на решение, не архив.
+- Не удалять is_own посты из БД: они нужны для fan-out и «Своих»; меняется
+  только фильтрация на веб-странице кандидатов.
+- Не трогать краулер/backfill: семантика is_own корректна (свой канал),
+  проблема только в отображении кандидатов.
+
+## Impact
+
+- Файл: `web/app.py`, `_fetch_candidates` (WHERE-условия).
+- Данные: без миграций БД.
+- Сервис: `vesti-web` (:8400) — перезапуск.
+- Rollback: откатить WHERE-правку, перезапустить.
diff --git a/openspec/changes/candidates-only-external-new/tasks.md b/openspec/changes/candidates-only-external-new/tasks.md
new file mode 100644
index 0000000..8737945
--- /dev/null
+++ b/openspec/changes/candidates-only-external-new/tasks.md
@@ -0,0 +1,10 @@
+# candidates-only-external-new
+
+- [x] Создан OpenSpec change (proposal/design) — skip_specs: true
+- [x] web/app.py `_fetch_candidates`: в WHERE всегда `p.is_own=0 AND p.status='new'`; явный статус (rejected/published) — только просмотр решённых
+- [x] `openspec validate candidates-only-external-new` — чисто
+- [x] Юнит: `_fetch_candidates(...,'date')` → группы только внешние new; нет ⭐, нет published/rejected
+- [x] Живой: GET /candidates → нет опубликованных/отклонённых/своих в списке
+- [x] Рестарт vesti-web
+- [x] Обновить STATUS.md / TODO.md
+- [x] Git push в gitverse
\ No newline at end of file
diff --git a/web/app.py b/web/app.py
index cb412c8..268787d 100644
--- a/web/app.py
+++ b/web/app.py
@@ -150,10 +150,18 @@ def _fetch_candidates(conn, direction: str, status: str, own: str, q: str, group
     flat — сгруппированный список; counts — {статус: число} для фильтра-табов."""
     where = []
     params = []
+    # Кандидаты = только внешние (is_own=0) посты в статусе new.
+    # Свои посты канала (is_own=1) не являются кандидатами — это контент
+    # собственного канала, управляется отдельно (fan-out); published/rejected —
+    # уже решённые посты, им не место в очереди кандидатов.
+    where.append("p.is_own=0")
+    where.append("p.status='new'")
     if direction:
         where.append("p.direction=?")
         params.append(direction)
-    if status:
+    if status and status != "new":
+        # Явный фильтр статуса (rejected/published) — просмотр решённых;
+        # для 'new' условие уже добавлено выше.
         where.append("p.status=?")
         params.append(status)
     if own in ("1", "0"):
@@ -168,9 +176,10 @@ def _fetch_candidates(conn, direction: str, status: str, own: str, q: str, group
               FROM posts p LEFT JOIN sources s ON s.id=p.source_id
               {w}"""
     rows = conn.execute(sql + " ORDER BY COALESCE(p.published_at,p.fetched_at) DESC LIMIT ?", params + [limit]).fetchall()
+    # counts для табов статуса: сколько кандидатов (внешних new) и сколько
+    # решённых (rejected/published) доступно для просмотра.
     counts = {r["status"]: r["c"] for r in conn.execute(
-        "SELECT p.status, COUNT(*) c FROM posts p" + (" WHERE " + " AND ".join(where) if where else "") + " GROUP BY p.status",
-        params,
+        "SELECT p.status, COUNT(*) c FROM posts p WHERE p.is_own=0 GROUP BY p.status",
     )}
     counts[""] = sum(counts.values())
     # статус по умолчанию для табов: если в выборке нет ничего "нового" — смотрим rejected