docs: decouple Collector delivery from production Gate - #866
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAGENTS.md now defines production Gate scope. Failed Gates block only protected runtime or collector transitions. Development and control-plane work uses targeted checks, followed by a full Gate immediately before the protected boundary. ChangesProduction Gate scope
Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to This documentation-only change does not alter production behavior, and no actionable merge-blocking risk remains after normal validation checks. Possibly related issues
Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Change
Make the repository-wide delivery rule explicit: ordinary development, review, CI, merge, artifact publication, and releases that do not cross a protected runtime boundary must not wait for a production Gate. Development and control-plane verification defaults to a focused check of no more than 15 minutes; the full Gate runs once at the actual collector/runtime transition.
The existing Polymarket 3600-second Gate implementation and fail-closed cutover policy are unchanged.
Issue relationship
Closes #865
Validation
git diff --checkRuntime impact and rollback
None. This changes repository delivery instructions only. Revert commit
5d11b3e9to roll back the rule.Summary by CodeRabbit