Repository navigation
fix(snapshot): separate read-only data refresh from trading activation - #607
Merged
Pigbibi merged 1 commit intoOct 24, 2026
Merged
Conversation
Co-Authored-By: Codex <noreply@openai.com>
Pigbibi
deleted the
codex/website-independent-account-snapshot-20261024
branch
October 24, 2026 07:40
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.
现有账户只读同步仍依赖交易probe,返回200却可能没有新快照,最终等待超时;独立GET观察器又只接受HK手动调用,并把最新部署模板当作现役版本。补丁让现有PAPER/HK/SG使用同一只读观察器,允许原定时或手动入口,核对实际100%流量revision的快照开关。目标、账户范围、来源绑定、原生金额、时间和接收ACK校验保留。
增加默认关闭的snapshot_only手动选项,用于首户有限验证,跳过heartbeat通知、运行投影及摘要候选。此PR不改Cloud Run服务、交易启停、IAM或主业务定时计划;各目标采用仍串行核验。
验证:两处现役失败条件已先复现;267项相关测试、ruff、actionlint、diff检查通过,覆盖三目标×手动/原schedule×交易启停、失败新模板与健康serving分离,以及精确revision读取失败时不重试。没有真实账户、金额或凭据进入源码/测试。