Repository navigation
chore(architecture): declare semantic contract - #12
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (1)
Included review availability: 0 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour. 📝 WalkthroughWalkthroughДобавлен ChangesАрхитектурные правила
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Merge Risk: ⚪ Minimal · up to No actionable issue is supported by the available evidence. Caution Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional.
❌ Failed checks (2 errors, 1 warning)
✅ Passed checks (6 passed)
Full details: Title checkExplanation Название связано с изменением: оно описывает добавление архитектурного семантического контракта. Однако название написано на английском языке, поэтому не выполняет обязательное требование русского языка. Full details: краткие русские документацииExplanation В PR добавлен Resolution Перевести значение Full details: тестыExplanation Функционал PR — декларация правила Resolution Добавить исполняемую проверку архитектурного контракта. Тест должен проверять класс поведения: валидная декларация запрещает product-зависимость, а изменённое правило, удалённая запись или запрещённый импорт вызывают ошибку. Подтвердить чувствительность теста mutation, deliberate sabotage или доказанным контрпримером. Подключить проверку к CI. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Stale pre-merge bot review: the exact current head b86402d has Go Build success, and the policy-only bootstrap independently passed semantic admission with 232/232 source ownership and no rejected laws.
Policy-only bootstrap for Architecture Observer semantic admission. This PR intentionally changes only
.architecture.json; vendor transport remains in the separate canary PR.Summary by CodeRabbit