Repository navigation
Conversation
… for (yetone#517 follow-up) — 重置前用完 read as the soonest renewal first, which is Smart's order, when the point is the opposite: each account's week is used up evenly until its own reset, so none runs out of its week early and the five-hour windows later on still have several accounts with quota. The English name and its pace wording stay; the provider editor's, the group editor's and the Routing tab's hints lead with that (uses each account's week up evenly until it renews), then how it ranks; zh takes 根据剩余周额度和重置时间分配使用，避免过早耗尽某个账号的周额度，让后续更多个 5 小时窗口仍有多个账号可用, with the details as before, and the Routing tab's stories begin 额度均衡. ja (週ペース) and de (Wochentempo) keep their names, their hints translated anew for the new English. The docs' routing tables (en, zh), README's routing= row and magpie group's help say the same; routing-save.test.cjs expects 额度均衡. No routing logic changes. go vet, go build -tags nogui, go test -tags nogui ./internal/gui/ . pass. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e, as the others are Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…eek used up evenly, the most left per hour first, so later five-hour windows still have several — about as long as they were before; ja and de follow, zh unchanged; README's row is the first half alone Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Note
Copilot was unable to run its full agentic suite in this review.
Copilot review overview
Review effort: Lite
Findings: 2
Open (3)
What changed in this PR
Updates the “pace” routing mode’s user-facing explanation and Chinese label to reflect a more “balance weekly quota” behavior, keeping docs/UI/help text in sync.
Changes:
- Updated the “Weekly pace (pace)” description across docs and GUI strings.
- Renamed the Chinese label for “Weekly pace” from “重置前用完” to “额度均衡” in docs/UI/i18n.
- Adjusted GUI tests to match the updated Chinese label.
| File | Description |
|---|---|
| site/public/docs/zh/start.html | Updates Chinese documentation text and route naming for pace. |
| site/public/docs/start.html | Updates English end-user docs for pace to the new explanation. |
| internal/gui/tests/routing-save.test.cjs | Updates expected Chinese label for pace to keep tests passing. |
| internal/gui/assets/routing.js | Updates routing tooltip/description strings for pace. |
| internal/gui/assets/i18n.js | Updates Chinese translations and related localized strings for pace. |
| internal/gui/assets/app.js | Updates GUI routing descriptions for pace in the app asset bundle. |
| groups_cli.go | Updates CLI help text for pace to match new behavior description. |
| README.md | Updates README table description for pace to match new behavior description. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| usage the account or key with the most of its allowance left first | ||
| pace the account with the most of its week left per hour until it renews first, so less | ||
| of a week is lost at its reset | ||
| pace each week used evenly up to its reset, so later five hours still have several accounts |
Contributor
Author
There was a problem hiding this comment.
就是在说每周均衡而非账号的均衡，后者是最少使用
| <ul> | ||
| <li><b>不会无限递归。</b>一个组不能包含它自己，无论是直接包含，还是经由它里面的组间接包含。Coder 已经包含 Fast 时，再把 Coder 放进 Fast 会被拒绝，选择器里也不会列出这样的组。最多嵌套 8 层。一个组被别的组包含时不能删除，要先从那个组里移除它。</li> | ||
| <li><b>每一层按自己的路由。</b>外层组不管用什么路由，都不会伸进它里面的组：智能、轮流、最少使用、重置前用完都把子组当作一个成员，拿子组自己会先用的那个账号或 Key 来比；排到它时，子组按自己的顺序整体尝试。Coder 设为智能、里面放 Claude（按顺序）和 GPT（按顺序）两个组时，比较的是 Claude 组的第一个和 GPT 组的第一个，不会把两组的账号混在一起排。</li> | ||
| <li><b>每一层按自己的路由。</b>外层组不管用什么路由，都不会伸进它里面的组：智能、轮流、最少使用、额度均衡都把子组当作一个成员，拿子组自己会先用的那个账号或 Key 来比；排到它时，子组按自己的顺序整体尝试。Coder 设为智能、里面放 Claude（按顺序）和 GPT（按顺序）两个组时，比较的是 Claude 组的第一个和 GPT 组的第一个，不会把两组的账号混在一起排。</li> |
| rotate: ["In turn", "In turn: each conversation's next turn goes to the account after the one that answered its last, and a new conversation starts one further along; the requests within a turn stay put, keeping the prompt cache."], | ||
| usage: ["Least used", "Least used first: the account with the most of its allowance left goes first; a key by the tokens magpie sent it lately."], | ||
| pace: ["Weekly pace", "Weekly pace: the account with the most of its week left per hour until it renews goes first — the one with the most to lose at its reset; an account with five hours and no week by what its five hours have left per hour. One at 90% or more waits until the others can't answer; a key by the tokens magpie sent it lately."], | ||
| pace: ["Weekly pace", "Weekly pace: uses each account's week up evenly until it renews — the most of its week left per hour first — so later five-hour windows still have several accounts; one with five hours and no week goes by its five hours. One at 90% or more waits until the others can't answer; a key by the tokens magpie sent it lately."], |
Contributor
Author
There was a problem hiding this comment.
原意更清楚，相信opus 5.5，copilot是gpt不会写文。前文短句更多好阅读，后文要到per hour再想想才知道意思
Owner
|
@pc418 谢谢这个 PR，文案确实比「重置前用完」清楚。我把 head 核对
测试
为什么先不合并 |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#517 合并后，中文名「重置前用完」带的意思并不明确直观，对功能的说明不容易理解：按紧迫度、先用快到期的，那是「智能」已经在做的事。
Weekly pace 真正的目的，是让每个账号的周额度都按各自的重置时间匀速用掉，不让某一个账号提前见底，这样后面更多的 5 小时窗口里都还有多个账号可用。该算法，在重置时间差不多的情况下相当于最少使用，差别较大时相当于智能，所以哪个都不准确。
这个 PR 只改文案，不动路由逻辑：
magpie group帮助文本同步；routing-save.test.cjs的中文名断言更新。中文名的备选，供 @yetone 定：
验证：
go vet、go build -tags nogui、go test -tags nogui ./internal/gui/ .通过；只改了文案，浏览器测试没有跑（routing-save.test.cjs里的中文名断言已同步更新）。