Skip to content

fix(claude): 恢复 #172 工具结果排序修复供重新审阅 - #180

Closed
3316891527 wants to merge 1 commit into
fix/revert-pr-170-176from
fix/reopen-pr-172-claude-tool-order
Closed

3316891527 wants to merge 1 commit into
fix/revert-pr-170-176from
fix/reopen-pr-172-claude-tool-order

Conversation

@3316891527

@3316891527 3316891527 commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner

变更说明 / 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

  • 已记录静态核验及未运行项原因。
  • 原生产补丁、测试修正、分支与备份完整保留。
  • 待 revert: 撤销 PR #170–#176,恢复合入前代码 #177 获准合入后将目标改为 dev。
  • 本轮固定候选 assembleDebug、testDebugUnitTest 与 Candidate checks 实际成功。
  • 等待维护者明确批准合并。

本项已随 集成 PR #191 合入 dev,并经 发布同步 PR #193 同步到 main。原来源提交 c0375d6de81c7e3b811effa89fbfcaf5fcb03f7f 已核对保留。

本 PR 原目标为 fix/revert-pr-170-176;由于 dev 已包含全部来源提交,GitHub 拒绝将目标改到 dev。关闭此重复待合并条目,代码与原始审阅记录均保留。

@3316891527 3316891527 added the ci:full-android Require assembleDebug and Android JVM tests for this PR candidate label Nov 9, 2026
Reapply the complete original merge diff from PR #172 on the reviewed rollback base.
Original merge: 798841d

Keep this work open for maintainer review. No merge or auto-merge without explicit permission.
@3316891527
3316891527 force-pushed the fix/reopen-pr-172-claude-tool-order branch from 76055fb to c0375d6 Compare November 9, 2026 03:16
@3316891527 3316891527 closed this Nov 10, 2026
@3316891527
3316891527 deleted the fix/reopen-pr-172-claude-tool-order branch November 10, 2026 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:full-android Require assembleDebug and Android JVM tests for this PR candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant