修复玩家脸部

.workbuddy-ai/memory/2026-10-03.md

@@ -0,0 +1,82 @@
+# 2026-10-03
+
+## 修「表情糊到肚子上」：脸部矩形越界 + 加护栏
+
+用户先发了一张截图问「这是为什么」（本会话 Read 工具读不了图片文件，只能靠像素统计猜），随后说「继续」。
+直接跑 `test_run` 就复现了问题：
+
+```
+表情矩形左下角 (0.695, 0.482) 跑出了头部 UV 孤岛 [P: (0.694607, 0.681596), S: (0.301457, 0.314468)]
+（中心 (0.8455, 0.639) 大小 (0.301, 0.314)）
+```
+
+用户在 shader 里把 `face_center.y` 从 0.839 调到了 **0.639**（想「往下挪」），矩形下边落到 0.482，
+而头部孤岛底边是 0.6816 —— **矩形伸进了正下方那段孤岛（u 0.695~0.935 / v 0.293~0.674，胸腹）**，
+所以表情同时糊到了肚子上。这就是截图里那个现象。
+
+修法（`face_paint.gdshader`）：
+- 顶部新增两个常量 `FACE_UV_BOUNDS_MIN = vec2(0.6946, 0.6816)` / `FACE_UV_BOUNDS_MAX = vec2(0.9961, 0.9961)`
+  —— 就是头部孤岛的实测范围，当护栏用。
+- `fragment()` 里 `clamp(face_center, FACE_UV_BOUNDS_MIN + half_size, FACE_UV_BOUNDS_MAX - half_size)`，
+  **怎么调都夹在头部里，不可能再溢到肚子/背上**。`min/max` 包一层是为了 size 大于孤岛时不出现 lo>hi。
+- 默认值改成 `face_center = (0.8455, 0.7816)`、`face_size = (0.301, 0.20)`：
+  孤岛高 0.3145，整高时根本没法往下挪（可移动范围 = 孤岛 − 大小），收到 0.20 后中心才能落到 0.7816，
+  表情落在头部下半段，符合用户「往下移」的诉求。
+
+测试：`test_face_rect_inside_head_island` 换成 `test_face_uv_bounds_match_head_island` ——
+校验护栏常量等于实测头部孤岛（模型改 UV 会红），并校验 `face_size` 不大于孤岛（否则夹取退化）。
+`_shader_vec2()` 的匹配前缀从 `uniform vec2 X = vec2(` 放宽成 `X = vec2(`，这样 const 也能解析。
+另加一条断言：shader 里必须出现 `clamp(face_center`。
+
+验证：`test_run` **32/32**；实机跑角色选择页无 shader/脚本报错，自己截了两帧确认表情只在头部、肚子上是干净的。
+
+## 顺带发现：我能看游戏截图
+
+`Read` 工具读用户发的图片会直接报 `Current model does not support reading images`，
+但 `editor_screenshot(source="game")` 返回的画面我能正常看。以后排查画面问题可以直接自己截图对照，
+不必让用户描述。
+
+## 再往下挪一点：只能靠压高度
+
+用户要求「往下移一点」。中心 0.7816 已经等于孤岛下边界 + 半个高度，直接减小 center.y 会被 clamp 吃掉（等于没反应），
+所以改成 **`face_size.y` 0.20 → 0.17、`face_center.y` → 0.7666**（0.7666 − 0.085 = 0.6816，仍然贴着下边界）。
+自己前后各截一帧对比，表情确实整体下移了一点，代价是纵向压扁 15%。
+
+记下这条边界：**头部孤岛下边界 0.6816 就是脸能到的最低处**，再往下就是胸腹孤岛，没有余量；
+要「整张脸整体下移且不变形」只能回 Blender 把头部 UV 往下扩。
+`test_run` 32/32。
+
+## 换成「贴花」：脸终于能放到身体正中间
+
+用户接着发图说「还需要往下移，脸部应该位于中间位置」。UV 方案已经到极限（再往下就是胸腹孤岛），
+所以直接把定位方式换成**贴花**：`face_paint.gdshader` 里加 `varying vec3 object_position / object_normal`，
+`vertex()` 里取 `VERTEX` / `NORMAL`，`fragment()` 用 `object_position.xy` 定位矩形，再用
+`-normalize(object_normal).z` 筛掉背面。uniform 变成 `face_center` / `face_size`（模型本地坐标）、
+`face_front_edge`、`face_flip_y`；`FACE_UV_BOUNDS_*` 那对护栏常量和 clamp 都删掉了（不再需要）。
+好处：**和 UV 孤岛布局彻底解耦**，以后 Blender 重排 UV 也不会再把脸画歪。
+
+踩到的坑：第一版把正面判成 `+normal.z`，结果表情画到了**背上**。
+`character_select.tscn` 的 `Model` 节点和 `player.tscn` 的 `mouse` 节点都把模型转了 180°，
+**模型自己的正面是本地 -Z**。测试里钉了 `-normalize(object_normal).z` 防止再写反。
+
+调参过程（用截图迭代）：
+- 为了看正面，临时把 `character_select.gd` 的 `SPIN_SPEED` 改成 0（模型初始朝正面），**调完已还原成 0.5**。
+- 内置表情在 tile 里的墨线只占 ~70%（face_01 68.9%×74.4%、face_02 83.9%×65.4%，且居中），
+  所以 `face_size` 要按这个比例估：4.0 的矩形 → 约 2.8 单位的墨线宽度，正好对上用户截图里脸占身体 ~55% 的感觉。
+- 最终 `face_center = (0.0, 0.0)`、`face_size = (4.0, 4.0)`：实测脸落在身体正中间，模型原点就是蛋形身体的中轴。
+- 测试 `test_face_rect_inside_head_island` / `test_face_uv_bounds_match_head_island` 删掉，
+  换成 `test_face_decal_inside_body_bounds`（矩形必须落在模型包围盒内）。
+
+验证：`test_run` **32/32**；主菜单 →「开始游戏」→ 角色选择实机截图确认 预设表情1 落在身体正中间。
+
+## 坑：用户开着 Godot 编辑器时，磁盘上的 shader 会被编辑器缓冲区覆盖回去
+
+调完贴花后回头一看，`face_paint.gdshader` 变成了「我的新注释 + 旧的 `+normal.z` 代码 + `face_center.y = 0.6`」的混合体：
+我改的 `-normalize(object_normal).z` 和 `face_center.y = 0.0` 被冲掉了，而 `0.6` 这个值我从没写过。
+判断是**用户在编辑器里开着这个 shader**（缓冲区还是我第一版的内容），他改了参数并保存，把我后来的改动覆盖了。
+
+处理：用整份 `Write` 重新写回验证过的版本（`face_center = (0.0, 0.0)`、`-normalize(object_normal).z`、
+并把顶部那句「法线朝 +Z」的旧注释一并改对，避免注释和代码互相矛盾），再跑 `test_run` 32/32。
+
+**教训**：改 `.gdshader` / `.tscn` 这类编辑器可能正开着的文件之后，最好提醒用户先关掉或重开那个文件再保存；
+否则他那边一保存就会把磁盘上的新版本覆盖掉，而且不会有任何提示。

.workbuddy-ai/memory/MEMORY.md

@@ -52,16 +52,19 @@
 
 - `face_paint.gdshader`：将 face surface 的 UV remap 到 0~1，再映射到内置图集 rect 或本地 PNG；贴图 alpha 只代表墨线覆盖率，混到 `skin_color` 上，**不写空间 `ALPHA`**（否则脸片变洞）。
 - `mouse_mouse_lambert1_BaseColor.png` 和 `mouse_表情测试.png` 的背景都精确为 RGB(137,179,255) / `#89B3FF`；表情图源 alpha 全 255，不能直接当透明度。
-- **最终做法：全身一个 shader，脸部靠 UV 矩形圈出来**。`face_paint.gdshader` 用 `body_tex`（= `mat_body` 的 BaseColor）采样全身底色，再在 `face_uv_min` / `face_uv_size` 圈出的矩形里，把表情贴图（内置图集或本地 PNG）和玩家手绘按 alpha 叠上去。透明处保持身体底色，所以**脸和周围不可能是两块不同的皮肤**。`FaceRig` 把 mat_body 的 `albedo_texture` / `albedo_color` / `roughness` / `metallic` 抄进 shader，并用 `set_surface_override_material(0, mat)`（surface 0，不再是 1）。
+- **最终做法：全身一个 shader，脸是「贴在身体正面的贴花」**。`face_paint.gdshader` 用 `body_tex`（= `mat_body` 的 BaseColor）采样全身底色，再在 `face_center` / `face_size` 圈出的矩形里，把表情贴图（内置图集或本地 PNG）和玩家手绘按 alpha 叠上去。透明处保持身体底色，所以**脸和周围不可能是两块不同的皮肤**。`FaceRig` 把 mat_body 的 `albedo_texture` / `albedo_color` / `roughness` / `metallic` 抄进 shader，并用 `set_surface_override_material(0, mat)`（surface 0，不再是 1）。
+- **贴花用模型本地坐标，不用 UV**：`varying vec3 object_position` / `object_normal` 在 `vertex()` 里取 `VERTEX` / `NORMAL`，`fragment()` 里用 `object_position.xy` 定位。**这样和 UV 孤岛怎么排无关** —— 之前用 UV 矩形时，矩形一越界就会把表情糊到肚子/背上（头部孤岛和胸腹孤岛只差 0.008），换贴花后这类问题从根上没了。
+- **模型自己的正面是本地 -Z**：`character_select.tscn` 的 `Model` 节点和 `player.tscn` 的 `mouse` 节点都把它转了 180°。所以正面遮罩写 `-normalize(object_normal).z`，**写反了表情会跑到背上**（踩过）。背面同 (x,y) 的点也会落进矩形，所以这个遮罩是必须的。
 - 不再需要 `skin_color` 当 shader 输入：`FaceRig` 反过来从身体贴图左上角取色写进 `skin_color`，只给画布和缩略图垫底用（导入贴图是压缩的，`get_pixel` 前要先 `decompress()`，否则报 "Can't get_pixel() on compressed image"）。
 - 表情遮罩用 `step()` 相乘而不是 `if`：采样留在统一控制流里，免得 mipmap 导数在分支里未定义。
 - **仍然不能写空间 `ALPHA`**：alpha 只代表墨线覆盖率，写进 ALPHA 会让面片变洞。
 - `face_atlas_transparent.png` 是我们自己的素材（从已删除的 `mouse_表情测试.png` 去蓝底、反混合出来的 1024² 副本），保留原图坐标排布；内置 ID `builtin:face_01/02` 对应各自的图集 rect。**原始表情图已随模型删除，这个副本现在是唯一来源，别删。**
-- **表情位置/大小/翻转只写在 `face_paint.gdshader` 顶部那段「调表情就改这一段」里**：`face_center`（中心，UV 单位，**往下挪就减小 .y**）、`face_size`（.x 宽 .y 高）、`face_flip_y`（默认 true）。
-  **FaceRig 故意不传这三个 uniform** —— 早期版本从 `@export` 传过去，会把 shader 里调好的值盖掉；现在 `face_rig.gd` 里没有对应的 export，测试里有一条断言禁止再传。
-- 头部孤岛实测：u 0.695~0.996（左右）、v 0.682~0.996（上下，**v 越大越靠头顶**）。贴图第 0 行在上边，所以默认必须 `face_flip_y = true`，否则表情上下颠倒（用户实测确认过）。
-- **矩形中心 ± 半个大小不能溢出头部孤岛**：正下方还有一段孤岛（u 0.695~0.935 / v 0.293~0.674），量下来是胸腹那一带，溢出去表情就会糊到肚子上。`tests/test_face.gd` 的 `test_face_rect_inside_head_island` 会直接从 shader 源码里读 `face_center` / `face_size` 再判定，**所以随手调这两个值不会被测试误报，只有溢出才会红**。
-- 两个 FaceRig 节点（`player.tscn` 的 `Player/FaceRig`、`character_select.tscn` 的预览 `FaceRig`）现在都不带表情矩形参数了 —— 值只有一份，在 shader 里。
+- **表情位置/大小/翻转只写在 `face_paint.gdshader` 顶部那段「调表情就改这一段」里**：`face_center`（中心，**模型本地坐标**，模型大约 x -3~3、y -3~3，**往下挪就减小 .y**）、`face_size`（.x 宽 .y 高，默认 4.0×4.0）、`face_front_edge`（法线朝前的阈值，调大只留最正面）、`face_flip_y`（默认 true）。
+  **FaceRig 故意不传这几个 uniform** —— 早期版本从 `@export` 传过去，会把 shader 里调好的值盖掉；现在 `face_rig.gd` 里没有对应的 export，测试里有一条断言禁止再传。
+- 贴图第 0 行在上边，而头部 UV 的 v 朝上，所以默认必须 `face_flip_y = true`，否则表情上下颠倒（用户实测确认过）。
+- 内置表情在各自图集 tile 里的墨线占比：`face_01` 68.9%×74.4%、`face_02` 83.9%×65.4%，且都居中 —— 所以 `face_size` 要按「墨线只占 tile 的七成」来估：4.0 的矩形给出约 2.8 单位的墨线宽度。
+- `test_face_decal_inside_body_bounds` 校验贴花矩形落在模型包围盒里（模型换尺度/原点会红）；`test_face_shader_has_expected_uniforms` 里钉住 `-normalize(object_normal).z` 和 `object_position.xy`，防止有人改回 UV 方案或写反正面。
+- 两个 FaceRig 节点（`player.tscn` 的 `Player/FaceRig`、`character_select.tscn` 的预览 `FaceRig`）都不带表情参数 —— 值只有一份，在 shader 里。
 - `FaceRig` 保存 `user://face_preset.json` 中的 expression ID 和 `user://face_paint.png` 暂存笔画；已保存的自绘表情用 `user://face_library/index.json` + 各自透明 PNG 持久化，角色选择页动态显示内置 atlas 表情和本地表情。
 - 联机当前表情同步：内置表情发送 ID；`user:` 本地表情发送当前 PNG 字节；笔画层另外发送。**不广播整份本地表情库**。
 - `FacePainter`：直接改 RGBA8 字节缓冲。画 = source-over alpha 混合，擦 = 把目标 alpha 乘掉；512² 一次 1MB，按一笔/动作同步一次。
@@ -81,6 +84,14 @@
 
 ## Godot MCP 踩坑
 
+- **看画面**：`Read` 工具读用户发的图片文件会报 `Current model does not support reading images`，
+  但 `editor_screenshot(source="game")` 返回的游戏画面能正常看 —— 排查画面问题直接自己截图对照，
+  比让用户描述快。没有画面时也可以用像素统计（PNG 手写解码 + 粗 ASCII 图）反推布局。
+- **用户在编辑器里开着的文件会覆盖磁盘改动**：改完 `face_paint.gdshader` 后，磁盘上出现过
+  「我的新注释 + 旧的 `+normal.z` + 我从未写过的 `face_center.y = 0.6`」这种混合体 ——
+  用户在 Godot 里开着这个 shader、缓冲区是旧版本，他一保存就把我后来的改动冲掉了，而且没有任何提示。
+  所以改 `.gdshader` / `.tscn` 之后要在回复里提醒对方先关掉/重开那个文件再保存。
+
 - **数值标量经 MCP 会被字符串化**：`node_set_property` 传 `0.15` 能写进去，但回显是 `"0.15"`；
   而 `batch_execute` 里包 `{"command": "set_property", "params": {...}}` 直接报
   `INTERNAL_ERROR … handler returned malformed result`（和旧版本文档里写的正好相反，以实测为准）。