Skip to content

fix(ci): heal main — tool-count 64, preview ruff-skip, gate-zero 900s - #9

Merged
ManSio merged 1 commit into
mainfrom
feat/lsp-thin-client
Aug 25, 2026
Merged

fix(ci): heal main — tool-count 64, preview ruff-skip, gate-zero 900s#9
ManSio merged 1 commit into
mainfrom
feat/lsp-thin-client

Conversation

@ManSio

@ManSio ManSio commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Фикс: зелёный main (после мержа #8)

PR #8 смёржен без финального фикс-коммита — main упал в CI на трёх багах. Этот PR несёт ровно один коммит (222bb1f5) поверх смёрженного состояния.

Что чинит

  1. Контракт счётчика тулов: test_auto_doc_updater::test_count_tools_real_project_guard ждал 63, стало 64 (добавлен predict_change; реальный core = 31 класс). Обновлены: тест-гвард, README, docs/ru/README, AGENTS.md, docstring server_tools.py.
  2. change_preview падал FileNotFoundError: 'ruff' в clean-state (там pip install -e . без dev-экстр → ruff не установлен). _run теперь ловит отсутствие бинарника (returncode 127), гейты трактуют 127 как skip окружения, а не провал изменения.
  3. ruff W292 (новая строка в конце файла) в tests/test_lock_guard.py — краснили ubuntu/windows джобы.
  4. Таймаут gate-zero: полный pytest честно идёт ~171с (CI) — кап 300с флакал дважды под нагрузкой; поднят до 900с в verify_diary.py и шаблоне хука.

Проверка

  • Локально: guard-тест (64) ✅, change_preview 4/4 ✅, ruff clean ✅, pre-commit 5/5 (gate-zero в 900с) ✅.
  • CI этого PR прогонит полный цикл (ubuntu+windows, clean-state, manifest-parity, transport-equiv, docker).

- test_auto_doc_updater guard: 63 -> 64 (predict_change added; real core = 31 classes).
- change_preview._run: FileNotFoundError -> returncode 127; gates treat 127 as skip (clean-state has no ruff without dev extras) - was FileNotFoundError crash.
- Sync counts: server_tools docstring (31 core / 64 / 65 with exec), README (64), docs/ru/README (64), AGENTS.md header + section B (64 / 31 core).
- tests/test_lock_guard.py: W292 trailing newline.
- Pre-commit gate-zero timeout 300 -> 900 in verify_diary.py + hook template (suite grew to ~1500 tests; clean-state CI runs 171s; 300s flaked twice today under parallel load).
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: bea31b9e-fc0f-42dd-a522-ca54cf3f3ce7


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ManSio
ManSio merged commit 115e640 into main Aug 25, 2026
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant