From e8ce82829613418b7f6cceca7abc4dd8fbf7ba82 Mon Sep 17 00:00:00 2001 From: Anton Vakhrushev Date: Fri, 10 Jul 2026 20:25:11 +0300 Subject: [PATCH] =?UTF-8?q?=D0=91=D0=B5=D0=BA=D0=BB=D0=BE=D0=B3:=20=D0=BF?= =?UTF-8?q?=D0=BE=D0=BB=D0=BD=D1=8B=D0=B9=20=D1=84=D0=BE=D1=80=D0=BC=D0=B0?= =?UTF-8?q?=D1=82=20=D1=8F=D1=80=D0=BB=D1=8B=D0=BA=D0=B0=20=C2=AB=D0=9E?= =?UTF-8?q?=D0=B1=D0=BD=D0=BE=D0=B2=D0=B8=D1=82=D1=8C=20=D0=B8=D0=BC=D1=8F?= =?UTF-8?q?=C2=BB=20+=20=D1=81=D0=BA=D1=80=D1=8B=D1=82=D0=B8=D0=B5=20cance?= =?UTF-8?q?lled=20=D0=B2=20=D1=81=D0=BF=D0=B8=D1=81=D0=BA=D0=B5?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Кнопка «Обновить имя» должна давать формат add-шага (Название (режиссёр, год), сезон; всё опц. кроме названия) вместо усечённого «Title (Year)». - Отменённые (cancelled) скрывать в общем списке наравне с удалёнными; тумблер «показать удалённые» раскрывает и cancelled, и deleted. Co-Authored-By: Claude Opus 4.8 (1M context) --- docs/backlog/README.md | 2 + .../knopka-obnovit-imya-polnyj-format.md | 37 +++++++++++++++++++ docs/backlog/skryvat-otmenennye-v-spiske.md | 35 ++++++++++++++++++ 3 files changed, 74 insertions(+) create mode 100644 docs/backlog/knopka-obnovit-imya-polnyj-format.md create mode 100644 docs/backlog/skryvat-otmenennye-v-spiske.md diff --git a/docs/backlog/README.md b/docs/backlog/README.md index a761078..456d206 100644 --- a/docs/backlog/README.md +++ b/docs/backlog/README.md @@ -40,6 +40,8 @@ Tududi (проект `jellybit`) больше **не** держит беклог ## Низкий +- [Кнопка «Обновить имя»: полный формат ярлыка (Название (режиссёр, год), сезон)](knopka-obnovit-imya-polnyj-format.md) — Перелив имени даёт усечённое «Title (Year)» вместо полного формата add-шага (режиссёр+сезон); нужный `render` уже есть +- [Скрывать отменённые (cancelled) в общем списке наравне с удалёнными](skryvat-otmenennye-v-spiske.md) — cancelled шумит в ленте; тумблер «показать удалённые» должен раскрывать и cancelled, и deleted - [Панель действий ревью вне htmx-свопа блока источника](panel-review-vne-swap.md) — При выборе источника одним кликом обновляется только блок источника (#source-block)… - [Мгновенные обновления через SSE](sse-obnovleniya.md) — Живые обновления прогресса сейчас на htmx-поллинге (фаза 2 веб-UI) — просто и работает… - [Шум ERROR фоновых циклов при недоступной зависимости](oshibki-klassifikaciya-i-konvencii-logirovaniya.md) — Остаток задачи логирования: ext.* ERROR-шторм при недоступном qBittorrent + эскалация устойчивого сбоя тика _(ревью Fable)_ diff --git a/docs/backlog/knopka-obnovit-imya-polnyj-format.md b/docs/backlog/knopka-obnovit-imya-polnyj-format.md new file mode 100644 index 0000000..f279bdd --- /dev/null +++ b/docs/backlog/knopka-obnovit-imya-polnyj-format.md @@ -0,0 +1,37 @@ +# Кнопка «Обновить имя»: полный формат ярлыка (Название (режиссёр, год), сезон) + +**Приоритет:** низкий · **Теги:** web-ui, naming, review + +Кнопка «Обновить имя» (перелив распознанного названия в `display_name` и ярлык +раздачи) сейчас генерирует усечённый формат `Title (Year)` через +`naming.FormatTitleYear(plan.Title, plan.Year)` +(`internal/worker/review.go:1110`). Ожидается тот же формат, что и «полное» +отображаемое имя add-шага: **`Название (режиссёр, год)`**, а для сериала — плюс +**`. Сезон N`**. Всё опционально, кроме названия. + +Нужный формат уже реализован приватной `render()` в +`internal/naming/naming.go:104-140` (`movie: "Title (Director, Year)"`, `series` ++ `". Сезон N"`), но `RefreshDisplayName` его не использует — комментарий у +`FormatTitleYear` (`naming.go:88-95`) прямо отмечает: «у перелива режиссёра и +сезона нет». Задача — устранить это расхождение: перелив должен давать тот же +ярлык, что и add-шаг. + +Шаги: +- Экспонировать `render`-логику (публичная обёртка в `internal/naming`) или + собрать `extracted` из плана и позвать её из `RefreshDisplayName`. +- **Сезон** — из плана (`recognize.PlanFile.Season`, `recognize.go:77`): для + сериала взять сезон (единый — подставить, разные — решить: опустить или взять + минимальный; свериться с логикой сводки сезонов на странице). +- **Режиссёр** — открытый вопрос: в `recognize.Plan` его сейчас нет. Найти + источник (матч метабазы TMDB/TVDB несёт режиссёра? recognition?) или, если + недоступен, опустить (формат допускает отсутствие). Уточнить, откуда add-шаг + берёт режиссёра, и переиспользовать тот же источник. +- Все части, кроме названия, опциональны — пустой режиссёр/год/сезон просто + выпадают из скобок/хвоста (как уже делает `render`). + +Проверить: единый формат ярлыка раздачи и `display_name`; санитайзинг и лимит +длины (`maxNameLen`) сохраняются; поведение на `done`/`orphaned` (кнопка гейтится +наличием распознавания, не состоянием — `internal/httpapi/download.go:142`). + +Связано: `openspec/specs/recognition`, `openspec/specs/review`, пакеты +`internal/naming`, `internal/worker`, `internal/recognize`. diff --git a/docs/backlog/skryvat-otmenennye-v-spiske.md b/docs/backlog/skryvat-otmenennye-v-spiske.md new file mode 100644 index 0000000..bb1619a --- /dev/null +++ b/docs/backlog/skryvat-otmenennye-v-spiske.md @@ -0,0 +1,35 @@ +# Скрывать отменённые (cancelled) в общем списке наравне с удалёнными + +**Приоритет:** низкий · **Теги:** web-ui + +Отменённые загрузки (`cancelled`) должны по умолчанию скрываться в общем списке +так же, как удалённые (`deleted`), а тумблер «Показать всё, включая удалённые» +(`web/templates/index.html:52`) должен показывать **и те, и те**. Сейчас под +группой `all` скрыт только `deleted`, а `cancelled` всегда виден и шумит в ленте +— тем более что новый стоп-кран «Закрыть» будет плодить `cancelled`-записи. + +Где править: +- `internal/store/list.go:62-66` — ветка `else if !f.IncludeDeleted` под группой + `all` скрывает только `StateDeleted` (`state <> ?`). Расширить до + `state NOT IN (deleted, cancelled)`. +- Осмыслить семантику флага `ListFilter.IncludeDeleted` (`list.go:45`): теперь он + «показывать скрытые терминальные» (deleted **и** cancelled). Либо переименовать + (`IncludeHidden`/`IncludeTerminal`), либо оставить имя и уточнить комментарий. +- Метка тумблера (`index.html:52`) и текст — при желании подправить («…включая + отменённые и удалённые»); `ShowAll`/`all=1` уже управляет показом. +- Комментарии-инварианты про «под all видны» в `internal/store/list.go:22-26` и + `statesInGroup` (cancelled/reverted/deleted не имеют чипа) — привести в + соответствие. + +Замечания: +- `cancelled` своей группы-чипа не имеет (`statesInGroup`, `list.go:29-40`) — под + группами `review/active/done/problem` он и так не появляется, правка нужна + только в ветке `all`. +- `reverted` пользователь не упоминал — по умолчанию оставляем видимым; при + желании обсудить симметрию отдельно. +- Обновить тест `TestListDownloadsPageFilterAndDeleted` + (`internal/store/list_test.go`) — добавить `cancelled` в скрытые по умолчанию и + видимые под `IncludeDeleted`. + +Связано: `openspec/specs/web-ui`, пакеты `internal/store`, `internal/httpapi`, +конвенция [web-ui](../conventions/web-ui.md).