fix(history): enforce delivery scope availability - #48
Conversation
Apply AI Review suggestion Verified with confidence: 100/100 AI-Review: #47 (comment) Resolves: #47 (comment)
|
Warning Review limit reached
Next review available in: 43 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
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 |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
问题
PR #47 已合并后,审查发现发送情况查询仍有三条提前返回路径会绕过当前成员范围检查:未追踪的旧记录、来源缺失记录和远端记录。因此当前成员范围不可用时,这些查询仍可能错误地返回成功。
修改
验证
cargo test -p uc-application --lib current_peer_scope_is_unavailable --locked -- --nocapture:3 passedcargo test -p uc-application --lib clipboard::sync::get_entry_delivery_view::tests --locked:20 passedcargo test -p uc-application --lib --locked:833 passedcargo check --workspace --all-targets --lockedcargo metadata --locked --format-version 1cargo fmt --all -- --checknode scripts/architecture/check-engine-repository.mjsgit diff --check审查来源
验证边界
真实双设备流程未执行。