Веб-UI: режиссёр в блоке «Распознано как» на странице загрузки

На /download/{id} поле «Режиссёр» было захардкожено прочерком, хотя экран ревью режиссёра уже выводит: слоистое разрешение полей ярлыка было заперто в неэкспортируемом worker.effectiveDisplayName. Из-за этого билдеры вью видели только слой распознавания+матч (rd.Plan.Director) без слоя контекста — то же на экране ревью. Вынес разрешение в экспортируемую naming.EffectiveFields(parsedContext, plan) LabelFields с методом Label(): выбор слоя по сырым значениям (как прежде), выбранные скаляры возвращаются очищенными (sanitize идемпотентен, display_name побайтно тот же). effectiveDisplayName стал тонкой обёрткой; страница загрузки и экран ревью берут режиссёра из той же функции — согласованно с заголовком. OpenSpec: web-ui (ADDED «Режиссёр в блоке распознавания страницы загрузки»), review (MODIFIED «Инфо и предпросмотр выбранного источника» — слоистое разрешение с фолбэком на контекст). Change заархивирован. Беклог: закрыта rezhisser-v-kartochke-zagruzki. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-07-17 21:21:27 +03:00
parent 0354a8c96b
commit 5948f4d219
16 changed files with 612 additions and 93 deletions
@@ -6,6 +6,7 @@ import (
 	"strconv"
 	"time"

+	"git.vakhrushev.me/av/jellybit/internal/naming"
 	"git.vakhrushev.me/av/jellybit/internal/recognize"
 	"git.vakhrushev.me/av/jellybit/internal/store"
 	"git.vakhrushev.me/av/jellybit/internal/worker"
@@ -38,6 +39,7 @@ type downloadDetailView struct {
 	OriginalTitle string
 	Season        string // сводка сезонов для сериала (пусто для фильма)
 	Year          int
+	Director      string // режиссёр эффективного источника (пусто — неизвестен)
 	Provider      string
 	ProviderID    string
 	MatchURL      string // ссылка на запись метабазы (пусто — показываем текстом)
@@ -145,6 +147,10 @@ func (s *server) buildDownloadView(id string, rd *worker.ReviewData) downloadDet
 			view.Season = recognize.SeasonSummary(rd.Plan)
 		}
 		view.Year = rd.Plan.Year
+		// Режиссёр — из того же слоистого разрешения, что и display_name в шапке
+		// (override → распознавание+матч → контекст), чтобы поле не расходилось с
+		// заголовком; пустой во всех слоях → прочерк в шаблоне.
+		view.Director = naming.EffectiveFields(d.ParsedContext, rd.Plan).Director
 		// Ручное обновление имени доступно, когда есть распознанное название,
 		// которое можно перелить (иначе FormatTitleYear даст пусто → no-op).
 		view.Nameable = rd.Plan.Title != ""
@@ -9,6 +9,7 @@ import (
 	"strings"
 	"testing"

+	"git.vakhrushev.me/av/jellybit/internal/recognize"
 	"git.vakhrushev.me/av/jellybit/internal/store"
 	"git.vakhrushev.me/av/jellybit/internal/worker"
 )
@@ -119,6 +120,55 @@ func TestRouterRendersPages(t *testing.T) {
 	}
 }

