已开启
PR !32 后续讨论:Implement 衔接约束放公共层还是 profile + 4 点修改建议 #28
LuBo创建于 6 天前
LuBo
6 天前 评论:
6 天前 评论:
@littlejerry1 这是 !32 的评审后续讨论(PR 合入后评论通道已关闭,内容迁移到此)。两件事需要你表态:①两条 Implement 衔接约束是否由你接手上浮公共层(execution-plan 模板 + ohos-implement 反模式表 + ohos-plan 指引);②第二类 4 点修改建议按上浮 PR 还是单独小 PR 处理。


PR !32 已于 2026-10-29 17:05 被合入(merged_by: sunfei2021)。因该仓 PR 合入后关闭评论通道(405),评审讨论内容迁移到本 issue,供大家讨论后续处理。
讨论稿(事后评论):评审期间本 PR 已被合入(2026-10-29 17:05),本评论不含合入结论,转为后续工作的讨论基础。两个断点均已核实为真实的公共层缺口;意见分两类——①两条约束的分层归属(是否上浮公共层);②无论落点在哪层都建议修改的 4 点(可当作后续 PR 的修改清单)。
断点核实(对照公共层现状)
workflow/workflow.md「先读仓库 Agent 指南」与gate-checklist.md「目标仓 Agent 指南已检查」只覆盖 Define/Specify 阶段;execution-plan.md模板的 Read-only Context 是自由填写的路径表;ohos-implementskill 不提编码规则ohos-validate有「缺口回ohos-review」;但ohos-implement反模式表没有「禁止自行创建 evidence」红线修复方向正确,checklist 措辞也精确(点名了三份文件名)。以下两类意见供讨论。
第一类:分层归属——放 profile 还是公共层(讨论项)
ability_runtime/AGENTS.md#6. 编码规则)。ohos-implement是唯一不加载 profile 的。「Implement 衔接约束」段实际在 Plan/Review/Validate 门禁被消费,靠 Plan 阶段把禁止项写进 execution-plan 间接传导到 Implement——能用,但比公共层落点多一跳。execution-plan.md模板禁止项固定一行 +ohos-implement反模式表两行 +ohos-plan补一条 Read-only Context 指引;上浮后删除 profile 内条目(沿用 !20 → issue #9 的处理模式)。第二类:无论在哪层合入,都建议先修改的 4 点
AGENTS.md或没有编码规则章节时该条无法满足。建议改为条件式——存在该章节时必须引用;不存在时记录N/A+ 已确认的替代规范来源。同时建议把「目标仓根 AGENTS.md」放宽为「目标仓 Agent 指南(AGENTS.md/CLAUDE.md)」,与本 profile 上下文来源表及gate-checklist.md措辞对齐。### Implement 衔接约束):没有任何 skill 在 Implement 阶段加载 profile(ohos-implement不读)。建议段内加一句传导说明(本段在 Plan 门禁核对、经 execution-plan 禁止项传导至 Implement),或改名「Plan→Implement 衔接约束」。若上浮公共层,同一条同样适用:落点应在ohos-implement反模式表(implement 时刻真正被读到),而不是只在 plan/review 侧声明。execution-plan.md模板的「禁止项」是“不得出现以下内容”的负面清单,而新要求是“必须声明一条红线”(声明式)。建议明确落点表述:公共层方案可直接在模板禁止项清单中固定一行;profile 方案建议表述为「禁止项清单后附加固定声明」,避免“在负面清单里写必须句”产生执行歧义。第二类 4 条都不阻塞分层讨论,可随本 PR 或上浮 PR 一并处理。