feat(archive): durable Maker strategy acceptance - #99
Conversation
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
|
Warning Review limit reached
Next review available in: 56 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (94)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 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 |
12af013 to
1bd2af1
Compare
Summary
strategy_dataMaker tables, with per-table retry isolation, fixed quota/retention, restart recovery, stable ClickHouse deduplication tokens, and explicit health/telemetryhb_runtimewire contract and v2 schema/DDL migrations against Maker commit563594435853c88cca5b187b8c999f845e31136bcanonical-cex-market-data-replay-archive; addmaker-archive-forwarder-conformancefor the cross-service boundary0.2.37; the tag is intentionally created only after this PR mergesContract boundaries
CEX_BROKER_*environment variables are introduced. The only new deployment option is the forwarder-localARCHIVE_FORWARDER_SPOOL_PATH; quota and retention are fixed contract constants.Verification
bun test— 593 pass, 0 failbunx tsc --noEmitbun run buildbunx @biomejs/biome lint .openspec validate maker-archive-forwarder-conformance --strictdocker compose -f services/archive-forwarder/docker-compose.yml config --quietgit diff --checkCross-service fixture
563594435853c88cca5b187b8c999f845e31136btest/fixtures/archive_forwarder_envelope.json784f647e048052a6c3382309b1a86abfbe08bc162363ead9fc88eaa1ba3d50c9Linear: FIET-901, FIET-903, FIET-909, FIET-937