Repository navigation
refactor(preview): 收敛重复检查与纯测试应用交付 - #159
Merged
Merged
Conversation
This branch was successfully deployed
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.
目的与范围
收敛验证与 Preview 的重复工作,保留数据、身份、凭据和关闭清理边界。
has_deployed_release输出;保留控制配置顺序的 PostgREST release 判断。pdm check,末尾只运行独有的 actionlint;本地完整 pre-commit hooks 不变。tests/、重命名新旧路径均符合、查询前后 head/base 一致时,跳过应用 Preview。API 失败、清单截断或无法确认时仍走原交付。这不是净减代码的通用框架:新增一个受信只读判定和对应测试,用于减少 wheel、镜像、Registry、Heroku 的无用交付;不承诺纯测试 PR 零资源成本。
验证证据
pdm run check通过:90 passed、60 skipped;未启动本地数据库,跳过项不计作通过。scripts/tests/test_preview_delivery.py:10 passed,覆盖部分交付报告、完整纯测试差异、混合变更、重命名、非根 tests 路径、截断、身份变化及 API 失败。pdm run pre-commit run actionlint --all-files、diff whitespace 和独立源码复核通过。合并与验证状态
用户已单独授权将
Hermetic repository contract、Portable peer database runtime分别迁移为impl-py checks、impl-py e2e tests,同步对应 branch protection。head1eef9e3的两个新名称检查和 inkcre-cli checks 已在 CI39951895469 通过,随后完成保护迁移。已核对 strict=true、GitHub Actions App15368 绑定、
Provision isolated branch及其余保护均未改变,没有 bypass。Neon 原HTTP422由配额占满导致,用户授权清理5个历史恢复分支后,供给run39951895442的attempt2成功,三个必需检查全部通过。旧main的可信Preview controller仍等待旧名称,无法在本PR合并前接受新检查名称。用户明确接受这一项非必需过渡失败后,取消等待中的run39951892903,未伪造状态或改用不可信controller。2026-10-25 14:29北京时间,在最新base389263c、无未解决评审且保护条件满足时正常squash merge为
99d7283a913fa0baad81e7fbf2ad7cd91245df57。关闭后的Neon清理40169256353及应用清理40169256406均成功;独立查询确认PR159分支和两个Heroku应用均不存在,Registry稳定别名已退役。交付影响、风险与回退
Draft 也可能按当前主分支的受信 controller 触发现有 Preview;本 PR 首次交付不能证明尚未合并的新判定生效。
主分支采纳后,新增 scope job 对每个内部 PR 多做只读查询。Neon 和关闭清理成本仍存在。以后新增本地 hook 时须明确其 CI 归属,不能假设全部 hooks 自动进入 CI。
回退对应提交恢复原执行与自动应用 Preview;不自动删除打开 PR 的已有资源。
关联
TS 的独立重复执行删减:InKCre/impl-ts#163 。无父子 PR 依赖。