ledger: CurrenciesIn's overflow-bound comment names takeLocked; foldLocked holds the reservation
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
core/cost/ledger/query.go の上限付き分数に関するコメントを読み、次に core/cost/ledger/writer.go の foldLocked にある予約条件を確認します。コメントが takeLocked ではなく foldLocked を示すように更新し、既存の上限の説明は維持してください。コメントが実装と一致し、レビュー確認 C14 に合格すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Found by the round-3 strict review of #1153, in a comment that PR's round 2 added.
core/cost/ledger/query.go says the capped minute keeps maxLabelsPerMinute-1 real rows, "Minus one because takeLocked reserves the last slot for overflowKey itself". The reservation is in foldLocked, not takeLocked — core/cost/ledger/writer.go carries if len(w.rows) >= maxLabelsPerMinute-1 in that function.
The number is right and the conclusion it supports (the overflow row is never the only row) holds. Only the function name is wrong — a mechanism stated without checking the symbol resolves, which is the specific mistake grep -rnE '^func +(\([^)]*\) *)?<Name>\b' catches.
Tracked red by the review harness as check C14, which re-derives the bound from writer.go rather than from the comment.
Assisted-By: Claude (Anthropic AI) [email protected]
- 主要言語
- Go
- スター
- 16
- フォーク
- 41
- 平均マージ
- 11時間 22分
- マージ済み PR（30日）
- 267
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
rossoctl/cortex のほかの issue
-
UI shows costs for all sessions even when drilled into a session対応中かも @esnible が 3 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
rossoctl/cortex#1116 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 22/100
メンテナーはふだん 1 日以内に返信
-
Add a coding-agent demo with cpex policy enforcement対応中かも @araujof が 1 日前に担当しました。 オープン
rossoctl/cortex#1428 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Release cortex-cpex binaries and add installer support対応中かも @araujof が 1 日前に担当しました。 オープン
rossoctl/cortex#1427 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Update the hr-cpex demo for CPEX 0.3対応中かも @araujof が 1 日前に担当しました。 オープン
rossoctl/cortex#1426 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
rossoctl/cortex の issue をすべて見る
似ている issue
-
Realtime: ASR EOU language tags leak into the LLM conversation (unintended reply-language switching)対応中かも @nexxtmobile が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 1 日以内に返信
-
effort:high model:sonnet phase:3 type:docs
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
OffchainLabs/nitro#4771 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
BurntSushi/toml#530 ·