fix(upload,models): 上传落盘补齐兜底 + 不可用模型归因准确(v0.7.5)

两个真实缺陷,都由本轮的「跨端边界验收」暴露:

一、上传落盘:去重命中时删掉了唯一副本(T-WEB-017)
persistTempFile(动作 / 模型 / 场景上传走的通路)在 sha256 去重命中时直接
safeUnlink(tempPath) 并返回旧记录 —— 一旦记录指向的路径不存在(换过 STORAGE_DIR、
只恢复数据库、手工删过文件),上传仍返回 201,但后续动作预处理 500
(实测 ENOENT .../<sha>.vmd)、模型分析与缩略图静默失败。持久化通路 persistBuffer
早在上一轮就补了「文件缺失则按原路径补写」,这条通路漏了;现在同样处理:
先 existsSync(absolutePath(existing)),缺失则按记录里的原路径 rename(跨设备退化为
copyFile)补写,只有确认目标存在才删临时文件。
新增 test/storage-persist.spec.ts(5 项):首次上传 / 去重命中且文件在 /
去重命中但文件已丢失(补写而非删除)/ persistBuffer 回归 / 空文件拒绝。
反向验证:临时还原修复后第 3 条立刻失败,恢复后 5 项全过。

二、不可用模型的原因归因不准确(T-WEB-017)
unusableReason() 只要 stats 里有 humanoid 数字就归因于 humanoid,于是一个直接上传的
.glb(骨架完全合规、drivable=true、只是没有 VRM 容器)会被写成
「humanoid 不达标(19/22)」,把审核引向"重新绑定骨骼"而不是"以 VRM 交付"。
实测(跨端验收用真实 PMX→Assimp→GLB,886 KB):
  修复前 → humanoid 不达标(19/22),未产出 VRM
  修复后 → 未产出 VRM(格式为 GLB;humanoid 覆盖 19/22 已达标),需重新以 VRM 交付才能进入客户端目录
现在按 drivable 区分:不满足 16 根必需骨才归因 humanoid;达标但没 VRM 容器则明确说"已达标"。
新增 2 项单测(model-usability 5 → 7 项)。

三、跨端边界验收(Windows × Web)新增两类场景,双方都断言
  * 不可用模型:真实 PMX 用 Assimp 转 GLB 后入库(等价"转换只产出 GLB")→
    usable=false、原因准确、**不进入 catalog/export**、defaultModelId 不变;
    包内客户端的 bundle.models 里没有任何非 VRM 模型;
  * 场景 config 只给 {physics:{gravity:{y:-9.8}}} → 服务端补齐默认
    (碰撞体 auto、地面 0.6/45°、静风、相机 follow、环境光 1、出生点 1 条),
    客户端解析后逐项一致。

实测:npm run verify 全绿(契约 6 项 / 35 路由 / 版本 0.7.5 / 27 文件 309 项测试 /
build / 产物级实体注册核对 / 启动冒烟);跨端集成验收在**发布包起的服务端**上全部通过。
版本 0.7.4 → 0.7.5。
This commit is contained in:
2026-09-18 03:03:50 +08:00
parent 5a78749e49
commit 12041f2e58
9 changed files with 290 additions and 10 deletions
+18 -1
View File
@@ -104,7 +104,24 @@ export class StorageService {
const existing = await this.assets.findOne({ where: { sha256 } });
if (existing) {
await this.safeUnlink(tempPath);
// 去重命中时**必须确认文件真的还在**:换过 STORAGE_DIR、或只恢复了数据库
// 而没恢复上传目录时,旧记录会指向一个已经不存在的路径。此时若直接
// `safeUnlink(tempPath)` 把刚上传的唯一副本删掉,上传仍返回 201,
// 但后续预处理 / 模型分析 / 下载 / 缩略图全部 ENOENT
// (实测:动作上传成功后 `POST /api/motions/:id/preprocess` 直接 500)。
// 这里按**记录里的原路径**补写一份 —— 与 persistBuffer 的兜底一致。
const absolute = this.absolutePath(existing);
if (existsSync(absolute)) {
await this.safeUnlink(tempPath);
return { asset: existing, url: this.url(existing) as string };
}
await mkdir(dirname(absolute), { recursive: true });
try {
await rename(tempPath, absolute);
} catch {
await copyFile(tempPath, absolute);
await this.safeUnlink(tempPath);
}
return { asset: existing, url: this.url(existing) as string };
}