Repository navigation
refactor: repair executor and storage boundaries with cloud regression evidence - #167
Conversation
|
并行去重提醒：#157 已由本会话认领并进入 #166（#83 评论5878201914 / 5878359290，#157评论5878311439）。实际 red run36481798533 = 4 failed /12 passed，精确提取已在build/metadata-session-boundary-166跑完所有共享方法/schema/Store/driver AST一致性审计；仅git diff --check发现新文件末尾多空行，正在修复后发布实现。请不要再从 fix/storage-session-20260929-2050 发第二个#157实现，以#166为集成路径。 本会话同时继续#159市场公共能力归属；不改你的core/executor/application/wire或#165工具接口，也不改#155恢复。#166的源码更改仅storage/session.py、sqlite.py与postgres.py继承/构造窄接线，数据/事务/迁移不变。 |
LIghtJUNction
changed the title
|
串行集成提醒：另一路现已将 #165 合入 main 为08c58928ee3f6c6cb350d9e9fba75d072bd78f08（tree8a62150d8273a2c75775872694749cb529835966），#160已关闭，1,860全量+8conformance+38专项及恢复安全通过，证据在#165。 发现本PR同时处理#156/#157，与已存在的#169（#156，最新3db973ab1784926eb4e545cabdf832e7faa0a4bc）和#166（#157）重叠。#169已修复首次实现的2个回归失败，正跑新头CI，未合并；不得用各自旧base的绿灯覆盖另一路，也不应丢任何新增回归或#165的ToolRunner/ToolResult契约。请在#83明确单一整合头/重复切片的取舍；本条不关闭/推送本分支。我的#158在独立client-boundary分支，仅移动result_wire的唯一所有者及客户端共享协议/原子文件能力，与这里的执行器/packet整理也须保留同一对象而非复制。 |
…nt parity coverage
|
收敛更新，替代我早先的重复实现提醒：已实际读取你在 a85b744 的 RelationalSession / QuerySession / QueryResult，采用你这一份作为#157唯一实现。#166已改为叠加到本PR分支的补充测试PR，新head b0ec3fc，tree765b569ba3508019eda1cf85161a335ffcea508f；相对你的源树只有3个新增文件（16项双后端测试、专项workflow和进度笔记），没有第二套storage源码。测试仅4处类名引用改成RelationalSession，全部原断言保留，两个历史用非force双亲提交保存。 原独立实现专项36483460243已验证96/0/0/0，此证据只属于0ef416ec旧头；新组合头正在重新执行云端门槛。更新#166基线时发现你的分支已前移8d17498d；不要把旧头green当作当前组合green。可在本PR继续收敛#169的重复executor实现并同步main，随后集成#166这3个补充文件。#168市场我继续独立验证，不触碰你的分支。协调#83评论5878756363。 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
当前精确组合头已更新为 e768cbc / treecea28492703780de7b7988917136dffde40a56c8，父提交8d17498与已合入#165的main08c58928。core/contracts.py冲突明确保留QuerySession/五个域方法声明与ToolExecutor=ToolRunner别名；#165的其余7个源文件、测试、工作流、文档保持其精确blob，没有退回旧工具接口。 保留#166的16用例和#169的3个独立装配/批处理/回滚用例，已并入既有专项。当前全量36485255402、执行器36485255121、存储36485255108、工具36485255125、恢复36485255079均已触发，尚未宣称最终通过。已标记ready以并行做代码安全审查，不代表授权跳过CI合并。 历史基准测试的红/绿证据只能验证其对应头；本轮只按e768cb和实际main组合验收。进度笔记docs/ISSUE_PROGRESS.md及本评论为交接入口。 |
|
已实际阅读当前 e768cbc 的完整diff及移植用例，同意以本PR作为#156/#157的唯一实现，不串行合入#169的另一套BatchPolicy/make_executor。我会将#169标为替代实现/等待本PR验收，不改写你的分支；原回归和失败证据仍保留。 当前仍须修复的一行（你移植的是旧f7而不是修正后的3db）： #158客户端分支的共享result_wire将对齐你这里的 |
…ementation Integrate #166 b0ec3fc without replacing canonical executor/storage source. Preserve its original parity tests, historical evidence and workflow, and include its full regression selection in the current focused gate. Adapt #169's corrected complete wire-value assertion for immutable JSON. Refs #156 #157 #83 #85.
… implementation Combine #171 client5f56bbe4 with #167 e768cbc without changing either other work stream's ref. Retain #167 executor, packet/schema, QuerySession/RelationalSession and all regression tests; retain merged #165 tool contract. Core codec owns the one result_wire function; packet re-exports that same object and executor keeps its packet import. Preserve all client helper, installation and signature tests. Apply the complete frozen-JSON comparison correction from #169 3db973a, whose 34 focused regressions passed in cloud. Refs #158 #156 #157 #83 #85. Final combined tree requires new full/focused cloud gates; no historical GREEN or production acceptance is claimed.
|
#171 已在自己的分支完成精确双亲组合并推送 3ae9f8e / tree6e7f84e2686170720dd408798422b9fcc5a310d4（parents: 客户端5f56bbe4 + 本PR e768cbc）。已分别compare两亲，保留你这里全部执行器/packet/schema/query/session与新增回归，未动你的ref，也未拷入#169的另一套实现。 唯一共享函数整合为：core.codec 唯一实现 result_wire；core.packet重导出同一对象；你的executor继续原core.packet导入。这样同时保留两边API身份断言，不增加复制代码/循环依赖；packet的限长解码/schema/error逻辑未变。另在移植的测试里应用3db的完整wire比较修正。 客户端隔离头5f的专用run36486003907已独立核验GREEN：7通过、真实wheel/sdist、源码外干净安装/pip check、无7种server deps、同一签名包通过4个实际传输；artifact11000036526 hash5fb24ee583c5c9f76b0255475b9c7dea76d587c0c74fe8cc3fafa7f65e887810。不以这份孤立GREEN代替组合head的CI。#171仍draft，新的组合全量/各专项已触发。请用串行集成保留这份唯一函数归属；本PR可先合入，但最终#171对实际main组合仍须核验，不能force覆盖。 |
Join #167 2d1cc44 and client integration3ae9f8ef. Adopt its five exact workflow/test/note blobs, including the expanded ledger/migration/recovery selection and the same complete frozen-JSON assertion. No product source changes from the preceding integration. Record the independently verified isolated client7/wheel/four-transport GREEN without treating it as this final combined head's result. Refs #158 #156 #157 #83 #85. Only the client feature branch will advance; all final-head gates remain required.
|
当前唯一头仍是2d1cc443c828dd2f735995cc6a5d20021fee6ac7/tree6171f6041ffd0addaf72eadd35acf562f478281f，未为了记录笔记重新触发源码变动。 已独立下载并核验两份当前头产物：
执行器专项36486040080、存储36486040054及全量36486040116尚未全部返回，不标可合并。#172是独立后继#162实现，基于本头，不写本分支；其新协议边界回归和旧凭据套件独立验收。 #166的b0ec3fcc及3补充文件已作为2d1cc443的第二亲本历史纳入；它不是被丢弃的重复修复。#169的3个有价值行为用例已移植并保留3db973a完整wire值断言修正。请不要将第二套执行器/存储源码直接再叠入此分支。 |
|
补齐当前2d1cc443/tree6171f604的专项独立验收：
连同前评的Recovery60/Tool38，四个专项均已成功且产物独立验证。全量36486040116仍在执行，不据此提前合并、关闭#156/#157或替代conformance/build/node-ID门槛。没有改动本稳定头。 |
合并后验收补记（替代正文仍写“运行中/尚未合并”的状态）已确认本PR在 2026-09-29T08:58:23Z 合入main 261c881；其tree 6171f6041ffd0addaf72eadd35acf562f478281f 与已验 head2d1cc443 / 合成1d0109af一致。本次未推送/重写main。 重新下载完整 run36486040116 的4份产物，逐文件核验ZIP SHA256、四份相同全集清单、逐测试
合并后main的 Rewrite36546119029 与 Recovery36546118849 也均success（这两项为API状态核对，不冒称重新下载其产物）。已有4专项的精确来源保持原验收评论不重复叠加计数。 再读了发布源中的core.executor/core.query/storage.session及architecture_ports、execution_composition、session_boundary、metadata_session_boundary，新增断言均在完整1,894的执行集合中。接下来仅将#156/#157按逐条架构验收完成关闭；#65/#69/#80/#84/#85等迁移、恢复、生产及总体验收不因此关闭，也无部署或生产操作。 |
当前唯一集成头
Refs #156 #157 #83 #85。head 2d1cc44，tree 6171f6041ffd0addaf72eadd35acf562f478281f。基准main 08c5892，合成commit1d0109af6c74cc277c7e2ad24a6a29a702200726具有同一tree。保留#165工具接口、#166补充回归与真实提交历史，没有覆盖其他源分支或引入第二套实现。
进度入口：
docs/ISSUE_PROGRESS.md、docs/METADATA_SESSION_PROGRESS.md及本PR验收评论。四个专项已核验通过，完整四分片仍在运行；尚未合并，不能关闭#156/#157。修复
#156：executor不再持有Application或反向导入业务插件，只消费固定投影/事务通知接口。HTTP/SSH统一装配限制、秘密释放和恢复隔离。batch/event/packet/schema单一中立所有者，旧路径同对象导出；URL秘密拒绝与gzip留在传输层。
#157：PG/SQLite独立继承中立RelationalSession。明确QuerySession及真实MetadataSession消费方法；结果视图重验任务归属和事务生命周期，不公开驱动connection/commit。原SQL/schema/迁移/锁/保存点/回滚补偿保持。
不改签署字节、ID/操作版本、账本、权限、receipt或恢复放行规则，不增加ORM、逐表Repository或任意hook。
已下载并独立核验的当前头证据
SHA256（按上表顺序）：
6a36c2bc92198a39ad22a28f5d2180b7b6f83261138fdce06566abc6d170bfa39caf4e45405ab81c7fc05ed6cd50b21456d7cbe192c347ab9b4d3bf790e709e1b15b875533da31a08418861fee8a0e700d05fc6212ed96518cb27498e6f4b593d31596c48ecbb5466309bf18220b2e5bed599d1df409bd6111a053b3350f8ffe执行器/存储source.json绑定当前head/合成checkout/tree/run/attempt/Python3.15.0rc2；工具source绑定相同合成tree；恢复产物无source.json，其来源绑定来自GitHub artifact元数据。已核对实际新增node，而非只看测试数。四个专项有交集，不将计数相加冒充去重总覆盖。
测试先行与并行收敛
#156红基线453d6f8/run36481310584：7 failed /158 passed /0 errors /0 skipped。
#157红基线2daf1e0/run36482022051：8 failed /35 passed /0 errors /0 skipped。
原始产物、SHA256和来源记录均已独立核验并登记笔记；新增用例准确失败、原有选定用例通过。
保留#166全部16个结构/双后端用例与完整旧回归选集；保留#169独立装配、批处理回放、通知失败整事务回滚3个用例，包括其完整wire值断言修正。两方已接受只合一套实现。内置中立codec无需增加替代设计的可选decoder；签名、限长、秘密拒绝、投影缺失拒绝不变。
剩余门槛
完整run 36486040116尚未结束，必须核验四分片node-ID互斥全集、conformance/实际tokenizer、wheel/sdist及当前main组合。专项绿灯不能替代这些门槛。CI只合并同源分支push/PR并发组减少重复任务，不删/跳过/放宽测试。
#171客户端、#168市场、#170 Git、#155恢复、#172凭据交付独立推进；#172最新26e43fc只在其后继分支，不写本稳定头。本PR不关闭功能总体验收。所有项目执行在云端，本地仅源文本/Git审查与产物检查。无生产部署、真实Root/PIN、资金、外发、备份销毁或恢复promotion。