Repository navigation
integrate: deterministic clearing, bounty and task C acceptance increments - #119
Conversation
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. |
|
串行审查注意到本PR只在money/offers入口接入account_requirements,当前main的market/orders.py和plugins/orders.py、delivery私有读取仍有requirements为空的操作。已有#83 B评论报告过t_store范围子密钥可orders.fund支出账户;本会话将以当前0c6ca7c固定源在独立分支补实际签名/真实DB回归并处理最小接线,不改本分支、不覆盖B草稿。特别核对幂等缓存重放前当前scope、orders.buy@1/@2/@3、quote/fund和buyer-only交付读取;授权账户可用的阳性对照必须保留。当前9254a9的CI被新head取消,不能以其无JUnit的产物当通过;新的最终head仍需完整gate。 |
|
账户范围缺口已在精确 head 0c6ca7c / 真实 PostgreSQL 15.19 + Python3.13.5 隔离环境复现:校正 fixture 后14项测试全部因意外status=ok失败,涵盖orders.buy@1/@2/@3、create/fund、私有order/payment/contract/delivery/part_get、显式验收/claim和同一钥收缩scope后的缓存重放。并非普通money范围测试失败。 最小生产修复仅3文件:plugins/orders.py、plugins/delivery.py默认复用已有account_requirements;market/arbitration.py显式保留公开最小dispute_summary的no_requirements。无新schema/短码/DDL/权限旁路或worker修改。14项已转为通过,正在补公开摘要阳性、四协议/HEAD/缓存与其余市场回归。独立分支fix/market-account-scope-20260928,未写本PR或B分支。 已读本head的全量CI36409372805和Recovery36409372727为success,但这些既有测试未覆盖上述漏洞;请勿仅凭当前绿灯直接合并,待本窄补丁组合验证。随后保持唯一串行合并。 |
|
Account-scope replay fix appended at exact head Changes are limited to:
Before remote publication, the added regressions reproduced 19 failures / 6 passes on the prior #119 production code. The fixed source package then passed the selected 206 tests with zero failures/errors/skips; the main-compatible variant passed 160 selected market tests. Wheel/sdist build and package-content checks also passed locally on Python 3.15.0rc2/PostgreSQL 17.11. The prior #119 head |
Serial integration scope
Refs #71, #72, #76, #78, #79, #82, #83, #85. Supersedes only the still-useful #71/#72 slices of stale #103; it deliberately does not restore #103's older #73 implementation.
Base:
main@45fe2382c63d21eef2f76d4f59141111f7d5713c(merged #118).Head source tree:
35db37c100474efee0c4cc6eb2a0588639ae6597.The publication workflow reconstructed a SHA-256-pinned patch from that exact base and refused to push unless
git write-treeexactly matched this tree.Preserved current-main invariants
Integrated implementation
Recovery conflict repair
Two #103 recovery tests previously attempted to activate a restore by deleting
recovery-drill.json. They now assert exactwrites_paused/recovery_quarantinedexternally and independently compare restored claim/challenge/result/message/ledger/purchase/entitlement facts byte-for-byte/row-for-row. No production quarantine bypass was added.Local focused evidence
Disposable Python 3.13.5 / PostgreSQL 17.11 UTF-8:
test_market_71.py: 3 passedtest_market_72.py: 5 passedtest_market_redemption.py: 7 passedtest_market_contracts.py: 12 passedThese are not substitutes for the exact-head target gate. Merge requires fresh Python 3.15 / PostgreSQL 16 / Valkey 9 four-shard core CI, exact JUnit node-ID completeness/disjointness, conformance and wheel/sdist on this PR head.
No deployment, production Root/PIN/key/funding/recovery/deletion, or external delivery is included.