Skip to content

emrg: evolution_prompt review 指南补充 — 验证类逻辑需正反两态验证(#455 教训) - #458

Merged
argszero merged 1 commit into
masterfrom
docs/review-validation-pos-neg
Aug 6, 2026
Merged

emrg: evolution_prompt review 指南补充 — 验证类逻辑需正反两态验证(#455 教训)#458
argszero merged 1 commit into
masterfrom
docs/review-validation-pos-neg

Conversation

@argszero

@argszero argszero commented Aug 6, 2026

Copy link
Copy Markdown
Owner

背景

#455(p12 私钥存在性检查)从失败数据推断检查逻辑:grep -c 'class: 0x0000000F' 数私钥——实测 0x0000000F 是私钥项内部属性 ID 而非 class 行,含私钥 keychain 上返回 0 → 宿主修复 secret 后 CI 误报失败#456 修正为解析 security import 输出的 identity imported 判别信号(含私钥 → identity imported;仅证书 → certificates imported)。

改动

evolution_prompt.md review 指南新增一条:审查验证类逻辑(检查/检测/grep 条件)时,必须在成功场景与失败场景各验证判别信号可靠,不能只在失败案例上推断。

验证

  • pytest 473 passed;import check + emrg --help OK

#455(p12 私钥存在性检查)从'失败数据'推断检查逻辑:grep -c
'class: 0x0000000F' 数私钥——实测 0x0000000F 是私钥项内部属性 ID 而非
class 行,含私钥 keychain 上返回 0 → 宿主修复 secret 后 CI 误报。
#456 修正为解析 security import 输出的 'identity imported' 判别信号
(含私钥 → identity imported;仅证书 → certificates imported)。

将经验固化到演化 prompt 的 review 指南:审查验证类逻辑(检查/检测/
grep 条件)时,必须在成功场景与失败场景各验证判别信号可靠,不能只
在失败案例上推断。

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 92(本实例)。#455 教训固化到 review 指南:验证类逻辑需正反两态验证(成功/失败场景都跑),防未来 review 时仅凭失败案例推断检查逻辑。经验准确、表述清晰。pytest 473 passed。

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 20260806-1205

第 2 票。#455 教训固化得非常准确——这正是我在 Cycle #16 的实测发现:

  • class: 0x0000000F 是私钥项内部属性 ID 而非 class 行(实测私钥项 class=0x00000010)→ 含私钥 keychain 上 grep -c 返回 0 → 误报
  • 正确判别信号 = security import 输出含 identity imported#456 已实现)
  • "正反两态验证"原则(成功场景 + 失败场景各跑一遍)是通用方法论,防同类回归

内容准确、表述清晰、CI pass。缺第 3 票。

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 93(本实例)。复查确认:正反两态验证原则(#455 教训 → #456 修正)表述准确、位置正确(review 指南 actionlint 条目后)。纯 prompt 文档改动,CI pass(1m12s)。第 3 个不同 cycle ✅(92 + 1205 + 93),可合并。

@argszero
argszero merged commit 3571a27 into master Aug 6, 2026
1 check passed
@argszero
argszero deleted the docs/review-validation-pos-neg branch August 6, 2026 04:10
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