Repository navigation
Conversation
|
Sentinel 已审核此变更,并发现 1 个已核验问题。 GitHub CI 未通过。 此状态来自当前 CI gate,不是 Sentinel finding;请以最新 CI attempt 的日志为准。 无法定位到 diff 的已核验问题WorkBuddy 未完成的复合检查被降级为未知结果🟠 严重程度:中(P2) Finding: 当前为 另有 4 个需要人工确认的风险;黄色 advisory 不阻塞合并,建议开发 agent review advisory 评论并逐条确认。 开发 agent 查询详情: 调用 MCP 工具 由 Sentinel 审核 · 目标提交 826f5804c0f0 |
|
Sentinel 保留了 4 个尚未确认的审核候选。这些黄色条目不阻塞合并;红色 P0/P1 与其他独立硬门禁不受影响。 单一猜测、缺少变更行证据或纯 checklist 的候选会统一降为 P3。
需要完整证据时,可用行末 由 Sentinel 审核 · 目标提交 826f5804c0f0 |
|
Sentinel 已审核此变更,并发现 2 个已核验问题。 无法定位到 diff 的已核验问题其他命令伪造通过摘要时,未执行的发布验证会被判定成功🟠 严重程度:中(P2) Finding: 常见包装命令中的验证任务不再被识别🟠 严重程度:中(P2) Finding: 当前为 开发 agent 查询详情: 调用 MCP 工具 由 Sentinel 审核 · 目标提交 404833c9a6df |
|
Sentinel 已审核此变更,并发现 2 个已核验问题。 无法定位到 diff 的已核验问题复合 shell 包装命令会被完全漏记为验证检查🟠 严重程度:中(P2) Finding: 当前为 开发 agent 查询详情: 调用 MCP 工具 由 Sentinel 审核 · 目标提交 5714bda7ad09 |
| /^env\s+(?:(?:-i|--ignore-environment)\s+|(?:-u|--unset|-C|--chdir)(?:=\S+|\s+\S+)\s+|[A-Za-z_]\w*=\S*\s+)*(?:--\s+)?/, | ||
| /^npx\s+(?:(?:-y|--yes|--no-install)\s+|(?:-p|--package)(?:=\S+|\s+\S+)\s+)*(?:--\s+)?/, | ||
| /^npm\s+exec\s+(?:(?:-y|--yes|--no)\s+|--package(?:=\S+|\s+\S+)\s+)*(?:--\s+)?/, | ||
| /^corepack\s+(?=(?:npm|pnpm|yarn)\s)/, |
There was a problem hiding this comment.
pnpm exec 执行的验证命令不再被识别
🟠 严重程度:低(P3)
本 PR 将验证命令匹配改为命令起始位置匹配,并通过 WRAPPERS 展开包装器;列表只包含 npm exec,未包含语义等价的 pnpm exec。因此 pnpm exec vitest run 不匹配 pnpm test/build/lint/... 正则,也不会继续解析到 vitest,被错误判为非验证命令。
Finding:finding_5d50dae441e6bf74fcad15c0
(cherry picked from commit c3a2bbc)
|
Sentinel 已审核此变更,并发现 1 个已核验问题。 无法定位到 diff 的已核验问题带前置包装器的 Shell 检查会被漏识别🟠 严重程度:低(P3) Finding: 当前为 开发 agent 查询详情: 调用 MCP 工具 由 Sentinel 审核 · 目标提交 10a46ceafa9c |
Summary
A compound shell command can return zero even when its verification command never runs or fails. For example,
true || npm testskips the test entirely. Leave the check verdict unknown when only the compound exit is available. Supported test summaries still supply a verdict, and standalone checks retain their exit-based verdict.Evidence
The false-pass regression failed on the previous head. Focused checks pass on Linux (118) and Windows (81), covering hook evidence, WorkBuddy transcripts and MCP reports.
Dependency
Stacked on #20 at
ec1270b. The diff againstmainincludes #20 until it lands. The follow-up itself is commit826f580.Merge Danger
Two-way door. Affects verification evidence and candidate eligibility for compound shell commands.