Skip to content

refactor(preview): 收敛重复检查与纯测试应用交付 - #159

Merged
xiaoland merged 3 commits into
mainfrom
refactor/validation-execution-trim
Oct 25, 2026
Merged

xiaoland merged 3 commits into
mainfrom
refactor/validation-execution-trim

Conversation

@xiaoland

@xiaoland xiaoland commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

目的与范围

收敛验证与 Preview 的重复工作,保留数据、身份、凭据和关闭清理边界。

  • 删除没有消费者的 Core release 查询和 has_deployed_release 输出;保留控制配置顺序的 PostgREST release 判断。
  • CI 已执行 pdm check,末尾只运行独有的 actionlint;本地完整 pre-commit hooks 不变。
  • 仅当整个 PR 的完整文件清单都位于根 tests/、重命名新旧路径均符合、查询前后 head/base 一致时,跳过应用 Preview。API 失败、清单截断或无法确认时仍走原交付。
  • 保留 Neon 实际供给及必需检查;关闭清理不依赖路径判定。已有现场不会因后续纯测试变更被删除。

这不是净减代码的通用框架:新增一个受信只读判定和对应测试,用于减少 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 和独立源码复核通过。
  • 使用历史 PR 的只读 GitHub API 验证分页输出形状;新的线上跳过流程尚未观察。

合并与验证状态

用户已单独授权将 Hermetic repository contract、Portable peer database runtime 分别迁移为 impl-py checks、impl-py e2e tests,同步对应 branch protection。head 1eef9e3 的两个新名称检查和 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 依赖。

@xiaoland
xiaoland marked this pull request as ready for review October 25, 2026 06:28
@xiaoland
xiaoland merged commit 99d7283 into main Oct 25, 2026
8 of 11 checks passed

This branch was successfully deployed

1 active deployment
preview — 1eef9e3f Deployed Oct 25, 2026 by xiaoland via impl-py preview cleanup #503
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant