Repository navigation
fix: preserve evidenced action availability across refreshes - #28
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
重复抓取未变化的ETF分红时,原实现会把全部行动的captured_at改成本次时间。该字段同时决定执行层的可用时间并参与前向账户历史前缀哈希,导致经济内容未变也无法继续观察;原revised_rows.actions仅比较事件ID集合,还会漏掉同ID的来源或字段修订。
本次在不可变快照链中,只有完整记录版本一致时才保留已验证父快照的首次采集时间。新鲜抓取结果和真实时间单独写入带SHA256校验的action-observations.json;金额、日期、来源、source_record或ID的变化保留新时间,完整live响应的删除也会保留为删除。派生history沿用行动版本的首次可用时间,分类声明保持首次观察时间。新增分项变更计数和时间倒退、缺失、无时区、未来时间的校验。
验证:18项数据集测试通过,覆盖重复刷新、同ID修订、金额变化、删除、回执篡改及非法时间。Ruff检查通过;完整Linux/Windows测试与纯分支覆盖率由本PR检查验收。
保持下游全字段历史前缀检查,不放宽门禁。原冻结账户、旧快照和登记不修改;使用新代码的账户需要单独验收和未来起点预登记。不能将修复后的历史回放计为原账户的前向观察。