Skip to content

refactor(plugin): 删除 setup helper v2 壳 - #4

Closed
kachofugetsu09 wants to merge 1 commit into
codex/setup-helper-v3-commandsfrom
codex/setup-helper-v3-cleanup
Closed

refactor(plugin): 删除 setup helper v2 壳#4
kachofugetsu09 wants to merge 1 commit into
codex/setup-helper-v3-commandsfrom
codex/setup-helper-v3-cleanup

Conversation

@kachofugetsu09

@kachofugetsu09 kachofugetsu09 commented Aug 14, 2026

Copy link
Copy Markdown
Member

结果

#3 的双入口行为等价与显式 Session write-set delta 基础上,删除 SetupHelper v2 class、ChatIdCommandModule、legacy command normalization 和 BeforeTurnCtx 适配,只保留 v3 Commands 实现。

边界

  • 本 PR 的相邻 diff 只删除 v2 壳并把测试 oracle 切到已固定的 v3 输出。
  • 不修改 command name/alias、Telegram catalog、QQ 配置优先级或回复文本。
  • 不修改正式安装、workspace、Session、plugin-data 或 manifest。
  • 必须在 feat(plugin): 迁移 setup helper 到 Commands #3 和 Core #414 之后评审/合入。

不可变身份

  • base: 6e15fdcc9f93f9fc44e7f841e55e69727738debe
  • head: fc22a5ad1a24b3d3998a0ba724e4b4fdefa11618
  • Core: 8c45ab7873aac8c9eebcd22e9d6ae123f8dcf616
  • Plugin contracts: 4dd69dd621e029e51e99aa428443fa3a4ec1f6cf

验证

  • pytest -q -W error tests/: 10 passed
  • BasedPyright: 0 errors;其余为外部 Core stub 与既有 strict warnings
  • Plugin API v3 static contract: passed,plugin_classes=[],source SHA-256 c43a341979ec77b880731321d5e453a30ec009dc2ce19953f37e681a3fb96cb3
  • compileall 与 git diff --check: passed
  • 真实 Manager/pipeline 仍证明 command short-circuit、Session/Context/Reasoner/append 零调用和 teardown 资源归零。
  • GitHub Actions:以本 PR 状态为准。

回滚

Revert 本 PR,或回到 backup/setup-helper-v3-cleanup-before-restack-20260814@ab9854afe806897d9ba519a86cb448eebd65374e#3 仍保留完整 v2 oracle 和已固定的持久写入差异。

@kachofugetsu09
kachofugetsu09 force-pushed the codex/setup-helper-v3-cleanup branch from ab9854a to fc22a5a Compare August 14, 2026 15:49
@kachofugetsu09

Copy link
Copy Markdown
Member Author

当前 main 已包含 pure-v3 最终实现与对应验收结果;此旧迁移 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