fix: Correct token calculation logic for merge_system_prompt option #141
Conversation
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
修复了 merge_system_prompt 功能中的以下问题： 1. 修复了 max_usable_tokens 在合并模式下的计算逻辑 2. 移除了重复的 input_tokens 计算 3. 确保所有 token 计算路径的一致性 这些问题可能导致： - Token 计算错误，导致 API 调用失败 - 分块逻辑错误，导致内容处理不正确 - 输出 token 限制计算错误，导致响应被截断
Deploying rss-translator-home with
|
| Latest commit: |
efcf875
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://b3029366.rss-translator-home.pages.dev |
| Branch Preview URL: | https://fix-token-calculation-merge.rss-translator-home.pages.dev |
This was referenced Dec 11, 2025
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.
问题描述
在 PR #139 合并后，发现
merge_system_prompt功能中存在 token 计算逻辑错误。修复的问题
max_usable_tokens在合并模式下的计算逻辑merge_system_prompt=True时，需要正确计算 system_prompt 的 token 占用input_tokens计算（第 335 行）潜在影响
这些问题可能导致：
测试
相关 PR
修复 PR #139 中发现的问题。