Repository navigation
feat: add generation control extension - #717
SunSi12138 wants to merge 1 commit into
Conversation
5df13e2 to
85d0563
Compare
|
Continuation after #706 was reverted by #716:
CI note:
Current-head PR CI is rerunning now. No merge action will be taken from this development pass. |
|
CI follow-up for the post-merge Release Gate failure from the reverted #706 landing: Failure cause:
What changed:
Verification:
Current #717 head remains |
SunSi12138
left a comment
There was a problem hiding this comment.
独立 Review 完成:通过,未发现需要阻塞或明显值得修改的问题。
重点核对:
- Issue #691 初始范围已覆盖:desired/actual inventory 作为 source of truth、stage/activate 分离、revision-based watch 仅作 invalidation、provider/materialization 边界,以及 NativeAOT 的 process-replacement 明确语义。
SharpLink.GenerationControl的运行时依赖保持为SharpLink.Abstractions;SharpLink.Generator仅为 analyzer 引用,Protocol v2 与普通 RPC data path 未被改动。- solution、生产项目引用边界、NuGet/package verification、reviewed public API baseline、package smoke 与 release publish 列表均已接入新包。
- #717 的 22 个 per-file patch 与此前 #706 最终版本一致;当前 base 与 #706 最终审核时的 base 树内容等价(中间仅发生合并及回滚),未发现重新提交造成的内容漂移。
- 当前 head
85d05634e13c79d59735634e6aed7bd7f0b2c965触发的 PR Fast、PR Package Smoke、CodeQL、Codec Padding Security Evidence 均已完成并为 success。PR Package Smoke 包含 build/pack、包依赖契约、reviewed public API 与 NuGet consumption smoke。
本审核只给出结论,不执行合并,也不修改 Issue 状态。PR 保持未合并,等待人工决定是否合并。
|
Final development verification on current head
Post-merge Release Gate flake follow-up from the reverted #706 landing:
The replacement PR is based directly on current |
|
设计方向已在 Issue #691 进一步收敛:不再采用“远端通过 Stage/Activate 控制对端”的模型,而改为“任一端 generation 变化后声明/通知 peer,peer 根据本地策略自行 reconcile,只更新自己”。 这意味着 #717 当前最核心的 public API(remote Stage/Activate + Desired/Actual control semantics)需要实质性重做,不适合在这个已经完成旧方案 review/CI 的 PR 上继续替换。 因此本 PR 作为旧方案 superseded 关闭,不合并。后续实现继续以 #691 为设计主线,另开新的 focused PR。#717 中可复用的独立包、descriptor/revision、provider boundary、NativeAOT 语义和 package/CI 接入经验仍保留作为参考。 |
Fixes #691
重新提交 #706 的内容。此前 #706 在未经审核确认的情况下被合并,已由 #716 回滚;分支
issue-691-generation-control已在原提交5df13e2c1c上恢复,本 PR 即该分支相对dev的完整改动(22 个文件)。说明