Repository navigation
Conversation
Give the post-fit over-budget check its own error code. Image block counts stay on content references.
|
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. Delightful! Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
Requirement or Bug
区分导入时「预算不够」和「装完再称仍超预算」两种失败,并去掉会被覆盖的图片块计数。
Bug Reproduction Steps
N/A
Root Cause
N/A
Code Changes
renderHistory if reserve or no turn fits throw INSUFFICIENT_TOKEN_BUDGET assemble message - if estimate exceeds budget - throw Error without code → CONVERSION_FAILED + if estimate exceeds budget + throw BUDGET_ACCOUNTING_FAILED makeSteps - image or document block: increment imageBlocks or documentBlocks + document block: increment documentBlocks + image block: do not increment readClaudeSession imageBlocks = imageReferences.contentReferencesBehavior Changes and Affected Users
convertSession的error.codeCONVERSION_FAILEDBUDGET_ACCOUNTING_FAILEDconvertSession调用方。CLI 导入页只显示原句,不看码Rendered import exceeded its token budget./import-sessions或kimi import-sessions的人stats.imageBlockscontentReferences覆盖contentReferences受影响模块:
packages/session-import。pnpm --filter @moonshot-ai/session-import test,195 项通过。没有能打中BUDGET_ACCOUNTING_FAILED的现有用例,这次也不新写。Checklist
/approve).gen-changesetsskill, or this PR needs no changeset.gen-docsskill, or this PR needs no doc update.