Skip to content

feat: balance dispatch by quota run rate#721

Open
alimohammad-123 wants to merge 19 commits into
kunchenguid:mainfrom
alimohammad-123:fm/quota-runrate-dispatch
Open

feat: balance dispatch by quota run rate#721
alimohammad-123 wants to merge 19 commits into
kunchenguid:mainfrom
alimohammad-123:fm/quota-runrate-dispatch

Conversation

@alimohammad-123

@alimohammad-123 alimohammad-123 commented Jul 19, 2026

Copy link
Copy Markdown

What Changed

  • Score quota-balanced dispatch candidates by projected run-rate capacity across general session and weekly windows, excluding model-specific and code-review quotas.
  • Add validated quotaBalanced tuning for reserves, reset capacity, stale-data margins, and private lock-safe usage history, with account-free diagnostics and safe fallback behavior.
  • Harden locale-safe composer glyph parsing and seed rollback cleanup, and stabilize bootstrap and watcher test synchronization.

Risk Assessment

✅ Low: Captain, the prior blocking defect is correctly fixed by excluding quota-axi's code-review window IDs, and the updated selector remains bounded by conservative fallback behavior.

Testing

Completed 1 recorded test check.

  • Outcome: 🔧 1 issue found → auto-fixed (3) ✅ across 4 runs (2h41m0s)

Pipeline

Updates from git push no-mistakes

⏭️ **intent** - skipped

✅ No issues found.

✅ **Rebase** - passed

✅ No issues found.

🔧 **Review** - 1 issue found → auto-fixed ✅
  • 🚨 bin/fm-dispatch-select.sh:379 - Exclude feature-scoped Codex windows from general quota scoring. quota-axi emits code_review_five_hour and code_review_weekly with kinds session and weekly, so this filter includes them and can make low code-review quota incorrectly disqualify Codex even when its general quota is healthy. Restrict Codex general windows to the base window IDs, or explicitly reject code_review_*, and cover that schema case.

🔧 Fix: Exclude code-review windows from general quota scoring
✅ Re-checked - no issues remain.

🔧 **Test** - 1 issue found → auto-fixed (3) ✅
  • 🚨 tests failed with exit code 1
  • command -v tmux >/dev/null || { echo "tmux is required for e2e tests" >&2; exit 1; }; tmux -V; rc=0; for t in tests/*.test.sh; do echo "== $t =="; bash "$t" || rc=1; done; exit "$rc"

🔧 Fix: Captain: stabilize secondmate seed rollback cleanup
1 error still open:

  • 🚨 tests failed with exit code 1
  • command -v tmux >/dev/null || { echo "tmux is required for e2e tests" >&2; exit 1; }; tmux -V; rc=0; for t in tests/*.test.sh; do echo "== $t =="; bash "$t" || rc=1; done; exit "$rc"

🔧 Fix: Stabilize watcher stale-absorb test synchronization
1 error still open:

  • 🚨 tests failed with exit code 1
  • command -v tmux >/dev/null || { echo "tmux is required for e2e tests" >&2; exit 1; }; tmux -V; rc=0; for t in tests/*.test.sh; do echo "== $t =="; bash "$t" || rc=1; done; exit "$rc"

🔧 Fix: Captain: stabilize bootstrap timeout fixture synchronization
✅ Re-checked - no issues remain.

  • command -v tmux >/dev/null || { echo "tmux is required for e2e tests" >&2; exit 1; }; tmux -V; rc=0; for t in tests/*.test.sh; do echo "== $t =="; bash "$t" || rc=1; done; exit "$rc"
✅ **Document** - passed

✅ No issues found.

✅ **Lint** - passed

✅ No issues found.

✅ **Push** - passed

✅ No issues found.

@alimohammad-123 alimohammad-123 changed the title feat: balance quota dispatch by sustainable run rate feat: balance dispatch by quota run rate Jul 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant