Skip to content

refactor: remove high-confidence redundant code - #518

Merged
liujuanjuan1984 merged 1 commit into
mainfrom
chore/redundancy-cleanup
Aug 24, 2026
Merged

refactor: remove high-confidence redundant code#518
liujuanjuan1984 merged 1 commit into
mainfrom
chore/redundancy-cleanup

Conversation

@liujuanjuan1984

@liujuanjuan1984 liujuanjuan1984 commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

概要

  • 删除全仓无消费者的 JSON-RPC extension URI 映射及其冗余 imports。
  • 删除 ExtensionMethodRegistry 中从未读取的 specs 缓存与属性。
  • 移除单层认证 scheme 查询包装,并在 Agent Card 构建时一次性生成已配置 scheme 集合,避免重复构建凭据列表。

审查边界

  • 本次仅清理有静态引用、运行覆盖与仓库历史共同支持的高确信冗余。
  • 保留框架路由、协议接口、测试注入点、迁移版本函数及应用层/接入层预留边界。
  • 保留 DatabaseTaskStoreCompat:当前仍承担 terminal-state 原子写入与 SDK shape 检查,不属于死代码。
  • 不改变 Agent Card、OpenAPI、JSON-RPC 或 REST wire contract,因此无需文档迁移。

独立审查结论

  • 221599e 中删除的三个内部表面均没有运行时、测试、文档或 package re-export 消费。
  • Agent Card 仍只声明已启用凭据对应的 bearerAuth / basicAuth,与修改前行为等价;区别仅在于凭据列表从重复构建两次收敛为一次。
  • registry 的重复方法检测、method 集合自检和运行时 resolve() 分发保持不变,删除的只是无法观测的 specs 副本。
  • 删除的符号虽然未使用下划线命名,但不属于仓库记录或导出的受支持 Python API;未发现需要保留兼容别名的证据。
  • 未发现偏离任务范围、遗漏行为或需要追加代码修复的问题。

Commits

  • 221599e refactor: remove high-confidence redundant code

验证

  • bash ./scripts/doctor.sh
    • 821 项测试通过
    • 覆盖率 93.35%
    • Ruff、Mypy、ShellCheck、secret detection、依赖兼容检查、构建与 wheel smoke test 通过

Issue 关联

@liujuanjuan1984

Copy link
Copy Markdown
Collaborator Author

独立审查已完成,未发现阻塞性问题或需要追加修改的分支内缺陷。

  • 三项删除均无运行时、测试、文档或 package re-export 消费,并有历史与覆盖率证据支持。
  • Agent Card 的启用凭据过滤及 bearer/basic 声明行为保持等价。
  • JSON-RPC registry 的重复注册检查、method 自检及运行时分发均保持不变。
  • 升级 a2a-sdk 至 1.1.3 并审计下游兼容层 #515 仅为相关审计背景,本 PR 不解决或关闭该 issue;未发现应使用 Closes 的关联 issue。
  • 本地 bash ./scripts/doctor.sh 通过:821 tests,coverage 93.35%,构建与 wheel smoke test 成功。
  • GitHub Actions 的 Default Toolchain、Python 3.11 与 Python 3.12 checks 均已通过。

PR 标题与中文描述已按最终 diff、commit 和 issue 关系更新。

@liujuanjuan1984

Copy link
Copy Markdown
Collaborator Author

补充对照 liujuanjuan1984/codex-a2a#358 的审查结果:

  • 本仓库没有其清理的三类同构执行层冗余:流读取已直接使用 anext(),session claim release 已直接绑定 release_preferred_session_claim(),也没有单调用的 stream-start 日志薄壳。
  • 对方 PR 曾误删带显式 __all__、随 py.typed 发布且由测试验证的 jsonrpc.params facade,随后恢复。refactor: remove high-confidence redundant code #518 删除的符号没有 __all__、package re-export、文档或 facade 测试证据,不属于同类发布兼容边界,无需恢复。
  • 本仓库的明确聚合入口已保留:client/__init__.py 有显式 __all__contracts.extensions 聚合入口有大量生产与契约测试消费者;jsonrpc.params 承载实际参数解析逻辑,并非无消费者的空壳。
  • 对照后修正 PR 描述中的 issue 关系:升级 a2a-sdk 至 1.1.3 并审计下游兼容层 #515 不由本 PR 交付,因此取消 Related 表述,只保留已审查但不推进/关闭的事实说明。

结论:未发现需要追加代码改动的问题,现有测试与 CI 结论继续有效。

@liujuanjuan1984
liujuanjuan1984 marked this pull request as ready for review August 24, 2026 08:32
@liujuanjuan1984
liujuanjuan1984 merged commit 8aa02a7 into main Aug 24, 2026
3 checks passed
@liujuanjuan1984
liujuanjuan1984 deleted the chore/redundancy-cleanup branch August 24, 2026 08:32
@liujuanjuan1984

Copy link
Copy Markdown
Collaborator Author

后续治理备忘已拆分为 #519:用于明确 Python API facade 导出边界,并在决定正式承诺 contracts.extensions 后再添加显式 __all__ 与 import contract tests。#519 不属于本 PR 的交付范围,不影响当前 Draft PR 的代码结论。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant