Skip to content

fix(codex): calibrate Business Premium display capacity - #7444

Closed
Ingwannu wants to merge 1 commit into
lidge-jun:devfrom
Ingwannu:fix/business-premium-capacity-7440
Closed

Ingwannu wants to merge 1 commit into
lidge-jun:devfrom
Ingwannu:fix/business-premium-capacity-7440

Conversation

@Ingwannu

@Ingwannu Ingwannu commented Oct 20, 2026 •

Copy link
Copy Markdown
Owner

Summary

Verification

  • Independent Bun 1.4.3 regression, bun test tests/providers/provider-capacity.test.ts: before implementation 18 pass / 1 fail; after implementation 19 pass / 0 fail. The added case checks two Premium labels (including case/space normalization), Team weighting, weighted weekly consumption, calibrated coverage and absence of fabricated burst windows. Existing unknown-plan, stale, pause/reauth, mixed-window and prototype-key negatives remain green.
  • bun scripts/structure-ssot.ts, bun scripts/privacy-scan.ts, and git diff --check pass. Ordinary privacy checks are allowed; no Codex Security scan was invoked.
  • bun node_modules/typescript/bin/tsc --noEmit reached the finite 300-second local limit and was terminated. Typecheck is unverified locally, not passed. The full suite is disproportionate for this four-file calibration on the shared host; focused behavior is verified and current-head hosted CI remains required.
  • Each local workload runs alone under hard CPU/RAM/swap/process/time limits and fresh private homes; the production configuration and live services are untouched. Root dependency map/lock matches the existing frozen install. No paid upstream calls, live account mutation, sync/restart or runtime installation occurred.
  • The isolated documentation frozen install completed with lifecycle scripts disabled. Local docs build rendered the static site/assets but reached the same finite 300-second limit during Pagefind indexing; it is not a successful local docs build. The unchanged candidate's current-head hosted docs site build job passes.
  • Current-head hosted gates passes, including Typecheck and Privacy scan; docs site build, structure, catalog, hygiene, target and Docker checks pass. All four selected Ubuntu shards and the aggregate ci now pass in current-head run 39421778705. macOS/Windows remain unselected. Keep Draft pending independent review. This PR is not a merge, deployment or release.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults (no credential/destination/auth boundary is changed).

@coderabbitai

coderabbitai Bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Deterministic PR hygiene checks passed.

@lidge-jun

Copy link
Copy Markdown
Owner

Carried onto current dev in #7557, with a Co-authored-by trailer for @Ingwannu. The code, test, and inventory note are unchanged; the docs paragraph moved below the section introduction. Closing this draft is left to a maintainer.

@lidge-jun

Copy link
Copy Markdown
Owner

Thank you for this work. It has been carried onto current dev in #7557, with your authorship kept as a Co-authored-by trailer, and review continues there. Closing this PR in favour of #7557.

@lidge-jun lidge-jun closed this Oct 22, 2026
lidge-jun added a commit that referenced this pull request Oct 24, 2026
* fix(codex): calibrate Business Premium display capacity

Business Premium seats report the plan string self_serve_business_prolite,
which missed the configured capacity map and counted at the baseline
weight 1. OpenAI documents Premium as 5x Standard included usage, so
calibrate it at 5 in the display-only pool capacity estimate.

Refs #7440
Carries #7444

Co-authored-by: Ingwannu <[email protected]>

* docs(codex): move capacity calibration rule to openai-tiers and correct unknown-plan wording

---------

Co-authored-by: Ingwannu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants