fix: recognize Codex Team plans in pool capacity - #1443
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Codex capacity estimator now assigns configured weight ChangesCodex capacity aggregation
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
|
✅ Deterministic PR hygiene checks passed. |
✅ READY
Review readiness checklist
✅ 4/4 boxes ticked. This pull request is already Ready for Review. |
Treat upstream plan 'team' as the Business-tier configured weight so valid Codex pools are not excluded from dashboard capacity estimates.
21b40ee to
b088583
Compare
Rate Limit Exceeded
|
Rate Limit Exceeded
|
Rate Limit Exceeded
|
Rate Limit Exceeded
|
|
Thanks @terrytan95 — this was useful because Codex still reports some accounts as plan Merging now. |
Summary
teamplan identifier as the Business-tier configured capacity weight.Closes #1442
Verification
bun test tests/provider-capacity.test.ts— 14 passed, 0 failed.PATH=/Users/terrytan/.nvm/versions/node/v24.3.0/bin:$PATH bun run typecheck— passed.bun run privacy:scan— passed.PATH=/Users/terrytan/.nvm/versions/node/v24.3.0/bin:$PATH bun run test— 10,813 passed, 7 skipped, 0 failed.git diff --check— passed.Checklist
Review readiness checklist
All CI tests are green on my local testing.
I pushed my PR to the latest dev commit.
I resolved all correct Codex and CodeRabbit findings.
My PR is ready for review.
Summary by CodeRabbit
Summary by CodeRabbit
New Features
Bug Fixes