Refactor output.rs into output/ submodule#129
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (8)
✨ Finishing Touches🧪 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 |
Break the 1,282-line monolithic output.rs into focused submodules: formatters, agents, definitions, execution, system, and tests. No behavior changes — all 93 tests pass unchanged. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The print_agent_def_detail and print_schedule_detail functions mirror the Response enum variant fields directly. Adding the allow attribute satisfies CI clippy (-D warnings) without restructuring the API. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Handle the new Response::AssignTriggerResult variant added in PR #131, delegating to agents::print_assign_trigger_result. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
0a38b24 to
9e030b8
Compare
Summary
output.rsinto a focusedoutput/submodule with 6 files:formatters.rs,agents.rs,definitions.rs,execution.rs,system.rs, andtests.rscheck_response+print_response)AssignTriggerResultresponse variant (from PR feat: Make triggers opt-in and fix inject submission #131)Test plan
cargo build -p pu-cli— no warnings or errorscargo clippy -p pu-cli --all-targets— no warningscargo test -p pu-cli— 93 tests pass🤖 Generated with Claude Code