Repository navigation
fix(claude): 恢复 #172 工具结果排序修复供重新审阅 - #180
Closed
3316891527 wants to merge 1 commit into
Closed
3316891527 wants to merge 1 commit into
3316891527 wants to merge 1 commit into
Conversation
2 of 5 tasks
3316891527
force-pushed
the
fix/reopen-pr-172-claude-tool-order
branch
from
November 9, 2026 03:16
76055fb to
c0375d6
Compare
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.
变更说明 / Description
将 Claude 历史中的 tool_result 按对应 tool_use 的原顺序排列,保持结果 ID 与内容一一对应,使等价请求的工具结果顺序和提示缓存前缀稳定。
背景与动机 / Context and motivation
维护者要求撤销 #170–#176 并恢复 open 审阅。原 #172 已合并,GitHub 无法重开已合并 PR,因此以本 PR 重新审阅。未经维护者明确允许不得合并或启用自动合并。
改动范围 / Changes
完整恢复 #172 的 2 文件补丁,包括 ClaudeProvider 和 ClaudeProviderHistoryTest。保留 private 方法及原 JVM 签名;测试保存、禁用并恢复 AppLogger 系统和文件日志设置,避免 JVM Android 日志干扰。
兼容性与风险 / Compatibility and risks
仅调整已匹配工具结果的顺序,不改写工具 ID 或结果内容。当前暂以撤销 PR #177 的分支为目标,仅展示本项差异;待 #177 获准进入 dev 后应改为 dev 并重新检查,不能合入临时基准分支。
关联 Issue / Related issue
原 #172;撤销与恢复审阅要求见 #177。
验证方式 / Verification
初次重建时已逐项核对原完整合并补丁。本轮在清理历史空白并补齐 CI 的共同回滚基准上重放该项,保持独立功能差异。当前 PR 启用 ci:full-android,要求固定合并候选实际运行 assembleDebug 与 :app:testDebugUnitTest,Candidate checks 必须检查两项真实成功。新的结果以本 PR Checks 为准;历史 CI 不代替本轮构建和测试。
证据 / Evidence
原实现及历史 JVM 修正见 #172,回滚基准见 #177。
检查清单 / Checklist
本项已随 集成 PR #191 合入
dev,并经 发布同步 PR #193 同步到main。原来源提交c0375d6de81c7e3b811effa89fbfcaf5fcb03f7f已核对保留。本 PR 原目标为
fix/revert-pr-170-176;由于dev已包含全部来源提交,GitHub 拒绝将目标改到dev。关闭此重复待合并条目,代码与原始审阅记录均保留。