Repository navigation
feat: skill hygiene validator [OMN-5200] - #60
Conversation
…rcement [OMN-5200] 7-check validator (underscore-names, no-duplicate-names, no-unindexed-nesting, skill-md-required, name-matches-dir, no-python-in-skills, no-orphan-topics) with --strict mode for post-migration CI enforcement. 34 unit tests.
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Comment |
Summary
scripts/validation/validate_skill_hygiene.pywith 7 checks enforcing skill directory naming, structure, and Python separationcheck_ai_slop.pypattern: standalone script, argparse CLI, exit codes 0/1/2--strictmode promotes migration warnings (M1: Implement Pydantic schema models v1 (Day Close + Ticket Contract) #4 skill-md-required, feat(export): implement JSON schema export script (OMN-963) #5 name-matches-dir) to errors for post-migration CIChecks
Test plan
uv run pytest tests/test_validate_skill_hygiene.py -v— 34/34 passing