+// TestDownloadPageShowsDirector — блок «Распознано как» на /download/{id}
+// выводит режиссёра слоистым разрешением: из плана (матч) и из сохранённого
+// контекста при распознавании без матча (ранее поле было захардкожено прочерком).
+func TestDownloadPageShowsDirector(t *testing.T) {
+	base := func(plan recognize.Plan, parsedContext string) *worker.ReviewData {
+		return &worker.ReviewData{
+			Download: store.Download{
+				ID: testULID, SourceRef: "Dune", State: store.StateReview,
+				ParsedContext: parsedContext,
+			},
+			Recognition: &store.Recognition{ID: "1", DownloadID: testULID, IsCurrent: true},
+			Plan:        plan,
+		}
+	}
+	moviePlan := func(director string) recognize.Plan {
+		return recognize.Plan{Type: recognize.MediaMovie, Title: "Дюна", Year: 2024, Director: director,
+			Files: []recognize.PlanFile{{Src: "dune.mkv", Role: recognize.RoleMain}}}
+	}
+
+	cases := []struct {
+		name       string
+		rd         *worker.ReviewData
+		wantInBody string
+	}{
+		{
+			name:       "режиссёр из плана (матч)",
+			rd:         base(moviePlan("Дени Вильнёв"), ""),
+			wantInBody: "Дени Вильнёв",
+		},
+		{
+			name:       "режиссёр из контекста при распознавании без матча",
+			rd:         base(moviePlan(""), `{"type":"movie","title":"Дюна","director":"Дени Вильнёв"}`),
+			wantInBody: "Дени Вильнёв",
+		},
+	}
+	for _, c := range cases {
+		t.Run(c.name, func(t *testing.T) {
+			h := testRouter(t, stubReader{one: &c.rd.Download}, stubReviewer{data: c.rd})
+			rr := get(t, h, "/download/"+testULID)
+			if rr.Code != http.StatusOK {
+				t.Fatalf("GET /download/{id} = %d, want 200", rr.Code)
+			}
+			if !strings.Contains(rr.Body.String(), c.wantInBody) {
+				t.Errorf("страница загрузки не содержит режиссёра %q", c.wantInBody)
+			}
+		})
+	}
+}
+
 // TestStaticServed проверяет отдачу встроенной статики с кэш-заголовком.
 func TestStaticServed(t *testing.T) {
 	h := testRouter(t, stubReader{}, stubReviewer{})
@@ -9,6 +9,7 @@ import (
 	"strings"

 	"git.vakhrushev.me/av/jellybit/internal/ident"
+	"git.vakhrushev.me/av/jellybit/internal/naming"
 	"git.vakhrushev.me/av/jellybit/internal/recognize"
 	"git.vakhrushev.me/av/jellybit/internal/store"
 	"git.vakhrushev.me/av/jellybit/internal/worker"
@@ -119,7 +120,10 @@ func buildReviewView(id string, rd *worker.ReviewData, errMsg string) reviewView
 		view.IsSeries = rd.Plan.Type == "series"
 		view.Title = rd.Plan.Title
 		view.OriginalTitle = rd.Plan.OriginalTitle
-		view.Director = rd.Plan.Director
+		// Режиссёр — из слоистого разрешения (override → распознавание+матч →
+		// контекст), а не только из плана: иначе слой контекста (режиссёр без
+		// матча) терялся бы, как на странице загрузки.
+		view.Director = naming.EffectiveFields(rd.Download.ParsedContext, rd.Plan).Director
 		view.Year = rd.Plan.Year
 		if view.IsSeries {
 			view.SeasonSummary = recognize.SeasonSummary(rd.Plan)
@@ -0,0 +1,112 @@
+package naming
+
+import (
+	"encoding/json"
+	"testing"
+
+	"git.vakhrushev.me/av/jellybit/internal/recognize"
+)
+
+func mustJSON(t *testing.T, f Fields) string {
+	t.Helper()
+	b, err := json.Marshal(f)
+	if err != nil {
+		t.Fatalf("marshal fields: %v", err)
+	}
+	return string(b)
+}
+
+func episode(season int) recognize.PlanFile {
+	s := season
+	return recognize.PlanFile{Role: recognize.RoleEpisode, Season: &s}
+}
+
+// EffectiveFields: слоистое разрешение полей ярлыка (план → контекст) и сводка
+// сезонов из плана/контекста. Проверяем и итоговый Label (совпадение с прежним
+// display_name), и отдельно очищенное поле Director (его печатает шаблон).
+func TestEffectiveFields(t *testing.T) {
+	cases := []struct {
+		name    string
+		parsed  Fields // пустой Title → parsed_context не задаётся
+		plan    recognize.Plan
+		want    string // ожидаемый Label()
+		wantDir string // ожидаемый Director
+	}{
+		{
+			name:    "матч даёт режиссёра и год",
+			plan:    recognize.Plan{Type: recognize.MediaMovie, Title: "Дюна", Director: "Дени Вильнёв", Year: 2024},
+			want:    "Дюна (Дени Вильнёв, 2024)",
+			wantDir: "Дени Вильнёв",
+		},
+		{
+			name:    "режиссёр из контекста переживает распознавание без матча",
+			parsed:  Fields{Type: "movie", Title: "Дюна", Director: "Дени Вильнёв", Year: 2021},
+			plan:    recognize.Plan{Type: recognize.MediaMovie, Title: "Дюна", Year: 2024}, // без director
+			want:    "Дюна (Дени Вильнёв, 2024)",                                           // год из плана, режиссёр из контекста
+			wantDir: "Дени Вильнёв",
+		},
+		{
+			name:    "режиссёр матча бьёт контекстного",
+			parsed:  Fields{Type: "movie", Title: "Дюна", Director: "Кто-то из контекста"},
+			plan:    recognize.Plan{Type: recognize.MediaMovie, Title: "Дюна", Director: "Дени Вильнёв", Year: 2024},
+			want:    "Дюна (Дени Вильнёв, 2024)",
+			wantDir: "Дени Вильнёв",
+		},
+		{
+			name:    "год из контекста, когда плана нет",
+			parsed:  Fields{Type: "movie", Title: "Брат", Year: 1997},
+			plan:    recognize.Plan{Type: recognize.MediaMovie, Title: "Брат"}, // year 0, без режиссёра
+			want:    "Брат (1997)",
+			wantDir: "",
+		},
+		{
+			name: "сериал: сводка сезонов из плана",
+			plan: recognize.Plan{Type: recognize.MediaSeries, Title: "Фарго",
+				Files: []recognize.PlanFile{episode(1), episode(2), episode(3)}},
+			want:    "Фарго. Сезоны 1–3",
+			wantDir: "",
+		},
+		{
+			name:    "сериал: сезон из контекста, когда в плане нет эпизодов",
+			parsed:  Fields{Type: "series", Title: "Сёгун", Season: ptr(2)},
+			plan:    recognize.Plan{Type: recognize.MediaSeries, Title: "Сёгун", Year: 2024},
+			want:    "Сёгун (2024). Сезон 2",
+			wantDir: "",
+		},
+		{
+			name:    "режиссёр из управляющих символов и лишних пробелов очищается",
+			plan:    recognize.Plan{Type: recognize.MediaMovie, Title: "Дюна", Director: "  Дени\tВильнёв\n ", Year: 2024},
+			want:    "Дюна (Дени Вильнёв, 2024)",
+			wantDir: "Дени Вильнёв",
+		},
+		{
+			name:    "режиссёр только из управляющих символов даёт пустое поле и выпадает из ярлыка",
+			plan:    recognize.Plan{Type: recognize.MediaMovie, Title: "Дюна", Director: "\x01\x02", Year: 2024},
+			want:    "Дюна (2024)",
+			wantDir: "",
+		},
+	}
+	for _, c := range cases {
+		t.Run(c.name, func(t *testing.T) {
+			parsed := ""
+			if c.parsed.Title != "" {
+				parsed = mustJSON(t, c.parsed)
+			}
+			ef := EffectiveFields(parsed, c.plan)
+			if got := ef.Label(); got != c.want {
+				t.Errorf("Label() = %q, want %q", got, c.want)
+			}
+			if ef.Director != c.wantDir {
+				t.Errorf("Director = %q, want %q", ef.Director, c.wantDir)
+			}
+		})
+	}
+}
+
+// Битый parsed_context не валит разрешение (best-effort): используется только план.
+func TestEffectiveFieldsBrokenParsedContext(t *testing.T) {
+	plan := recognize.Plan{Type: recognize.MediaMovie, Title: "Дюна", Year: 2024}
+	if got := EffectiveFields("{не json", plan).Label(); got != "Дюна (2024)" {
+		t.Errorf("Label() = %q, want «Дюна (2024)»", got)
+	}
+}
@@ -16,12 +16,14 @@ package naming

 import (
 	"context"
+	"encoding/json"
 	"log/slog"
 	"strconv"
 	"strings"
 	"unicode/utf8"

 	"git.vakhrushev.me/av/jellybit/internal/llm"
+	"git.vakhrushev.me/av/jellybit/internal/recognize"
 )

 // maxNameLen — ограничение длины отображаемого имени (символов/рун).
@@ -45,6 +47,61 @@ type Fields struct {
 	IsRussian     bool   `json:"is_russian"`
 }

+// LabelFields — эффективные скалярные поля отображаемого ярлыка после слоистого
+// разрешения (план → контекст). Значения уже очищены (sanitize): управляющие
+// символы вырезаны, пробелы схлопнуты — те же, что попадут внутрь Label, поэтому
+// присутствие/отсутствие поля определяется одинаково при прямом выводе (шаблон)
+// и внутри Label. Season — готовая строка-сводка сезонов (для фильма пусто).
+type LabelFields struct {
+	Title    string
+	Director string
+	Year     int
+	Season   string
+}
+
+// Label собирает полный ярлык display_name из эффективных полей. sanitize внутри
+// идемпотентен, поэтому на уже очищенных полях результат тот же.
+func (f LabelFields) Label() string {
+	return Label(f.Title, f.Director, f.Year, f.Season)
+}
+
+// EffectiveFields разрешает поля отображаемого ярлыка слоями: план (override →
+// распознавание+матч, уже свёрнут вызывающим) с фолбэком на извлечённый из
+// контекста слой (parsedContext, схема Fields) для полей, которых план не дал
+// (например режиссёр из контекста при распознавании без матча). Сводка сезонов —
+// recognize.SeasonSummary(plan) с фолбэком на контекстный скаляр. parsedContext
+// недоверен: битый JSON → пустой слой (best-effort). Выбор слоя идёт по сырым
+// значениям (как прежде), выбранные скаляры возвращаются очищенными (sanitize),
+// чтобы прямой вывод поля во вью совпадал с тем, что даёт Label в шапке.
+func EffectiveFields(parsedContext string, plan recognize.Plan) LabelFields {
+	var ctxf Fields
+	if s := strings.TrimSpace(parsedContext); s != "" {
+		_ = json.Unmarshal([]byte(s), &ctxf)
+	}
+	title := plan.Title
+	if title == "" {
+		title = ctxf.Title
+	}
+	director := plan.Director
+	if director == "" {
+		director = ctxf.Director
+	}
+	year := plan.Year
+	if year == 0 {
+		year = ctxf.Year
+	}
+	season := recognize.SeasonSummary(plan)
+	if season == "" && plan.Type == recognize.MediaSeries {
+		season = ctxf.SeasonLabel()
+	}
+	return LabelFields{
+		Title:    sanitize(title),
+		Director: sanitize(director),
+		Year:     year,
+		Season:   sanitize(season),
+	}
+}
+
 // SeasonLabel — сводка сезона из контекстного скаляра: «Сезон N» для сериала с
 // заданным номером, иначе пусто. Для контекста сезон скалярный (в отличие от
 // плана распознавания, где он per-file и сводится recognize.SeasonSummary).
@@ -1135,35 +1135,12 @@ func (w *Worker) effectivePlan(ctx context.Context, id string) (plan recognize.P
 	return applyOverrides(plan, overrides), prov, pid, nil
 }

-// effectiveDisplayName собирает полный ярлык display_name из эффективных полей:
-// плана (override → распознавание+матч, уже свёрнуто effectivePlan) с fallback на
-// сохранённый контекст (parsed_context) для полей, которых план не дал (например
-// режиссёр из контекста при распознавании без матча). Сводка сезонов —
-// recognize.SeasonSummary(plan) с fallback на контекстный скаляр. Единый формат с
-// шагом добавления (общий naming.Label). parsed_context недоверен: битый JSON →
-// пустой слой (best-effort), очистка полей — на рендере.
+// effectiveDisplayName собирает полный ярлык display_name из эффективных полей.
+// Тонкая обёртка над naming.EffectiveFields (слоистое разрешение план →
+// контекст) + Label — единый источник разрешения полей ярлыка, общий со
+// страницей загрузки и экраном ревью.
 func effectiveDisplayName(d store.Download, plan recognize.Plan) string {
-	var ctxf naming.Fields
-	if s := strings.TrimSpace(d.ParsedContext); s != "" {
-		_ = json.Unmarshal([]byte(s), &ctxf)
-	}
-	title := plan.Title
-	if title == "" {
-		title = ctxf.Title
-	}
-	director := plan.Director
-	if director == "" {
-		director = ctxf.Director
-	}
-	year := plan.Year
-	if year == 0 {
-		year = ctxf.Year
-	}
-	season := recognize.SeasonSummary(plan)
-	if season == "" && plan.Type == recognize.MediaSeries {
-		season = ctxf.SeasonLabel()
-	}
-	return naming.Label(title, director, year, season)
+	return naming.EffectiveFields(d.ParsedContext, plan).Label()
 }

 // RefreshDisplayName — внешняя точка входа обновления отображаемого имени