Skip to content

test: смоук-проверка AI-ревью пайплайна - #14

Open
Sermage wants to merge 1 commit into
feat/pr-ai-reviewfrom
test/ai-review-smoke
Open

test: смоук-проверка AI-ревью пайплайна#14
Sermage wants to merge 1 commit into
feat/pr-ai-reviewfrom
test/ai-review-smoke

Conversation

@Sermage

@Sermage Sermage commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Тестовый PR для проверки, что self-hosted runner подхватывает ai-review.yml,
гоняет RAG+LLM и постит AI-ревью комментарием.

Добавлен временный examples/smoke_review.py с функцией divide без проверки
деления на ноль — чтобы у ревьюера было что заметить.

После проверки PR и файл можно удалить.

Временный examples/smoke_review.py, чтобы убедиться, что self-hosted runner
подхватывает ai-review.yml и постит ревью. Удалить после проверки.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WPCiJi7DVBdV2fjN2x3HMd
@github-actions

Copy link
Copy Markdown

🤖 AI-ревью

🐞 Потенциальные баги

— замечаний нет. Код тривиален, логических ошибок не содержит.

🏛 Архитектурные проблемы

— замечаний нет. Файл examples/smoke_review.py не является частью основной бизнес-логики приложения и не нарушает архитектурных правил проекта.

💡 Рекомендации

  • examples/smoke_review.py:10-14: Убедитесь, что этот файл удаляется после завершения проверки, чтобы не оставлять мусорные артефакты в репозитории, как указано в docstring.

Контекст (RAG): infra/rerankers.py — Мелкие незначащие слова — чтобы «как что в и на» не давали ложных совпадений. · app/retrieval_pipeline.py · CLAUDE.md — Jarvis CLI > Архитектурные правила > Тесты

Сгенерировано jarvis-cli · review_pr.py

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