lekss361/gendesign

Fork 0

chore(tooling): pre-commit пинит ruff v0.7.4, в venv 0.15.12 — хук переформатирует чужие строки #2864

New issue

Closed

opened 2026-08-13 11:38:20 +00:00 by bot-backend · 1 comment

bot-backend commented

2026-08-13 11:38:20 +00:00

Collaborator

Симптом

Любой коммит, трогающий питон-файл, где есть assert ..., "сообщение", получает в дифф
хунки на нетронутых строках. Поймал на PR #2863: три посторонних хунка в
backend/tests/api/v1/test_analyze_competitors_status.py (строки 90, 110, 234).

Причина

.pre-commit-config.yaml пинит ruff-pre-commit v0.7.4 (ноябрь 2024).
backend/pyproject.toml объявляет ruff>=0.5.0, и в backend/.venv разрешается
0.15.12. Они расходятся в форматировании длинных assert-сообщений:

# 0.15.x (так лежит в main)
assert val is None or isinstance(val, datetime.date), (
    f"ready_dt имеет неожиданный тип {type(val)}: {val}"
)

# 0.7.4 (так переписывает хук)
assert val is None or isinstance(
    val, datetime.date
), f"ready_dt имеет неожиданный тип {type(val)}: {val}"

Файлы в main отформатированы новым ruff → старый хук их «чинит» обратно на каждом
коммите, который их касается. Пинг-понг: следующий, кто запустит uv run ruff format,
вернёт как было.

Почему это не поймали раньше

ruff format --check намеренно не в CI-гейте (.forgejo/workflows/ci.yml:210-212:
«format-pass = future enhancement»). Гейтится только ruff check .. Поэтому расхождение
форматтеров не краснеет нигде — оно просто добавляет шум в каждый дифф.

Что сделать

Поднять rev: в .pre-commit-config.yaml до версии, совпадающей с той, что
разрешается в backend/.venv (сейчас 0.15.12), и прогнать
uv run ruff format . по backend/ + tradein-mvp/backend/ одним коммитом-нормализацией.
Заодно решить, пинить ли ruff точной версией в pyproject.toml вместо >=0.5.0 —
иначе дрейф повторится, как только выйдет следующий релиз с изменением стиля.

Оценка: S. Шумный, но безопасный коммит — трогает только форматирование.

## Симптом Любой коммит, трогающий питон-файл, где есть `assert ..., "сообщение"`, получает в дифф хунки на **нетронутых строках**. Поймал на PR #2863: три посторонних хунка в `backend/tests/api/v1/test_analyze_competitors_status.py` (строки 90, 110, 234). ## Причина `.pre-commit-config.yaml` пинит `ruff-pre-commit` **v0.7.4** (ноябрь 2024). `backend/pyproject.toml` объявляет `ruff>=0.5.0`, и в `backend/.venv` разрешается **0.15.12**. Они расходятся в форматировании длинных assert-сообщений: ```python # 0.15.x (так лежит в main) assert val is None or isinstance(val, datetime.date), ( f"ready_dt имеет неожиданный тип {type(val)}: {val}" ) # 0.7.4 (так переписывает хук) assert val is None or isinstance( val, datetime.date ), f"ready_dt имеет неожиданный тип {type(val)}: {val}" ``` Файлы в `main` отформатированы новым ruff → старый хук их «чинит» обратно на каждом коммите, который их касается. Пинг-понг: следующий, кто запустит `uv run ruff format`, вернёт как было. ## Почему это не поймали раньше `ruff format --check` **намеренно не в CI-гейте** (`.forgejo/workflows/ci.yml:210-212`: «format-pass = future enhancement»). Гейтится только `ruff check .`. Поэтому расхождение форматтеров не краснеет нигде — оно просто добавляет шум в каждый дифф. ## Что сделать 1. Поднять `rev:` в `.pre-commit-config.yaml` до версии, совпадающей с той, что разрешается в `backend/.venv` (сейчас 0.15.12), и прогнать `uv run ruff format .` по `backend/` + `tradein-mvp/backend/` одним коммитом-нормализацией. 2. Заодно решить, пинить ли ruff точной версией в `pyproject.toml` вместо `>=0.5.0` — иначе дрейф повторится, как только выйдет следующий релиз с изменением стиля. Оценка: S. Шумный, но безопасный коммит — трогает только форматирование.

Light1YT referenced this issue from a commit

2026-08-13 11:44:11 +00:00

fix(ptica): фильтр класса в velocity ссылался на алиас, которого нет в CTE

bot-backend referenced this issue

2026-08-13 11:44:36 +00:00

fix(ptica): фильтр класса в velocity ссылался на алиас, которого нет в CTE (#2464-G) #2865

Light1YT referenced this issue from a commit

2026-08-13 16:59:37 +00:00

fix(ptica): отказ слоя НСПД больше не выглядит как «здесь ничего нет»

Light1YT referenced this issue from a commit

2026-08-13 17:36:23 +00:00

test(ci): сторож пропусков кричит ::error:: под Actions

Light1YT referenced this issue from a commit

2026-08-13 17:44:46 +00:00

docs(ptica): фактор уверенности называл не ту величину

Light1YT referenced this issue from a commit

2026-08-14 05:58:39 +00:00

fix(ptica): загрузка ЦП больше не пишется в категориальную колонку

lekss361 added the

chore

priority/p3

tech-debt

labels 2026-08-16 10:25:27 +00:00

Light1YT referenced this issue from a commit

2026-08-19 08:12:01 +00:00

fix(tradein): фильтр выдачи Яндекса мёртв — slug ЖК ищем по странице объекта

Light1YT referenced this issue from a commit

2026-08-19 09:10:14 +00:00

feat(tradein): позиция квартиры внутри когорты аналогов (перцентиль)

Light1YT referenced this issue from a commit

2026-08-19 10:07:25 +00:00

fix(ptica): защита таймаутом теперь реально ограничивает время

Light1YT referenced this issue from a commit

2026-08-19 10:37:43 +00:00

fix(ptica): соединение БД отпускается до похода за фотографией наружу

Light1YT referenced this issue from a commit

2026-08-19 11:33:04 +00:00

test(ptica): покрыть переиспользование сессии после close() в отдаче фотографий (#2464-C)

bot-backend referenced this issue

2026-08-19 11:33:25 +00:00

test(ptica): покрыть переиспользование сессии после close() в отдаче фотографий (#2464-C) #2930

Light1YT referenced this issue from a commit

2026-08-19 12:21:09 +00:00

fix(ptica): шумовой скорер перестаёт считать водоёмы источниками шума (#2464-G)

Light1YT referenced this issue from a commit

2026-08-19 13:09:14 +00:00

fix(ptica): достоверность видит зону ПЗЗ из НСПД, а не только из пустой таблицы (#2464)

Light1YT referenced this issue from a commit

2026-08-19 16:05:57 +00:00

fix(ptica): пустой слой риск-зон перестаёт означать «рисков нет» (#2934)

bot-backend referenced this issue

2026-08-19 16:06:26 +00:00

fix(ptica): пустой слой риск-зон перестаёт означать «рисков нет» (#2934) #2940

Light1YT referenced this issue from a commit

2026-08-21 10:26:29 +00:00

chore(tooling): pre-commit, pyproject и uv.lock — один ruff 0.15.20 (#2864)

bot-backend referenced this issue

2026-08-21 10:26:32 +00:00

chore(tooling): pre-commit, pyproject и uv.lock — один ruff 0.15.20 (#2864) #3021

bot-backend referenced this issue from a commit

2026-08-21 11:37:24 +00:00

chore(tooling): pre-commit, pyproject и uv.lock — один ruff 0.15.20 (#2864) (#3021)

Light1YT referenced this issue from a commit

2026-08-21 11:39:33 +00:00

chore(format): нормализация под ruff 0.15.20 — 161 файл, только формат (#2864)

bot-backend referenced this issue

2026-08-21 11:39:35 +00:00

chore(format): нормализация под ruff 0.15.20 — 161 файл, только формат (#2864) #3022

bot-backend referenced this issue from a commit

2026-08-21 12:01:55 +00:00

chore(format): нормализация под ruff 0.15.20 — 161 файл, только формат (#2864) (#3022)

bot-backend commented

2026-08-21 14:07:28 +00:00

Author

Collaborator

Закрываю: хук, CI и lock'и на одном ruff 0.15.20, main нормализован

PR #3021 (1782dae0): rev v0.15.20, ruff==0.15.20 в обоих pyproject, uv lock (в lock'ах изменился только ruff); гейт test_2864_ruff_version_alignment.py держит три источника истины равными (на старом состоянии красный по значению: v0.7.4 vs 0.15.12/0.15.20).
PR #3022 (cdf493f3): ruff format 0.15.20 по backend/ и tradein-mvp/backend/ — 161 файл, только формат.
Проверка на свежем origin/main (21.08 14:0x UTC): uvx ruff@0.15.20 format --check backend/ tradein-mvp/backend/ → 1002 files already formatted; ruff check — зелёный. Коммиты, сделанные после через хук (#3023, #3024), прошли без переформатирования чужих строк.

Пункт 2 постановки («пинить ли точной версией») решён пином ==: с >= любой uv lock --upgrade снова уводил бы lock от хука; теперь бамп — осознанное действие втроём (rev + pyproject×2 + lock), и гейт напоминает, если забыли одно из трёх.

## Закрываю: хук, CI и lock'и на одном ruff 0.15.20, main нормализован - PR #3021 (`1782dae0`): rev `v0.15.20`, `ruff==0.15.20` в обоих pyproject, `uv lock` (в lock'ах изменился только ruff); гейт `test_2864_ruff_version_alignment.py` держит три источника истины равными (на старом состоянии красный по значению: `v0.7.4` vs `0.15.12`/`0.15.20`). - PR #3022 (`cdf493f3`): `ruff format` 0.15.20 по `backend/` и `tradein-mvp/backend/` — 161 файл, только формат. - Проверка на свежем `origin/main` (21.08 14:0x UTC): `uvx ruff@0.15.20 format --check backend/ tradein-mvp/backend/` → **1002 files already formatted**; `ruff check` — зелёный. Коммиты, сделанные после через хук (#3023, #3024), прошли без переформатирования чужих строк. Пункт 2 постановки («пинить ли точной версией») решён пином `==`: с `>=` любой `uv lock --upgrade` снова уводил бы lock от хука; теперь бамп — осознанное действие втроём (rev + pyproject×2 + lock), и гейт напоминает, если забыли одно из трёх.

bot-backend closed this issue

2026-08-21 14:07:29 +00:00