记录首批4个错误码迁移的落地结果 [09-05-error-explainability-payment-readiness]
task.json 与 implement.md §15 补齐 commit cd01e4bf 的实施记录:基线
591→587,验证结果,剩余范围未扩大。
Co-Authored-By: Claude Sonnet 5 <[email protected]>
这个提交包含在:
@@ -379,6 +379,20 @@ A=优先审查/选迁移;B=业务影响复核;C=延期理由逐模块交审
|
||||
第一批只解决四个具体分支,不等于73文件都已形成迁移决定。员工资料、登录、审批解析、票据等优先域继续按真实业务影响细分;paymentAccounts共享validationError需先追到实际调用语义,不能套一个固定原因码完成所谓“1条迁移”。余下记录仍须在既定日期逐模块确认迁移或提出有理由延期;本轮未批准任何延期。
|
||||
|
||||
完整四分支记录见原主工作区artifacts/qa-flow/project-roadmap-execution-20260909/error-first-batch-branches.json;原始673条台账不覆盖,保留当前与未来验收预期的区别。
|
||||
|
||||
## 15. 2026-09-25 首批 4 个分支已落码(commit cd01e4bf)
|
||||
|
||||
按上节方案原样落地,四处均改为调用 `buildBusinessValidationError(reasonCode, details, statusCode)`,保留原 HTTP 状态码与 `featureFlag`/`reason` 等既有 details 字段,只新增精确 reasonCode;不改功能开关判断逻辑、不改权限校验顺序、不调整守卫顺序。
|
||||
|
||||
- `ap_operations.feature_disabled`(409)、`ap_operations.query_invalid`(400)、`todo.feature_disabled`(409)、`portal.feature_disabled`(409)四条原因码已注册进服务端 `errorReasons.js` 与客户端 `errorReasons.ts`,中文文案逐字一致,并同步进 `server/tests/unit/fixtures/error-reason-messages.json` 的内容断言。
|
||||
- `scripts/qa/error-reason-legacy-baseline.json` 精确移除这 4 条已迁移记录(591→587),未按文件整体删除、未改其余 expiresOn;`check-error-reason-contract.mjs` 主检查与 `--self-test` 均 PASS,无 `new_legacy`/`stale_exception`/`unregistered_reason`。
|
||||
- `apOperations.permissions.test.js`:扩展"功能关闭"断言校验 reasonCode 与原 details 共存;新增"功能开启但查询非法"400 用例(断言 `ap_operations.query_invalid` 且台账业务查询为 0);新增合法查询仍 200 用例。
|
||||
- `portal.test.js`:补齐此前完全空白的"工作台关闭"路径——409、`portal.feature_disabled`、零业务查询。
|
||||
- `todos.test.js`:新建(此前 todos 路由零测试覆盖,覆盖 GET/POST 两个消费同一 guard 的入口:关闭时 409+reasonCode+零 `todoItem` 查询,开启时正常放行)。
|
||||
|
||||
验证:server 5 个相关测试文件 286/286 通过;client typecheck/build 通过;`check-ui-design-rules.mjs --strict`、`check-trellis-governance.mjs`、`git diff --check` 通过。`routes/` 全量回归 872/874,2 个失败在 `paymentBatches.reimbursement.test.js` 的 BIFF8 导出用例,用 `git stash` 验证改动前 HEAD 同样失败(既有环境问题,疑似 BIFF8/Python 依赖),与本次改动无关,未在本次范围处理。
|
||||
|
||||
剩余 587 条例外记录不变;A/16、B/25、C/32 三档文件仍需逐模块确认迁移或提出有理由延期,本次未扩大范围、未批准任何延期。
|
||||
# 2026-09-16 上线后契约补强
|
||||
|
||||
后续日期批次方案:共享date工具新增纯isValidDateOnlyString谓词(严格YYYY-MM-DD、有限UTC日期、回读同日),保持旧parseDateOnlyString行为以免无关服务突然抛500;先接报销/付款/差旅申请/资产四路由的既有Zod日期schema,optional/nullable外层语义不变,失败沿原输入错误合同且写入前拒绝。催办复用前端displayFormat的Asia/Shanghai格式并明确“北京时间”,旧details分支也校验UTC形状/有限性,不回显任意字符串。验证单测正反边界、各域实际入口/解析器、无副作用及前端typecheck/build;采购、合同、考勤等剩余消费者继续单列,不冒称所有日期解析债务清零。不迁移或改生产历史日期,不提交/部署本轮未审代码。
|
||||
|
||||
@@ -15,7 +15,7 @@
|
||||
"branch": null,
|
||||
"base_branch": "main",
|
||||
"worktree_path": null,
|
||||
"commit": "a83791263833547fdab7ba6910f190fcccd5e34b",
|
||||
"commit": "cd01e4bf",
|
||||
"pr_url": null,
|
||||
"subtasks": [],
|
||||
"children": [],
|
||||
@@ -26,8 +26,11 @@
|
||||
"client/src/api/errorMessages.ts",
|
||||
"client/src/lib/resolveErrorMessage.ts",
|
||||
"client/src/pages/PaymentBatchesPage.tsx",
|
||||
"server/src/routes/apOperations.js",
|
||||
"server/src/routes/todos.js",
|
||||
"server/src/routes/portal.js",
|
||||
"scripts/qa"
|
||||
],
|
||||
"notes": "来源:真实单据 0014 创建付款批次时第三次出现‘参数校验失败’;生产只读诊断确认未创建批次,员工付款资料缺失且错误未被客户端解释。2026-09-05 最终规划已获用户批准实施;交接复核重点:门禁六类 self-test 真失败、主链路零例外且禁止新增例外、付款候选与创建共享同一 readiness resolver、20 处旧断言逐项处置且不得使用宽泛正则。0014 真实资料补录与付款不属于本 task 自动操作范围。",
|
||||
"notes": "来源:真实单据 0014 创建付款批次时第三次出现‘参数校验失败’;生产只读诊断确认未创建批次,员工付款资料缺失且错误未被客户端解释。2026-09-05 最终规划已获用户批准实施;交接复核重点:门禁六类 self-test 真失败、主链路零例外且禁止新增例外、付款候选与创建共享同一 readiness resolver、20 处旧断言逐项处置且不得使用宽泛正则。0014 真实资料补录与付款不属于本 task 自动操作范围。2026-09-25 完成 10-05 到期例外首批 4 个迁移(AP 台账/查询、todo、portal 三个 feature_disabled + AP 查询校验),基线 591→587,commit cd01e4bf;剩余 587 条按 §13 的 A/16/B25/C32 分流表逐批处理,未扩大本次范围。",
|
||||
"meta": {}
|
||||
}
|
||||
|
||||
在新工单中引用
屏蔽一个用户