Repository navigation
fix: verify persisted artifacts before settling distillation - #56
likaho991007-design wants to merge 1 commit into
Conversation
Bugbot is paused — on-demand spend limit reachedBugbot uses usage-based billing for this team and has hit its on-demand spend limit. A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue. |
|
Sentinel 已审核此变更,未发现已核验问题,但发现需要人工确认的风险。 另有 2 个需要人工确认的风险;黄色 advisory 不阻塞合并,建议开发 agent review advisory 评论并逐条确认。 开发 agent 查询详情: 调用 MCP 工具 由 Sentinel 审核 · 目标提交 898028e7da71 |
|
Sentinel 保留了 2 个尚未确认的审核候选。这些黄色条目不阻塞合并;红色 P0/P1 与其他独立硬门禁不受影响。 单一猜测、缺少变更行证据或纯 checklist 的候选会统一降为 P3。
需要完整证据时,可用行末 由 Sentinel 审核 · 目标提交 898028e7da71 |
With
persist:true, a successful/conversation/distillHTTP envelope can still contain a rejected or unstored result. Treating transport success as a successful distillation removes the candidate even when no reusable Gene/Capsule was saved.Require a stored business receipt before returning
sent:ok:true,status:stored, non-empty Gene and Capsule asset IDs, and no explicit quality rejection. Preserve the response, error reason and model usage on failure so existing candidate retry handling can work. Optional quality metadata may be absent; an ID-only reply is not persistence proof.Validation on the Linux SSH test server with Node.js 22.23.3:
f7b4cdewithsentinstead offailed.node --test --test-concurrency=1 test/*.test.jsafter installing the declared development dependencies (dsh-llm,dsh-skill,dsh-tools0.1.5-rc.3 and evolver-core 2.0.38).Success fixtures now return the real stored-receipt shape, including the mock HTTP server previously returning only
recorded:truefor distillation. Initial missing-dependency/fixture failures and the final passing logs are retained. This PR changes neither recall, distillation prompts, publication policy, lifecycle timing nor engine code.