mirror of
https://github.com/rennf93/roboco.git
synced 2026-08-03 07:23:24 +02:00
The CI 'Python quality gate' (make quality) failed on PR #780's head because foundation-check detected lifecycle-artifact drift: the committed panel/lib/lifecycle.json and docs/rag/lifecycle/status-transitions.md still carried an `awaiting_pm_review -> claimed` claim transition that the current lifecycle spec no longer emits. make gate (format/lint/mypy/xenon) does not run foundation-check, so the reviewer could not reproduce locally. Fix: ran `make lifecycle` to regenerate the artifacts from the spec and commit the diff — no hand-written spec change, just stale generated output brought current. Second finding: tests/integration/test_notification_reescalation_dedup_exemption.py:164 had a tautological `assert target.id is not None` (target.id was uuid4() at construction, so it could never fail). Replaced with a real DB query that fetches all BLOCKER_ESCALATION rows for the sender/task, filters to those addressed to target.id, and asserts exactly 2 re-escalation rows exist — proving both _re_escalate_recipient calls delivered to the right recipient. Verified: make gate green (format/lint/mypy/xenon); quality-fast suite shows 7779 passed (same count as pre-change), the only failure is the pre-existing local-only cloud_auth test (needs localhost:5432 which CI provides); no regression in the notification dedup tests.