fix(agent): bypass chat budget for image routes#1636
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)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThis PR implements model-aware context budget bypass logic in AgentRuntimePresenter. DeepChat's chat context budget preflight is now skipped for image-generation, TTS, and non-chat endpoints while preserving existing behavior for chat models and ACP-bypassed providers. The change flows through message processing, manual compaction, streaming requests, and resume handling, with a regression test validating image endpoint bypass. ChangesImage Generation Context Budget Bypass
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add 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 |
* fix(agent): bypass chat budget for image routes (#1636) * Initial plan * merge(gen-video): resolve agent runtime conflict Agent-Logs-Url: https://github.com/ThinkInAIXYZ/deepchat/sessions/7c9b17d6-272d-44a2-ab2d-57a554c773ab Co-authored-by: zhangmo8 <43628500+zhangmo8@users.noreply.github.com> --------- Co-authored-by: duskzhen <zerob13@gmail.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
…ngs (#1637) * feat: implement OpenAI-compatible video generation features and settings * Copilot/fix review feedback pr 1637 (#1639) * fix(agent): bypass chat budget for image routes (#1636) * Initial plan * merge(gen-video): resolve agent runtime conflict Agent-Logs-Url: https://github.com/ThinkInAIXYZ/deepchat/sessions/7c9b17d6-272d-44a2-ab2d-57a554c773ab Co-authored-by: zhangmo8 <43628500+zhangmo8@users.noreply.github.com> --------- Co-authored-by: duskzhen <zerob13@gmail.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> * fix: review issue --------- Co-authored-by: zhangmo8 <zhangmo8@users.noreply.github.com> Co-authored-by: duskzhen <zerob13@gmail.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Summary
Tests
Summary by CodeRabbit
Documentation
Bug Fixes
Tests