Fix legacy tool model textures 修复旧版共振器与重戟模型材质

[WhereisFff]

• fix: 修复旧版共振器和重戟模型的材质继承 ([Bug] 共振器旧模型资源包中所有模型都是余烬材质 #5149)
• fixed [Bug] 共振器旧模型资源包中所有模型都是余烬材质 #5149

[Gugle2308]

PR #5154 状态为 closed，按审查规则（closed/labeled → 停止，不发布评论），本次不做审查、不输出评论内容。

如需继续，请 reopen 该 PR 后再触发审查。

[Gugle2308]

⚠️ Dangerous command requires approval:

python3 -c "import json;d=json.load(open('/tmp/p5154.json'));print('merged_by:',(d.get('merged_by') or {}).get('login'));print('base:',d['base']['ref'],d['base']['sha'][:8]);print('head:',d['head']['r...

Reason: script execution via -e/-c flag

Reply /approve to execute, /approve session to approve this pattern for the session, /approve always to approve permanently, or /deny to cancel.

[Gugle2308]

代码审查摘要 — PR #5154

⚠️ 状态提示：该 PR 已于 2026-10-03T04:48Z 被 squash 合并（merge commit 3f7492463，标题 fix: 修复旧版共振器和重戟模型的材质继承 (#5149) (#5154)），并且修复后的内容已存在于当前 dev/26.1/1.6 tip（1a17b1db9）。因此以下为 post-merge 复核结论，不需要再"改后合并"。

范围: 9 个文件（0 Java，全部为资源包模型 JSON）/ 144 行 diff / 9 个 hunk，No newline 标记 0 个

🔍 根因核对（与 issue #5149 对齐）

Issue #5149 标题为「[Bug] 共振器旧模型资源包中所有模型都是余烬材质」——这正是"子模型覆盖贴图变量失效"的典型症状。核对目标分支后确认根因与修复方向正确：

层级	模型	贴图变量约定
基础模组（新模型）	assets/anvilcraft/models/item/ember_metal_resonator.json 等	几何体面引用 #1，"1": "anvilcraft:item/ember_metal_gear"
frost/transcendence 子模型	{"parent":"anvilcraft:item/ember_metal_resonator","textures":{"1":"anvilcraft:item/frost_metal_resonator"}}	只覆盖 "1"
资源包旧模型（修复前）	resourcepacks/old_resonator_heavy_halberd/.../ember_metal_resonator.json	面引用 #0，"0": "anvilcraft:item/ember_metal_resonator"（硬编码）

旧模型把贴图写死在 "0" 且面直接引用 #0，而 frost/transcendence 子模型覆盖的是 "1" → 覆盖落空，全部退化为余烬材质。本次改为 "0": "#1" + "1": <原硬编码贴图> + "particle": "#1"，让旧几何体重新走 #1 这一"材质槽"，子模型的 "1" 覆盖得以生效。

"particle": "#1" 还顺带修正了破坏粒子曾使用余烬贴图的问题。

✅ 覆盖完整性（关键，逐文件对账）

该资源包共 30 个模型，逐文件核对后无遗漏，且未改动的 21 个文件本就不需要改：

分组	文件数	角色	本次改动
ember_metal_* 几何模型（resonator + resonance_{axe,hoe,pickaxe,shovel} + heavy_halberd{,_mace,_spear,_sword}）	9	完整模型（含 elements），须定义 "1"	✅ 9/9 全部改到
ember_metal_heavy_halberd_throwing	1	仅 display 的子模型	✅ 正确未动（父即 ember_metal_heavy_halberd）
frost_metal_*	10	子模型，已覆盖 "1"	✅ 正确未动
transcendence_*	10	子模型，已覆盖 "1"	✅ 正确未动

自动化校验（30 个文件全量）结果：JSON 全部合法，0 个未解析的 # 引用，每个 throwing 子模型的 parent 都指向本材质变体（frost_metal_heavy_halberd / transcendence_heavy_halberd），无"跨材质继承"的悬挂链；包内 6 个被引用贴图（ember/frost/transcendence × resonator/heavy_halberd）均存在。包内已无任何残留的 "0": "anvilcraft:item/..." 硬编码写法。

🟢 机制正确性

"0": "#1" 属于两跳引用链（元素 #0 → "#1" → 具体贴图）。已用原版资源证据确认该链路受支持：原版 assets/minecraft/models/block/cube_bottom_top.json 即写成 "down": "#bottom"，而父模型 block/cube 的几何体引用 #down（原版同类写法共 236 处），与本次改法完全同构 —— 不存在"引用链不被解析"的风险。

💡 建议（非阻塞，已合并，可另开 issue / PR）

• _old 模板仍是旧写法且已无引用：assets/anvilcraft/models/item/ 下的 ember_metal_resonator_old.json、ember_metal_heavy_halberd_old.json、ember_metal_resonance_{axe,hoe,pickaxe,shovel}_old.json（共 6 个）内容就是本次被改的资源包模型的"原件"，仍保留 "0": "anvilcraft:item/..." + 面引用 #0 的旧写法。在 src/main 全仓 grep 未发现任何引用（git grep -n "_old" -- src/main 仅命中 active_silencer_old 配方与 gem_old.snbt），属于孤儿资源。若后续从这些 _old 重新同步资源包，该 bug 会原样复现；顺便一提 ember_metal_heavy_halberd_old.json 还缺 format_version。建议清理或同步改成 #1 形式。
• 合并后人工验收清单（贴图继承这类问题单测/CI 覆盖不到）：分别给余烬/霜/超然三种材质的共振器与重戟（含 mace/spear/sword/throwing 及 4 种共振工具）在资源包开启状态下确认显示各自材质、破坏粒子正确，尤其确认 frost/transcendence 不再是余烬外观。

结论: APPROVE —— 修复精准且最小（只改 9 个几何父模型、21 个子模型无需改动），根因、机制、覆盖完整性三项均已离线核对通过；标题已为规范的中英双语，无需修改（另：本机 gh token 已失效、且 PR 已合并，故未执行 gh pr edit）。

由 Hermes Agent 审查

[Gugle2308]

💾 Self-improvement review: Patched references/builtin-pack-slot-override-verification.md in skill 'texture-asset-pr-review' (1 replacement).

[Gugle2308]

代码审查摘要 — PR #5154

⚠️ 此 PR 已合并（post-merge 评论）。 事件送达时线上已完成 squash 合并：dev/26.1/1.6 ← commit 3f749246（"fix: 修复旧版共振器和重戟模型的材质继承 (#5149) (#5154)"，2026-10-03 04:48 UTC）。当前分支 tip 为 1a17b1db9（#5150，紧随其后 24s 落地），其树中已含本修复（已用 git show 实证）。因此下文以「合并后观察」为主，不含要求改后再合并的动作项。

范围： 9 个文件（纯资源 JSON，无 Java）/ +27 −18（9×3 增 / 9×2 删，与 diff 逐项吻合）/ 全部位于 resourcepacks/old_resonator_heavy_halberd
关联： issue #5149「共振器旧模型资源包中所有模型都是余烬材质」

✅ 修复机制核对（结论：正确）

改动把 pack 内顶层模型的贴图变量改成间接引用：

"textures": { "0": "#1", "1": "anvilcraft:item/ember_metal_resonator", "particle": "#1" }   // faces 仍用 #0

• 该 pack 的 18 个"薄子模型"（frost_metal_* / transcendence_*，仅 6 行：parent + "textures": {"1": ...}）早就声明了覆盖 1；但迁移到旧几何版本时，父模型（ember_metal_*）既没声明 1、faces 也没用 #1（全部 #0 并且 "0" 直接硬编码贴图）⇒ 子模型的覆盖是无效写入（死配置），所有金属变体于是都渲染余烬贴图，与 [Bug] 共振器旧模型资源包中所有模型都是余烬材质 #5149 报告完全一致。本修复让覆盖重新生效。✅
• 与主资源包既有约定对齐：assets/anvilcraft/models/item/ember_metal_resonator.json / ember_metal_heavy_halberd.json 本就是 "1": "anvilcraft:item/ember_metal_gear" + faces #1，子模型（frost_metal_*.json）覆盖 1。此 PR 把 pack 侧改成同一写法，方向正确。✅
• 物品侧链路可达性已核对：src/generated/resources/assets/anvilcraft/items/ember_metal_resonator.json 用 minecraft:select（component anvilcraft:resonate_mode / heavy_halberd 形态）指向 anvilcraft:item/ember_metal_resonance_axe 等 → 薄子模型 frost_metal_resonance_axe →（被 pack 覆盖的）ember_metal_resonance_axe → #0→#1。首尾贯通。✅

✅ 完整性核对（9/9 全覆盖，无漏项）

以脚本对 base（d30d52e8，合并前）与 head（ac6634af）全量对账：

检查项	结果
base 上"faces 用到硬编码贴图变量"的 pack 模型数	恰 9 个，与改动集合逐一对应（ember_metal_heavy_halberd{,_mace,_spear,_sword}、ember_metal_resonance_{axe,hoe,pickaxe,shovel}、ember_metal_resonator）
第 10 个 ember_metal_heavy_halberd_throwing.json 未改	✅ 正确——它无 textures/elements（仅 display），靠父链继承，无需改动
30 个 pack 模型 JSON 语法	全部可解析，无错误 ✅
faces 用到的变量是否都有定义（含父链）	全部可解析，无悬空变量、无未解引用 # ✅
子模型覆盖值指向的贴图存在性	pack 内 textures/item/{ember_metal_heavy_halberd, ember_metal_resonator, frost_metal_heavy_halberd, frost_metal_resonator, transcendence_heavy_halberd, transcendence_resonator}.png 均在 ✅
投掷形态链	pack 内 frost/transcendence_heavy_halberd_throwing 的 parent 是同金属模型（覆盖 1），不是 ember 基模型 ✅ 不会串材质
因果/时机反向核对	合并前 "0": "anvilcraft:item/..."（硬编码）→ 合并后 "0": "#1" ✅ 声称与 diff 一致

⚠️ 观察（post-merge，非阻塞）

1. 💡 同源遗留缺陷（未被引用的归档副本）：src/main/resources/assets/anvilcraft/models/item/ 下 6 个 *_old.json（ember_metal_resonator_old、ember_metal_resonance_{axe,hoe,pickaxe,shovel}_old、ember_metal_heavy_halberd_old）仍是硬编码 "0" + 硬编码 particle，没有 #1 间接层。git grep 全树（含 src/generated）确认当前零引用，属历史归档。风险在于将来有人把它们当作「旧模型 pack 模板」复用时会原样复现 [Bug] 共振器旧模型资源包中所有模型都是余烬材质 #5149。建议后续二选一：同步加 #1 间接层，或直接删除。
2. 💡 "0" 现在只是 "#1" 的转发别名（faces 全用 #0），比主资源包模型多一跳。以「最小 diff、不动 elements」为由保留是合理的；若将来允许改 elements，直接让 faces 用 #1 并删掉 "0" 会更贴近主资源包写法。
3. 🟢 兼容性：变量 0 仍保留 ⇒ 任何外部/第三方 pack 显式覆盖 "0" 依旧生效，行为向后兼容；particle 改为跟随 1，与字段语义（该物品的贴图）一致，属改进。
4. 🟢 无副作用面：改动仅限 pack 下模型 JSON；pack 不含 items/ 定义，物品→模型映射仍来自主资源包与生成资源；pack.mcmeta（min/max format [84,0]）未变；无 Java/注册/序列化/网络影响。

🧪 合并后回归建议

场景	期望结果
pack 开启 + 余烬金属（ember）共振器/重戟各形态	旧几何 + 余烬贴图（与修复前一致，无回归）
pack 开启 + 霜冻金属（frost）：共振器 5 形态、重戟 spear/sword/mace/投掷(using_item)	旧几何 + 霜冻贴图（修复前错误显示余烬贴图）
pack 开启 + 超越金属（transcendence）同上	旧几何 + 超越贴图
pack 关闭	现模型完全不变
破坏/使用粒子（particle）	跟随该金属贴图，不再是基贴图

手工验证一句话即可：开 pack 后 /give 三种金属的共振器 + 重戟，切换 axe/hoe/pickaxe/shovel 与 spear/sword/mace/投掷形态看贴图是否随金属变化。

📋 声称验证表

声称	状态	证据
修复旧版共振器模型材质继承	✅	5 个 ember_metal_resonance_*/resonator 加 #1 间接层，18 个薄子模型覆盖生效
修复旧版重戟模型材质继承	✅	4 个 ember_metal_heavy_halberd{,_mace,_spear,_sword} 同改
fixed #5149	✅	issue 标题「共振器旧模型资源包中所有模型都是余烬材质」，症状与本修复（覆盖变量失效）一致

结论：APPROVE（合并后追认） — 修复方向、范围与实现均正确，9/9 覆盖且已实证落库；唯一遗留是未被引用的 *_old.json 归档副本（建议后续清理，不影响本次修复）。

标题： 当前标题 Fix legacy tool model textures 修复旧版共振器与重戟模型材质 已符合 <English> <中文> 规范，无需修改（且 PR 已合并，未执行 gh pr edit）。

---

由 Hermes Agent 审查

[Gugle2308]

💾 Self-improvement review: Patched references/pr-class-routing.md in skill 'github-webhook-pr-review' (1 replacement). · Patched references/builtin-pack-slot-override-verification.md in skill 'texture-asset-pr-review' (1 replacement).