lanrtop
|
c9b951dd30
|
fix(onboarding/parity): 混合文件预警收窄空白归一、失败时不丢预警、测住改写判断（P10 审查修复）
Push & PR Check / check (push) Successful in 1m2s
Push & PR Check / check (pull_request) Successful in 1m5s
P10 审查核实的三处缺陷：
- 块外比较原先逐行 trim_end 并丢弃全部空行，项目在块外删空行（Markdown 两段
并成一段）或加行尾空格（硬换行）时判为无改动，升级后不预警却产生混合文件。
改为 CRLF 统一为 LF、托管块原位换成占位符，只把文末占位符连同其前空白按
trim_end 归一（正好抵消炼境追加托管块时的 trim_end + 空行）；文中托管块两侧
的空行与行尾空格照常比较——炼境只原位替换文中的块，不碰两侧。
- dispatch_templates_with_mixed_file_check 先 `then()?` 后标注：buff 先同步蓝图
规则改写了 AGENTS.md/CLAUDE.md、再写 DB 失败时，预警随 Err 一起丢失。改为
无论成败先标注，失败时把混合文件名并入错误信息；无混合文件则原样返回错误。
- 「块外有未提交改动但托管块已是最新、本次不改写」组合没有测试，删掉 annotate
的改写判断测试照样全绿。主场景测试补情形三。
测试：新增 outside_managed_whitespace_edits_count_as_changes、
mixed_file_warning_survives_failure_after_write，主场景补情形三；纯函数测试的
期望值随归一化收窄更新（"# 标题\n项目内容" → "# 标题\n\n项目内容"，保留「之前
追加的块尚未提交、只多出块前空行」不算改动的断言）。变异自检：三处修复逐一回退，
对应测试各自失败，已还原。cargo test --locked：lib 272 passed / 5 ignored，0 failed；
pnpm typecheck 通过。契约文档 project-context.yaml 与蓝图 P10 卡同步。
Rules-Applied: R07,R12
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
2026-10-03 04:32:39 +09:00 |
|
lanrtop
|
13aa3a1a8c
|
feat(onboarding/parity): 接入包改写托管块前检测块外未提交改动，给出混合文件预警（P10）
Push & PR Check / check (push) Successful in 1m2s
Push & PR Check / check (pull_request) Successful in 59s
2026-10-01 enterprise 的 git-workflow.md 在有项目未提交改写时被升级了托管块，
同一文件混有炼境与项目两种来源的改动，按「禁止字符串手术」无法拆分提交，只能交人定案。
管家应在制造混合文件时明确告知，而不是事后由人发现。
- TemplateResult 新增可选 warning（序列化时缺省省略，前端类型同步）
- apply_onboarding_pack 在模板分发与 buff 蓝图规则同步前后夹一层检测：写入前对
AGENTS.md、docs/ai-context/git-workflow.md、CLAUDE.md 跑 git status，有改动再取
HEAD:./<文件>（不在 HEAD 中按空内容算），两边去掉 ONBOARDING_MANAGED 与
BLUEPRINT_MANAGED 块后比较（忽略行尾与空行）；写入后只对本次确被改写的文件预警，
文案含文件名与「块外有未提交改动，升级后将成为混合文件」
- 只提示不阻断，块外内容照旧不动；git 不可用、非仓库或命令失败一律静默；
git status 带 --no-optional-locks，不抢 index.lock
- CLAUDE.md 只由蓝图规则同步改写，不在模板结果里，预警时补一条结果
- BLUEPRINT_MANAGED 标记提为 blueprint.rs 的 pub(crate) 常量，检测与 upsert 共用
- MCP 单项目输出在该文件行下加预警行，批量输出追加混合文件清单；工具描述同步；
炼境 UI 接入包结果行下显示预警
- 契约文档 project-context.yaml 同步；蓝图 P10 标 ✅ 并记录验证与已知限制
验证：cargo test --locked lib 270 passed / 5 ignored；新增 3 个测试（纯函数、临时 git
仓库三种情形、非 git 目录），变异自检（探测强制返回 true 时两个场景测试均失败）；
pnpm typecheck 与 pnpm test 通过。
Rules-Applied: R01,R06,R12
Feature-Confirmed: true
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
2026-10-03 03:58:28 +09:00 |
|