Repository navigation
fix: preserve replacement drafts and captured native owners - #70
Conversation
|
Sentinel 已审核此变更,未发现已核验问题,但发现需要人工确认的风险。 GitHub CI 未通过。 此状态来自当前 CI gate,不是 Sentinel finding;请以最新 CI attempt 的日志为准。 另有 1 个需要人工确认的风险;黄色 advisory 不阻塞合并,建议开发 agent review advisory 评论并逐条确认。 开发 agent 查询详情: 调用 MCP 工具 由 Sentinel 审核 · 目标提交 254f72af3e7d |
|
Sentinel 保留了 1 个尚未确认的审核候选。这些黄色条目不阻塞合并;红色 P0/P1 与其他独立硬门禁不受影响。 单一猜测、缺少变更行证据或纯 checklist 的候选会统一降为 P3。
需要完整证据时,可用行末 由 Sentinel 审核 · 目标提交 254f72af3e7d |
|
Sentinel 已审核此变更,未发现需要报告的问题。 开发 agent 查询详情: 调用 MCP 工具 由 Sentinel 审核 · 目标提交 d947fbefa0ef |
The captured-owner change targeted lib/app/native-turn.js, which EvoMap#68 removed; main's shared late-terminal flow takes precedence, as the PR description anticipated. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Merged with maintainer changes: main merged in, dropping the shared "retain unknown native owners during finalization" commit (it targets native-turn code #68 removed; main's |
|
Sentinel 已审核此变更,未发现已核验问题,但发现需要人工确认的风险。 另有 1 个需要人工确认的风险;黄色 advisory 不阻塞合并,建议开发 agent review advisory 评论并逐条确认。 开发 agent 查询详情: 调用 MCP 工具 由 Sentinel 审核 · 目标提交 06903d328765 |
|
Sentinel 保留了 1 个尚未确认的审核候选。这些黄色条目不阻塞合并;红色 P0/P1 与其他独立硬门禁不受影响。 单一猜测、缺少变更行证据或纯 checklist 的候选会统一降为 P3。
需要完整证据时,可用行末 由 Sentinel 审核 · 目标提交 06903d328765 |
Summary
A successful reply to an older draft submission could delete a newer draft with the same ID. Cleanup now compares the saved draft with the submitted snapshot inside the existing locked update, keeping a replacement even if its creation timestamp is unchanged and only its receipt differs.
The Windows CI failure also exposed a native finalization bug. An explicitly unknown captured owner (pid: null) triggered another ancestry lookup inside the tightly bounded Kimi Notification hook. Finalization now preserves that captured value and still uses the existing fallback when no owner was captured.
Evidence
Merge Danger
Two-way. The publication request and receipt are unchanged. The owner fix applies to the existing native mechanism; #68 replaces that mechanism, so its shared late-terminal flow should take precedence when the branches are synchronized.