feat: rank marketplace offers with effective prices#26
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughMarketplace candidate pricing in llm_policy.lua now prefers effective price fields from offers, falling back to legacy price fields, and adds a price_multiplier from ranking_price_multiplier. The execution result's chosen payload now also includes this price_multiplier. ChangesEffective pricing and multiplier propagation
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 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 |
Summary
effective_price_*when presentprice_multiplierinto the chosen route so the host can report cost against the raw quoteValidation
unhardcodedfocused test:tests/test_live_wiring.py::test_marketplace_offers_rank_with_effective_prices_when_presentSummary by CodeRabbit
New Features
Bug Fixes