feat(candidates): флаг прочитанности (is_read), жирные непрочитанные, голубая заливка выбранного, дефолтная группировка по дате

This commit is contained in:
kpa39l
2026-09-15 16:42:55 +00:00
parent 562d787f1b
commit c09dff08ac
8 changed files with 211 additions and 5 deletions
+1
View File
@@ -159,6 +159,7 @@ CLASSIFY_TIMEOUT=20 .venv/bin/python -m classifier.classify --db db/vesti.db --l
- Тихие OpenSpec-change: `candidates-only-external-new`, `keep-candidates-context` — задачи [x], валидны; архивировать после подтверждения.
- Правый блок действий: кнопки «🤖 Обработать моделью / ✍️ Переписать / 🚫 Отклонить» вынесены в ОДИН ряд НАД полем комментария, «✅ Опубликовать» — в начале approve-формы, поле комментария + «💾 Сохранить комментарий» — ниже (change `candidates-buttons-order`).
- После «🚫 Отклонить» (и bulk reject/reject-old) — остаёмся на странице кандидатов в текущей группировке, НЕ уходим в «🗑 Отклонённые» (change `reject-stay-candidates`); отклонённые — через фильтр статуса.
- Список кандидатов: флаг «прочитан» — непрочитанные карточки жирные, выбранный пост — светло-голубая заливка (класс active-row), открытие поста помечает is_read=1; группировка по умолчанию — «дата» (хронология получения) (change `candidates-read-flag`).
## Ключевые артефакты
- /opt/vesti/AGENT.MD — правила проекта (обязательно: все изменения через OpenSpec)
@@ -0,0 +1,3 @@
schema: spec-driven
skip_specs: true
created: 2026-09-15
@@ -0,0 +1,117 @@
## Design
### 1. БД: колонка is_read
`db/schema.sql` — в CREATE TABLE posts добавить:
```sql
is_read INTEGER DEFAULT 0, -- 1 = пост просмотрен (открыт в правой панели)
```
Миграция существующей БД — при старте веб-приложения (идемпотентно):
в начале `web/app.py` (после `DB_PATH`) или в `_db()`:
```python
def _migrate():
"""Идемпотентные миграции при старте (веб-сервис)."""
conn = sqlite3.connect(DB_PATH, timeout=10)
cols = [r[1] for r in conn.execute("PRAGMA table_info(posts)").fetchall()]
if "is_read" not in cols:
conn.execute("ALTER TABLE posts ADD COLUMN is_read INTEGER DEFAULT 0")
conn.commit()
conn.close()
```
Вызвать один раз при импорте (перед `app = FastAPI(...)` или после).
### 2. Дефолтная группировка — дата
- `GET /candidates`: параметр `group_by: str = "date"` (было `"source"`).
- `POST /candidates/select`: `group_by: str = Form("date")` (было `"source"`).
- `POST /candidates/bulk`: `group_by: str = Form("date")` — тоже, чтобы
формы левой панели (bulk) сохраняли выбранную группировку по умолчанию.
### 3. Прочтение при выборе поста
В `GET /candidates`: когда `selected` задан и пост найден — пометить
прочитанным ДО рендера (чтобы карточка сразу стала обычной):
```python
if selected:
conn.execute("UPDATE posts SET is_read=1 WHERE id=? AND is_read=0", (selected,))
conn.commit()
```
(селект уже найден выше; UPDATE до `conn.close()`).
В `POST /candidates/select` — после редиректа на
`/candidates?selected=id&group_by=...` это покроет выбор; но select —
обычная форма, редирект идёт на GET, который и помечает. Отдельный UPDATE
в select не нужен (GET сделает).
### 4. Шаблон: карточки списка
`web/templates/candidates.html`, блок `{% for p in g.posts %}`:
Сейчас (строки 78–92):
```html
<div class="list-group-item d-flex align-items-start">
<a class="flex-grow-1 text-decoration-none{% if selected and selected.id == p.id %} fw-bold text-primary{% endif %}"
href="...">
<span class="d-block text-truncate">{{ (p.text or '')[:90] }}</span>
...
</a>
<input type="checkbox" ...>
</div>
```
Стало:
```html
<div class="list-group-item d-flex align-items-start{% if selected and selected.id == p.id %} active-row{% endif %}{% if not p.is_read %} fw-bold{% endif %}">
<a class="flex-grow-1 text-decoration-none"
href="...">
<span class="d-block text-truncate">{{ (p.text or '')[:90] }}</span>
...
</a>
<input type="checkbox" ...>
</div>
```
- `is_read=0` → `fw-bold` на всей карточке (текст жирный);
- `is_read=1` → обычный;
- выбранный → класс `active-row` → светло-голубая заливка.
Класс `active-row` — в base.html или candidates.html `<style>`:
```css
.list-group-item.active-row {
background-color: #e3f2fd; /* светло-голубой */
/* не перебиваем hover/оригинал: border, text — наследуются */
}
```
(используем кастомный класс, т.к. bootstrap `active` меняет цвет текста на
белый — не нужно; `bg-info`/`bg-light` — слишком сильные/неоднородные.)
### 5. Жирный шрифт только для непрочитанного
`fw-bold` на контейнере карточки делает жирным и текст, и метаданные —
это нормально для почтового клиента (непрочитанное письмо — весь блок
жирный). Ссылка остаётся `text-decoration-none`, цвет «primary» у
выбранного убираем (заливка вместо цвета).
## Верификация
- `openspec validate candidates-read-flag` — чисто.
- Миграция: запуск приложения (TestClient/import) создаёт колонку is_read;
существующие посты получают 0.
- Тест: GET /candidates?selected=N → в HTML карточка N имеет `active-row`
и НЕ `fw-bold` (после прочтения); у непрочитанного поста — `fw-bold`.
- Дефолт: GET /candidates (без group_by) → в шаблоне `group_by == 'date'`,
группы подписаны «Сегодня/Вчера/…».
- Перезапуск `sudo systemctl restart vesti-web`.
- Ручная проверка: список по умолчанию по дате; непрочитанные жирные;
открытый пост — голубая заливка; после открытия карточка перестаёт быть
жирной.
@@ -0,0 +1,45 @@
## Why
Список кандидатов — «почтовый клиент»: слева список карточек, справа детали.
Сейчас все карточки выглядят одинаково, не видно, какие посты уже смотрели,
а какой пост выбран (открыт) — не выделен, кроме цвета текста ссылки.
Пользователь хочет как в почтовых клиентах:
- непрочитанные посты — жирным шрифтом;
- прочитанные — обычным (как сейчас);
- выбранный (открытый) пост — светло-голубая заливка всей карточки.
Плюс: по умолчанию группировка должна быть ПО ДАТЕ (хронология получения),
а не по источнику.
## What Changes
- БД: колонка `posts.is_read INTEGER DEFAULT 0` (0 = не прочитан, 1 = прочитан).
- Web:
- выбор поста (GET /candidates?selected=N или POST /candidates/select)
помечает его `is_read=1`;
- карточка непрочитанного поста в списке — жирный текст (`fw-bold` на
заголовке/тексте), прочитанного — обычный (как сейчас);
- карточка выбранного поста — светло-голубая заливка всей карточки
(вместо текущего `fw-bold text-primary` на ссылке);
- дефолт `group_by` в GET /candidates и POST /candidates/select — `date`
вместо `source` (хронологический порядок получения: сегодня → вчера →
неделя → раньше).
## Why Not
- Не менять статус при публикации/отклонении: прочитанность — отдельный
флаг просмотра, не связан со статусом.
- Не делать авто-прочтение всех при заходе на страницу: только выбранный
пост становится прочитанным (как в почте).
- Группировка по дате — дефолт, но переключатель (источник/статус) остаётся.
## Impact
- Файлы: `db/schema.sql` (колонка), `web/app.py` (миграция при старте +
дефолт группировки + прочтение при выборе), `web/templates/candidates.html`
(классы карточек).
- Данные: ALTER TABLE (новые посты is_read=0 — непрочитанные; старые
после миграции тоже 0).
- Сервис: `vesti-web` (:8400) — перезапуск.
- Rollback: убрать колонку/правки и перезапустить.
@@ -0,0 +1,15 @@
# candidates-read-flag
- [x] Создан OpenSpec change (proposal/design) — `skip_specs: true`
- [x] `openspec validate candidates-read-flag` — чисто
- [x] db/schema.sql: колонка posts.is_read INTEGER DEFAULT 0
- [x] web/app.py: миграция is_read при старте (_migrate, идемпотентно)
- [x] web/app.py: GET /candidates и POST select/bulk: group_by по умолчанию 'date'
- [x] web/app.py: при selected → UPDATE posts SET is_read=1
- [x] candidates.html: непрочитанные — fw-bold, выбранный — active-row (светло-голубая заливка)
- [x] base.html: CSS .list-group-item.active-row (#dcecfc)
- [x] Тест (TestClient): миграция, прочтение при selected, дефолт date, классы карточек
- [x] Перезапуск `sudo systemctl restart vesti-web`
- [x] Ручная проверка в браузере: дата по умолчанию, жирные непрочитанные, голубая заливка выбранного
- [x] Обновить STATUS.md / TODO.md
- [x] Бэкап (крон 2:45 делает сам, вручную НЕ запускать)
+23 -3
View File
@@ -75,6 +75,21 @@ def _db() -> sqlite3.Connection:
return conn
def _migrate():
"""Идемпотентные миграции при старте веб-сервиса."""
conn = sqlite3.connect(DB_PATH, timeout=10)
try:
cols = [r[1] for r in conn.execute("PRAGMA table_info(posts)").fetchall()]
if "is_read" not in cols:
conn.execute("ALTER TABLE posts ADD COLUMN is_read INTEGER DEFAULT 0")
conn.commit()
finally:
conn.close()
_migrate()
def _session_ok(request: Request) -> bool:
return request.cookies.get(SESSION_COOKIE) == os.getenv("VESTI_WEB_SESSION", "")
@@ -277,7 +292,7 @@ def _fetch_candidates(conn, direction: str, status: str, own: str, q: str, group
@app.get("/candidates", response_class=HTMLResponse)
def candidates(request: Request, direction: str = "", status: str = "", own: str = "",
q: str = "", group_by: str = "source", selected: int = 0, error: str = ""):
q: str = "", group_by: str = "date", selected: int = 0, error: str = ""):
_require_auth(request)
conn = _db()
grouped, counts, status = _fetch_candidates(conn, direction, status, own, q, group_by)
@@ -294,6 +309,11 @@ def candidates(request: Request, direction: str = "", status: str = "", own: str
else:
continue
break
# открытие поста = прочтение (как в почте): сразу помечаем, чтобы карточка перестала быть жирной
if sel is not None and sel["id"]:
conn.execute("UPDATE posts SET is_read=1 WHERE id=? AND is_read=0", (sel["id"],))
sel["is_read"] = 1
conn.commit()
conn.close()
# строка параметров фильтров для ссылок (выбор поста, смена группировки)
qs = []
@@ -313,7 +333,7 @@ def candidates(request: Request, direction: str = "", status: str = "", own: str
@app.post("/candidates/select")
def candidates_select(request: Request, id: int = Form(...), direction: str = Form(""),
status: str = Form(""), own: str = Form(""), q: str = Form(""), group_by: str = Form("source")):
status: str = Form(""), own: str = Form(""), q: str = Form(""), group_by: str = Form("date")):
"""Выбор кандидата левой панели (обычная форма без JS)."""
_require_auth(request)
url = f"/candidates?selected={id}&group_by={group_by}"
@@ -326,7 +346,7 @@ def candidates_select(request: Request, id: int = Form(...), direction: str = Fo
@app.post("/candidates/bulk")
def candidates_bulk(request: Request, action: str = Form(...), ids: list[int] = Form(default_factory=list),
direction: str = Form(""), status: str = Form(""), own: str = Form(""),
q: str = Form(""), group_by: str = Form("source")):
q: str = Form(""), group_by: str = Form("date")):
"""Bulk-операции: approve/reject/reject-old/process по отмеченным ids."""
_require_auth(request)
back = f"/candidates?group_by={group_by}"
+5
View File
@@ -9,6 +9,9 @@
body { padding-top: 4.5rem; background: #f5f6fa; }
.card { box-shadow: 0 1px 4px rgba(0,0,0,.08); }
.post-text { white-space: pre-wrap; max-height: 140px; overflow-y: auto; font-size: .9rem; }
/* список кандидатов: выбранный пост — светло-голубая заливка карточки */
.list-group-item.active-row { background-color: #dcecfc; }
.list-group-item.active-row:hover { background-color: #d0e4f8; }
</style>
</head>
<body>
@@ -19,6 +22,8 @@
<a class="nav-link" href="/candidates">Кандидаты</a>
<a class="nav-link" href="/published">Опубликовано</a>
<a class="nav-link" href="/metrics">Метрики</a>
<a class="nav-link" href="/crawlers">Краулеры</a>
<a class="nav-link" href="/sources">Источники</a>
<a class="nav-link text-danger" href="/logout">Выйти</a>
</div>
</div>
+2 -2
View File
@@ -75,8 +75,8 @@
</summary>
<div class="list-group list-group-flush small">
{% for p in g.posts %}
<div class="list-group-item d-flex align-items-start">
<a class="flex-grow-1 text-decoration-none{% if selected and selected.id == p.id %} fw-bold text-primary{% endif %}"
<div class="list-group-item d-flex align-items-start{% if selected and selected.id == p.id %} active-row{% endif %}{% if not p.is_read %} fw-bold{% endif %}">
<a class="flex-grow-1 text-decoration-none"
href="/candidates?selected={{ p.id }}&group_by={{ group_by }}{% if filters_qs %}&{{ filters_qs }}{% endif %}">
<span class="d-block text-truncate">{{ (p.text or '')[:90] }}</span>
<span class="d-block text-muted">