fix: исправить условие «нуждается в привязке» для проектов-сериалов

[claude]

Что сделано

• Исправлен метод GetKogdaIgraMissingGames() в ProjectRepository.cs

Проблема: Проекты-сериалы пропадали из выборки «надо привязать», если хотя бы одна из привязанных КИ-игр завершилась менее 60 дней назад. Условие hasNonStaleKogdaIgra (игра с End > now - 60 дней) исключало весь проект, даже если все игры фактически уже прошли.

Новое условие: Показывать проект как кандидата привязки если:

1. Нет КИ-привязки вообще, ИЛИ
2. Все привязанные КИ-игры уже завершились (End < now) И в проекте продолжается активность (обновление за последние 60 дней)

Closes #3860

Generated with Claude Code

[leotsarev]

@claude мне кажется ошибка в логике. Давай напишем юнит тесты на условие фильтрации, и так разберемся все ли верно.

[claude]

Claude finished @leotsarev's task in 6m 36s —— View job

---

Юнит-тесты добавлены

• Прочитать текущий код фильтрации
• Найти существующие тестовые проекты
• Вынести предикат фильтрации в KogdaIgraMissingGamesPredicate.NeedsBinding() (internal static)
• Написать 16 юнит-тестов на все граничные случаи
• Запустить тесты — все прошли ✅
• Закоммитить и запушить

---

Что сделано:

Логика фильтрации вынесена из ProjectRepository в отдельный internal static метод KogdaIgraMissingGamesPredicate.NeedsBinding(), который тестируется напрямую в JoinRpg.DataModel.Mocks (этот проект уже настроен через InternalsVisibleTo).

16 тестов, все зелёные:

Сценарий	Ожидание
Нет привязок, активный проект	включить ✅
Нет привязок, устаревший проект	включить ✅
Будущая игра (End > now), активный проект	исключить ✅
Будущая игра, устаревший проект	исключить ✅
End == now (граница)	исключить ✅
Прошедшая игра (End < now), активный проект	включить ✅
Прошедшая игра, устаревший проект	исключить ✅
Обновлено ровно 60 дней назад (граница)	исключить ✅
Обновлено 59 дней назад (чуть меньше порога)	включить ✅
Только неактивная привязка (Active=false), активный	включить ✅
Только неактивная привязка, устаревший	включить ✅
Смешанный: одна прошедшая + одна будущая	исключить ✅
Несколько прошедших игр, активный проект	включить ✅
Несколько прошедших игр, устаревший	исключить ✅
Игра с End=null, активный проект	включить ⚠️
Игра с End=null, устаревший проект	исключить ⚠️

⚠️ Потенциальный баг — End=null:

Если у КИ-игры End=null (дата окончания не заполнена), то null >= now в C# возвращает false, и hasActiveOrFutureGame = false. Это значит, что проект с привязанной игрой без даты окончания попадёт в выборку как "нуждающийся в привязке" (если недавно обновлялся). Скорее всего, логичнее считать такую игру «будущей» и исключать проект из выборки. Нужно ваше решение: фикс (g.End == null || g.End >= now) или такое поведение ожидаемо?
| Branch

[leotsarev]

Так работать не будет — нам нужно возвращать из этого метода Expression<Func<<>> иначе результат не будет транслироваться на сторону EF

[leotsarev]

Используй LinqKit чтобы переиспользовать предикаты. Добавь информацию о использовании LINQKIT в документацию (создай отдельный файлик который описывает как писать Linq запросы с помощью LinqKit и сошлись на него в CLAUDE.MD

[leotsarev]

Давай условие DisableKogdaIgraMapping занесем внутрь этого метода

[leotsarev]

@claude fix

[claude]

Claude finished @leotsarev's task in 6m 27s —— View job

---

Исправление по замечаниям ревью

• Прочитать текущий код
• Проверить наличие LinqKit в проекте (уже есть LinqKit.EntityFramework)
• Рефакторинг: GetPredicate() возвращает Expression<Func<Project, DateTime, bool>>
• Перенести условия Active и DisableKogdaIgraMapping внутрь предиката
• Обновить ProjectRepository.cs — использовать предикат через Invoke
• Обновить тесты под новый API
• Создать документацию по LinqKit (docs/linq-queries.md)
• Обновить CLAUDE.md со ссылкой на документацию
• Запустить тесты — 18/18 зелёных ✅
• Закоммитить и запушить

---

Что сделано

KogdaIgraMissingGamesPredicate.cs — метод NeedsBinding(IEnumerable, DateTime, DateTime) заменён на GetPredicate(DateTime now), возвращающий Expression<Func<Project, DateTime, bool>>. Условия project.Active и project.Details.DisableKogdaIgraMapping перенесены внутрь предиката.

ProjectRepository.cs — GetKogdaIgraMissingGames теперь использует predicate.Invoke(project, update.LastUpdated) через LinqKit — выражение транслируется в SQL.

Тесты — обновлены: создают Project-объекты через вспомогательный CreateProject(), вызывают GetPredicate(Now).Compile()(project, lastUpdated). Добавлены 2 новых теста: DisableKogdaIgraMapping=true и Active=false.

docs/linq-queries.md — новый файл с описанием паттерна Expression + LinqKit: зачем, как определять, как использовать в EF, как тестировать.
| Branch