Skip to content

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

Draft
xiaoland wants to merge 3 commits into
mainfrom
refactor/validation-execution-trim
Draft

xiaoland wants to merge 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。PR 仍保持草稿:Neon 实际供给在两次运行中均返回 HTTP422,尚未满足必需检查。已取消只会继续等待的非必需应用交付 run39951892903,未取消或绕过数据库检查。

只读 Neon 项目查询确认 owner.branches_limit=10,现有分支正好10个,本 PR 的 preview/core-py/pr-159 不存在。需释放一个分支名额或提高配额后重跑;未删除其他任务、备份或维护分支,未改套餐。

交付影响、风险与回退

Draft 也可能按当前主分支的受信 controller 触发现有 Preview;本 PR 首次交付不能证明尚未合并的新判定生效。
主分支采纳后,新增 scope job 对每个内部 PR 多做只读查询。Neon 和关闭清理成本仍存在。以后新增本地 hook 时须明确其 CI 归属,不能假设全部 hooks 自动进入 CI。
回退对应提交恢复原执行与自动应用 Preview;不自动删除打开 PR 的已有资源。

关联

TS 的独立重复执行删减:InKCre/impl-ts#163 。无父子 PR 依赖。

This branch was successfully deployed

No deployments
preview — 1eef9e3f Deployed Oct 23, 2026 by xiaoland via deploy #501
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