Fix model pricing refresh without PowerToys integration - #447
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughLocal usage enrichment now selects enabled Codex and Claude providers directly. PowerToys status-pipe availability no longer controls enrichment. Tests cover disabled PowerToys integration and supported-provider filtering. ChangesLocal usage enrichment
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to Local pricing refresh now works when PowerToys integration is disabled while continuing to refresh only enabled Codex and Claude providers. The covered behavior introduces no identified merge-blocking risk. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Comment |
Maintainer replacement for #442 because the original head is in an external fork. Tree is the reviewed #442 implementation unchanged.\r\n\r\nSupersedes #442.\r\n
Summary by CodeRabbit